explicitly set processing times to null for failed jobs

apps/check-processing-time/src/check_processing_time.py

@@ -1,4 +1,4 @@
-from typing import Union
+from typing import Union, Optional
 
 
 def get_time_from_result(result: Union[list, dict]) -> Union[list, float]:
@@ -13,9 +13,9 @@ def get_time_from_result(result: Union[list, dict]) -> Union[list, float]:
     return processing_time
 
 
-def lambda_handler(event, _) -> list[Union[list, float]]:
+def lambda_handler(event, _) -> Optional[list[Union[list, float]]]:
     if event['processing_failed']:
-        raise ValueError('refusing to calculate processing times for failed job')
+        return None
 
     processing_results = event['processing_results']
     result_list = [processing_results[key] for key in sorted(processing_results.keys())]

tests/test_check_processing_time.py

@@ -88,5 +88,4 @@ def test_lambda_handler_failed_job():
             },
         }
     }
-    with pytest.raises(ValueError, match=r'^refusing to calculate processing times for failed job$'):
-        check_processing_time.lambda_handler(event, None)
+    assert check_processing_time.lambda_handler(event, None) is None