From b0a6adf3b23dc417f26274ad0b348cb7d67231a0 Mon Sep 17 00:00:00 2001
From: Omar Abdelwahab <omara@fb.com>
Date: Mon, 6 Oct 2025 13:49:02 -0700
Subject: [PATCH] Added minor fix

---
 tests/integration/conftest.py | 26 ++++++++------------------
 1 file changed, 8 insertions(+), 18 deletions(-)

diff --git a/tests/integration/conftest.py b/tests/integration/conftest.py
index 0e553eef4..42015a608 100644
--- a/tests/integration/conftest.py
+++ b/tests/integration/conftest.py
@@ -87,9 +87,7 @@ def pytest_configure(config):
     suite = config.getoption("--suite")
     if suite:
         if suite not in SUITE_DEFINITIONS:
-            raise pytest.UsageError(
-                f"Unknown suite: {suite}. Available: {', '.join(sorted(SUITE_DEFINITIONS.keys()))}"
-            )
+            raise pytest.UsageError(f"Unknown suite: {suite}. Available: {', '.join(sorted(SUITE_DEFINITIONS.keys()))}")
 
     # Apply setups (global parameterizations): env + defaults
     setup = config.getoption("--setup")
@@ -127,9 +125,7 @@ def pytest_addoption(parser):
             """
         ),
     )
-    parser.addoption(
-        "--env", action="append", help="Set environment variables, e.g. --env KEY=value"
-    )
+    parser.addoption("--env", action="append", help="Set environment variables, e.g. --env KEY=value")
     parser.addoption(
         "--text-model",
         help="comma-separated list of text models. Fixture name: text_model_id",
@@ -169,7 +165,9 @@ def pytest_addoption(parser):
     )
 
     available_suites = ", ".join(sorted(SUITE_DEFINITIONS.keys()))
-    suite_help = f"Single test suite to run (narrows collection). Available: {available_suites}. Example: --suite=responses"
+    suite_help = (
+        f"Single test suite to run (narrows collection). Available: {available_suites}. Example: --suite=responses"
+    )
     parser.addoption("--suite", help=suite_help)
 
     # Global setups for any suite
@@ -241,11 +239,7 @@ def pytest_generate_tests(metafunc):
 
     # Generate test IDs
     test_ids = []
-    non_empty_params = [
-        (i, values)
-        for i, values in enumerate(param_values.values())
-        if values[0] is not None
-    ]
+    non_empty_params = [(i, values) for i, values in enumerate(param_values.values()) if values[0] is not None]
 
     # Get actual function parameters using inspect
     test_func_params = set(inspect.signature(metafunc.function).parameters.keys())
@@ -262,9 +256,7 @@ def pytest_generate_tests(metafunc):
             if parts:
                 test_ids.append(":".join(parts))
 
-    metafunc.parametrize(
-        params, value_combinations, scope="session", ids=test_ids if test_ids else None
-    )
+    metafunc.parametrize(params, value_combinations, scope="session", ids=test_ids if test_ids else None)
 
 
 pytest_plugins = ["tests.integration.fixtures.common"]
@@ -277,9 +269,7 @@ def pytest_ignore_collect(path: str, config: pytest.Config) -> bool:
         return False
 
     sobj = SUITE_DEFINITIONS.get(suite)
-    roots: list[str] = (
-        sobj.get("roots", []) if isinstance(sobj, dict) else getattr(sobj, "roots", [])
-    )
+    roots: list[str] = sobj.get("roots", []) if isinstance(sobj, dict) else getattr(sobj, "roots", [])
     if not roots:
         return False