-
Notifications
You must be signed in to change notification settings - Fork 26
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
Add test configuration using Jest and RTL #14
Comments
@LucasAndrad I like RTL. But I'm not sure if we can somehow mock browser API. I need to perform some research.) |
Good point, I didn't thought about it. I'll comment here if I find a solution for that. |
Hey @ElForastero, there is a package to mock browser api, and works just fine: I'm facing some problems with @testing/preact. Is there an specific reason to use preact instead react? Just asking because I was able to test everything on my project after added react on it. Also if you want to take a look, here is what I did so far: |
@LucasAndrad Great work 👍 I think we can switch the defaults from Preact to React. Using Preact can really lead to some strange problems, and would be better to leave this at the discretion of the user. |
I really liked this boilerplate, so I would to add some contribution if you agree. Here is my idea about tests:
Having a test configuration will allow us developers to build more consistent and "security" extensions with this boilerplate.
package.json
and also the scripts to run testsI'm already working on this one, just need to fix a little problem with the coverage report, but let me know what you think about this please. Thank you.
The text was updated successfully, but these errors were encountered: