aboutsummaryrefslogtreecommitdiffstats
path: root/framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst
diff options
context:
space:
mode:
authorAshlee Young <ashlee@onosfw.com>2015-10-30 09:20:45 -0700
committerAshlee Young <ashlee@onosfw.com>2015-10-30 09:20:45 -0700
commit89a574196783f93e32d81244f37f726be11bbf30 (patch)
tree408c721579d29d49eebf50c78b7f4a8da7a6489e /framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst
parent62bb467ca10e4fd0ca23499953bc7f7f413dee16 (diff)
New ML2/L3 plugin to support SFC
Change-Id: Ie778a2b2e09a29972e28d70c8eedee407b1d8eb6 Signed-off-by: Ashlee Young <ashlee@onosfw.com>
Diffstat (limited to 'framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst')
-rw-r--r--framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst33
1 files changed, 33 insertions, 0 deletions
diff --git a/framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst b/framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst
new file mode 100644
index 00000000..06d3f2c1
--- /dev/null
+++ b/framework/src/openstack/neutron/plugin/networking-onos/networking-onos/HACKING.rst
@@ -0,0 +1,33 @@
+Neutron Style Commandments
+==========================
+
+- Step 1: Read the OpenStack Style Commandments
+ http://docs.openstack.org/developer/hacking/
+- Step 2: Read on
+
+Neutron Specific Commandments
+--------------------------
+
+- [N319] Validate that debug level logs are not translated
+- [N320] Validate that LOG messages, except debug ones, have translations
+- [N321] Validate that jsonutils module is used instead of json
+- [N322] We do not use @authors tags in source files. We have git to track
+ authorship.
+- [N323] Detect common errors with assert_called_once_with
+
+Creating Unit Tests
+-------------------
+For every new feature, unit tests should be created that both test and
+(implicitly) document the usage of said feature. If submitting a patch for a
+bug that had no unit test, a new passing unit test should be added. If a
+submitted bug fix does have a unit test, be sure to add a new one that fails
+without the patch and passes with the patch.
+
+All unittest classes must ultimately inherit from testtools.TestCase. In the
+Neutron test suite, this should be done by inheriting from
+neutron.tests.base.BaseTestCase.
+
+All setUp and tearDown methods must upcall using the super() method.
+tearDown methods should be avoided and addCleanup calls should be preferred.
+Never manually create tempfiles. Always use the tempfile fixtures from
+the fixture library to ensure that they are cleaned up.