Skip to content

set up paperwork before intake tests#5270

Merged
OmarZubaidi merged 87 commits intodevelopfrom
omar/optimize-intake-tests-prep-step
Dec 10, 2025
Merged

set up paperwork before intake tests#5270
OmarZubaidi merged 87 commits intodevelopfrom
omar/optimize-intake-tests-prep-step

Conversation

@OmarZubaidi
Copy link
Contributor

@OmarZubaidi OmarZubaidi commented Nov 26, 2025

  • created setup steps before all tests instead of beforeAll in each test to benefit from parallel runners and reusing patients
  • refactored almost all tests to use new setup patients and run in parallel
  • deleted test suites that are now covered by setup steps, especially since booking an appointment using a new patient is the happy path anyway.
    • one setup test has a flag to account for tests that aren't worth being covered in another suite
  • renamed and moved things around to make more sense
  • updated cleanup steps to ensure no more resource littering

locally, i'm seeing a reduction of up to 47% (10:39 -> 5:38).

qa team: please note new way of setting up tests. if you are working on any intake tests, please wait for this pr to merge in first

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 37 out of 37 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@OmarZubaidi OmarZubaidi merged commit cf1bf12 into develop Dec 10, 2025
13 checks passed
@OmarZubaidi OmarZubaidi deleted the omar/optimize-intake-tests-prep-step branch December 10, 2025 17:49
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.

3 participants

Comments