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

Jest setup for demo website #19

Merged
merged 2 commits into from
Feb 24, 2024
Merged

Conversation

TheTallJerry
Copy link
Collaborator

@TheTallJerry TheTallJerry commented Feb 16, 2024

Overall

  • Jest tests for demo website, 100% coverage for both components.
  • Multi-project Jest setup. This allows Jest to be run the respective configs for different projects, namely:
    • jsdom for demo website
    • node for memory models scripts
    • any other env as needed.
  • Prop types.

Miscellaneous

displayName Jest config helps identify the context of the tests when run:
image

Jest currently outputs the following warnings to stdout:
image
this is (likely a false positive) due to https://togithub.com/jestjs/jest/issues/13576. The option is recognized regardless in test runs (npm run test does not output the coverage table). The warnings should disappear once that issue is fixed.

@TheTallJerry TheTallJerry changed the title Jest setup Jest setup for demo website Feb 23, 2024
Copy link
Owner

@david-yz-liu david-yz-liu left a comment

Choose a reason for hiding this comment

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

Nice job, @TheTallJerry!

@david-yz-liu david-yz-liu merged commit ca6ea52 into david-yz-liu:main Feb 24, 2024
3 checks passed
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