-
Notifications
You must be signed in to change notification settings - Fork 7
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 Automated Tests #3
Comments
Adding unit tests gives you better sleep. Also they can be integrated into the CI/CD pipe-line too. |
@wsameer sounds good. Do you have a preferred library in mind? (Ideally, one that works well with React + TypeScript) |
|
@wsameer - sounds good to me. Do you want to take this one? |
I could give JEST testing library a shot. Will PR if successfull. |
@babu1998 - any update here? |
Other than doing some smoke tests via the interactive demo site, I have no other testing method currently for the code in this repo 😅
I'm partial to end-to-end testing tools for a project like this (like cypress for example) because our tests can truly observe the changes in
document.title
and the<link rel="...">
values.I'm looking for the following:
This PR could potentially expand into multiple PRs - setting up tests first locally, then adding it to a CI / CD process (which also doesn't exist yet 😅 ) etc. etc.
The text was updated successfully, but these errors were encountered: