Skip to content

test(calling): added playwright tests for call settings #5012

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.

Sign up for GitHub

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

Merged
vivekv1504 merged 8 commits into from
Jun 12, 2026
Merged

test(calling): added playwright tests for call settings #5012

Show file tree
Hide file tree
Changes from 6 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
fea1944
test(calling): added playwright tests for call settings
vivekv1504 May 20, 2026
1609b55
fix(calling): improve DND button click handling to avoid eventual con…
vivekv1504 May 20, 2026
e68b9a7
test(calling): add comprehensive call settings and voicemail E2E tests
vivekv1504 May 29, 2026
c3e809a
test(calling): enhance call settings tests with new utility functions…
vivekv1504 Jun 5, 2026
5f5ffbb
test(calling): address PR feedback on call-settings Playwright helpers
vivekv1504 Jun 5, 2026
5b1228e
test(calling): address PR review comments on call settings E2E tests
vivekv1504 Jun 10, 2026
b41dfbd
Merge branch 'next' into callingSDK-callsettings
vivekv1504 Jun 10, 2026
f40b29d
Merge branch 'next' into callingSDK-callsettings
vivekv1504 Jun 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions docs/samples/calling.min.js
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/samples/calling.min.js.map
View file
Open in desktop

Large diffs are not rendered by default.

13 changes: 3 additions & 10 deletions docs/samples/calling/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ <h2 class="collapsible">Call initialization</h2>
class="btn-code">getMediaStreams()</button>
</div>
<div class="u-mv">
<button id="bnr-button" type="button" onclick="toggleNoiseReductionEffect()"
<button id="bnr-button" type="button" onclick="toggleNoiseReductionEffect()"
class="btn-code">Enable BNR</button>
</div>
</fieldset>
Expand Down Expand Up @@ -293,14 +293,7 @@ <h3 style="color: #1a73e8; margin-bottom: 0.25rem;">Call Transfer</h3>
</div>
</fieldset>

<!-- calling / incoming -->
<fieldset id="incomingsection" class="hidden">
<legend>Incoming Call</legend>
<div class="u-mv">
<button onclick="answer(this)" id="answer" class="btn--green">Answer</button>
<button onclick="reject(this)" id="reject" class="btn--red">Reject</button>
</div>
</fieldset>

