diff --git a/debug_toolbar/panels/sql/tracking.py b/debug_toolbar/panels/sql/tracking.py index 53804195b..69e7ed8e0 100644 --- a/debug_toolbar/panels/sql/tracking.py +++ b/debug_toolbar/panels/sql/tracking.py @@ -93,7 +93,7 @@ def _quote_expr(self, element): except DjangoUnicodeDecodeError: return repr(element) elif isinstance(element, six.binary_type): - return '(binary data)' + return "(binary data)" else: return repr(element) diff --git a/tests/panels/test_sql.py b/tests/panels/test_sql.py index def14e4a2..c1179c62a 100644 --- a/tests/panels/test_sql.py +++ b/tests/panels/test_sql.py @@ -113,23 +113,29 @@ def test_param_conversion(self): ('["Foo", true, false]', "[10, 1]", '["2017-12-22 16:07:01"]'), ) - @unittest.skipIf(connection.vendor in ('sqlite', 'postgresql'), - 'Mixing bytestrings and text is not allowed on PostgreSQL and' - 'SQLite') + @unittest.skipIf( + connection.vendor in ("sqlite", "postgresql"), + "Mixing bytestrings and text is not allowed on PostgreSQL and SQLite" + ) def test_binary_param_force_text(self): self.assertEqual(len(self.panel._queries), 0) with connection.cursor() as cursor: - cursor.execute("SELECT * FROM auth_user WHERE username = %s", [b'\xff']) + cursor.execute( + "SELECT * FROM auth_user WHERE username = %s", + [b'\xff'] + ) self.assertEqual(len(self.panel._queries), 1) - self.assertEqual(self.panel._queries[0][1]['sql'], - "SELECT * FROM auth_user WHERE username = '\ufffd'") + self.assertEqual( + self.panel._queries[0][1]["sql"], + "SELECT * FROM auth_user WHERE username = '\ufffd'" + ) self.panel.process_response(self.request, self.response) self.panel.generate_stats(self.request, self.response) - @unittest.skipUnless(connection.vendor != 'sqlite', 'Test invalid for SQLite') + @unittest.skipUnless(connection.vendor != "sqlite", "Test invalid for SQLite") def test_raw_query_param_conversion(self): self.assertEqual(len(self.panel._queries), 0)