Skip to content

Commit

Permalink
Fix IngressClass test
Browse files Browse the repository at this point in the history
Previously, the test would try to send a request to NGINX without
ensuring the connection to NGINX first. This could lead to a
request timeout, which in turn would lead to a test run hang.

The commit ensures the connection to NGINX.
  • Loading branch information
pleshakov committed Sep 17, 2021
1 parent 77c3f71 commit 9a87419
Showing 1 changed file with 7 additions and 0 deletions.
7 changes: 7 additions & 0 deletions tests/suite/test_ingress_class.py
Original file line number Diff line number Diff line change
Expand Up @@ -99,10 +99,17 @@ def test_response_codes_117_plus(
backend_setup,
expected_responses,
ingress_controller_prerequisites,
ingress_controller_endpoint,
):
"""
Checks for ingressClass behaviour
"""
ensure_connection_to_public_endpoint(
ingress_controller_endpoint.public_ip,
ingress_controller_endpoint.port,
ingress_controller_endpoint.port_ssl,
)

for item in ingresses_under_test:
ensure_response_from_backend(
backend_setup.req_url, backend_setup.ingress_hosts[item]
Expand Down

0 comments on commit 9a87419

Please sign in to comment.