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

feat(tests): add e2e testing to catch up to Week 13 #237

Merged
merged 12 commits into from
Dec 4, 2023

Conversation

connordoman
Copy link
Contributor

@connordoman connordoman commented Dec 4, 2023

Welcome to PrivacyPal! 👋

Resolves: #229

Description of the change:

Pages and components present in develop as of this morning have tests written and fixes implemented.

This is mostly the following pages:

  • /login
  • /logout
  • /user/dashboard
  • /staff
  • /upload
  • /upload/status

Motivation for the change:

Gotta test everything, baby.

Try it yourself:

You'll need to at least run make setup-db run from app/web/db and then spin up the PrivacyPal server and then from app/web run npm run cypress:open (or npm run test npm run cypress:run to run all Jest & Cypress tests headlessly).

@connordoman connordoman added area/front-end Front-end work test Unit/Integration tests labels Dec 4, 2023
@connordoman connordoman added this to the Term 1 Week 13 ❗️ milestone Dec 4, 2023
@connordoman connordoman self-assigned this Dec 4, 2023
app/web/package.json Outdated Show resolved Hide resolved
@tthvo
Copy link
Contributor

tthvo commented Dec 4, 2023

Rebase first pls.

@tthvo
Copy link
Contributor

tthvo commented Dec 4, 2023

Rebase pls :))

@connordoman connordoman removed the request for review from nganphan123 December 4, 2023 04:45
@connordoman connordoman merged commit 82c38b8 into develop Dec 4, 2023
8 checks passed
@tthvo tthvo deleted the gh-229-e2e-week-13 branch December 4, 2023 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/front-end Front-end work test Unit/Integration tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants