Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

Jest testing code #3

Closed
wants to merge 1 commit into from
Closed

Conversation

Weiiirrr
Copy link

Please review the Jest testing code.

@Weiiirrr Weiiirrr requested review from hzeng2 and jessestorbeck June 12, 2023 23:17
@jkhartshorne
Copy link
Contributor

@hzeng2 @jessestorbeck -- reminding you of this review request. You should be able to test now.

Copy link

@hzeng2 hzeng2 left a comment

Choose a reason for hiding this comment

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

All tests passed, by running yarn add -D jest-environment-jsdom first.

@jkhartshorne
Copy link
Contributor

@Weiiirrr -- you should be able to add jest-environment-jsdom to the repo, following your instructions. Can you please make that update, commit, and push before we merge? Thanks!

@jkhartshorne
Copy link
Contributor

@jkhartshorne
Copy link
Contributor

We're going to leave testing for later, as part of the migration to the newer react-redux, react-router, and to Vite.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants