This repository has been archived by the owner on Oct 20, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Superseeds: #480
In preparation for #461
Command to test adders locally will slightly change with this:
will now be
You can also use
pnpm test:ui
which will also spin up vitest ui.In the future this will allow us to use snapshots to test the outputs of the adders. As vitest now uses proper multi-threading this may also help with speeding up the testing. Currently most time is spent preparing the
storybook
tests.Here a few crucial notes:
vitest
in@svelte-add/testing-library
. But apparentlyvitest
only detects tests in the package you run it in, and not in other dependencies. That's why I'm just passing down thedescribe
andtest
methods.-t tailwindcss
it will still apply all adders, but only run the tests for the adder specified. I did not find an api that allows us to get the current testcase filter-t
option is just a regex filter you could in theory also test only thekit-js
templates with-t kit-js
ast-manipulation
package.