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

Add Cypress and basic tests #56

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dgading
Copy link
Contributor

@dgading dgading commented Jan 28, 2025

This PR adds basic tests to make sure pages are loading with the correct data. It is a bit brittle right now since I had to use data from my local and if that changes, some of the tests could break.

Since I still have the pending ticket for updating the interactivity of the goal search, I decided not to include those tests in this PR since all of them would need to be rewritten when the changes are added.

To run the PR:

  1. In a terminal start your site at localhost:3000 (can be changed in the cypress.config file if needed)
  2. In a second terminal navigate to the tests folder and run npm install
  3. After installation, run npm run cy:smoke to run all the tests.
  4. To use the visual tester, you can run npm run cy:open to open the Cypress application.
  5. All tests should be passing.

@dgading dgading requested a review from acabouet January 28, 2025 21:11
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.

1 participant