From b80d018c079f39d5d7325231dedd311635d8a2c5 Mon Sep 17 00:00:00 2001 From: Timothy Crosley Date: Sun, 10 Mar 2019 21:57:30 -0700 Subject: Fix handling of whitespace character --- CHANGELOG.md | 2 ++ isort/isort.py | 4 ++-- test_isort.py | 13 +++++++++++++ 3 files changed, 17 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 66e88cc1..85638132 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,8 @@ Changelog ========= ### 4.3.15 - March 10, 2019 - hot fix release - Fixed a regression with handling streaming input from pipes (Issue #895) +- Fixed handling of \x0c whitespace character (Issue #811) +- Improved CLI documentation ### 4.3.14 - March 9, 2019 - hot fix release - Fixed a regression with */directory/*.py style patterns diff --git a/isort/isort.py b/isort/isort.py index ac21230f..adc64ab3 100644 --- a/isort/isort.py +++ b/isort/isort.py @@ -223,8 +223,8 @@ class SortImports(object): check_output = self.output check_against = file_contents if self.config['ignore_whitespace']: - check_output = check_output.replace(self.line_separator, "").replace(" ", "") - check_against = check_against.replace(self.line_separator, "").replace(" ", "") + check_output = check_output.replace(self.line_separator, "").replace(" ", "").replace("\x0c", "") + check_against = check_against.replace(self.line_separator, "").replace(" ", "").replace("\x0c", "") if check_output.strip() == check_against.strip(): if self.config['verbose']: diff --git a/test_isort.py b/test_isort.py index 58d27d17..8c157c4e 100644 --- a/test_isort.py +++ b/test_isort.py @@ -2875,3 +2875,16 @@ def test_to_ensure_empty_line_not_added_to_file_start_issue_889(): '# comment2\n' 'import sys\n') assert SortImports(file_contents=test_input).output == test_input + + +def test_to_ensure_correctly_handling_of_whitespace_only_issue_811(capsys): + test_input = ('import os\n' + 'import sys\n' + '\n' + '\x0c\n' + 'def my_function():\n' + ' print("hi")\n') + SortImports(file_contents=test_input, ignore_whitespace=True) + out, err = capsys.readouterr() + assert out == '' + assert err == '' -- cgit v1.2.1