Skip to content

fix: use await in test#2640

Merged
cexbrayat merged 1 commit intovuejs:mainfrom
cexbrayat:fix/await
Mar 25, 2025
Merged

fix: use await in test#2640
cexbrayat merged 1 commit intovuejs:mainfrom
cexbrayat:fix/await

Conversation

@cexbrayat
Copy link
Member

Removes the warning:

stderr | tests/trigger.spec.ts > trigger > errors > throws error if options contains a target value
Promise returned by `expect(actual).rejects.toThrowError(expected)` was not awaited. Vitest currently auto-awaits hanging assertions at the end of the test, but this will cause the test to fail in Vitest 3. Please remember to await the assertion.

Removes the warning:
```
stderr | tests/trigger.spec.ts > trigger > errors > throws error if options contains a target value
Promise returned by `expect(actual).rejects.toThrowError(expected)` was not awaited. Vitest currently auto-awaits hanging assertions at the end of the test, but this will cause the test to fail in Vitest 3. Please remember to await the assertion.
```
@netlify
Copy link

netlify bot commented Mar 25, 2025

Deploy Preview for vue-test-utils-docs ready!

Name Link
🔨 Latest commit 413a893
🔍 Latest deploy log https://app.netlify.com/sites/vue-test-utils-docs/deploys/67e2e03f175b070008ce602e
😎 Deploy Preview https://deploy-preview-2640--vue-test-utils-docs.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 configuration.

@cexbrayat cexbrayat merged commit c5482b4 into vuejs:main Mar 25, 2025
7 checks passed
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.

1 participant