Skip to content

Align scheduled-call E2E with the shared date/time picker - #76

Merged
dnlbui merged 1 commit into
mainfrom
agent/align-scheduled-call-picker-main
Aug 5, 2026
Merged

Align scheduled-call E2E with the shared date/time picker#76
dnlbui merged 1 commit into
mainfrom
agent/align-scheduled-call-picker-main

Conversation

@dnlbui

@dnlbui dnlbui commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

What Changed

This PR aligns the scheduled video-call E2E flow with the shared date/time picker used by the current dev client.

  • Waits for dateTimePickerModal and verifies its contextual title is Schedule Call.
  • Uses the shared date, hour, minute, AM/PM, and confirmation controls.
  • Matches the current minute-precision success toast and reuses that expected time for downstream message and call-list assertions.

Fixed Flow

  1. Set up the users and open the call scheduling choices.
  2. Choose Schedule.
  3. Wait for the shared date/time picker, verify its call-specific title, fill the shared controls, and submit.
  4. Verify the success toast, sender and recipient scheduled messages, and both users' call lists.

Why

The test still targeted the previous call-specific picker IDs even though the dev client now opens a shared date/time picker. The dialog was open, but the obsolete locator reported it as missing. The current client also displays scheduled times to the minute rather than including seconds.

This replaces #75, which was merged into an already-merged parent branch and therefore never reached main.

Validation

  • CI=true npx playwright test tests/videoCall.e2e.test.js --project=chromium --grep "schedule call: scheduled message appears and call listed for both users" --workers=1 --retries=0 --reporter=line
  • node --check tests/videoCall.e2e.test.js
  • git diff --check origin/main...HEAD

Closes #74

@dnlbui
dnlbui marked this pull request as ready for review August 5, 2026 21:50
@dnlbui
dnlbui merged commit 2fc3036 into main Aug 5, 2026
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.

Align scheduled video-call E2E with the shared date/time picker

1 participant