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

test: update scaffold CT test to not have chaining that will cause detached dom flake #28960

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

jennifer-shehane
Copy link
Member

Additional details

We had some flake in the scaffold-component-testing.cy.ts file where the .contains command would fail with a 'detached DOM' error essentially, saying the DOM had refreshed before getting to that chain. This rewrites those chains so that they'll query for the element once and ensure they're visible (not all buttons on the page).

Failure
Screenshot 2024-02-16 at 2 53 40 PM

Screenshot 2024-02-16 at 2 58 33 PM

Now
Screenshot 2024-02-16 at 2 59 30 PM

Passing

Steps to test

Run scaffold-component-testing.cy.ts file locally

How has the user experience changed?

N/A

@jennifer-shehane jennifer-shehane self-assigned this Feb 16, 2024
Copy link

cypress bot commented Feb 16, 2024

13 flaky tests on run #54079 ↗︎

0 28947 1341 0 Flakiness 13

Details:

empty commit
Project: cypress Commit: 1e70ccdb6c
Status: Passed Duration: 20:16 💡
Started: Feb 16, 2024 8:08 PM Ended: Feb 16, 2024 8:29 PM
Flakiness  project-setup.cy.ts • 1 flaky test • launchpad-e2e

View Output

Test Artifacts
... > can reconfigure config after CT has been set up Test Replay Screenshots
Flakiness  e2e/origin/config_env.cy.ts • 1 flaky test • 5x-driver-firefox

View Output

Test Artifacts
cy.origin- Cypress.config() > serializable > overwrites different values in secondary if one exists in the primary
    </td>
  </tr></table>
Flakiness  specs_list_latest_runs.cy.ts • 1 flaky test • app-e2e

View Output

Test Artifacts
App/Cloud Integration - Latest runs and Average duration > when no runs are recorded > shows placeholders for all visible specs Test Replay Screenshots
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-chrome

View Output

Test Artifacts
network stubbing > waiting and aliasing > yields the expected interception when two requests are raced Test Replay
Flakiness  commands/querying/querying.cy.js • 1 flaky test • 5x-driver-chrome

View Output

Test Artifacts
... > throws when alias property is `0` Test Replay

The first 5 flaky specs are shown, see all 10 specs in Cypress Cloud.

Review all test suite changes for PR #28960 ↗︎

@mschile mschile changed the title chore: update scaffold CT test to not have chaining that will cause detached dom flake test: update scaffold CT test to not have chaining that will cause detached dom flake Feb 19, 2024
@jennifer-shehane jennifer-shehane merged commit e7d13cf into develop Feb 22, 2024
81 of 83 checks passed
@jennifer-shehane jennifer-shehane deleted the fix-flake-scaffold-ct-0 branch February 22, 2024 15:23
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Feb 22, 2024

Released in 13.6.6.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.6.6, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants