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

Creation of jest testing framework, along with basic integration/unit tests #509

Merged
merged 9 commits into from
May 2, 2023
Merged

Creation of jest testing framework, along with basic integration/unit tests #509

merged 9 commits into from
May 2, 2023

Conversation

stevenewald
Copy link
Collaborator

@stevenewald stevenewald commented Apr 30, 2023

This pull request adds a framework for testing using Jest and Playwright for CRACO. It also includes an example integration test and several basic unit tests to demonstrate how to use the framework. While basic, these tests will fail if basic CRACO functionality fails to execute.

The Jest and Playwright combination provides a powerful testing solution for CRACO, allowing us to test our code from end-to-end and catch any potential issues early on. Additionally, the example unit tests showcase how to write effective unit tests and integrate them into our testing suite.

I have designed the framework around making it painless to add additional tests or modify/run existing ones. For example, setup.js and teardown.js handle individual test installation, package/file management, and compilation, significantly reducing the amount of boilerplate required for additional integration tests.

Copy link
Owner

@dilanx dilanx left a comment

Choose a reason for hiding this comment

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

Nice looks amazing!

@dilanx dilanx merged commit 42c756c into dilanx:main May 2, 2023
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.

None yet

2 participants