Skip to content

Conversation

@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Apr 18, 2025

Additional details

This signature was removed from sinon in v3. This removes the signature that we deprecated long ago in cy.stub.

This did require some updates in places where we were still using it, so could break some users.

Steps to test

Test pass. Using cy.stub(obj, method, fn) should no longer work properly.

How has the user experience changed?

There would be TypeScript errors for TS users using this signature.

It looks like the stub will just silently not apply with the updates. Here's an example of using the old signature - the assertion now fails since the stub wasn't properly applied.

Screenshot 2025-04-21 at 11 59 38 AM

PR Tasks

@jennifer-shehane jennifer-shehane changed the base branch from develop to release/15.0.0 April 18, 2025 18:13
@jennifer-shehane jennifer-shehane self-assigned this Apr 18, 2025
@cypress
Copy link

cypress bot commented Apr 18, 2025

cypress    Run #62109

Run Properties:  status check passed Passed #62109  •  git commit c345b2a0e1: Merge branch 'release/15.0.0' into remove-deprecated-cy-stub-sig
Project cypress
Branch Review remove-deprecated-cy-stub-sig
Run status status check passed Passed #62109
Run duration 18m 56s
Commit git commit c345b2a0e1: Merge branch 'release/15.0.0' into remove-deprecated-cy-stub-sig
Committer Jennifer Shehane
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 11
Tests that did not run due to a developer annotating a test with .skip  Pending 1099
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 26561
View all changes introduced in this branch ↗︎
UI Coverage  45.92%
  Untested elements 187  
  Tested elements 163  
Accessibility  92.7%
  Failed rules  3 critical   8 serious   2 moderate   2 minor
  Failed elements 886  

@jennifer-shehane jennifer-shehane requested review from AtofStryker and removed request for AtofStryker April 21, 2025 16:01
@mschile mschile requested a review from cacieprins April 22, 2025 14:41
@jennifer-shehane jennifer-shehane merged commit f58aabd into release/15.0.0 May 2, 2025
1 of 4 checks passed
@jennifer-shehane jennifer-shehane deleted the remove-deprecated-cy-stub-sig branch May 2, 2025 17:43
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Aug 20, 2025

Released in 15.0.0.

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

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Aug 20, 2025
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.

Remove deprecated cy.stub signature

3 participants