fix: address CodeRabbit review feedback

- Use module import for shared fixtures to avoid F811 lint collision
  with fixture argument names
- Move yield outside exception handlers so test assertion failures
  aren't masked as "deployment errors"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: mpk-droid

agents/langgraph/agentic_rag/tests/integration/conftest.py

@@ -4,7 +4,7 @@
 import os
 
 import pytest
-from integration.conftest import cluster_auth, repo_root  # noqa: F401
+import integration.conftest  # noqa: F401 — re-exports cluster_auth, repo_root
 from integration.utils import (
     MakeTargetError,
     RouteNotFoundError,
@@ -68,24 +68,23 @@ def deployed_agent(cluster_auth, agent_dir, agent_name):
 
     deployed = False
     try:
-        logger.info("Building image on cluster via build-openshift...")
-        run_make("build-openshift", cwd=agent_dir, timeout=600)
+        try:
+            logger.info("Building image on cluster via build-openshift...")
+            run_make("build-openshift", cwd=agent_dir, timeout=600)
 
-        logger.info("Deploying to cluster...")
-        run_make("deploy", cwd=agent_dir, timeout=300)
-        deployed = True
+            logger.info("Deploying to cluster...")
+            run_make("deploy", cwd=agent_dir, timeout=300)
+            deployed = True
 
-        route_url = get_route(agent_name, namespace=namespace)
-        logger.info("Agent deployed at %s", route_url)
+            route_url = get_route(agent_name, namespace=namespace)
+            logger.info("Agent deployed at %s", route_url)
+        except (MakeTargetError, RouteNotFoundError) as exc:
+            pytest.fail(f"Deployment failed: {exc}")
+        except Exception as exc:
+            pytest.fail(f"Unexpected error during deployment setup: {exc}")
 
         yield route_url
 
-    except (MakeTargetError, RouteNotFoundError) as exc:
-        pytest.fail(f"Deployment failed: {exc}")
-
-    except Exception as exc:
-        pytest.fail(f"Unexpected error during deployment: {exc}")
-
     finally:
         if deployed:
             logger.info("Tearing down deployment...")