Conversation
Expand test suite from 17 tests (22% coverage) to 269 tests (94% coverage). New test files: - test_link_traverser.py (~50 tests) - test_report_generator.py (22 tests) - test_visual_comparison.py (~57 tests) Rewritten test files with comprehensive coverage: - test_cli.py (38 tests) - test_diff_engine.py (41 tests) - test_fetcher.py (16 tests) - test_wayback_cleaner.py (26 tests) Bug fixes found during testing: - diff_engine.py: fix compare_structures TypeError on Python 3.14 (SequenceMatcher rejects unhashable dict elements) - wayback_cleaner.py: fix remove_wayback_header fallback path skipping content past the meta tag boundary
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 8 minutes and 38 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (12)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
visual_comparison.py imports PIL which requires Pillow. It was in setup.py but not in requirements.txt used by CI.
Resolves NameError when selenium is not installed by making type annotations lazily evaluated via __future__.annotations.
Tests import selenium directly (TimeoutException, webdriver mock targets), so it must be present in the test environment.
Tests mock ChromeDriverManager/GeckoDriverManager and import numpy directly, so both must be present in the test environment.
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
Summary
test_link_traverser.py,test_report_generator.py,test_visual_comparison.pytest_cli.py,test_diff_engine.py,test_fetcher.py,test_wayback_cleaner.pydiff_engine.py:compare_structuresTypeError on Python 3.14 (SequenceMatcher rejects unhashable dict elements)wayback_cleaner.py:remove_wayback_headerfallback path skipping content past the<meta>tag boundaryPer-module coverage
__init__.pycli.pydiff_engine.pyfetcher.pylink_traverser.pyreport_generator.pyvisual_comparison.pywayback_cleaner.pyTest plan
pytest --cov=wayback_diff