-
Notifications
You must be signed in to change notification settings - Fork 3
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
Feature/front end tests #108
Conversation
This pull request is being automatically deployed with Vercel (learn more). eligibility-estimator – ./🔍 Inspect: https://vercel.com/eligibility-estimator/eligibility-estimator/5n3iDJ9D3JmDEAVPLRVnyRJ8gKFk |
3cbbd6c
to
261fa97
Compare
1e09a69
to
eb0e19c
Compare
There seems to be an issue where Edit: Our version of node on this project does not have replaceAll(), gonna try and use a shim in test Edit x2: That did it! |
eb0e19c
to
2863479
Compare
2863479
to
7662a6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine to me!
This is the first pass at getting better front-end code coverage, it's getting a little large so I think I'll get this half reviewed and then setup the next set of tests in a separate PR.