Issue wemake-services#1111 - Forbid to use \r in line breaks: some mi…

…nor processing [step2]
jigi-33 · Feb 22, 2020 · 23a4c21 · 23a4c21
1 parent 3e380a5
commit 23a4c21
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/tests/test_visitors/test_tokenize/test_consistency/test_string_newlines.py b/tests/test_visitors/test_tokenize/test_consistency/test_string_newlines.py
@@ -18,7 +18,9 @@
 
 wrong_newline_single = 'print(1)\rprint(2)'
 wrong_newline_sequenced = 'print(1)\r\nprint(2)'
-wrong_newline_in_multiline = """print(1)\rprint(2)."""
+wrong_newline_in_multiline = """print(1)\rprint(2)\r
+print(3).
+"""
 
 
 @pytest.mark.parametrize('code', [

diff --git a/wemake_python_styleguide/visitors/tokenize/syntax.py b/wemake_python_styleguide/visitors/tokenize/syntax.py
@@ -61,7 +61,9 @@ def _check_line_starts_with_dot(self, token: tokenize.TokenInfo) -> None:
         if line.startswith('.') and not line.startswith('...'):
             self.add_violation(LineStartsWithDotViolation(token))
 
-    def _check_line_comprise_carriage_return(self, token: tokenize.TokenInfo) -> None:
+    def _check_line_comprise_carriage_return(
+        self, token: tokenize.TokenInfo
+    ) -> None:
         line = token.line.find('\r')
         if line != -1 and line == (len(token.line) - 2):
             self.add_violation(LineCompriseCarriageReturnViolation(token))