Changes:
- list() declarations replaced with []
- dict() declarations replaced with {}
- Added N351 to hacking for literal rule along with checks, test hacking, and to readme
- Minor fix to incorrect path in hacking guidelines pointing to test_hacking.py
Change-Id: Ie7b9201d61b6f2024217b6f4117605eb52fa916a
Closes-Bug: #1437515
Rally Style Commandments
- Step 1: Read the OpenStack Style Commandments http://docs.openstack.org/developer/hacking/
- Step 2: Read on
Rally Specific Commandments
- * [N30x] - Reserved for rules related to
mocklibrary -
- [N301] - Ensure that
assert_*methods frommocklibrary is used correctly - [N302] - Ensure that nonexistent "assert_called" is not used
- [N303] - Ensure that nonexistent "assert_called_once" is not used
- [N301] - Ensure that
- * [N310-N314] - Reserved for rules related to logging
-
- [N310] - Ensure that
rally.common.logis used as logging module - [N311] - Validate that debug level logs are not translated
- [N312] - Validate correctness of debug on check.
- [N310] - Ensure that
- * [N32x] - Reserved for rules related to assert* methods
-
- [N320] - Ensure that
assertTrue(isinstance(A, B))is not used - [N321] - Ensure that
assertEqual(type(A), B)is not used - [N322] - Ensure that
assertEqual(A, None)andassertEqual(None, A)are not used - [N323] - Ensure that
assertTrue/assertFalse(A in/not in B)are not used with collection contents - [N324] - Ensure that
assertEqual(A in/not in B, True/False)andassertEqual(True/False, A in/not in B)are not used with collection contents
- [N320] - Ensure that
- [N340] - Ensure that we are importing always
from rally import objects - [N341] - Ensure that we are importing oslo_xyz packages instead of deprecated oslo.xyz ones
- [N350] - Ensure that single quotes are not used
- [N351] - Ensure that data structs (i.e Lists and Dicts) are declared literally rather than using constructors