Add retry logic with 30s timeout for evidence file downloads

dvacca-onfido · dvacca-onfido · commit dd8c3d9937ad · 2026-02-19T10:38:16.000+01:00
diff --git a/tests/test_workflow_runs.py b/tests/test_workflow_runs.py
@@ -72,7 +72,9 @@ def test_find_workflow_run(onfido_api, workflow_run):
 
 
 def test_download_evidence_file(onfido_api, workflow_run):
-    file = onfido_api.download_signed_evidence_file(workflow_run.id)
+    file = repeat_request_until_http_code_changes(
+        onfido_api.download_signed_evidence_file, [workflow_run.id], sleep_time=2
+    )
 
     assert len(file) > 0
     assert file[:4] == b"%PDF"
@@ -88,7 +90,7 @@ def test_download_evidence_folder(onfido_api, applicant_id):
     )
 
     file = repeat_request_until_http_code_changes(
-        onfido_api.download_evidence_folder, [workflow_run_id]
+        onfido_api.download_evidence_folder, [workflow_run_id], sleep_time=2
     )
 
     assert len(file) > 0

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,9 @@ def test_find_workflow_run(onfido_api, workflow_run):`
`72`	`72`
`73`	`73`
`74`	`74`	`def test_download_evidence_file(onfido_api, workflow_run):`
`75`		`- file = onfido_api.download_signed_evidence_file(workflow_run.id)`
	`75`	`+ file = repeat_request_until_http_code_changes(`
	`76`	`+ onfido_api.download_signed_evidence_file, [workflow_run.id], sleep_time=2`
	`77`	`+ )`
`76`	`78`
`77`	`79`	`assert len(file) > 0`
`78`	`80`	`assert file[:4] == b"%PDF"`
`@@ -88,7 +90,7 @@ def test_download_evidence_folder(onfido_api, applicant_id):`
`88`	`90`	`)`
`89`	`91`
`90`	`92`	`file = repeat_request_until_http_code_changes(`
`91`		`- onfido_api.download_evidence_folder, [workflow_run_id]`
	`93`	`+ onfido_api.download_evidence_folder, [workflow_run_id], sleep_time=2`
`92`	`94`	`)`
`93`	`95`
`94`	`96`	`assert len(file) > 0`