openstack
/
placement
Code Issues Proposed changes

Merge "Added hacking rule for assertEqual(a in b, True/False)."

This commit is contained in:
Jenkins 2015-02-05 17:13:04 +00:00 committed by Gerrit Code Review
parent a6b44c07e1 9c7932d39f
commit 64df58a713
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
HACKING.rst
View File

@ -47,6 +47,9 @@ Nova Specific Commandments
- [N335] Check for usage of deprecated assertRaisesRegexp
- [N336] Must use a dict comprehension instead of a dict constructor with a sequence of key-value pairs.
- [N337] Don't import translation in tests
- [N338] Change assertEqual(A in B, True), assertEqual(True, A in B),
assertEqual(A in B, False) or assertEqual(False, A in B) to the more specific
assertIn/NotIn(A, B)
Creating Unit Tests
-------------------