[talelboussetta]

Summary

tests/docker/test_pool_recycle_config.py (added in #2232) imports crawl4ai at the top, above its pytest.importorskip("crawler_pool", reason="docker server deps not installed") guard. In an environment where the package's dependencies are missing, that import raises ModuleNotFoundError during collection and pytest stops with "Interrupted: 1 error during collection", instead of skipping the file as the guard intends.

This moves the crawl4ai import below the guard. Follow-up to #2232 (I noted this in review there).

List of files changed and why

• tests/docker/test_pool_recycle_config.py - import BrowserConfig after importorskip, so a missing dependency skips the file.

How Has This Been Tested?

Bare venv (pytest, pyyaml, repo installed with --no-deps), pytest --collect-only tests/docker/test_pool_recycle_config.py:

• before: ModuleNotFoundError: No module named 'pydantic', "Interrupted: 1 error during collection"
• after: SKIPPED ... docker server deps not installed, "1 skipped"

With crawl4ai's deps but no docker server deps: 1 skipped before and after. With everything installed (docker requirements + pytest-asyncio): 5 passed before and after, rebased on current develop, so behaviour with deps present is unchanged.

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added/updated unit tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.