Repoint starwars dance flappy UI tests#73392
Open
davidsbailey wants to merge 10 commits into
Open
Conversation
Use allthethingscourse L24/9 (Star wars #9, free-play, skin hoc2015) in place of courses/starwars/units/1/lessons/1/levels/15. The ar-sa RTL Applitools snapshot will need a one-time re-baseline on drone (course-header chrome differs). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Use allthethingscourse L24/7 (Star wars #7) in place of courses/starwars/units/1/lessons/1/levels/5; pairing is course-independent. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Navigate to allthethingscourse unit 1 lesson 37 level 8 (Dance_Party_12) instead of dance unit 1 lesson 1 level 13 -- the same Dance Lab level. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Use allthethingscourse L7/2 (flappy_11, freePlay) in place of /flappy/10, which resolves to the same flappy_11 freeplay level. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Background navigates to allthethingscourse L7/2 (flappy_11) instead of /flappy/10 (same freeplay level). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Use allthethingscourse L7/1 (flappy_1) for all three /flappy/1 references; same level, so the hardcoded YouTube id still matches. The @eyes 'Fallback player' snapshot needs a one-time drone re-baseline. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Use ui-test-csf/units/1 as the non-AI unit page in place of courses/flappy/units/1. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Second navigation (cookie-persistence check) now uses allthethingscourse L37/5 (Dance Lab #5) instead of courses/dance/units/1/lessons/1/levels/9. Line 3 was already on allthethings L37/1; this clears the last live courses/dance reference. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
No running UI test references either course any more: all courses/<slug> and tutorial-root URLs were repointed onto allthethingscourse (and ui-test-csf). The surviving references are seeding-independent -- /projects/{starwars,dance} standalone routes (global template levels), the create-dropdown / make-project project-type options, a @Skip footer scenario, an eyes snapshot label, and an href-string check. The dance/starwars levels that allthethings reuses stay seeded globally via custom_levels regardless of this script removal.
[reset db] so drone wipes and reseeds the DB, the only way a seed removal is actually validated.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.
This PR is part of the effort to move UI tests off of production courses.
Done in this PR:
Links
Testing story
Deployment notes
starwarsanddancecourses from test-studio, which may cause some eyes diffs