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

Restore sinon sandbox #187476

Merged
merged 1 commit into from
Jul 10, 2023
Merged

Restore sinon sandbox #187476

merged 1 commit into from
Jul 10, 2023

Conversation

lramos15
Copy link
Member

Fixes #187471

I couldn't figure out how to get afterEach to work in this context so I called restore at the end of each test using a stub. Similar to how service disposal is handled.

@lramos15 lramos15 self-assigned this Jul 10, 2023
@lramos15 lramos15 enabled auto-merge (squash) July 10, 2023 14:06
@VSCodeTriageBot VSCodeTriageBot added this to the July 2023 milestone Jul 10, 2023
@lramos15 lramos15 merged commit db16e86 into main Jul 10, 2023
7 checks passed
@lramos15 lramos15 deleted the lramos15/delighted-caribou branch July 10, 2023 14:33
@alexr00
Copy link
Member

alexr00 commented Jul 10, 2023

@lramos15 there is also a teardown that you might be able to use.

@github-actions github-actions bot locked and limited conversation to collaborators Aug 24, 2023
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.

Memory leak in tests: sinon.stub() without sinon.restore()
4 participants