-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
custom eslint rule for e2e test length #8431
Conversation
No loom links were found in the first post. Please add one there if you'd like to it to appear on Slack. Do not edit this comment manually. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8431 +/- ##
==========================================
- Coverage 73.47% 73.43% -0.05%
==========================================
Files 1334 1352 +18
Lines 41259 41518 +259
Branches 7686 7772 +86
==========================================
+ Hits 30316 30487 +171
- Misses 10943 11031 +88 ☔ View full report in Codecov by Sentry. |
Playwright test results - MV2Details Open report ↗︎ Flaky testschromeSetup › auth.setup.ts › authenticate Skipped testschrome › tests/extensionConsoleActivation.spec.ts › can activate a mod with built-in integration |
Playwright test results - MV3Details Open report ↗︎ Flaky testschrome › tests/regressions/doNotCloseSidebarOnPageEditorSave.spec.ts › #8104: Do not automatically close the sidebar when saving in the Page Editor |
This rule will encourage us to break up long tests using refactored page methods or
test.step
Primary reviewer @grahamlangford