fix: cosmetic changes.

Signed-off-by: Alexander Cristurean <acristur@redhat.com>

Author: crstrn13

Makefile

@@ -71,7 +71,6 @@ disruptive: poetry-no-dev  ## Run disruptive tests
 egress-gateway: poetry-no-dev  ## Run egress gateway tests
 	$(PYTEST) -n4 -m 'egress_gateway' --dist loadfile --enforce $(flags) testsuite/tests/singlecluster/egress/
 
-
 kuadrantctl: poetry-no-dev  ## Run Kuadrantctl tests
 	$(PYTEST) -n4 --dist loadfile --enforce $(flags) testsuite/tests/kuadrantctl/
 

testsuite/backend/mockserver.py

@@ -126,7 +126,7 @@ def commit(self):
             self.cluster,
             self.name,
             selector=self.match_labels,
-            ports=[ServicePort(name="api", port=8080, targetPort="api")],
+            ports=[ServicePort(name="http", port=8080, targetPort="api")],
             labels={"app": self.label},
         )
         self.service.commit()

testsuite/gateway/gateway_api/gateway.py

@@ -15,7 +15,7 @@
 from testsuite.utils import check_condition, asdict, domain_match
 
 
-class KuadrantGateway(KubernetesObject, Gateway):  # pylint: disable=too-many-ancestors
+class KuadrantGateway(KubernetesObject, Gateway):
     """Gateway object for Kuadrant"""
 
     # Name of the GatewayClass that is to be used for all the instances

testsuite/gateway/gateway_api/hostname.py

@@ -56,7 +56,7 @@ def base_domain(self) -> str:
     def expose_hostname(self, name, exposable) -> Hostname:
         return StaticHostname(
             f"{name}.{self.base_domain}",
-            exposable.get_tls_cert if self.verify is None else lambda: self.verify,  # type: ignore
+            exposable.get_tls_cert if self.verify is None else lambda _hostname: self.verify,  # type: ignore
         )
 
     def commit(self):

testsuite/tests/singlecluster/conftest.py

@@ -175,17 +175,20 @@ def pytest_runtest_makereport(item, call):  # pylint: disable=unused-argument
     outcome = yield
     report = outcome.get_result()
 
-    if call.when != "call" or not report.passed:
+    if call.when != "call":
         return
 
     client = item.funcargs.get("client")
-    backend = item.funcargs.get("backend")
-    if client is None or backend is None or backend.external_hostname is None:
+    if not isinstance(client, KuadrantClient):
         return
 
     denied_ids = set(client.denied_request_ids)
     client.denied_request_ids.clear()
-    if not denied_ids:
+
+    backend = item.funcargs.get("backend")
+    if not report.passed or backend is None or not hasattr(backend, "external_hostname"):
+        return
+    if backend.external_hostname is None or not denied_ids:
         return
 
     backend_client = backend.external_hostname.client()