Rename more Status to RequestStatus

opensafely-core · Mar 21, 2024 · 86eaf67 · 86eaf67
1 parent 33034dc
commit 86eaf67
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 9 deletions.
diff --git a/tests/integration/views/test_request.py b/tests/integration/views/test_request.py
@@ -515,26 +515,38 @@ def test_requests_release_files_404(airlock_client, release_files_stubber):
             "/requests/view/request-id/default/",
             None,
             "output_checker",
-            Status.PENDING,
+            RequestStatus.PENDING,
             False,
         ),
         (
             "/requests/view/request-id/default/file.txt",
             None,
             "output_checker",
-            Status.PENDING,
+            RequestStatus.PENDING,
             False,
         ),
         (
             "/requests/content/request-id/default/file.txt",
             None,
             "output_checker",
-            Status.PENDING,
+            RequestStatus.PENDING,
             False,
         ),
-        ("/requests/submit/request-id", {}, "author", Status.PENDING, False),
-        ("/requests/reject/request-id", {}, "output_checker", Status.SUBMITTED, False),
-        ("/requests/release/request-id", {}, "output_checker", Status.SUBMITTED, True),
+        ("/requests/submit/request-id", {}, "author", RequestStatus.PENDING, False),
+        (
+            "/requests/reject/request-id",
+            {},
+            "output_checker",
+            RequestStatus.SUBMITTED,
+            False,
+        ),
+        (
+            "/requests/release/request-id",
+            {},
+            "output_checker",
+            RequestStatus.SUBMITTED,
+            True,
+        ),
     ],
 )
 def test_request_view_tracing_with_request_attribute(

diff --git a/tests/unit/test_business_logic.py b/tests/unit/test_business_logic.py
@@ -565,9 +565,12 @@ def test_release_request_add_same_file(bll):
 
 def _get_current_file_reviews(bll, release_request, path, author):
     """Syntactic sugar to make the tests a little more readable"""
-    return bll.get_release_request(release_request.id, author).filegroups["default"].files[
-        path
-    ].reviews
+    return (
+        bll.get_release_request(release_request.id, author)
+        .filegroups["default"]
+        .files[path]
+        .reviews
+    )
 
 
 def test_approve_file_not_submitted(bll):