test: disables the FoxLoader on E2E#30451
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection:
Selected tags cover the most critical user flows that depend on app startup completing correctly:
These tags represent a broad cross-section of tests that will validate the FoxLoader E2E path works correctly across different app flows. Not selecting all tags since the change is production-safe and only affects E2E test execution behavior. Performance Test Selection: |
|



Description
This PR disables the FoxLoader on E2E context. This animation has been causing extra timers on E2E runs.
Changelog
CHANGELOG entry:
Related issues
Fixes: -
Manual testing steps
N/A
Screenshots/Recordings
Before
After
Pre-merge author checklist
Performance checks (if applicable)
trace()for usage andaddTokenfor an exampleFor performance guidelines and tooling, see the Performance Guide.
Pre-merge reviewer checklist
Note
Low Risk
Low risk: change is gated behind the
isE2Eenvironment flag and mainly affects startup splash behavior during automated tests.Overview
Disables the
FoxLoaderanimation in E2E builds. WhenisE2Eis true,FoxLoadernow rendersnull, immediately callshideAsync(), marks the session animation as complete, and triggersonAnimationCompleteto avoid extra timers during E2E runs.Updates
FoxLoadertests to mockisE2E, assert the component renders nothing in E2E, and verify the completion callback and splash hide are invoked.Reviewed by Cursor Bugbot for commit 372a8a9. Bugbot is set up for automated code reviews on this repo. Configure here.