Merge branch '25_2' into 25_2_reduce_runners

Author: EugeniyKiyashko

e2e/testcafe-devextreme/tests/dataGrid/common/keyboardNavigation/keyboardNavigation.functional.ts

@@ -2605,23 +2605,6 @@ test('Empty row should lose focus on Tab (T941246)', async (t) => {
     }
 
     await checkNavigationOfAllCells();
-
-    await t
-      .pressKey('tab');
-
-    const $buttonElement = isCommandColumnFixed
-      ? dataGrid.getFixedDataRow(1).getCommandCell(2).getButton(0)
-      : dataGrid.getDataRow(1).getCommandCell(2).getButton(0);
-
-    await t
-      .expect($buttonElement.focused)
-      .notOk()
-      .pressKey('shift+tab')
-      .expect($buttonElement.focused)
-      .ok()
-      .pressKey('shift+tab')
-      .expect(dataGrid.getDataCell(1, 1).element.focused)
-      .ok();
   }).before(async () => {
     await createWidget('dxDataGrid', {
       dataSource: [

e2e/testcafe-devextreme/tests/dataGrid/common/scrolling.ts

@@ -1833,34 +1833,31 @@ test('DataGrid - The "row" parameter in the FocusedRowChanged event refers to a
 [true, false].forEach((nativeScroll) => {
   type TestCaseWindow = typeof window & { dataGridScrollableEventValues?: number[] };
 
-  test(
-    `Should not scroll back on top with virtual scrolling and adaptive master detail (nativeScroll: ${nativeScroll}) [T1278804]`,
-    async (t) => {
-      // NOTE: idx + 1 logic inside POM
-      const adaptiveCellIdx = 101;
-      const scrollValuesThreshold = 100;
-
-      const dataGrid = new DataGrid('#container');
-      const firstRow = dataGrid.getDataRow(0);
-      const firstDataCell = firstRow.getDataCell(0);
-      const adaptiveCell = firstRow.getCommandCell(adaptiveCellIdx);
+  test.meta({ unstable: true })(`Should not scroll back on top with virtual scrolling and adaptive master detail (nativeScroll: ${nativeScroll}) [T1278804]`, async (t) => {
+    // NOTE: idx + 1 logic inside POM
+    const adaptiveCellIdx = 101;
+    const scrollValuesThreshold = 100;
 
-      await t
-        .click(firstDataCell.element)
-        .click(adaptiveCell.element);
+    const dataGrid = new DataGrid('#container');
+    const firstRow = dataGrid.getDataRow(0);
+    const firstDataCell = firstRow.getDataCell(0);
+    const adaptiveCell = firstRow.getCommandCell(adaptiveCellIdx);
+
+    await t
+      .click(firstDataCell.element)
+      .click(adaptiveCell.element);
 
-      await dataGrid
-        .scrollBy(t, { y: 1000 });
-      await dataGrid
-        .scrollBy(t, { y: 1000 });
+    await dataGrid
+      .scrollBy(t, { y: 1000 });
+    await dataGrid
+      .scrollBy(t, { y: 1000 });
 
-      const scrollOffsets = await t
-        .eval(() => (window as TestCaseWindow).dataGridScrollableEventValues) as number[];
+    const scrollOffsets = await t
+      .eval(() => (window as TestCaseWindow).dataGridScrollableEventValues) as number[];
 
-      const hasSmallScrollValues = scrollOffsets.some((offset) => offset < scrollValuesThreshold);
-      await t.expect(hasSmallScrollValues).notOk();
-    },
-  ).before(async () => {
+    const hasSmallScrollValues = scrollOffsets.some((offset) => offset < scrollValuesThreshold);
+    await t.expect(hasSmallScrollValues).notOk();
+  }).before(async () => {
     await createWidget('dxDataGrid', {
       dataSource: getData(3, 100).map((item, idx) => ({ ...item, id: idx })),
       keyExpr: 'id',

e2e/testcafe-devextreme/tests/editors/textBox/etalons/Textbox validation message (fluent.blue.light).png

@@ -12,7 +12,7 @@ const TEXTEDITOR_INPUT_CLASS = 'dx-texteditor-input';
 fixture.disablePageReloads`ValidationMessage`
   .page(url(__dirname, '../../container.html'));
 
-test.meta({ browserSize: [300, 200] })('Validation Message position should be correct after change visibility of parent container (T1095900)', async (t) => {
+test.meta({ browserSize: [600, 600] })('Validation Message position should be correct after change visibility of parent container (T1095900)', async (t) => {
   const { takeScreenshot, compareResults } = createScreenshotsComparer(t);
 
   addFocusableElementBefore('#container');
@@ -21,7 +21,8 @@ test.meta({ browserSize: [300, 200] })('Validation Message position should be co
     .click(Selector(`.${TEXTEDITOR_INPUT_CLASS}`))
     .pressKey('backspace')
     .pressKey('enter')
-    .pressKey('tab');
+    .pressKey('tab')
+    .wait(500);
 
   await setAttribute('#container', 'hidden', 'true');
   await removeAttribute('#container', 'hidden');

e2e/testcafe-devextreme/tests/editors/textBox/validationMessage.ts

@@ -3,23 +3,25 @@ import { createScreenshotsComparer } from 'devextreme-screenshot-comparer';
 import Scheduler from 'devextreme-testcafe-models/scheduler';
 import { createWidget } from '../../../../helpers/createWidget';
 import url from '../../../../helpers/getPageUrl';
-import { testScreenshot } from '../../../../helpers/themeUtils';
+import { isMaterial, isMaterialBased, testScreenshot } from '../../../../helpers/themeUtils';
 
 fixture.disablePageReloads`Scheduler header: material theme`
   .page(url(__dirname, '../../../container.html'));
 
 // visual: material.blue.light
-test('dateNavigator buttons should have "text" styling mode with material theme', async (t) => {
+test('dateNavigator buttons should have "text" styling mode', async (t) => {
   const { toolbar } = new Scheduler('#container');
 
+  const expectedClass = isMaterialBased() ? 'dx-button-mode-text' : 'dx-button-mode-contained';
+
   await t
-    .expect(toolbar.navigator.prevButton.hasClass('dx-button-mode-text'))
+    .expect(toolbar.navigator.prevButton.hasClass(expectedClass))
     .ok()
 
-    .expect(toolbar.navigator.caption.hasClass('dx-button-mode-text'))
+    .expect(toolbar.navigator.caption.hasClass(expectedClass))
     .ok()
 
-    .expect(toolbar.navigator.nextButton.hasClass('dx-button-mode-text'))
+    .expect(toolbar.navigator.nextButton.hasClass(expectedClass))
     .ok();
 }).before(async () => createWidget('dxScheduler', {
   currentView: 'day',
@@ -36,7 +38,7 @@ test('viewSwitcher dropdown button popup should have a specified class', async (
   await t
     .click(dropDownButton.element)
     .expect(Selector(viewSwitcherDropDownButtonContent).count)
-    .eql(1);
+    .eql(isMaterial() ? 1 : 0);
 }).before(async () => createWidget('dxScheduler', {
   currentView: 'day',
   views: ['day', 'week'],

e2e/testcafe-devextreme/tests/scheduler/common/header/header_material.ts

@@ -0,0 +1,71 @@
+/**
+ * @jest-environment jsdom
+ */
+import {
+  beforeEach, describe, expect, it,
+} from '@jest/globals';
+
+import { renderTrialPanel } from './trial_panel.client';
+
+describe('trial panel client', () => {
+  beforeEach(() => {
+    document.body.innerHTML = '';
+  });
+
+  it('should not display subscription text when subscriptions parameter is empty string', () => {
+    renderTrialPanel(
+      'https://example.com/buy',
+      'https://example.com/docs',
+      '25.2.0',
+      '',
+    );
+
+    const triggerElement = document.querySelector('dx-license-trigger');
+    expect(triggerElement).not.toBeNull();
+
+    const panelElement = document.querySelector('dx-license');
+    if (panelElement) {
+      const text = panelElement.textContent ?? '';
+      expect(text).not.toContain('Included in Subscriptions:');
+    }
+  });
+
+  it('should set subscriptions attribute when parameter is a valid string', () => {
+    renderTrialPanel(
+      'https://example.com/buy',
+      'https://example.com/docs',
+      '25.2.0',
+      'Universal, DXperience',
+    );
+
+    const triggerElement = document.querySelector('dx-license-trigger');
+    expect(triggerElement).not.toBeNull();
+
+    const subscriptionsAttr = triggerElement?.getAttribute('subscriptions');
+    expect(subscriptionsAttr).toBe('Universal, DXperience');
+  });
+
+  it('should not display subscription text when subscriptions parameter is actual null', () => {
+    renderTrialPanel(
+      'https://example.com/buy',
+      'https://example.com/docs',
+      '25.2.0',
+      null as any, // Actual null value
+    );
+
+    const triggerElement = document.querySelector('dx-license-trigger');
+    expect(triggerElement).not.toBeNull();
+
+    // When null is passed, it gets converted to string "null" as HTML attribute
+    const subscriptionsAttr = triggerElement?.getAttribute('subscriptions');
+    expect(subscriptionsAttr).toBe('');
+
+    // Ensure the panel element doesn't show "Included in Subscriptions: null"
+    // This is the key test - our fix should prevent displaying "null"
+    const panelElement = document.querySelector('dx-license');
+    if (panelElement) {
+      const text = panelElement.textContent ?? '';
+      expect(text).not.toContain('Included in Subscriptions: null');
+    }
+  });
+});

packages/devextreme/js/__internal/core/license/trial_panel.client.test.ts

@@ -304,10 +304,11 @@ class DxLicenseTrigger extends SafeHTMLElement {
       const license = document.createElement(componentNames.panel);
 
       Object.values(attributeNames).forEach((attrName) => {
-        license.setAttribute(
-          attrName,
-          this.getAttribute(attrName) as string,
-        );
+        const attrValue = this.getAttribute(attrName);
+
+        if (attrValue) {
+          license.setAttribute(attrName, attrValue);
+        }
       });
 
       license.setAttribute(DATA_PERMANENT_ATTRIBUTE, '');
@@ -338,7 +339,7 @@ export function renderTrialPanel(
   buyNowUrl: string,
   licensingDocUrl: string,
   version: string,
-  subscriptions = '',
+  subscriptions: string | undefined | null,
   customStyles?: CustomTrialPanelStyles,
 ): void {
   registerCustomComponents(customStyles);
@@ -348,7 +349,7 @@ export function renderTrialPanel(
   trialPanelTrigger.setAttribute(attributeNames.buyNow, buyNowUrl);
   trialPanelTrigger.setAttribute(attributeNames.licensingDoc, licensingDocUrl);
   trialPanelTrigger.setAttribute(attributeNames.version, version);
-  trialPanelTrigger.setAttribute(attributeNames.subscriptions, subscriptions);
+  trialPanelTrigger.setAttribute(attributeNames.subscriptions, subscriptions ?? '');
 
   document.body.appendChild(trialPanelTrigger);
 }

packages/devextreme/js/__internal/core/license/trial_panel.client.ts

@@ -9,7 +9,7 @@ export function showTrialPanel(
   buyNowUrl: string,
   licensingDocUrl: string,
   version: string,
-  subscriptions?: string,
+  subscriptions?: string | null,
   customStyles?: CustomTrialPanelStyles,
 ): void {
   if (isClient()) {