Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

Author: cfm

API2.md

@@ -155,7 +155,7 @@ direction TB
 [*] --> CacheLookup : process(event)
 CacheLookup: status = redis.get(event.id)
 
-CacheLookup --> IdempotentBranch : status in {102 Processing, 200 OK, 207 MultiStatus}
+CacheLookup --> IdempotentBranch : status in {102 Processing, 200 OK}
 CacheLookup --> StartBranch : status == None
 
 state "Enforce idempotency" as IdempotentBranch {

securedrop/tests/test_journalist_api2.py

@@ -1170,10 +1170,10 @@ def test_api2_source_conversation_truncated(
 
         status_code, msg = response.json["events"][event.id]
         # Because some deletes may fail (simulated) and some succeed, the handler
-        # returns 200 if all succeed, or 207 (MultiStatus) if any fail.
+        # returns 200 if all succeed.
         # The test_files fixtures never cause delete_file_object() to raise,
         # so OK (200) is expected.
-        assert status_code in (200, 207)
+        assert status_code == 200
 
         # Verify item-wise results
         returned_items = response.json["items"]