Repository navigation
Fix main's lint and tests after two merges crossed - #1569
Conversation
The project-setup test added in #1413 uses mock.patch.object, but the file has no module-level `from unittest import mock`, so flake8 reports F821 and the test errors with NameError on main. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014T3x9mgq9C4Pvq7PDkfi1g
✅ Deploy Preview for antenna-preview canceled.
|
✅ Deploy Preview for antenna-ssec canceled.
|
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 52 minutes. View limit details
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 |
mihow
changed the title
Summary
Main is red on lint and on two test errors. #1515 removed the module-level
mockimport fromami/main/tests.pywhile #1413, which uses it, was still open; each passed CI alone, and they merged 11 minutes apart. This adds the import back so main, and every open PR's CI, can pass again.List of Changes
ami/main/tests.pyimportsfrom unittest import mock.Review focus
NameError: name 'mock' is not definedinTestProjectSetup, and flake8 reported F821 at the same line.Testing
pre-commit run --files ami/main/tests.py: black, isort and flake8 pass.manage.py test ami.main.tests.TestProjectSetup: 8 tests pass.🤖 Generated with Claude Code
https://claude.ai/code/session_014T3x9mgq9C4Pvq7PDkfi1g