From 328cf93003aad97ea14752f82bfe65a7e6b2bc41 Mon Sep 17 00:00:00 2001
From: Ryunosuke O'Neil <r.oneil@cern.ch>
Date: Mon, 10 Feb 2025 18:37:55 +0100
Subject: [PATCH] pre-commit

---
 .../src/diracx/routers/jobs/status.py         |  1 -
 diracx-routers/tests/test_job_manager.py      | 21 +++++++++----------
 2 files changed, 10 insertions(+), 12 deletions(-)

diff --git a/diracx-routers/src/diracx/routers/jobs/status.py b/diracx-routers/src/diracx/routers/jobs/status.py
index fb973fd7..c1776329 100644
--- a/diracx-routers/src/diracx/routers/jobs/status.py
+++ b/diracx-routers/src/diracx/routers/jobs/status.py
@@ -178,4 +178,3 @@ async def patch_metadata(
                 int(job_id),
                 p_updates_,
             )
-
diff --git a/diracx-routers/tests/test_job_manager.py b/diracx-routers/tests/test_job_manager.py
index a54573f7..34d90e54 100644
--- a/diracx-routers/tests/test_job_manager.py
+++ b/diracx-routers/tests/test_job_manager.py
@@ -1602,25 +1602,24 @@ def test_patch_metadata(normal_user_client: TestClient, valid_job_id: int):
         assert j["MinorStatus"] == "Job accepted"
         assert j["ApplicationStatus"] == "Unknown"
 
-
-    before = r.json()
-
     # Act
     hbt = str(datetime.now(timezone.utc))
     r = normal_user_client.patch(
         "/api/jobs/metadata",
         json={
             valid_job_id: {
-                    "UserPriority": 2,
-                    "HeartBeatTime": hbt,
-                    # set a parameter
-                    "JobType": "VerySpecialIndeed",
+                "UserPriority": 2,
+                "HeartBeatTime": hbt,
+                # set a parameter
+                "JobType": "VerySpecialIndeed",
             }
         },
     )
 
     # Assert
-    assert r.status_code == 204, "PATCH metadata should return 204 No Content on success"
+    assert (
+        r.status_code == 204
+    ), "PATCH metadata should return 204 No Content on success"
     r = normal_user_client.post(
         "/api/jobs/search",
         json={
@@ -1636,9 +1635,9 @@ def test_patch_metadata(normal_user_client: TestClient, valid_job_id: int):
     )
     assert r.status_code == 200, r.json()
 
-    after =r.json()
-
     assert r.json()[0]["JobID"] == valid_job_id
     assert r.json()[0]["JobType"] == "VerySpecialIndeed"
-    assert datetime.fromisoformat(r.json()[0]["HeartBeatTime"]) == datetime.fromisoformat(hbt)
+    assert datetime.fromisoformat(
+        r.json()[0]["HeartBeatTime"]
+    ) == datetime.fromisoformat(hbt)
     assert r.json()[0]["UserPriority"] == 2