Skip to content

fix: make qwik-testing-library agnostic from vitest runner #36

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

Merged
merged 1 commit into from
Mar 29, 2025

Conversation

ianlet
Copy link
Owner

@ianlet ianlet commented Mar 29, 2025

As the experimental mock$ function is currently relying on vi.fn() from vitest to create mock functions, we cannot make it part of qwik-testing-library to provide mocking utilities. We need to extract it as a separate lib.

BREAKING CHANGE: extract qwik-mock from qwik-testing-library as a separate lib

As `mock$` is currently relying on `vi.fn()` from `vitest` to create mock functions,
we cannot make it part of `qwik-testing-library` to provide mocking utilities.
We need to extract it as a separate lib.

BREAKING CHANGE: extract qwik-mock from qwik-testing-library as a separate lib
@ianlet ianlet force-pushed the fix-vitest-dependency branch from 6dda4a6 to c837f33 Compare March 29, 2025 18:15
Copy link

@thejackshelton thejackshelton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ianlet ianlet merged commit 909037f into main Mar 29, 2025
7 checks passed
@ianlet ianlet deleted the fix-vitest-dependency branch March 29, 2025 18:47
Copy link

🎉 This PR is included in version @noma.to/qwik-mock-v1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version @noma.to/qwik-testing-library-v1.3.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ianlet
Copy link
Owner Author

ianlet commented Mar 29, 2025

@all-contributors please add @thejackshelton for review

Copy link
Contributor

@ianlet

I've put up a pull request to add @thejackshelton! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants