diff options
Diffstat (limited to 'osprofiler/hacking/checks.py')
| -rw-r--r-- | osprofiler/hacking/checks.py | 23 |
1 files changed, 12 insertions, 11 deletions
diff --git a/osprofiler/hacking/checks.py b/osprofiler/hacking/checks.py index c77e91d..b31a47d 100644 --- a/osprofiler/hacking/checks.py +++ b/osprofiler/hacking/checks.py @@ -166,8 +166,8 @@ def assert_equal_none(logical_line, filename): N322 """ - res = (re_assert_equal_start_with_none.search(logical_line) or - re_assert_equal_end_with_none.search(logical_line)) + res = (re_assert_equal_start_with_none.search(logical_line) + or re_assert_equal_end_with_none.search(logical_line)) if res: yield (0, "N322 assertEqual(A, None) or assertEqual(None, A) " "sentences not allowed, you should use assertIsNone(A) " @@ -185,8 +185,9 @@ def assert_true_or_false_with_in(logical_line, filename): N323 """ - res = (re_assert_true_false_with_in_or_not_in.search(logical_line) or - re_assert_true_false_with_in_or_not_in_spaces.search(logical_line)) + res = (re_assert_true_false_with_in_or_not_in.search(logical_line) + or re_assert_true_false_with_in_or_not_in_spaces.search( + logical_line)) if res: yield (0, "N323 assertTrue/assertFalse(A in/not in B)sentences not " "allowed, you should use assertIn(A, B) or assertNotIn(A, B)" @@ -204,8 +205,8 @@ def assert_equal_in(logical_line, filename): N324 """ - res = (re_assert_equal_in_end_with_true_or_false.search(logical_line) or - re_assert_equal_in_start_with_true_or_false.search(logical_line)) + res = (re_assert_equal_in_end_with_true_or_false.search(logical_line) + or re_assert_equal_in_start_with_true_or_false.search(logical_line)) if res: yield (0, "N324: Use assertIn/NotIn(A, B) rather than " "assertEqual(A in/not in B, True/False) when checking " @@ -226,8 +227,8 @@ def check_quotes(logical_line, filename): check_tripple = ( lambda line, i, char: ( - i + 2 < len(line) and - (char == line[i] == line[i + 1] == line[i + 2]) + i + 2 < len(line) + and (char == line[i] == line[i + 1] == line[i + 2]) ) ) @@ -291,9 +292,9 @@ def check_dict_formatting_in_string(logical_line, tokens): # NOTE(stpierre): Can't use @skip_ignored_lines here because it's # a stupid decorator that only works on functions that take # (logical_line, filename) as arguments. - if (not logical_line or - logical_line.startswith("#") or - logical_line.endswith("# noqa")): + if (not logical_line + or logical_line.startswith("#") + or logical_line.endswith("# noqa")): return current_string = "" |