Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

[Issue #1757]: setup e2e tests for search #1

Merged
merged 35 commits into from
May 14, 2024

Conversation

rylew1
Copy link
Collaborator

@rylew1 rylew1 commented May 14, 2024

Summary

Fixes HHS#1757

Time to review: 3 min

Changes proposed

  • Add e2e tests for search page
  • Add e2e test (with FE and API) to its own CI job (ci-frontend-ci.yml)
    • invokes shell script to wait until API is loaded

Context

  • run with npm run test:e2e or npm run test:e2e:ui

  • to run just the search test npm run test:e2e -- tests/e2e/search.spec.ts

Screen.Recording.2024-05-08.at.8.59.02.PM.mov

Copy link

Coverage report for ./frontend

St.
Category Percentage Covered / Total
🟢 Statements 84.14% 870/1034
🟡 Branches 65.01% 223/343
🟡 Functions 75.58% 164/217
🟢 Lines 84.18% 809/961

Test suite run success

164 tests passing in 56 suites.

Report generated by 🧪jest coverage report action from ac2bf35

@coilysiren
Copy link
Collaborator

👀

@rylew1 rylew1 merged commit 6bfd75e into main May 14, 2024
24 checks passed
@rylew1 rylew1 deleted the rylew/1757-e2e-tests-for-search branch May 14, 2024 19:50
acouch pushed a commit that referenced this pull request Sep 18, 2024
Fixes #

- Add e2e tests for search page
- Add e2e test (with FE and API) to its own CI job
(`ci-frontend-ci.yml`)
    - invokes shell script to wait until API is loaded
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Task]: E2E tests for search page
2 participants