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

chore: unskip tests #25676

Merged
merged 2 commits into from
Feb 2, 2023
Merged

chore: unskip tests #25676

merged 2 commits into from
Feb 2, 2023

Conversation

lmiller1990
Copy link
Contributor

@lmiller1990 lmiller1990 commented Feb 1, 2023

Unskipped some tests that were skipped.

@cypress
Copy link

cypress bot commented Feb 1, 2023

5 flaky tests on run #43720 ↗︎

0 4284 879 0 Flakiness 5

Details:

Apply suggestions from code review
Project: cypress Commit: 78b1d69029
Status: Passed Duration: 15:15 💡
Started: Feb 2, 2023 3:31 AM Ended: Feb 2, 2023 3:47 AM
Flakiness  commands/net_stubbing.cy.ts • 2 flaky tests • 5x-driver-webkit

View Output Video

Test
... > with `times` > only uses each handler N times
... > stops waiting when an xhr request is canceled
Flakiness  cypress/cypress.cy.js • 3 flaky tests • 5x-driver-webkit

View Output Video

Test
... > correctly returns currentRetry
... > correctly returns currentRetry
... > correctly returns currentRetry

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@lmiller1990 lmiller1990 marked this pull request as ready for review February 2, 2023 01:09
@@ -259,8 +259,6 @@ describe('Launchpad: Setup Project', () => {
cy.findByRole('button', { name: 'Continue' })
.should('not.have.disabled')
.click()

verifyChooseABrowserPage()
Copy link
Contributor Author

Choose a reason for hiding this comment

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

covered in many other places - covering it here would require us to add dependencies to the pristine project, which would slow down the run for no additional coverage

@lmiller1990 lmiller1990 requested review from flotwig and a team February 2, 2023 01:10
Co-authored-by: Zach Bloomquist <git@chary.us>
Copy link
Contributor

@ZachJW34 ZachJW34 left a comment

Choose a reason for hiding this comment

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

🚀

@flotwig flotwig changed the title chore: try unskip tests chore: unskip tests Feb 2, 2023
@flotwig flotwig merged commit ac99a5c into develop Feb 2, 2023
@flotwig flotwig deleted the lmiller/fixing-skipped branch February 2, 2023 17:12
tgriesser added a commit that referenced this pull request Feb 8, 2023
* develop: (28 commits)
  chore: update changelog validation example (#25742)
  fix: Improve error handling around calls to `this.next` in middleware (#25702)
  chore: debug page tooltip distance and artifact border (#25727)
  fix: update newProject ref when switching between organizations in SelectCloudProjectModal (#25730)
  misc: Add max widths to debug page message states (#25725)
  chore: export types (#25714)
  chore: release @cypress/webpack-preprocessor-v5.16.3
  chore: release @cypress/vue-v5.0.4
  chore: release @cypress/grep-v3.1.4
  chore: Fix flaky test (#25726)
  dependency(deps): update dependency debug to ^4.3.4 🌟 (#25699)
  feat: openInIDE for failed debug spec (#25691)
  test: fix flaky CT test by relying on query (#25706)
  test: fix flaky migration test (#25672)
  misc: style change for responsiveness (#25687)
  misc: set min widths for icons (#25684)
  chore(deps): update dependency markdown-it to v11.0.1 🌟 (#25698)
  chore: Fix flaky origin .wait() test (#25693)
  chore: unskip tests (#25676)
  chore: release @cypress/webpack-preprocessor-v5.16.2
  ...
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.

4 participants