<!-- Call history section -->
<fieldset id="callHistorysection">
<legend>Call History</legend>
Expand Down Expand Up @@ -432,7 +425,7 @@ <h4>Forward calls to voicemail</h4>
<div class="u-mv">
<label>When not answered
<input type="checkbox" id="vmNotAnsweredCb" name="vmNotAnsweredCb" checked="" onchange="changeElementVisibility(document.getElementById('vmNotAnsweredCb'),
document.getElementById('notAnsweredRings'))" disabled="true">
document.getElementById('vmNotAnsweredRings'))" disabled="true">
<input type="string" id="vmNotAnsweredRings" name="vmNotAnsweredRings"
placeholder="Number of rings" disabled="true">
</label>
Expand Down
69 changes: 53 additions & 16 deletions packages/calling/playwright.config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -86,22 +86,24 @@ export default defineConfig({
},

// Single-user registration sets (generated from USER_SETS, depend on OAuth)
...['SET_REGISTRATION_1', 'SET_REGISTRATION_2', 'SET_REGISTRATION_3', 'SET_CONTACTS'].flatMap((key) => [
{
name: `${key} - PROD`,
dependencies: ['OAuth - PROD'],
testDir: './playwright/suites',
testMatch: USER_SETS[key].testSuite,
use: browserOptions[PW_BROWSER],
},
{
name: `${key} - INT`,
dependencies: ['OAuth - INT'],
testDir: './playwright/suites',
testMatch: USER_SETS[key].testSuite,
use: {...browserOptions[PW_BROWSER], testEnv: 'int'} as any,
},
]),
...['SET_REGISTRATION_1', 'SET_REGISTRATION_2', 'SET_REGISTRATION_3', 'SET_CONTACTS'].flatMap(
(key) => [
{
name: `${key} - PROD`,
dependencies: ['OAuth - PROD'],
testDir: './playwright/suites',
testMatch: USER_SETS[key].testSuite,
use: browserOptions[PW_BROWSER],
},
{
name: `${key} - INT`,
dependencies: ['OAuth - INT'],
testDir: './playwright/suites',
testMatch: USER_SETS[key].testSuite,
use: {...browserOptions[PW_BROWSER], testEnv: 'int'} as any,
},
]
),

// 2-user call tests (PROD uses USER_4+USER_5, parallel with registration sets)
{
Expand Down Expand Up @@ -139,5 +141,40 @@ export default defineConfig({
testMatch: USER_SETS.SET_CALL_TRANSFER_CONSULT.testSuite,
use: {...browserOptions[PW_BROWSER], testEnv: 'int'} as any,
},
// Single-user Call Settings tests — shares USER_1/2/3 with registration and transfer
{
name: 'SET_CALL_SETTINGS - PROD',
Comment thread
Shreyas281299 marked this conversation as resolved.
// Depends on SET_CALL so USER_4 (the CF forward destination) is deregistered
// before the CF tests run, preventing forwarded calls from ringing a live
// device in another suite. The dependency is explicit (not just transitive
// via SET_CALL_TRANSFER_CONSULT) so the ordering survives future refactors.
dependencies: [
'OAuth - PROD',
'SET_REGISTRATION_1 - PROD',
'SET_REGISTRATION_2 - PROD',
'SET_REGISTRATION_3 - PROD',
'SET_CALL - PROD',
'SET_CALL_TRANSFER_CONSULT - PROD',
],
testDir: './playwright/suites',
testMatch: USER_SETS.SET_CALL_SETTINGS.testSuite,
use: browserOptions[PW_BROWSER],
},
{
name: 'SET_CALL_SETTINGS - INT',

@eigengravy eigengravy Jun 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Account isolation violation: SET_CALL_SETTINGS shares USER_1/2/3 with registration tests

SET_CALL_SETTINGS uses accounts USER_1, USER_2, and USER_3 but only depends on OAuth - PROD. Since SET_REGISTRATION_1/2/3 also depend only on OAuth and use these same accounts, Playwright can schedule them concurrently — violating the constraint at the top of test-data.ts:

the same Webex account must NEVER be active in two browser sessions simultaneously

This needs dependencies: ['SET_REGISTRATION_1 - PROD', 'SET_REGISTRATION_2 - PROD', 'SET_REGISTRATION_3 - PROD'] at minimum, or should wait until after SET_CALL (which already serializes through the registration projects).

@vivekv1504 vivekv1504 Jun 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

added dependencies on SET_REGISTRATION_1/2/3 and SET_CALL_TRANSFER_CONSULT for both PROD and INT so SET_CALL_SETTINGS won’t run concurrently with other suites using USER_1/2/3.

// See PROD note above: explicit SET_CALL dependency guarantees USER_4 is
// deregistered before the CF tests forward calls to it.
dependencies: [
'OAuth - INT',
'SET_REGISTRATION_1 - INT',
'SET_REGISTRATION_2 - INT',
'SET_REGISTRATION_3 - INT',
'SET_CALL - INT',
'SET_CALL_TRANSFER_CONSULT - INT',
],
testDir: './playwright/suites',
testMatch: USER_SETS.SET_CALL_SETTINGS.testSuite,
use: {...browserOptions[PW_BROWSER], testEnv: 'int'} as any,
},
],
});
44 changes: 41 additions & 3 deletions packages/calling/playwright/constants/selectors.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,12 @@ export const CALLING_SELECTORS = {
DESTINATION_INPUT: '#destination',
MAKE_CALL_BTN: '#create-call-action',
END_CALL_BTN: '#end-call',
/** Prefer over raw #answer — the sample app has a duplicate hidden #answer; this targets the visible incoming strip only. */
INCOMING_ANSWER_BTN: '#incomingsection:not(.hidden) #answer',
ANSWER_BTN: '#answer',
/**
* Answer button in the incoming-call section. Starts disabled; becomes enabled when a
* line:incoming_call event fires. Use .toBeEnabled() to wait for an incoming call,
* .toBeDisabled() to assert no incoming call (e.g. call forwarded / DND active).
*/
INCOMING_ANSWER_BTN: '#incomingsection #answer',
MUTE_BTN: '#mute_button',
HOLD_BTN: '#hold_button',
DTMF_INPUT: '#dtmf_digit',
Expand Down Expand Up @@ -52,4 +55,39 @@ export const CALLING_SELECTORS = {
CALL_QUALITY_METRICS: '#call-quality-metrics',

END_BTN: '#end',

// Call Settings
FETCH_SETTINGS_BTN: '#fetch-setting',
DND_BTN: '#DND-button',
CALL_WAITING_BTN: '#CallWaiting-button',

// Call Forward — scoped to avoid collision with duplicate IDs in the voicemail form
CF_SAVE_BTN: '#callForwardForm #cfButton',
CF_ALWAYS_CB: '#callForwardForm #alwaysCb',
CF_ALWAYS_DEST: '#alwaysDest',
CF_BUSY_CB: '#callForwardForm #busyCb',
CF_BUSY_DEST: '#busyDest',
CF_NO_ANSWER_CB: '#notAnsweredCb',
CF_NO_ANSWER_DEST: '#notAnsweredDest',
CF_NOT_REACHABLE_CB: '#notReachableCb',
CF_NOT_REACHABLE_DEST: '#notReachableDest',
CF_DIRECTORY_NUMBER: '#directoryNumber',
Comment thread
Shreyas281299 marked this conversation as resolved.
CF_ALWAYS_BTN: '#CallForwardAlways-button',
CF_ALWAYS_DATA: '#callforwardalways-data',

// CF No Answer — number of rings field
CF_NO_ANSWER_RINGS: '#notAnsweredRings',

// Voicemail — scoped to #voicemailForm to avoid collision with CF form IDs
VM_ENABLED_CB: '#vmCb',
VM_SAVE_BTN: '#voicemailForm #cfButton',
VM_SEND_ALL_CB: '#voicemailForm #alwaysCb',
VM_SEND_BUSY_CB: '#voicemailForm #busyCb',
VM_UNANSWERED_CB: '#voicemailForm #vmNotAnsweredCb',
VM_UNANSWERED_RINGS: '#vmNotAnsweredRings',
VM_MWI_CB: '#voicemailForm #notifCb',
VM_NOTIF_EMAIL_CB: '#voicemailForm #notifEmailCb',
VM_NOTIF_EMAIL_ID: '#notifEmailId',
VM_EMAIL_COPY_CB: '#voicemailForm #vmEmailCb',
VM_EMAIL_COPY_ID: '#vmEmailId',
};
6 changes: 6 additions & 0 deletions packages/calling/playwright/suites/set-call-settings.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
import {callSettingsTests, callSettingsCallTests} from '../test-groups/call-settings';

// Account roles resolved from testInfo.project.name → USER_SETS.
// SET_CALL_SETTINGS: accounts[0] = USER_3 (settings owner), accounts[1] = USER_2 (caller).
callSettingsTests();
callSettingsCallTests();
10 changes: 10 additions & 0 deletions packages/calling/playwright/test-data.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,16 @@ export const USER_SETS: Record<string, UserSet> = {
accounts: ['USER_6'],
testSuite: 'contacts.spec.ts',
},

// Call Settings E2E tests:
// accounts[0] = USER_3 — settings owner / callee
// accounts[1] = USER_2 — primary caller (also used for busy-state first call)
// accounts[2] = USER_1 — second caller (places call while USER_3 is already busy)
SET_CALL_SETTINGS: {
name: 'SET_CALL_SETTINGS',
accounts: ['USER_3', 'USER_2', 'USER_1'],
testSuite: 'set-call-settings.spec.ts',

@eigengravy eigengravy Jun 4, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

CRITICAL: Missing closing }, — this file won't parse

The SET_CALL_SETTINGS object is missing its closing brace+comma. Playwright test discovery will fail immediately with a syntax error.

  SET_CALL_SETTINGS: {
    name: 'SET_CALL_SETTINGS',
    accounts: ['USER_3', 'USER_2', 'USER_1'],
    testSuite: 'set-call-settings.spec.ts',
+ },
  // Single-user Contacts supplementary service E2E tests
  SET_CONTACTS: {

@vivekv1504 vivekv1504 Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done chnages applied ,rebase the branch and appiled the changes

},
};

/**
Expand Down
Loading
Loading