Skip to content
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

Create lab2 showcase progression on allthethings and add to bubble choice UI test #58538

Merged
merged 17 commits into from May 14, 2024

Conversation

sanchitmalhotra126
Copy link
Contributor

This creates a "Lab2 showcase" progression on allthethings as a place to test various cross-lab2 functionality. The immediate need for this was to add a UI test for bubble choice sublevels on lab2, which has been added to bubble_choice.feature. Also includes a related convenience fix to handle invalid sources in Weblab2.

Demo:
(note that the light mode footer for music lab showing up in this video is a known issue, only manifesting when music lab is part of a progression with other light mode lab2 labs. This will not manifest on any production music lab levels).

Lab2Showcase.mp4

Testing story

Ran UI tests against localhost with saucelabs connect.

@sanchitmalhotra126 sanchitmalhotra126 requested a review from a team as a code owner May 10, 2024 21:02
@sanchitmalhotra126 sanchitmalhotra126 requested review from lfryemason and removed request for a team May 10, 2024 21:02
@@ -39,3 +39,68 @@ Feature: BubbleChoice
When I click selector ".teacher-panel table td:contains(Alice)" once I see it to load a new page
And I wait for 4 seconds
Then I verify progress for the sublevel with selector ".uitest-bubble-choice:eq(0) .progress-bubble:first" is "perfect"

Scenario: Lab2 BubbleChoice progress
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tagged @code-dot-org/teacher-tools just as an FYI that I updated this UI test

Base automatically changed from sanchit/lab2-bubble-choice to staging May 13, 2024 17:19
Copy link
Contributor

@molly-moen molly-moen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One note on casing, otherwise looks good! Thanks for doing this!

@sanchitmalhotra126 sanchitmalhotra126 merged commit 27a3004 into staging May 14, 2024
2 checks passed
@sanchitmalhotra126 sanchitmalhotra126 deleted the sanchit/lab2-showcase branch May 14, 2024 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants