File / lines: Multiple test gaps across tests/
Problem: Several important error paths and fallback paths called out by AGENTS.md are not covered:
- HTTP artifact collection failure path (
scraper.py, lines 267-268)
- Cookie jar
LoadError path (scraper.py, lines 539-553)
- LLM repair exception handling (
place_scraper.py, lines 1335-1338)
- Unsupported LLM policy
ValueError (place_scraper.py, lines 2692-2693)
- Place preview enrichment network-failure fallback (
place_scraper.py, lines 2736-2778)
- Saved-list parser HTML fallback path (
parser.py, lines 137-138)
- Consent "More options" two-step flow (
scraper.py, lines 384-415)
Suggested fix: Add focused unit tests for each path. Use fixtures or mocked dependencies (e.g., patch _import_curl_requests, fake page objects) to exercise error handling without requiring network access.
Difficulty: good first issue
Impact: Medium — prevents regressions in failure handling and improves confidence in refactors.
File / lines: Multiple test gaps across
tests/Problem: Several important error paths and fallback paths called out by
AGENTS.mdare not covered:scraper.py, lines 267-268)LoadErrorpath (scraper.py, lines 539-553)place_scraper.py, lines 1335-1338)ValueError(place_scraper.py, lines 2692-2693)place_scraper.py, lines 2736-2778)parser.py, lines 137-138)scraper.py, lines 384-415)Suggested fix: Add focused unit tests for each path. Use fixtures or mocked dependencies (e.g., patch
_import_curl_requests, fake page objects) to exercise error handling without requiring network access.Difficulty: good first issue
Impact: Medium — prevents regressions in failure handling and improves confidence in refactors.