Skip to content

Fixing flakyness in MultipleSelect test#3374

Closed
olemartinorg wants to merge 1 commit intomainfrom
chore/flaky-test-multipleselect
Closed

Fixing flakyness in MultipleSelect test#3374
olemartinorg wants to merge 1 commit intomainfrom
chore/flaky-test-multipleselect

Conversation

@olemartinorg
Copy link
Contributor

Description

This test failed for me locally sometimes, and I suspect it's the same reason for why it failed in #3336 (cypress cloud link). Checking these values earlier makes Cypress wait until the value has been selected before proceeding.

Related Issue(s)

  • closes #{issue number}

Verification/QA

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

@olemartinorg olemartinorg added ignore-for-release Pull requests to be ignored in release notes backport-ignore This PR is a new feature and should not be cherry-picked onto release branches labels May 20, 2025
@olemartinorg olemartinorg requested a review from paal2707 May 21, 2025 08:21
@olemartinorg
Copy link
Contributor Author

Awaiting #3377 to see if that includes these changes instead.

@paal2707
Copy link
Contributor

Closing, added this fix in #3377

@paal2707 paal2707 closed this May 21, 2025
@olemartinorg olemartinorg deleted the chore/flaky-test-multipleselect branch August 27, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-ignore This PR is a new feature and should not be cherry-picked onto release branches ignore-for-release Pull requests to be ignored in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments