Compare tuples to tuples when annotating tests

1 files changed, 30 insertions, 16 deletions

diff --git a/tests/unittest_checker_logging.py b/tests/unittest_checker_logging.py
index fd29d6fab..fe813cbe6 100644
--- a/tests/unittest_checker_logging.py
+++ b/tests/unittest_checker_logging.py
@@ -92,11 +92,11 @@ class TestLoggingModuleDetection(CheckerTestCase):
         )
         self.checker.visit_module(None)
         self.checker.visit_import(stmts[0])
-        messages = [
-            Message(msg, node=stmts[1], args=args, confidence=UNDEFINED)
-        ]
+        messages = [Message(msg, node=stmts[1], args=args, confidence=UNDEFINED)]
         if with_too_many:
-            messages.append(Message("logging-too-many-args", node=stmts[1], confidence=UNDEFINED))
+            messages.append(
+                Message("logging-too-many-args", node=stmts[1], confidence=UNDEFINED)
+            )
         with self.assertAddsMessages(*messages):
             self.checker.visit_call(stmts[1])
 
@@ -120,7 +120,9 @@ class TestLoggingModuleDetection(CheckerTestCase):
     def test_brace_format_style_too_few_args(self):
         self._assert_logging_format_too_few_args("('{}, {}', 1)")
         self._assert_logging_format_too_few_args("('{0}, {1}', 1)")
-        self._assert_logging_format_too_few_args("('{named1}, {named2}', {'named1': 1})")
+        self._assert_logging_format_too_few_args(
+            "('{named1}, {named2}', {'named1': 1})"
+        )
         self._assert_logging_format_too_few_args("('{0}, {named}', 1)")
         self._assert_logging_format_too_few_args("('{}, {named}', {'named': 1})")
         self._assert_logging_format_too_few_args("('{0}, {named}', {'named': 1})")
@@ -131,36 +133,48 @@ class TestLoggingModuleDetection(CheckerTestCase):
         self._assert_logging_format_too_many_args("('{}', 1, 2)")
         self._assert_logging_format_too_many_args("('{0}', 1, 2)")
         self._assert_logging_format_too_many_args("('{}, {named}', 1, 2, {'named': 1})")
-        self._assert_logging_format_too_many_args("('{0}, {named}', 1, 2, {'named': 1})")
+        self._assert_logging_format_too_many_args(
+            "('{0}, {named}', 1, 2, {'named': 1})"
+        )
 
-    @pytest.mark.skipif(sys.version_info[0] < (3, 6), reason="F-string require >=3.6")
+    @pytest.mark.skipif(sys.version_info < (3, 6), reason="F-string require >=3.6")
     @set_config(logging_format_style="new")
     def test_fstr_not_new_format_style_matching_arguments(self):
         msg = "logging-format-interpolation"
-        args=('{', '')
+        args = ("{", "")
         self._assert_logging_format_message(msg, "(f'{named}')", args)
 
     @set_config(logging_format_style="fstr")
     def test_modulo_not_fstr_format_style_matching_arguments(self):
         msg = "logging-format-interpolation"
-        args=('f-string', '')
+        args = ("f-string", "")
         with_too_many = True
         self._assert_logging_format_message(msg, "('%s', 1)", args, with_too_many)
-        self._assert_logging_format_message(msg, "('%(named)s', {'named': 1})", args, with_too_many)
-        self._assert_logging_format_message(msg, "('%s %(named)s', 1, {'named': 1})", args, with_too_many)
+        self._assert_logging_format_message(
+            msg, "('%(named)s', {'named': 1})", args, with_too_many
+        )
+        self._assert_logging_format_message(
+            msg, "('%s %(named)s', 1, {'named': 1})", args, with_too_many
+        )
 
     @set_config(logging_format_style="fstr")
     def test_brace_not_fstr_format_style_matching_arguments(self):
         msg = "logging-format-interpolation"
-        args=('f-string', '')
+        args = ("f-string", "")
         with_too_many = True
         self._assert_logging_format_message(msg, "('{}', 1)", args, with_too_many)
         self._assert_logging_format_message(msg, "('{0}', 1)", args, with_too_many)
-        self._assert_logging_format_message(msg, "('{named}', {'named': 1})", args, with_too_many)
-        self._assert_logging_format_message(msg, "('{} {named}', 1, {'named': 1})", args, with_too_many)
-        self._assert_logging_format_message(msg, "('{0} {named}', 1, {'named': 1})", args, with_too_many)
+        self._assert_logging_format_message(
+            msg, "('{named}', {'named': 1})", args, with_too_many
+        )
+        self._assert_logging_format_message(
+            msg, "('{} {named}', 1, {'named': 1})", args, with_too_many
+        )
+        self._assert_logging_format_message(
+            msg, "('{0} {named}', 1, {'named': 1})", args, with_too_many
+        )
 
-    @pytest.mark.skipif(sys.version_info[0] < (3, 6), reason="F-string require >=3.6")
+    @pytest.mark.skipif(sys.version_info < (3, 6), reason="F-string require >=3.6")
     @set_config(logging_format_style="fstr")
     def test_fstr_format_style_matching_arguments(self):
         self._assert_logging_format_no_messages("(f'constant string')")