dimsum-e2e-tests 3.80.0-next.4 → 3.80.0-next.40

package/ds-modal-slide/inside-dialog/ModalSlide.insideDialog.func.spec.js

@@ -0,0 +1,45 @@
+import { Key } from 'webdriverio';
+import ModalSlideCO from '../ModalSlideCO';
+import DSDialogCO from '../../ds-dialog/DSDialogCO';
+
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  describe('PUI-19268 - ModalSlide:: inside Dialog, focus and Escape', () => {
+    beforeEach(async () => {
+      const errorOnGo = await ModalSlideCO.insideDialogURL.go();
+      if (errorOnGo) throw errorOnGo;
+      await ModalSlideCO.openDialog();
+      await ModalSlideCO.openModalSlide();
+    });
+    it('01: Cancel should close only the slide and focus the button that opened it', async () => {
+      await (await ModalSlideCO.getRejectedBtn()).click();
+      await (await ModalSlideCO.getModalSlide()).waitForDisplayed({ reverse: true });
+      await expect(await DSDialogCO.getDialog()).toBeDisplayed();
+      await expect(await ModalSlideCO.getOpenModalSlideBtn()).toBeFocused();
+    });
+    it('02: Confirm should close only the slide and focus the button that opened it', async () => {
+      await (await ModalSlideCO.getModalButton()).click();
+      await (await ModalSlideCO.getModalSlide()).waitForDisplayed({ reverse: true });
+      await expect(await DSDialogCO.getDialog()).toBeDisplayed();
+      await expect(await ModalSlideCO.getOpenModalSlideBtn()).toBeFocused();
+    });
+    it('03: Escape should close only the slide and focus the button that opened the slide', async () => {
+      // the story moves the focus into the slide 500ms after it opens
+      await browser.waitUntil(async () => (await $('[data-testid="back-id"]')).isFocused(), {
+        timeoutMsg: 'the focus should move into the slide',
+      });
+      await browser.keys(Key.Escape);
+      await (await ModalSlideCO.getModalSlide()).waitForDisplayed({ reverse: true });
+      await expect(await DSDialogCO.getDialog()).toBeDisplayed();
+      await expect(await ModalSlideCO.getOpenModalSlideBtn()).toBeFocused();
+    });
+    it('04: a second Escape, with the slide already closed, should close the dialog', async () => {
+      await browser.waitUntil(async () => (await $('[data-testid="back-id"]')).isFocused(), {
+        timeoutMsg: 'the focus should move into the slide',
+      });
+      await browser.keys(Key.Escape);
+      await (await ModalSlideCO.getModalSlide()).waitForDisplayed({ reverse: true });
+      await browser.keys(Key.Escape);
+      await (await DSDialogCO.getDialog()).waitForDisplayed({ reverse: true });
+    });
+  });
+}

package/ds-panel-header/DSPanelHeader.func.spec.js

@@ -0,0 +1,24 @@
+import DSPanelHeaderCO from './DSPanelHeaderCO';
+/* this is the syntax for only enabling test for desktop only
+ if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  describe('PUI-XXXX - PanelHeader:: example test - data-testid exists', () => {
+    before('loading page', async () => {
+      // your code goes here
+    });
+    it('01: wrapper should be displayed', async () => {
+      // your code goes here
+    });
+
+  });
+}
+*/
+describe('PUI-XXXX - PanelHeader:: example test - data-testid exists', () => {
+  before('loading page', async () => {
+    const errorOnGo = await DSPanelHeaderCO.basic.go();
+    if (errorOnGo) throw errorOnGo;
+  });
+  it('01: wrapper should be displayed', async () => {
+    const wrapper = await DSPanelHeaderCO.getWrapper();
+    await expect(wrapper).toBeDisplayedInViewport();
+  });
+});

package/ds-panel-header/DSPanelHeaderCO.js

@@ -0,0 +1,17 @@
+import { PANEL_HEADER_DATA_TESTID } from '@elliemae/ds-panel-header';
+import { PATH_PANEL_HEADER } from '../../environments/storybook/paths';
+import { PageObject, Urlbuilder } from '../helpers';
+
+export default class DSPanelHeader extends PageObject {
+  static basic = new Urlbuilder(PATH_PANEL_HEADER, 'basic');
+
+  // Selectors
+  static async getWrapper() {
+    return $(`[data-testid="${PANEL_HEADER_DATA_TESTID.ROOT}"]`);
+  }
+
+  // Snapshots
+  static snapshotPath(example = 'basic') {
+    return PageObject.getSnapshotPathBuilder('DSPanelHeader', example, 'ds-panel-header');
+  }
+}

package/ds-pills-v2/DSPillsV2CO.js

@@ -1,7 +1,21 @@
-import { PATH_E2E_PILLS_V2 } from '../paths';
+import { PATH_E2E_PILLS_V2, PATH_E2E_PILLS_V2_PITFALLS } from '../paths';
 import { PageObject, Urlbuilder, getElementByIndex } from '../helpers';
 
 export default class DSPillsV2CO extends PageObject {
+  /**
+   * Every id in the InputWithFormItemBlockTest story is derived by DSFormLayoutBlockItem from the
+   * `inputID` it is given. Derived once here so neither the selectors below nor the spec rebuild
+   * the `_feedback_message` / `_error_message` suffixes by hand.
+   */
+  static IDS = {
+    input: 'normal',
+    feedback: 'normal_feedback_message',
+    error: 'normal_error_message',
+  };
+
+  /** What the input's aria-describedby must equal, in the order the story supplies the messages. */
+  static DESCRIBED_BY = `${this.IDS.feedback} ${this.IDS.error}`;
+
   // Desktop
   static removableURL = new Urlbuilder(PATH_E2E_PILLS_V2, 'removable-pill-test');
 
@@ -27,6 +41,15 @@ export default class DSPillsV2CO extends PageObject {
 
   static multiSelectMB = new Urlbuilder(PATH_E2E_PILLS_V2, 'multi-select-menu-button-test');
 
+  // Pitfalls
+  static inputWithFormItemBlockURL = new Urlbuilder(PATH_E2E_PILLS_V2_PITFALLS, 'input-with-form-item-block-test');
+
+  // Distinct from `inputURL` above: same story name, different path. Both are hand-copies of the
+  // Input Pill pitfall story's 12 pills; this one is the Pitfalls-scoped twin used by the attribute
+  // spec, `inputURL` is the older copy carrying the axe (PUI-11496) and Percy (PUI-8356) baselines.
+  // The E2E copy renders PendingReviewDisclaimer; the canonical component pitfall story additionally renders InputPillLabelRequirementDisclaimer.
+  static inputPillPitfallURL = new Urlbuilder(PATH_E2E_PILLS_V2_PITFALLS, 'input-pill-test');
+
   static getUrl(component = 'basic') {
     return PageObject.getUrl(PATH_E2E_PILLS_V2, component);
   }
@@ -93,6 +116,35 @@ export default class DSPillsV2CO extends PageObject {
     return $('[data-testid="dropdown-menu-listbox"]');
   }
 
+  // --- InputWithFormItemBlockTest: label / error association selectors ---
+
+  /** The pill's inner <input>, reached by the id the story overrides onto it. */
+  static async getAssociatedInput() {
+    return $(`#${this.IDS.input}`);
+  }
+
+  /** `$$` on purpose — the count is what proves the override did not duplicate the id. */
+  static async getNodesWithInputId() {
+    return $$(`[id="${this.IDS.input}"]`);
+  }
+
+  /** The DSFormLayoutBlockItem label, which must point at the input via `for`. */
+  static async getAssociatedInputLabel() {
+    return $(`label[for="${this.IDS.input}"]`);
+  }
+
+  static async getFeedbackMessage() {
+    return $(`#${this.IDS.feedback}`);
+  }
+
+  static async getErrorMessage() {
+    return $(`#${this.IDS.error}`);
+  }
+
+  static async getClearIcon() {
+    return $('[data-testid="ds-pill-clear-icon"]');
+  }
+
   // Snapshots
   static snapshotPath(example = 'basic') {
     return PageObject.getSnapshotPathBuilder('Pills', example);

package/ds-pills-v2/pitfalls/DSPills.inputPill.func.spec.js

@@ -0,0 +1,26 @@
+import DSPillsV2CO from '../DSPillsV2CO';
+
+if (
+  (!browser.capabilities['ice:options'].isPhone &&
+    !browser.capabilities['ice:options'].isTablet &&
+    browser.capabilities.browserName === 'chrome') ||
+  browser.capabilities.browserName === 'Chrome'
+) {
+  describe('PUI-19148 - DSPillsV2:: input pill attribute contract - Func', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSPillsV2CO.inputPillPitfallURL.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should render the input with the attributes', async () => {
+      const input = await DSPillsV2CO.getInputByIndex(0);
+      await input.waitForDisplayed();
+      await expect(await input.getTagName()).toBe('input');
+      await expect(input).toHaveAttribute('data-testid', 'ds-pill-input');
+      const generatedId = await input.getAttribute('id');
+      await expect(generatedId).toMatch(/^ds-pill-/);
+      await expect(input).not.toHaveAttribute('aria-describedby');
+      await expect(input).toHaveAttribute('aria-invalid', 'false');
+    });
+  });
+}

package/ds-pills-v2/pitfalls/DSPills.inputWithFormItemBlockURL.axe-core.func.spec.js

@@ -0,0 +1,38 @@
+import DSPillsV2CO from '../DSPillsV2CO';
+import { axeCoreCheck, type } from '../../helpers';
+
+// Accessibility scan for the fix introduced in PUI-18617 ([PillsV2] Pitfalls enhancements, item 2).
+//
+// Note on what this does and does not prove: axe has no rule that requires an input to be described by
+// an error message, so a clean scan here is a regression guard against the association *breaking the
+// markup* (orphaned aria-describedby ids, duplicate ids, an unlabelled input) — not evidence that the
+// label and error are announced. The announcement itself is covered by the functional spec and by
+// manual screen reader verification on the ticket.
+if (
+  (!browser.capabilities['ice:options'].isPhone &&
+    !browser.capabilities['ice:options'].isTablet &&
+    browser.capabilities.browserName === 'chrome') ||
+  browser.capabilities.browserName === 'Chrome'
+) {
+  describe('PUI-19150 - DSPillsV2:: input pill label and error association - Axe-Core', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSPillsV2CO.inputWithFormItemBlockURL.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should have no accessibility violations, empty and filled', async () => {
+      const input = await DSPillsV2CO.getAssociatedInput();
+      await input.waitForDisplayed();
+      const emptyResult = await axeCoreCheck();
+      expect(emptyResult.length).toBe(0);
+      // Filling the input mounts the clear button, which adds an extra focusable control inside the
+      // associated form item — worth scanning in that state too.
+      await input.click();
+      await type('abcd');
+      const clearIcon = await DSPillsV2CO.getClearIcon();
+      await clearIcon.waitForDisplayed();
+      const filledResult = await axeCoreCheck();
+      expect(filledResult.length).toBe(0);
+    });
+  });
+}

package/ds-pills-v2/pitfalls/DSPills.inputWithFormItemBlockURL.func.spec.js

@@ -0,0 +1,72 @@
+import DSPillsV2CO from '../DSPillsV2CO';
+import { type } from '../../helpers';
+
+// Tests for the fix introduced in PUI-18617 ([PillsV2] Pitfalls enhancements, item 2): the input pill
+// inside a DSFormLayoutBlockItem announced neither its label nor its error message. The fix exposed
+// `dsPillInput` as a real slot on the default (non-inputRender) input, so the association is now made
+// by passing id / aria-describedby through the nested `dsInputtextInput` slot.
+//
+// Assertions are pure DOM attributes, so this is scoped to Chrome — see the skill's
+// "DOM-Attribute Tests Run Chrome-Only" rule. The `|| 'Chrome'` fallback is mandatory: BrowserStack
+// reports the capitalized name and a lowercase-only guard registers 0 tests (false green).
+if (
+  (!browser.capabilities['ice:options'].isPhone &&
+    !browser.capabilities['ice:options'].isTablet &&
+    browser.capabilities.browserName === 'chrome') ||
+  browser.capabilities.browserName === 'Chrome'
+) {
+  describe('PUI-19149 - DSPillsV2:: input pill label and error association - Func', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSPillsV2CO.inputWithFormItemBlockURL.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should associate the input with its label and its feedback/error messages', async () => {
+      const input = await DSPillsV2CO.getAssociatedInput();
+      await input.waitForDisplayed();
+
+      // The overridden id must land on the pill's real <input>, not on a wrapper. Checking the tag and
+      // the pill's own data-testid on the same element is what distinguishes the two.
+      await expect(await input.getTagName()).toBe('input');
+      await expect(input).toHaveAttribute('data-testid', 'ds-pill-input');
+
+      // hasError on the pill is forwarded to DSInputText as aria-invalid.
+      await expect(input).toHaveAttribute('aria-invalid', 'true');
+
+      // Both message ids, in the order the story supplies them.
+      await expect(input).toHaveAttribute('aria-describedby', DSPillsV2CO.DESCRIBED_BY);
+
+      // aria-describedby is only meaningful if the referenced nodes exist.
+      const feedbackMessage = await DSPillsV2CO.getFeedbackMessage();
+      await expect(feedbackMessage).toBeExisting();
+      const errorMessage = await DSPillsV2CO.getErrorMessage();
+      await expect(errorMessage).toBeExisting();
+
+      // The label side of the association: DSFormLayoutBlockItem renders label[for] from inputID.
+      const label = await DSPillsV2CO.getAssociatedInputLabel();
+      await expect(label).toBeExisting();
+      await expect(await label.getText()).toContain('Item');
+
+      // Duplicate ids would make the label and aria-describedby resolve to the wrong node.
+      const elementsWithId = await DSPillsV2CO.getNodesWithInputId();
+      await expect(elementsWithId.length).toBe(1);
+    });
+
+    it('02: should keep the association intact once the clear button mounts', async () => {
+      const input = await DSPillsV2CO.getAssociatedInput();
+      await input.click();
+      await type('abcd');
+
+      // The clear button only renders once the input has a value, and it must not reuse the input's id.
+      const clearIcon = await DSPillsV2CO.getClearIcon();
+      await clearIcon.waitForDisplayed();
+      await expect(clearIcon).not.toHaveAttribute('id', DSPillsV2CO.IDS.input);
+
+      await expect(input).toHaveAttribute('aria-describedby', DSPillsV2CO.DESCRIBED_BY);
+      await expect(input).toHaveAttribute('aria-invalid', 'true');
+
+      const elementsWithId = await DSPillsV2CO.getNodesWithInputId();
+      await expect(elementsWithId.length).toBe(1);
+    });
+  });
+}

package/ds-progress-indicator/DSProgressIndicator.axe-core.func.spec.js

@@ -1,4 +1,3 @@
-/* eslint-disable wdio/no-pause */
 import DSProgressIndicatorCO from './DSProgressIndicatorCO';
 import { axeCoreCheck } from '../helpers';
 
@@ -8,24 +7,28 @@ if (
     browser.capabilities.browserName === 'chrome') ||
   browser.capabilities.browserName === 'Chrome'
 ) {
-  describe('PUI-17521: Progress Indicator:: Basic - Axe Core', () => {
+  describe('PUI-17521 - DSProgressIndicator:: Basic - Axe-Core', () => {
     before('loading page', async () => {
       const errorOnGo = await DSProgressIndicatorCO.basicURL.go();
       if (errorOnGo) throw errorOnGo;
     });
     it('01: should display basic progress indicator and pass axe-core', async () => {
-      await (await DSProgressIndicatorCO.getUploadLabel()).waitForDisplayed();
-      await browser.pause(500);
+      await (await DSProgressIndicatorCO.getTitle()).waitForDisplayed();
+
       const result = await axeCoreCheck();
       expect(result.length).toBe(0);
     });
   });
-  describe('PUI-17524: Progress Indicator:: Others Test - Axe Core', () => {
+
+  describe('PUI-17524 - DSProgressIndicator:: Others Test - Axe-Core', () => {
     before('loading page', async () => {
       const errorOnGo = await DSProgressIndicatorCO.othersURL.go();
       if (errorOnGo) throw errorOnGo;
     });
     it('01: should display progress indicator and pass axe-core', async () => {
+      // Replaces a blind browser.pause(500): the story animates, so wait on the real end state.
+      await DSProgressIndicatorCO.waitForAnimatedCountersToSettle();
+
       const result = await axeCoreCheck();
       expect(result.length).toBe(0);
     });

package/ds-progress-indicator/DSProgressIndicator.func.spec.js

@@ -6,14 +6,41 @@ if (
     browser.capabilities.browserName === 'chrome') ||
   browser.capabilities.browserName === 'Chrome'
 ) {
-  describe('PUI-17522 - Progress Indicator:: Basic - Func', () => {
+  describe('PUI-17522 - DSProgressIndicator:: Basic - Func', () => {
     before('loading page', async () => {
       const errorOnGo = await DSProgressIndicatorCO.basicURL.go();
       if (errorOnGo) throw errorOnGo;
     });
-    it('01: should show basic progress indicator', async () => {
-      const progressIndicator = await DSProgressIndicatorCO.getUploadLabel();
-      await expect(progressIndicator).toBeDisplayed();
+
+    // The naming, aria-valuetext and invalid-range contracts live in the aria-contract spec; this
+    // one stays on the Basic story's own value contract.
+    it('01: should show basic progress indicator with a valid progressbar value range', async () => {
+      const title = await DSProgressIndicatorCO.getTitle();
+      await expect(title).toBeDisplayed();
+      await expect(title).toHaveText('upload');
+
+      const root = await DSProgressIndicatorCO.getRoot();
+      await expect(root).toHaveAttribute('aria-disabled', 'false');
+
+      const progress = await DSProgressIndicatorCO.getProgress();
+      await expect(progress).toHaveAttribute('role', 'progressbar');
+      await expect(progress).toHaveAttribute('aria-valuenow', '50');
+      await expect(progress).toHaveAttribute('aria-valuemin', '0');
+      await expect(progress).toHaveAttribute('aria-valuemax', '100');
+      await expect(await progress.getAttribute('style')).toContain('width: 50%');
+    });
+
+    // PUI-18942: title and counter text used to be raw text inside plain spans.
+    it('02: should render title and counter text through DSTypography', async () => {
+      const { TITLE, COUNTER } = DSProgressIndicatorCO.SLOT_TESTID;
+
+      const titleText = await DSProgressIndicatorCO.getTypographyIn(TITLE);
+      await expect(titleText).toBeExisting();
+      await expect(titleText).toHaveText('upload');
+
+      const counterText = await DSProgressIndicatorCO.getTypographyIn(COUNTER);
+      await expect(counterText).toBeExisting();
+      await expect(counterText).toHaveText('50 of 100');
     });
   });
 }

package/ds-progress-indicator/DSProgressIndicator.visual.spec.js

@@ -1,24 +1,32 @@
 import DSProgressIndicatorCO from './DSProgressIndicatorCO';
 
-if (!browser.capabilities['ice:options'].isPhone) {
-  describe('PUI-17520 - Progress Indicator:: Basic - Visual', () => {
+// Migrated off the pre-Percy browser.checkSnapshot API as part of the PUI-18856 compliance work,
+// which rewrote the class system these baselines were taken against.
+if (!browser.capabilities['ice:options'].isPhone && !browser.capabilities['ice:options'].isTablet) {
+  describe('PUI-17520 - DSProgressIndicator:: Basic - Visual', () => {
     before('loading page', async () => {
       const errorOnGo = await DSProgressIndicatorCO.basicURL.go();
       if (errorOnGo) throw errorOnGo;
     });
     it('01: should display basic progress indicator', async () => {
-      await (await DSProgressIndicatorCO.getUploadLabel()).waitForDisplayed();
-      const snapshot = await browser.checkSnapshot(DSProgressIndicatorCO.snapshotPath('progressindicator-basic'));
+      await (await DSProgressIndicatorCO.getTitle()).waitForDisplayed();
+
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
   });
-  describe('PUI-17523 - Progress Indicator:: Others Test - Visual', () => {
+
+  describe('PUI-17523 - DSProgressIndicator:: Others Test - Visual', () => {
     before('loading page', async () => {
       const errorOnGo = await DSProgressIndicatorCO.othersURL.go();
       if (errorOnGo) throw errorOnGo;
     });
     it('01: should display progress indicator', async () => {
-      const snapshot = await browser.checkSnapshot(DSProgressIndicatorCO.snapshotPath('progressindicator-others'));
+      // Two of the four indicators are driven by setInterval and only stop once their counters
+      // reach the maximum. Snapshotting before that lands produces a different image every run.
+      await DSProgressIndicatorCO.waitForAnimatedCountersToSettle();
+
+      const snapshot = await browser.percyCheckScreenshot();
       await expect(snapshot).toEqual(0);
     });
   });

package/ds-progress-indicator/DSProgressIndicatorCO.js

@@ -1,26 +1,104 @@
 import { PATH_E2E_PROGRESS_INDICATOR } from '../paths';
 import { PageObject, Urlbuilder } from '../helpers';
 
+// Slot testids owned by the component (packages/atom/ds-progress-indicator/src/constants/index.ts).
+// Note the lowercase `i` in "progressindicator": it comes from DSProgressIndicatorName.
+const SLOT_TESTID = {
+  ROOT: 'ds-progressindicator-root',
+  TITLE: 'ds-progressindicator-title',
+  COUNTER: 'ds-progressindicator-counter',
+  PROGRESS_BAR: 'ds-progressindicator-progress-bar',
+  PROGRESS: 'ds-progressindicator-progress',
+};
+
+// In segmented mode every segment reuses the PROGRESS testid, and the stories render several
+// indicators per page, so selectors are scoped by the story's per-scenario `*-case` wrapper.
+// Omit `caseTestId` for single-indicator stories such as BasicTest.
+const scoped = (testId, caseTestId) =>
+  caseTestId ? `[data-testid="${caseTestId}"] [data-testid="${testId}"]` : `[data-testid="${testId}"]`;
+
 export default class DSProgressIndicatorCO extends PageObject {
   // Desktop
   static basicURL = new Urlbuilder(PATH_E2E_PROGRESS_INDICATOR, 'basic-test');
 
   static othersURL = new Urlbuilder(PATH_E2E_PROGRESS_INDICATOR, 'others-test');
 
-  static getUrl(component = 'basic') {
-    return PageObject.getUrl(PATH_E2E_PROGRESS_INDICATOR, component);
+  static ariaContractURL = new Urlbuilder(PATH_E2E_PROGRESS_INDICATOR, 'aria-contract-test');
+
+  static slotsURL = new Urlbuilder(PATH_E2E_PROGRESS_INDICATOR, 'slots-test');
+
+  static singleBarVariantURL = new Urlbuilder(PATH_E2E_PROGRESS_INDICATOR, 'single-bar-variant-test');
+
+  static SLOT_TESTID = SLOT_TESTID;
+
+  // selectors
+  static async getRoot(caseTestId) {
+    return $(scoped(SLOT_TESTID.ROOT, caseTestId));
   }
 
-  static async getUploadLabel() {
-    return $('//span[contains(text(), "upload")]');
+  static async getTitle(caseTestId) {
+    return $(scoped(SLOT_TESTID.TITLE, caseTestId));
   }
 
-  // Snapshots
-  static snapshotPath(example = 'basic') {
-    return PageObject.getSnapshotPathBuilder('ProgressIndicator', example);
+  static async getCounter(caseTestId) {
+    return $(scoped(SLOT_TESTID.COUNTER, caseTestId));
+  }
+
+  static async getProgressBar(caseTestId) {
+    return $(scoped(SLOT_TESTID.PROGRESS_BAR, caseTestId));
+  }
+
+  // First (or only) progressbar of a case. Use getProgressBars for the segmented ones.
+  static async getProgress(caseTestId) {
+    return $(scoped(SLOT_TESTID.PROGRESS, caseTestId));
+  }
+
+  static async getProgressBars(caseTestId) {
+    return $$(scoped(SLOT_TESTID.PROGRESS, caseTestId));
+  }
+
+  // Text node DSTypography renders inside a slot, e.g. getTypographyIn(SLOT_TESTID.TITLE).
+  static async getTypographyIn(slotTestId, caseTestId) {
+    return $(`${scoped(slotTestId, caseTestId)} [data-dimsum-slot="dsTypographyRoot"]`);
+  }
+
+  // Slot-level lookups, for asserting the data-dimsum-slot contract independently of the testids.
+  static async getSlot(slotName, caseTestId) {
+    const prefix = caseTestId ? `[data-testid="${caseTestId}"] ` : '';
+    return $(`${prefix}[data-dimsum-slot="${slotName}"]`);
+  }
+
+  static async getSlots(slotName, caseTestId) {
+    const prefix = caseTestId ? `[data-testid="${caseTestId}"] ` : '';
+    return $$(`${prefix}[data-dimsum-slot="${slotName}"]`);
+  }
+
+  // aria-labelledby points at a React.useId() value containing ":", which is not a valid CSS id
+  // selector, so the target has to be resolved through the attribute selector instead of "#id".
+  static async getByIdAttribute(id) {
+    return $(`[id="${id}"]`);
   }
 
-  // selectors
-  // actions
   // utils
+  // OthersTest drives two indicators with setInterval counters. They stop once each counter reaches
+  // its max, so the story is only deterministic after both have landed on their final totals.
+  //
+  // The texts are read in a single browser.execute rather than by mapping over $$: WebdriverIO's
+  // element array overrides .map() to return a Promise, so Promise.all(list.map(...)) throws
+  // "object is not iterable". One round trip per poll is also cheaper than four.
+  static async waitForAnimatedCountersToSettle() {
+    await browser.waitUntil(
+      async () => {
+        const texts = await browser.execute(
+          (testId) => Array.from(document.querySelectorAll(`[data-testid="${testId}"]`), (el) => el.textContent),
+          SLOT_TESTID.COUNTER,
+        );
+        return texts.length >= 4 && texts[1] === '400 of 400' && texts[2] === '330 of 330';
+      },
+      {
+        timeout: 15000,
+        timeoutMsg: 'animated progress indicators never reached their final value',
+      },
+    );
+  }
 }

package/ds-progress-indicator/aria-contract/DSProgressIndicator.aria-contract.func.spec.js

@@ -0,0 +1,104 @@
+import DSProgressIndicatorCO from '../DSProgressIndicatorCO';
+
+// Coverage for the accessibility remediation shipped in PUI-18856 (PR #8353):
+//   PUI-18671 the progressbar used to carry a hardcoded aria-label="progress bar". It is now named
+//             by the visible title through aria-labelledby, and that string survives only as the
+//             fallback for an untitled indicator.
+//   PUI-18673 aria-valuetext now mirrors whatever the visible counter renders, default or custom.
+//   PUI-18675 each segment accepts a `label` that becomes its own accessible name. Only the Dimsum
+//             half of that defect is assertable here; the visible non-colour alternative is the
+//             consumer's, see stories/docs/principles/a11y/tradesoff.
+//   PUI-18672 an invalid `max` no longer produces width: NaN% and an inverted ARIA range. `max` was
+//             deliberately NOT made required (KNOWN_INTENTIONAL_DEVIATIONS.md, deviation 6), so what
+//             is asserted below is the shipped fail-safe: an indeterminate progressbar.
+//
+// Every assertion is a DOM attribute, which is identical across engines, so this runs Chrome only.
+// The `|| 'Chrome'` branch is mandatory: BrowserStack reports the capitalised name and a
+// lowercase-only guard would register zero tests while still exiting 0.
+if (
+  (!browser.capabilities['ice:options'].isPhone &&
+    !browser.capabilities['ice:options'].isTablet &&
+    browser.capabilities.browserName === 'chrome') ||
+  browser.capabilities.browserName === 'Chrome'
+) {
+  describe('PUI-19209 - DSProgressIndicator:: progressbar ARIA contract - Func', () => {
+    before('loading page', async () => {
+      const errorOnGo = await DSProgressIndicatorCO.ariaContractURL.go();
+      if (errorOnGo) throw errorOnGo;
+    });
+
+    it('01: should name the progressbar from the visible title and fall back when untitled', async () => {
+      const titled = await DSProgressIndicatorCO.getProgress('titled-case');
+      await titled.waitForDisplayed();
+      await expect(titled).not.toHaveAttribute('aria-label');
+
+      // The id is generated with React.useId(), so it is resolved at runtime rather than hardcoded.
+      const labelledBy = await titled.getAttribute('aria-labelledby');
+      const title = await DSProgressIndicatorCO.getTitle('titled-case');
+      await expect(await title.getAttribute('id')).toEqual(labelledBy);
+      await expect(await DSProgressIndicatorCO.getByIdAttribute(labelledBy)).toHaveText('Uploading documents');
+
+      const titleless = await DSProgressIndicatorCO.getProgress('titleless-case');
+      await expect(titleless).toHaveAttribute('aria-label', 'progress bar');
+      await expect(titleless).not.toHaveAttribute('aria-labelledby');
+    });
+
+    it('02: should mirror the visible counter in aria-valuetext for default and custom renderers', async () => {
+      const defaultCounter = await DSProgressIndicatorCO.getCounter('titled-case');
+      await expect(defaultCounter).toHaveText('50 of 100');
+      const defaultBar = await DSProgressIndicatorCO.getProgress('titled-case');
+      await expect(defaultBar).toHaveAttribute('aria-valuetext', '50 of 100');
+
+      const customCounter = await DSProgressIndicatorCO.getCounter('custom-counter-case');
+      await expect(customCounter).toHaveText('3 files remaining');
+      const customBar = await DSProgressIndicatorCO.getProgress('custom-counter-case');
+      await expect(customBar).toHaveAttribute('aria-valuetext', '3 files remaining');
+    });
+
+    it('03: should give each labelled segment its own accessible name', async () => {
+      const labelled = await DSProgressIndicatorCO.getProgressBars('labelled-segments-case');
+      await expect(labelled).toBeElementsArrayOfSize(2);
+
+      // A segment label wins over the indicator title, so aria-labelledby is dropped entirely.
+      await expect(labelled[0]).toHaveAttribute('aria-label', 'Approved documents');
+      await expect(labelled[0]).not.toHaveAttribute('aria-labelledby');
+      await expect(labelled[1]).toHaveAttribute('aria-label', 'Rejected documents');
+      await expect(labelled[1]).not.toHaveAttribute('aria-labelledby');
+
+      // Each segment reports its own range rather than the aggregate shown by the counter.
+      await expect(labelled[0]).toHaveAttribute('aria-valuetext', '50 of 100');
+      await expect(labelled[1]).toHaveAttribute('aria-valuetext', '100 of 200');
+
+      // An unlabelled segment still falls back to the indicator title.
+      const mixed = await DSProgressIndicatorCO.getProgressBars('mixed-segments-case');
+      await expect(mixed).toBeElementsArrayOfSize(2);
+      await expect(mixed[0]).toHaveAttribute('aria-label', 'Completed');
+      await expect(mixed[1]).not.toHaveAttribute('aria-label');
+      const fallbackId = await mixed[1].getAttribute('aria-labelledby');
+      await expect(await DSProgressIndicatorCO.getByIdAttribute(fallbackId)).toHaveText('Batch upload');
+    });
+
+    it('04: should render indeterminate for an invalid max and clamp an overflowing value', async () => {
+      const expectIndeterminate = async (caseId) => {
+        const bar = await DSProgressIndicatorCO.getProgress(caseId);
+        await expect(bar).toHaveAttribute('role', 'progressbar');
+        await expect(bar).not.toHaveAttribute('aria-valuenow');
+        await expect(bar).not.toHaveAttribute('aria-valuemin');
+        await expect(bar).not.toHaveAttribute('aria-valuemax');
+        await expect(bar).not.toHaveAttribute('aria-valuetext');
+        // The pre-fix behaviour was width: NaN%; the guard now yields a valid zero-length fill.
+        await expect(await bar.getAttribute('style')).toContain('width: 0%');
+        // The indicator is still named, so the invalid range degrades without losing the label.
+        await expect(bar).toHaveAttribute('aria-labelledby');
+      };
+
+      await expectIndeterminate('omitted-max-case');
+      await expectIndeterminate('zero-max-case');
+
+      const overflow = await DSProgressIndicatorCO.getProgress('overflow-case');
+      await expect(overflow).toHaveAttribute('aria-valuenow', '100');
+      await expect(overflow).toHaveAttribute('aria-valuemax', '100');
+      await expect(await overflow.getAttribute('style')).toContain('width: 100%');
+    });
+  });
+}