dimsum-e2e-tests 3.70.0-next.62 → 3.70.0-next.69

package/ds-apppicker/DSAppPickerCO.js

@@ -18,6 +18,13 @@ export default class DSAppPickerCO extends PageObject {
 
   static reflowURL = new Urlbuilder(`${PATH_E2E_APP_PICKER}`, 'reflow-test');
 
+  // PUI-18647 — the two custom-trigger APIs side by side. Storybook kebab-cases the
+  // `PUI18647CustomTriggers...` export with a hyphen after the prefix, hence `pui-18647`.
+  static customTriggerFocusURL = new Urlbuilder(
+    `${PATH_E2E_APP_PICKER}`,
+    'pui-18647-custom-triggers-focus-parity-with-deprecated',
+  );
+
   // Snapshots
   static snapshotPath(example = 'basic') {
     return PageObject.getSnapshotPathBuilder('AppPicker', example);
@@ -126,8 +133,28 @@ export default class DSAppPickerCO extends PageObject {
     return $$('[data-testid="ds-chip-label"]')[index];
   }
 
+  // Selection is exposed as `aria-current="true"`, not `aria-selected`. The chips are `role="button"`,
+  // which does not support `aria-selected` — PUI-17213/PUI-17860 (#8260) swapped it for `aria-current`
+  // plus the "X of Y" position announcement, signed off with the A11y SME.
+  // The attribute is omitted (not `false`) when unselected, so `getAttribute` resolves to null: compare
+  // the value instead of calling a string method on it, so an unselected chip returns false rather than
+  // throwing.
   static async isSelected(element) {
-    return (await element.getAttribute('aria-selected')).includes('true');
+    return (await element.getAttribute('aria-current')) === 'true';
+  }
+
+  // PUI-18647 selectors
+
+  // The supported API: DSAppPicker hands the trigger renderer `innerRef` plus the internal handlers,
+  // so it can drive the first-app/selected-app focus logic on open.
+  static async getSupportedCustomTrigger() {
+    return $('[data-testid="pui18647-trigger-component"]');
+  }
+
+  // The deprecated API: `renderTrigger` receives only `ref`, so it cannot reach those handlers. It
+  // keeps its pre-existing behaviour by design — this is not a bug, it is the documented delta.
+  static async getDeprecatedCustomTrigger() {
+    return $('[data-testid="pui18647-render-trigger"]');
   }
 
   // Slots

package/ds-apppicker/basic/DSAppPicker.basic.keyboard-nav.func.spec.js

@@ -1,11 +1,7 @@
 import { Key } from 'webdriverio';
 import DSAppPickerCO from '../DSAppPickerCO';
 
-if (
-  !browser.capabilities['ice:options'].isPhone &&
-  !browser.capabilities['ice:options'].isTablet &&
-  browser.capabilities.browserName !== 'Safari'
-) {
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
   describe('PUI-8030 - AppPicker:: Basic - keyboard nav func', () => {
     before('loading page', async () => {
       const errorOnGo = await DSAppPickerCO.basicURL.go();
@@ -34,12 +30,27 @@ if (
       const appPickerChip = await DSAppPickerCO.getChipByIndex(1);
       await expect(appPickerChip).toBeFocused();
     });
-    it('05: should close the menu - Escape', async () => {
+    // PUI-17950 made Escape the dismissal mechanism for a chip's expanded label, satisfying the
+    // "Dismissible" clause of WCAG 2.1 AA 1.4.13. Escape on an open panel is therefore two-stage
+    // whenever the focused chip's label truncates: the first press dismisses the expansion, only
+    // the second reaches the dialog. Chip 1 here truncates, so both presses are needed. The
+    // dismissal is conditional, not blanket — on a chip whose label fits, Escape still closes the
+    // panel in a single press, so this is not a rule that can be applied by rote to every spec.
+    it('05: should dismiss the expanded chip label and keep the menu open - Escape', async () => {
+      const appPickerChip = await DSAppPickerCO.getChipByIndex(1);
+      expect(await appPickerChip.getAttribute('data-expanded')).toEqual('true');
+      await browser.keys(Key.Escape);
+      const appPicker = await DSAppPickerCO.getAppPickerRoot();
+      await expect(appPicker).toBeDisplayedInViewport();
+      await expect(appPickerChip).toBeFocused();
+      expect(await appPickerChip.getAttribute('data-expanded')).not.toEqual('true');
+    });
+    it('06: should close the menu - Escape', async () => {
       const openAppPickerBtn = await DSAppPickerCO.getOpenAppPicker();
       await browser.keys(Key.Escape);
       await expect(openAppPickerBtn).toBeFocused();
     });
-    it('06: should reopen the menu and focus the first enabled chip - Space', async () => {
+    it('07: should reopen the menu and focus the first enabled chip - Space', async () => {
       await browser.keys(Key.Space);
       const appPicker = await DSAppPickerCO.getAppPickerRoot();
       const appPickerChip = await DSAppPickerCO.getChipByIndex(1);

package/ds-apppicker/closeAfterSelection/DSAppPicker.closeAfterSelection.func.spec.js

@@ -20,7 +20,7 @@ if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:o
       await expect(appPicker).not.toBeDisplayedInViewport();
       await expect(triggerBtn).toBeFocused();
     });
-    it('03: should open appPicker and have the third chip still focused', async () => {
+    it('03: should reopen appPicker with the selected app still marked as current', async () => {
       const openAppPickerBtn = await DSAppPickerCO.getOpenAppPicker();
       await openAppPickerBtn.click();
       const reportChip = await DSAppPickerCO.getChipByIndex(6);

package/ds-apppicker/custom-trigger-focus/DSAppPicker.custom-trigger-focus.func.spec.js

@@ -0,0 +1,82 @@
+import { Key } from 'webdriverio';
+import DSAppPickerCO from '../DSAppPickerCO';
+
+// Tests for the change introduced in PUI-18647 (GlobalHeader -> DSAppPicker contract passthrough).
+// Widening the hand-off meant a consumer can finally supply a custom trigger, which exposed two
+// different focus stories that now need pinning:
+//
+//  - `TriggerComponent` (supported) receives `innerRef` AND the internal open handlers, so a keyboard
+//    open can run the selected-app focus logic and land on the selected chip.
+//  - `renderTrigger` (deprecated) receives only `ref`, so it cannot reach those handlers and falls
+//    back to focusing the panel on every path. Unchanged by design.
+//
+// The mouse path is the RE-BASELINE: a TriggerComponent picker opened with the mouse used to land on
+// the FIRST app and now lands on the panel, matching the built-in trigger. Kept on the full browser
+// matrix because pointer-driven focus is exactly where engines disagree.
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  describe('PUI-19014 - DSAppPicker:: custom trigger open focus destinations - Func', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSAppPickerCO.customTriggerFocusURL.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should focus the panel when the supported trigger is opened with the mouse', async () => {
+      const trigger = await DSAppPickerCO.getSupportedCustomTrigger();
+      await trigger.waitForDisplayed();
+      await trigger.click();
+
+      const panel = await DSAppPickerCO.getAppPickerRoot();
+      await panel.waitForDisplayed();
+      await expect(panel).toBeFocused();
+    });
+
+    it('02: should focus the selected app when the supported trigger is opened with Enter', async () => {
+      // Escape closes the panel opened in 01 and returns focus to the trigger, which is also the
+      // starting state this case needs.
+      await browser.keys(Key.Escape);
+      const trigger = await DSAppPickerCO.getSupportedCustomTrigger();
+      await expect(trigger).toBeFocused();
+
+      await browser.keys(Key.Enter);
+      await (await DSAppPickerCO.getAppPickerRoot()).waitForDisplayed();
+
+      // 'Reports' is deliberately the third app, so "landed on the selected app" cannot be mistaken
+      // for "landed on the first app".
+      const selectedChip = await DSAppPickerCO.getChipByLabel('Reports');
+      await expect(selectedChip).toBeFocused();
+      await expect(await DSAppPickerCO.isSelected(selectedChip)).toBe(true);
+    });
+
+    it('03: should focus the panel when the deprecated trigger is opened with the mouse', async () => {
+      await browser.keys(Key.Escape);
+
+      const trigger = await DSAppPickerCO.getDeprecatedCustomTrigger();
+      await trigger.waitForDisplayed();
+      await trigger.click();
+
+      const panel = await DSAppPickerCO.getAppPickerRoot();
+      await panel.waitForDisplayed();
+      await expect(panel).toBeFocused();
+    });
+
+    it('04: should focus the panel when the deprecated trigger is opened with Enter', async () => {
+      // The panel 03 left open is controlled from the story and has no Escape handler, so it is
+      // dismissed by clicking its trigger again. Where a dismissal click leaves focus is NOT what
+      // this case is about and is not consistent across engines — clicking a <button> does not focus
+      // it on macOS, and the panel unmounting mid-click can blur. So the keyboard starting state is
+      // established explicitly instead of being inherited from the click.
+      const trigger = await DSAppPickerCO.getDeprecatedCustomTrigger();
+      await trigger.click();
+      await (await DSAppPickerCO.getAppPickerRoot()).waitForDisplayed({ reverse: true });
+
+      await browser.execute((el) => el.focus(), trigger);
+      await expect(trigger).toBeFocused();
+
+      await browser.keys(Key.Enter);
+
+      const panel = await DSAppPickerCO.getAppPickerRoot();
+      await panel.waitForDisplayed();
+      await expect(panel).toBeFocused();
+    });
+  });
+}

package/ds-apppicker/label-expansion-dismissal/DSAppPicker.label-expansion-dismissal.visual.spec.js

@@ -0,0 +1,108 @@
+import { Key } from 'webdriverio';
+import DSAppPickerCO from '../DSAppPickerCO';
+import { mouseOver } from '../../helpers';
+
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  const PARKED_X = 10;
+  const PARKED_Y = 10;
+
+  // Enabled and long enough to truncate. Chip 1 of the apps list is used rather than the first one,
+  // which is natively disabled and tracked separately under PUI-18981.
+  const TRUNCATING_CHIP = 'Pipeline but with a really long label that will truncate';
+
+  const ARIA_DISABLED_CHIP = 'Dashboard but with a really long label that will truncate';
+
+  describe('PUI-18986 - DSAppPicker:: Escape dismisses the expanded chip label - Visual', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSAppPickerCO.basicURL.go();
+      if (errorOnGo) throw errorOnGo;
+      await browser.maximizeWindow();
+      await DSAppPickerCO.openPanel();
+    });
+
+    it('01: truncated chip expands its label on hover', async () => {
+      await mouseOver(await DSAppPickerCO.getChipByLabel(TRUNCATING_CHIP));
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-hover-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('02: Escape collapses the hover-expanded label and keeps the panel open', async () => {
+      await browser.keys(Key.Escape);
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-hover-collapsed'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('03: label expands again once the pointer leaves and returns', async () => {
+      await mouseOver(null, PARKED_X, PARKED_Y);
+      await mouseOver(await DSAppPickerCO.getChipByLabel(TRUNCATING_CHIP));
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-rehover-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('04: truncated chip expands its label on focus alone', async () => {
+      await mouseOver(null, PARKED_X, PARKED_Y);
+      await browser.keys(Key.Tab);
+      await expect(await DSAppPickerCO.getChipByLabel(TRUNCATING_CHIP)).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-focus-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('05: Escape collapses the focus-expanded label without moving focus', async () => {
+      await browser.keys(Key.Escape);
+      await expect(await DSAppPickerCO.getChipByLabel(TRUNCATING_CHIP)).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-focus-collapsed'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('06: a further Escape closes the panel once the label is collapsed', async () => {
+      await browser.keys(Key.Escape);
+      await (await DSAppPickerCO.getAppPickerRoot()).waitForDisplayed({ reverse: true });
+      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('esc-dismiss-panel-closed'));
+      await expect(snapshot).toEqual(0);
+    });
+  });
+
+  describe('PUI-18987 - DSAppPicker:: Escape dismisses the expanded aria-disabled chip label - Visual', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSAppPickerCO.applyAriaDisabledURL.go();
+      if (errorOnGo) throw errorOnGo;
+      await browser.maximizeWindow();
+      await DSAppPickerCO.openPanel();
+    });
+
+    it('01: aria-disabled truncated chip expands its label on hover', async () => {
+      await mouseOver(await DSAppPickerCO.getChipByLabel(ARIA_DISABLED_CHIP));
+      const snapshot = await browser.percyCheckScreenshot(
+        DSAppPickerCO.snapshotPath('aria-disabled-esc-dismiss-hover-expanded'),
+      );
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('02: Escape collapses the hover-expanded label and keeps the panel open', async () => {
+      await browser.keys(Key.Escape);
+      const snapshot = await browser.percyCheckScreenshot(
+        DSAppPickerCO.snapshotPath('aria-disabled-esc-dismiss-hover-collapsed'),
+      );
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('03: aria-disabled truncated chip expands its label on keyboard focus', async () => {
+      await mouseOver(null, PARKED_X, PARKED_Y);
+      await browser.keys(Key.Tab);
+      await expect(await DSAppPickerCO.getChipByLabel(ARIA_DISABLED_CHIP)).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(
+        DSAppPickerCO.snapshotPath('aria-disabled-esc-dismiss-focus-expanded'),
+      );
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('04: Escape collapses the focus-expanded label without moving focus', async () => {
+      await browser.keys(Key.Escape);
+      await expect(await DSAppPickerCO.getChipByLabel(ARIA_DISABLED_CHIP)).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(
+        DSAppPickerCO.snapshotPath('aria-disabled-esc-dismiss-focus-collapsed'),
+      );
+      await expect(snapshot).toEqual(0);
+    });
+  });
+}

package/ds-apppicker/reflow/DSAppPicker.reflow.visual.spec.js

@@ -1,18 +1,19 @@
 /* eslint-disable wdio/no-pause */
 import DSAppPickerCO from '../DSAppPickerCO';
+import { setTextSize } from '../../helpers';
 
 if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
   describe('PUI-17867 - AppPicker:: Reflow test -visual', () => {
     before('loading page', async () => {
       const errorOnGo = await DSAppPickerCO.reflowURL.go();
       if (errorOnGo) throw errorOnGo;
-      await browser.maximizeWindow();
+      await setTextSize('26');
     });
 
     it('01: should show appPicker open', async () => {
       const openAppPickerBtn = await DSAppPickerCO.getOpenAppPicker();
       await openAppPickerBtn.click();
-      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('reflow-appPicker-open'));
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
 
@@ -20,16 +21,14 @@ if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:o
       const secondPipelineChip = await DSAppPickerCO.getChipByLabelAndIndex('Pipeline', 1);
       await secondPipelineChip.scrollIntoView();
       await browser.pause(300);
-      const snapshot = await browser.percyCheckScreenshot(DSAppPickerCO.snapshotPath('reflow-appPicker-scrolled'));
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
 
     it('03: should select the 2nd Pipeline chip', async () => {
       const secondPipelineChip = await DSAppPickerCO.getChipByLabelAndIndex('Pipeline', 1);
       await secondPipelineChip.click();
-      const snapshot = await browser.percyCheckScreenshot(
-        DSAppPickerCO.snapshotPath('reflow-appPicker-pipeline-selected'),
-      );
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
   });

package/ds-banner/DSBanner.text-resize.visual.spec.js

@@ -1,14 +1,17 @@
 import DSBannerCO from './DSBannerCO';
+import { setTextSize } from '../helpers';
 
 if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
   describe('PUI-17535 - DSBanner - Text Resize In Different Layouts - Visual Test', () => {
     before('loading page', async () => {
       const errorOnGo = await DSBannerCO.textResizeInDifferentLayoutsTest.go();
       if (errorOnGo) throw errorOnGo;
+      await setTextSize('26');
     });
     it('01: should display text resize in different layouts properly', async () => {
-      await browser.eyesOpen();
-      const snapshot = await browser.eyesCheckSnapshot(DSBannerCO.snapshotPath('text-resize-in-different-layouts'));
+      const banner = await DSBannerCO.getBannerByIndex();
+      await banner.waitForDisplayed();
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
   });

package/ds-banner/DSBannerCO.js

@@ -62,6 +62,10 @@ export default class DSBannerCO extends PageObject {
     return $('[data-testid="ds-banner"');
   }
 
+  static async getBannerByIndex(index = 0) {
+    return $$('[data-testid="ds-banner"]')[index];
+  }
+
   static async getBannerContainer() {
     return $('[data-testid="ds-banner-container"]');
   }

package/ds-card-v3/reflow/DSCardV3.reflow.visual.spec.js

@@ -1,10 +1,12 @@
 import DSCardV3CO from '../DSCardV3CO';
+import { setTextSize } from '../../helpers';
 
 if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
   describe('PUI-13981 - CardV3: Reflow -Visual Test', () => {
     before('loading page', async () => {
       const errorOnGo = await DSCardV3CO.reflowURL.go();
       if (errorOnGo) throw errorOnGo;
+      await setTextSize('26');
     });
     it('01: should display cards after a 200% of text resize', async () => {
       const expandContentButton = await DSCardV3CO.getExpandContentButton();

package/ds-chip/label-expansion-dismissal/DSChip.label-expansion-dismissal.visual.spec.js

@@ -0,0 +1,84 @@
+import { Key } from 'webdriverio';
+import DSChipCO from '../DSChipCO';
+import { mouseOver } from '../../helpers';
+
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  const PARKED_X = 10;
+  const PARKED_Y = 10;
+
+  // Indexes into the basic-test story grid:
+  // 0 Large | 1 Small | 2 Selected | 3 Truncated with large label | 4 Disabled | 5 wrapText with large label
+  const TRUNCATING_CHIP = 3;
+  const WRAP_TEXT_CHIP = 5;
+
+  // DSChipCO deliberately has no label lookup, so the tab target is reached by walking the tab order.
+  // The natively disabled chip drops out of it on its own, which keeps this independent of the story's
+  // exact number of focus stops.
+  const MAX_TAB_STOPS = 12;
+  const tabUntilFocused = async (index) => {
+    const target = await DSChipCO.getChipButtonByIndex(index);
+    for (let i = 0; i < MAX_TAB_STOPS; i += 1) {
+      // eslint-disable-next-line no-await-in-loop
+      if (await target.isFocused()) return target;
+      // eslint-disable-next-line no-await-in-loop
+      await browser.keys(Key.Tab);
+    }
+    throw new Error(`chip ${index} never took focus within ${MAX_TAB_STOPS} tab stops`);
+  };
+
+  describe('PUI-19009 - DSChip:: Escape dismisses the expanded truncated chip label - Visual', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSChipCO.basicURL.go();
+      if (errorOnGo) throw errorOnGo;
+      await browser.maximizeWindow();
+      await (await DSChipCO.getChipByIndex(TRUNCATING_CHIP)).waitForDisplayed();
+    });
+
+    it('01: truncated chip expands its label on hover', async () => {
+      await mouseOver(await DSChipCO.getChipByIndex(TRUNCATING_CHIP));
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-hover-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('02: Escape collapses the hover-expanded label without moving the pointer', async () => {
+      await browser.keys(Key.Escape);
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-hover-collapsed'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('03: label stays collapsed while the pointer remains on the chip', async () => {
+      await mouseOver(await DSChipCO.getChipByIndex(TRUNCATING_CHIP), 12, 12);
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-hover-still-collapsed'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('04: label expands again once the pointer leaves and returns', async () => {
+      await mouseOver(null, PARKED_X, PARKED_Y);
+      await mouseOver(await DSChipCO.getChipByIndex(TRUNCATING_CHIP));
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-rehover-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('05: truncated chip expands its label on focus alone', async () => {
+      await mouseOver(null, PARKED_X, PARKED_Y);
+      const chipButton = await tabUntilFocused(TRUNCATING_CHIP);
+      await expect(chipButton).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-focus-expanded'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('06: Escape collapses the focus-expanded label without moving focus', async () => {
+      await browser.keys(Key.Escape);
+      await expect(await DSChipCO.getChipButtonByIndex(TRUNCATING_CHIP)).toBeFocused();
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-focus-collapsed'));
+      await expect(snapshot).toEqual(0);
+    });
+
+    it('07: a wrapText chip never expands, so Escape leaves it untouched', async () => {
+      await mouseOver(await DSChipCO.getChipByIndex(WRAP_TEXT_CHIP));
+      await browser.keys(Key.Escape);
+      const snapshot = await browser.percyCheckScreenshot(DSChipCO.snapshotPath('esc-dismiss-wrap-text-unchanged'));
+      await expect(snapshot).toEqual(0);
+    });
+  });
+}

package/ds-dataviz/DSDataViz.axis.visual.spec.js

@@ -65,8 +65,7 @@ if (!browser.capabilities['ice:options'].isPhone) {
 
   // fails on firefox due to fake-positive of infinte loop
   if (browser.capabilities.browserName !== 'Firefox') {
-    // Skipped until PUI-16334 gets fixed
-    describe.skip('PUI-9401 - DSDataViz, TimeLinear axis -Visual', () => {
+    describe('PUI-9401 - DSDataViz, TimeLinear axis -Visual', () => {
       before('loading page', async () => {
         const errorOnGo = await DSDataVizCO.timeLinearAxis.go();
         if (errorOnGo) throw errorOnGo;

package/ds-dataviz/DSDataViz.ticks.visual.spec.js

@@ -62,8 +62,7 @@ if (!browser.capabilities['ice:options'].isPhone) {
     });
   });
 
-  // Skipped until PUI-16334 gets fixed
-  describe.skip('PUI-10321 - DSDataViz, Force ticks -Visual', () => {
+  describe('PUI-10321 - DSDataViz, Force ticks -Visual', () => {
     before('loading page', async () => {
       const errorOnGo = await DSDataVizCO.forceTicks.go();
       if (errorOnGo) throw errorOnGo;

package/ds-form-combobox-multi/aria-disabled/DSComboboxMulti.aria-disabled.visual.spec.js

@@ -11,8 +11,10 @@ if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:o
     });
     it('01: The first aria-disabled combobox should be focused', async () => {
       await browser.eyesOpen();
-      const firstCombobox = await DSComboboxMultiCO.getComboboxByIndex(0);
-      await firstCombobox.click();
+      // the container centre lands on the aria-hidden overlay input, which never focuses the
+      // combobox; the selected-values area is the only pointer target that reliably focuses it
+      const firstComboboxValues = await DSComboboxMultiCO.getSelectedValuesByIndex(0);
+      await firstComboboxValues.click();
       const snapshot = await browser.eyesCheckSnapshot(
         DSComboboxMultiCO.snapshotPath('combobox-multi-aria-disabled-click'),
       );

package/ds-form-date-range-picker/DateRangePickerCO.js

@@ -30,6 +30,8 @@ export default class DateRangePickerCO extends PageObject {
     'apply-aria-disabled-invalid-selection-test',
   );
 
+  static pickerCallbacks = new Urlbuilder(PATH_E2E_CONTROLLED_FORM_DATE_RANGE_PICKER, 'picker-callbacks-test');
+
   // Story elements
 
   static getNextMonthBtn = async () => $('[data-testid="ds-controlled-date-time-picker-calendar-next-month"]');
@@ -68,6 +70,10 @@ export default class DateRangePickerCO extends PageObject {
   static getClearableByIndex = async (index = 0) =>
     $$(`[data-testid="ds-controlled-date-time-picker-clear-btn"]`)[index];
 
+  static getOpenCallbackCount = async () => $('[data-testid="picker-open-count"]');
+
+  static getCloseCallbackCount = async () => $('[data-testid="picker-close-count"]');
+
   // TimeInput
   static getHourInput = async () => $('[data-testid="ds-controlled-date-time-picker-hour-input"]');
 

package/ds-form-date-range-picker/picker-callbacks/DateRangePicker.picker-callbacks.func.spec.js

@@ -0,0 +1,36 @@
+import { Key } from 'webdriverio';
+import DateRangePickerCO from '../DateRangePickerCO';
+
+if (
+  (!browser.capabilities['ice:options'].isPhone &&
+    !browser.capabilities['ice:options'].isTablet &&
+    browser.capabilities.browserName === 'chrome') ||
+  browser.capabilities.browserName === 'Chrome'
+) {
+  // Tests for the fix introduced in PUI-18907 (forward picker open and close callbacks).
+  describe('PUI-19082 - DSControlledDateRangePicker:: picker open and close callbacks - Func', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DateRangePickerCO.pickerCallbacks.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should invoke onPickerOpen when the calendar opens', async () => {
+      const calendarTrigger = await DateRangePickerCO.getPickerBtn();
+      const openCallbackCount = await DateRangePickerCO.getOpenCallbackCount();
+
+      await expect(openCallbackCount).toHaveText('0');
+      await calendarTrigger.click();
+      await expect(await DateRangePickerCO.getCalendarDialog()).toBeDisplayedInViewport();
+      await expect(openCallbackCount).toHaveText('1');
+    });
+
+    it('02: should invoke onPickerClose when the calendar closes', async () => {
+      const closeCallbackCount = await DateRangePickerCO.getCloseCallbackCount();
+
+      await expect(closeCallbackCount).toHaveText('0');
+      await browser.keys(Key.Escape);
+      await expect(await DateRangePickerCO.getCalendarDialog()).not.toBeDisplayedInViewport();
+      await expect(closeCallbackCount).toHaveText('1');
+    });
+  });
+}