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

docs(api): fix the explanation for 'mockRestore' #1441

Merged
merged 1 commit into from Jun 6, 2022
Merged

docs(api): fix the explanation for 'mockRestore' #1441

merged 1 commit into from Jun 6, 2022

Conversation

kazuma1989
Copy link
Contributor

Hi maintainers 👋
I found a tiny typo in the docs. I'd be happy to merge them!

The explanation for 'mockRestore' says "Does what mockRestore does", but this should be "Does what mockReset does" based on the code.

stub.mockRestore = () => {
stub.mockReset()

The explanation says "Does what `mockRestore` does", but this should be "Does what `mockReset` does" based on the code.

https://github.com/vitest-dev/vitest/blob/17dfd39e51b33a02458aaab8bf75d42dd680e72b/packages/vitest/src/integrations/spy.ts#L200-L201
@netlify
Copy link

netlify bot commented Jun 6, 2022

Deploy Preview for vitest-dev ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit f977262
🔍 Latest deploy log https://app.netlify.com/sites/vitest-dev/deploys/629d92914fb971000888ad40
😎 Deploy Preview https://deploy-preview-1441--vitest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@antfu antfu merged commit d25a3f2 into vitest-dev:main Jun 6, 2022
@kazuma1989 kazuma1989 deleted the patch-2 branch June 6, 2022 06:12
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.

None yet

2 participants