From: Łukasz Langa <lukasz@langa.pl>
Date: Thu, 14 Mar 2019 16:08:45 +0000 (+0100)
Subject: Mention tab comment fixes, extend tests
X-Git-Url: https://git.madduck.net/etc/vim.git/commitdiff_plain/6af55d885132828be3775f6a7fd1652fdf30ec43?ds=sidebyside

Mention tab comment fixes, extend tests
---

diff --git a/README.md b/README.md
index c34fc29..87ac757 100644
--- a/README.md
+++ b/README.md
@@ -966,6 +966,9 @@ More details can be found in [CONTRIBUTING](CONTRIBUTING.md).
 * *Black* now creates cache files atomically which allows it to be used
   in parallel pipelines (like `xargs -P8`) (#673)
 
+* *Black* now correctly indents comments in files that were previously
+  formatted with tabs (#262)
+
 * `blackd` now supports CORS (#622)
 
 ### 18.9b0
diff --git a/tests/test_black.py b/tests/test_black.py
index db46499..7c99f00 100644
--- a/tests/test_black.py
+++ b/tests/test_black.py
@@ -526,18 +526,27 @@ class BlackTestCase(unittest.TestCase):
         black.assert_equivalent(source, actual)
         black.assert_stable(source, actual, black.FileMode())
 
-    def test_comment_indentation(self) -> None:
+    def test_tab_comment_indentation(self) -> None:
         contents_tab = "if 1:\n\tif 2:\n\t\tpass\n\t# comment\n\tpass\n"
         contents_spc = "if 1:\n    if 2:\n        pass\n    # comment\n    pass\n"
-
-        self.assertFormatEqual(fs(contents_spc), contents_spc)
-        self.assertFormatEqual(fs(contents_tab), contents_spc)
+        self.assertFormatEqual(contents_spc, fs(contents_spc))
+        self.assertFormatEqual(contents_spc, fs(contents_tab))
 
         contents_tab = "if 1:\n\tif 2:\n\t\tpass\n\t\t# comment\n\tpass\n"
         contents_spc = "if 1:\n    if 2:\n        pass\n        # comment\n    pass\n"
+        self.assertFormatEqual(contents_spc, fs(contents_spc))
+        self.assertFormatEqual(contents_spc, fs(contents_tab))
+
+        # mixed tabs and spaces (valid Python 2 code)
+        contents_tab = "if 1:\n        if 2:\n\t\tpass\n\t# comment\n        pass\n"
+        contents_spc = "if 1:\n    if 2:\n        pass\n    # comment\n    pass\n"
+        self.assertFormatEqual(contents_spc, fs(contents_spc))
+        self.assertFormatEqual(contents_spc, fs(contents_tab))
 
-        self.assertFormatEqual(fs(contents_tab), contents_spc)
-        self.assertFormatEqual(fs(contents_spc), contents_spc)
+        contents_tab = "if 1:\n        if 2:\n\t\tpass\n\t\t# comment\n        pass\n"
+        contents_spc = "if 1:\n    if 2:\n        pass\n        # comment\n    pass\n"
+        self.assertFormatEqual(contents_spc, fs(contents_spc))
+        self.assertFormatEqual(contents_spc, fs(contents_tab))
 
     def test_report_verbose(self) -> None:
         report = black.Report(verbose=True)