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

misc: remove component testing API stubs that were removed in Cypress 11 #30696

Merged
merged 4 commits into from
Dec 3, 2024

Conversation

mschile
Copy link
Contributor

@mschile mschile commented Dec 2, 2024

Additional details

With Cypress 11, we made breaking changes to the component testing API but left in some stubs that threw errors if they were accessed. Since this will be the 3rd major release since those breaking changes, we are removing the code that was throwing those errors.

Steps to test

n/a

How has the user experience changed?

Errors are no longer thrown.

PR Tasks

@mschile mschile requested a review from AtofStryker December 2, 2024 19:49
@mschile mschile changed the title misc: remove CT stub API code that was removed in Cypress 11 misc: remove component testing API stubs that were removed in Cypress 11 Dec 2, 2024
Copy link

cypress bot commented Dec 2, 2024

cypress    Run #58624

Run Properties:  status check passed Passed #58624  •  git commit bbe89e0986: Merge branch 'release/14.0.0' into mschile/remove_ct_11_code
Project cypress
Branch Review mschile/remove_ct_11_code
Run status status check passed Passed #58624
Run duration 17m 52s
Commit git commit bbe89e0986: Merge branch 'release/14.0.0' into mschile/remove_ct_11_code
Committer Matt Schile
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 4
Tests that did not run due to a developer annotating a test with .skip  Pending 1326
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 29379
View all changes introduced in this branch ↗︎
UI Coverage  46.07%
  Untested elements 187  
  Tested elements 164  
Accessibility  92.55%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 891  

@mschile mschile merged commit dcbbe4a into release/14.0.0 Dec 3, 2024
80 of 82 checks passed
@mschile mschile deleted the mschile/remove_ct_11_code branch December 3, 2024 17:44
@jennifer-shehane jennifer-shehane mentioned this pull request Dec 11, 2024
28 tasks
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.

3 participants