Skip to content

test(e2e): add e2e #20

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 9 commits into from
Feb 1, 2025
Merged

test(e2e): add e2e #20

merged 9 commits into from
Feb 1, 2025

Conversation

mikededo
Copy link
Owner

Adds e2e tests for the plugin. At the same time, it also fixes a couple of
issues and bugs that have been found while implementing the tests.

@mikededo mikededo self-assigned this Jan 11, 2025
Copy link

changeset-bot bot commented Jan 11, 2025

🦋 Changeset detected

Latest commit: 3dc75ad

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-plugin-svelte-tailwindcss Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Instead of having e2e test per rule, (which did not make much sense as we
already have unit tests), we now simulate a SvelteKit app structure,
with a `tailwind.config.ts` file and other "real-world" assets.
We then still work with snapshots, as it's the easiest way to see the
issues in case a rule has been changed and there's an error due to that
change. It also allows to test a mix of rules in the same test.
@mikededo mikededo marked this pull request as ready for review February 1, 2025 07:55
@mikededo mikededo merged commit 722eb4a into main Feb 1, 2025
1 check passed
@mikededo mikededo deleted the chore/add-e2e branch February 1, 2025 07:56
@github-actions github-actions bot mentioned this pull request Apr 28, 2025
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