Fixed lint warning in useForm test#28046
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThe test file moves its global test setup/teardown into the describe('useForm') block: beforeEach now calls useFakeTimers and sets up mocks inside that describe, and afterEach runs useRealTimers, clearAllMocks, and restoreAllMocks inside the same describe, scoping these operations to the useForm test suite. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
eedb013 to
ae7ba27
Compare
E2E Tests FailedTo view the Playwright test report locally, run: REPORT_DIR=$(mktemp -d) && gh run download 26238479364 -n playwright-report -D "$REPORT_DIR" && npx playwright show-report "$REPORT_DIR" |
no ref
This test-only change fixes a lint warning about
beforeEachandafterEachbeing too high up.