From 4c50e6e94d2679673bfc31d525bcc4bf92d1b16e Mon Sep 17 00:00:00 2001 From: Chris Clark Date: Tue, 13 Aug 2024 13:50:23 -0400 Subject: [PATCH] ruff --- explorer/tests/test_tasks.py | 4 ++-- explorer/tests/test_views.py | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/explorer/tests/test_tasks.py b/explorer/tests/test_tasks.py index 1fccf038..93f31f28 100644 --- a/explorer/tests/test_tasks.py +++ b/explorer/tests/test_tasks.py @@ -123,7 +123,7 @@ def test_remove_unused_sqlite_dbs(self, MockDatabaseConnection, MockQueryLog): mock_queryset.first.return_value = query_log1 def filter_side_effect(*args, **kwargs): - if kwargs.get("database_connection") == 3: + if kwargs.get("database_connection") == 3: # noqa: PLR2004 return mock_queryset return MagicMock(first=MagicMock(return_value=None)) @@ -163,7 +163,7 @@ def test_do_not_remove_recently_used_db(self, MockDatabaseConnection, MockQueryL mock_queryset.first.return_value = query_log1 def filter_side_effect(*args, **kwargs): - if kwargs.get("database_connection") == 3: + if kwargs.get("database_connection") == 3: # noqa: PLR2004 return mock_queryset return MagicMock(first=MagicMock(return_value=None)) diff --git a/explorer/tests/test_views.py b/explorer/tests/test_views.py index 4f2b05b2..387f1bd0 100644 --- a/explorer/tests/test_views.py +++ b/explorer/tests/test_views.py @@ -1055,7 +1055,8 @@ def test_database_connection_error(self, mock_load): mock_load.side_effect = DatabaseError("Connection error") response = self.client.post(self.url, data=self.valid_data) self.assertEqual(response.status_code, 200) - self.assertJSONEqual(response.content, {"success": False, "error": "Failed to create explorer connection: Connection error"}) + self.assertJSONEqual(response.content, {"success": False, + "error": "Failed to create explorer connection: Connection error"}) # The idea is to render all of these views, to ensure that errors haven't been introduced in the templates