[daily-playwright] Add tests for Appointments page - #16811
Closed
github-actions[bot] wants to merge 1 commit into
Closed
github-actions[bot] wants to merge 1 commit into
github-actions[bot] wants to merge 1 commit into
Conversation
- 10 comprehensive test cases covering Appointments page functionality - Tests cover: page load, view switching, filtering, responsive design - Uses established patterns from existing facility/service tests - Graceful fallbacks for optional UI elements - Follows role-based selector best practices Closes: #16803 Component: Appointments/AppointmentsPage.tsx
Contributor
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Deploying care-preview with
|
| Latest commit: |
adfff4f
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://35b20e59.care-preview-a7w.pages.dev |
| Branch Preview URL: | https://daily-playwright-2026-09-14.care-preview-a7w.pages.dev |
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
| await page.waitForTimeout(500); | ||
|
|
||
| // Verify we're in table view (look for table element) | ||
| const table = page.locator("table"); |
| await page.waitForTimeout(1000); | ||
|
|
||
| // Get initial state of page title/content | ||
| const initialContent = await page.locator("main, [role='main']").innerHTML(); |
Contributor
|
Closing as part of resetting the daily Playwright workflow: its open PRs overlapped heavily and were written under older authoring rules. The reworked workflow will pick this area up again. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://entire.io/gh/ohcnetwork/care_fe/trails/3
Overview
Added comprehensive E2E test coverage for the Appointments page, which was previously completely untested (0 tests).
What's being tested
Test approach
facilityAdmin.jsonauth state)getByRole,getByLabel,getByText) with graceful fallbacksisVisible()checks with timeouts; tests don't assume specific contentRun tests locally
Coverage impact
tests/appointments/appointments.spec.ts(181 lines)Related tracking
Closes #16803(Daily Playwright Test Coverage Tracker)Next priorities for future runs:
Generated by Daily Playwright Test Improver