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

feat: setup jest in Stimulus variants #465

Merged
merged 5 commits into from
Aug 14, 2023
Merged

feat: setup jest in Stimulus variants #465

merged 5 commits into from
Aug 14, 2023

Conversation

G-Rath
Copy link
Contributor

@G-Rath G-Rath commented Aug 5, 2023

This should make it easier to write tests directly for Stimulus controllers (and JavaScript in general) which can be nice as they're faster than the current alternative (system tests).

@G-Rath G-Rath force-pushed the frontend-test-variant branch 2 times, most recently from 9d9fca2 to f69989f Compare August 6, 2023 01:02
@G-Rath G-Rath marked this pull request as ready for review August 6, 2023 02:00
Copy link
Contributor

@robotdana robotdana left a comment

Choose a reason for hiding this comment

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

looks good

can we have this also with the installed github actions ci checks? i think bin/ci-run is only for testing when in this repo

@G-Rath
Copy link
Contributor Author

G-Rath commented Aug 7, 2023

can we have this also with the installed github actions ci checks? i think bin/ci-run is only for testing when in this repo

Yup that's tracked by #422 but I want to do it in another PR

@G-Rath G-Rath force-pushed the frontend-test-variant branch 3 times, most recently from f6e00c1 to 1655045 Compare August 10, 2023 21:31
@lukeify lukeify self-requested a review August 11, 2023 00:04
@G-Rath
Copy link
Contributor Author

G-Rath commented Aug 14, 2023

tbh I feel like this should be pulled into either its own variants or into the frontend-base variants, but this is still an improvement 🤷

@G-Rath G-Rath merged commit 963aac6 into main Aug 14, 2023
7 of 22 checks passed
@G-Rath G-Rath deleted the frontend-test-variant branch August 14, 2023 19:05
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.

3 participants