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: add test setup based on jest and testing-library #12

Merged
merged 4 commits into from
May 24, 2020

Conversation

marcosvega91
Copy link
Member

I think that I have added what we need for testing.

In setupTests.js I have assigned to window object the export from testing library.

@smeijer
Copy link
Member

smeijer commented May 23, 2020

Thanks. Looking great!

One dependency should be moved though.

Also what do you think? Should we add one single "app doesn't throw on render" test just to verify that the setup is working?

@smeijer
Copy link
Member

smeijer commented May 23, 2020

And last one, do we need to add https://github.com/testing-library/user-event?

to make App.js work I added a mock for createRange because it caused an exception with CodeMirror. I also add a mock for images
@marcosvega91
Copy link
Member Author

I have added the other things that we need. I also create a test, as you suggest, for App.js :)

@smeijer
Copy link
Member

smeijer commented May 24, 2020

Awesome! Thanks.

@smeijer smeijer changed the title feat: add configuration for tests feat: add test setup based on jest and testing-library May 24, 2020
@smeijer smeijer merged commit 022743d into testing-library:master May 24, 2020
@marcosvega91 marcosvega91 deleted the feature/test branch May 24, 2020 10:08
@smeijer smeijer mentioned this pull request Jun 15, 2020
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.

2 participants