Merge pull request Kuadrant#407 from pehala/retry_codes_more

Add retry codes to additional tests
averevki · May 27, 2024 · 631de11 · 631de11
2 parents 288e85a + f270150
commit 631de11
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/testsuite/tests/kuadrant/authorino/multiple_hosts/test_remove_host.py b/testsuite/tests/kuadrant/authorino/multiple_hosts/test_remove_host.py
@@ -18,5 +18,6 @@ def test_removing_host(client, client2, auth, route, second_hostname):
     response = client.get("/get", auth=auth)
     assert response.status_code == 200
 
-    response = client2.get("/get", auth=auth)
-    assert response.status_code == 404
+    with second_hostname.client(retry_codes={200}) as failing_client:
+        response = failing_client.get("/get", auth=auth)
+        assert response.status_code == 404
diff --git a/testsuite/tests/kuadrant/reconciliation/test_httproute_matches.py b/testsuite/tests/kuadrant/reconciliation/test_httproute_matches.py
@@ -25,5 +25,6 @@ def test_matches(client, backend, route, hostname):
         response = failing_client.get("/get")
         assert response.status_code == 404, "Matches were not reconciled"
 
-    response = client.get("/anything/get")
-    assert response.status_code == 200
+    with hostname.client(retry_codes={404, 503}) as new_client:
+        response = new_client.get("/anything/get")
+        assert response.status_code == 200