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

Implement first set of simple UI tests and run in CI 🧪 #571

Closed
JamieSlome opened this issue May 21, 2024 · 5 comments · Fixed by #581
Closed

Implement first set of simple UI tests and run in CI 🧪 #571

JamieSlome opened this issue May 21, 2024 · 5 comments · Fixed by #581
Assignees
Labels
enhancement New feature or request

Comments

@JamieSlome
Copy link
Member

Is your feature request related to a problem? Please describe.

Currently, we do not have any UI tests that run in our CI pipeline. This results in issues like #570, where we are unsure whether a pull request breaks the user interface.

Describe the solution you'd like

  1. Choose a frontend/UI testing library - Jest appears to be the most common for testing React
  2. Write some simple tests to get the ball rolling; i.e. each page of the UI site renders as expected (snapshot testing is quite good for this)
  3. Run these tests in the existing CI pipeline (editable in .github/workflows)
@JamieSlome JamieSlome added the enhancement New feature or request label May 21, 2024
@JamieSlome
Copy link
Member Author

@tt-gideonaryeetey and @divinetettey - as mentioned, does this issue interest you?

@JamieSlome JamieSlome changed the title Choose a frontend testing library, implement first set of UI tests and run in CI Implement first set of simple UI tests and run in CI May 21, 2024
@JamieSlome JamieSlome changed the title Implement first set of simple UI tests and run in CI Implement first set of simple UI tests and run in CI 🧪 May 21, 2024
@divinetettey
Copy link
Contributor

Yes, we can take a look, my colleague from turntabl; @bill-n would be picking up this issue

@JamieSlome
Copy link
Member Author

@divinetettey - sure, and more than welcome @bill-n 👍

@bill-n - are you able to drop a comment on this issue so I can assign the ticket to you?

@bill-n
Copy link
Contributor

bill-n commented May 22, 2024

@JamieSlome I am happy to take this task. Is it okay to use Cypress?

@JamieSlome
Copy link
Member Author

@bill-n - absolutely; let's go with Cypress 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants