Skip to content

Run cypress tests/fix user course session test #1394

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

Merged
merged 7 commits into from
Mar 24, 2025

Conversation

kyleecodes
Copy link
Member

@kyleecodes kyleecodes commented Mar 24, 2025

Resolves #enter-issue-number

N/A

What changes did you make and why did you make them?

Cypress test user-course-session-behaviour is exclusively failing in the GHA staging environment but passing locally.
This PR also updates the cypress-test.yml workflow to upgrade the nodejs version in the cypress-run job to 22 (the default for ubuntu-24.04 is 20, so this needed explicitly set)

Potential causes: (bold is solution)

  • race condition in redux store update (marking session complete -> redux store updating with completion status -> feedback form appearing) may take longer or occur in a different order in the staging GHA environment.
  • syntax error in radio selection button feedback-radio-buttons
  • timing issues with feedback form interaction

Did you run tests? Share screenshot of results:

N/A

How did you find us? (GitHub, Google search, social media, etc.):

Copy link

vercel bot commented Mar 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bloom-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 24, 2025 10:46pm

@kyleecodes kyleecodes changed the title Run cypress tests/fix user course session Run cypress tests/fix user course session test Mar 24, 2025
Copy link

cypress bot commented Mar 24, 2025

Bloom frontend    Run #1111

Run Properties:  status check passed Passed #1111  •  git commit d44c97b21a: Fix: user-course-session-behaviour test
Project Bloom frontend
Branch Review develop
Run status status check passed Passed #1111
Run duration 08m 47s
Commit git commit d44c97b21a: Fix: user-course-session-behaviour test
Committer Kylee Fields
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 9
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 77
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

@kyleecodes kyleecodes force-pushed the run-cypress-tests/fix-user-course-session branch from 9ddbcca to 7b95215 Compare March 24, 2025 22:41
@kyleecodes kyleecodes self-assigned this Mar 24, 2025
@kyleecodes kyleecodes merged commit d44c97b into develop Mar 24, 2025
49 checks passed
@kyleecodes kyleecodes deleted the run-cypress-tests/fix-user-course-session branch March 24, 2025 23:05
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.

1 participant