diff --git a/test/normalizer_issue_files/E101.py b/test/normalizer_issue_files/E101.py index 2790932..53b5c5f 100644 --- a/test/normalizer_issue_files/E101.py +++ b/test/normalizer_issue_files/E101.py @@ -88,10 +88,10 @@ if length > options.max_line_length: if os.path.exists(os.path.join(path, PEP8_BIN)): cmd = ([os.path.join(path, PEP8_BIN)] + self._pep8_options(targetfile)) -#: E999+1 +# TODO Tabs in docstrings shouldn't be there, use \t. ''' multiline string with tab in it''' -#: E999+1 +# Same here. '''multiline string with tabs and spaces diff --git a/test/normalizer_issue_files/E22.py b/test/normalizer_issue_files/E22.py index ce454cd..c2c8010 100644 --- a/test/normalizer_issue_files/E22.py +++ b/test/normalizer_issue_files/E22.py @@ -94,7 +94,7 @@ c = (a+b) * (a-b) norman = True+False x = x*2 - 1 x = x/2 - 1 -#: E999 +# TODO whitespace should be the other way around according to pep8. x = x / 2-1 hypot2 = x*x + y*y diff --git a/test/normalizer_issue_files/E25.py b/test/normalizer_issue_files/E25.py index 98052fb..5b0d540 100644 --- a/test/normalizer_issue_files/E25.py +++ b/test/normalizer_issue_files/E25.py @@ -19,7 +19,7 @@ parser.add_argument('--long-option', parser.add_argument('--long-option', default= "/rather/long/filesystem") -#: E999+1:44 +# TODO this looks so stupid. parser.add_argument('--long-option', default ="/rather/long/filesystem/path/here/blah/blah/blah") #: E251+2:7 E251+2:9 diff --git a/test/test_normalizer_issues_files.py b/test/test_normalizer_issues_files.py index 373a924..0cd610d 100644 --- a/test/test_normalizer_issues_files.py +++ b/test/test_normalizer_issues_files.py @@ -44,8 +44,6 @@ def collect_errors(code): code, _, add_line = code.partition('+') l = line_nr + 1 + int(add_line or 0) - if code == 'E999': continue # TODO delete - yield WantedIssue(code[1:], l, column)