Fix all E2E test failures (333/333 passing)#17
Merged
pratikbodkhe merged 3 commits intomainfrom May 3, 2026
Merged
Conversation
Root causes fixed: - "Meeting history" renamed to "Timeline" across 5 test files - Back navigation: tests expected <link> but app uses <button> with router.back() - Escape key: navigate to "/" first to establish browser history for router.back() - Guest share: add storageState override and exact text matching for strict mode - Dashboard outstanding items: use high-priority seed issue (Migrate CI) to avoid being hidden behind PREVIEW_COUNT by CSV-imported higher-priority issues - Quick-add verification: navigate to series page since new medium-priority issues are hidden behind "+N more" due to priority sorting - Series timeline: use href-based locator to target correct meeting's "Open meeting details" link when multiple meetings are expanded - Inline tasks: use dblclick() to match component's onDoubleClick handler - Timeline test data: rewrite to match actual seed data (4 meetings, not 7) - Data pollution: add afterAll cleanup to reset issue status after keyboard tests - Send brief: conditional mailto assertion, resilient copy button check - Navigation flow: expand meeting section then click "Open meeting details" link - Onboarding: increase timeouts, use flexible locators for redirect flow - CI workflow: fix supabase env extraction and migration step
- Add explicit wait for wizard visibility in beforeEach (15s timeout) to handle slower CI environments where profile fetch takes longer - Increase widget removal assertion timeout from 5s to 10s for animation completion on slower runners
issue-detail: add .first() to avoid matching both the title and the Next.js route announcer div. widgets: use .rounded-xl parent locator instead of fragile ../.. traversal to reliably target the widget shell for hover events.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
--workers=1(serial execution, worst-case for data pollution)