Update eslint config for e2e tests

Author: anna-shakhova

e2e/testcafe-devextreme/docker/run-testcafe.js

@@ -30,6 +30,7 @@ const matrix = [
 ];
 (async() => {
 
+    // eslint-disable-next-line no-undef
     const parsedArgs = parseArgs(process.argv);
     const componentFolderName = parsedArgs.componentFolder;
     let testParts = matrix;
@@ -45,7 +46,7 @@ const matrix = [
 
     // eslint-disable-next-line no-restricted-syntax
     for (const { name, ...args } of testParts) {
-        // eslint-disable-next-line no-console
+        // eslint-disable-next-line no-console,no-undef
         console.log(`Started test: ${name}`);
 
         const startupParams = Object.entries(args).map(([key, value]) => `--${key}=${value}`);

e2e/testcafe-devextreme/eslint.config.mjs

@@ -9,6 +9,9 @@ import js from '@eslint/js';
 import { FlatCompat as FlatCompatibility } from '@eslint/eslintrc';
 import stylistic from '@stylistic/eslint-plugin';
 import { changeRulesToStylistic } from 'eslint-migration-utils';
+import spellCheckConfig from 'eslint-config-devextreme/spell-check';
+import typescriptConfig from 'eslint-config-devextreme/typescript';
+import testcafeConfig from 'eslint-config-devextreme/testcafe';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
@@ -24,8 +27,8 @@ export default [
       'node_modules/**',
     ],
   },
-  ...compatibility.extends('devextreme/spell-check'),
-  ...compatibility.extends('devextreme/testcafe'),
+  ...spellCheckConfig,
+  ...testcafeConfig,
   {
     plugins: {
       'no-only-tests': noOnlyTests,
@@ -128,7 +131,6 @@ export default [
       'space-before-function-paren': ['error', 'never'],
       'space-in-parens': 'error',
       'space-infix-ops': 'error',
-      'space-unary-ops': 'error',
       '@stylistic/space-infix-ops': 'error',
       'space-unary-ops': 'error',
       'spaced-comment': ['error', 'always', {
@@ -155,7 +157,7 @@ export default [
       'import/no-duplicates': 2,
     }
   },
-  ...compatibility.extends('devextreme/typescript').map(config => {
+  ...typescriptConfig.map(config => {
     const newConfig = {
       ...config,
       files: ['**/*.ts?(x)'],
@@ -216,10 +218,11 @@ export default [
       'no-await-in-loop': 'off',
       '@stylistic/no-extra-parens': 'off',
       '@typescript-eslint/require-await': 'off',
-      '@typescript-eslint/no-base-to-string': 'off'
+      '@typescript-eslint/no-base-to-string': 'off',
+      'require-await': 'off',
     },
   },
-  ...compatibility.extends('devextreme/typescript').map(config => {
+  ...typescriptConfig.map(config => {
     const newConfig = {
       ...config,
       files: ['**/*.d.ts'],

e2e/testcafe-devextreme/helpers/domUtils.ts

@@ -35,7 +35,7 @@ export const getStyleAttribute = ClientFunction((selector) => {
 export const setStyleAttribute = ClientFunction((selector, styleValue) => {
   const element = selector();
 
-  const styles = element.getAttribute('style') || '';
+  const styles = element.getAttribute('style') ?? '';
   const updatedStyles = `${styles} ${styleValue}`;
 
   element.setAttribute('style', updatedStyles);
@@ -44,7 +44,7 @@ export const setStyleAttribute = ClientFunction((selector, styleValue) => {
 export const setClassAttribute = ClientFunction((selector, styleValue) => {
   const element = selector();
 
-  const styles = element.getAttribute('class') || '';
+  const styles = element.getAttribute('class') ?? '';
   const updatedClasses = `${styles} ${styleValue}`;
 
   element.setAttribute('class', updatedClasses);
@@ -53,7 +53,7 @@ export const setClassAttribute = ClientFunction((selector, styleValue) => {
 export const removeClassAttribute = ClientFunction((selector, styleValue) => {
   const element = selector();
 
-  const styles = element.getAttribute('class') || '';
+  const styles = element.getAttribute('class') ?? '';
   const updatedClasses = `${styles.replace(styleValue, '')}`;
 
   element.setAttribute('class', updatedClasses);

e2e/testcafe-devextreme/helpers/shadowDom/shadowDomExtension.js

@@ -1,4 +1,4 @@
-
+/* eslint-disable no-undef */
 function getRoot() {
     return document.querySelector('#parentContainer').shadowRoot;
 }

e2e/testcafe-devextreme/runner.ts

@@ -274,7 +274,7 @@ async function main() {
 
               await t.hover('html');
 
-              const [width, height] = meta?.browserSize || DEFAULT_BROWSER_SIZE;
+              const [width, height] = meta?.browserSize ?? DEFAULT_BROWSER_SIZE;
               await t.resizeWindow(width, height);
             } else {
               await loadAxeCore(t);

e2e/testcafe-devextreme/tests/common/pivotGrid/contextMenu.ts

@@ -38,7 +38,7 @@ test.meta({ themes: [Themes.genericLight] })('ContextMenu width should be adjust
     visible: true,
   },
   onContextMenuPreparing(e) {
-    if (e.field && e.field.dataField === 'amount') {
+    if (e.field?.dataField === 'amount') {
       const menuItems = [] as any;
 
       e.items.push({ text: 'Summary Type', items: menuItems });

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

@@ -15,7 +15,7 @@ const getVisibleColumns = (dataGrid: DataGrid): Promise<string[]> => {
   return ClientFunction(
     () => (getInstance() as any)
       .getVisibleColumns()
-      .map((column: any) => column.dataField || column.name),
+      .map((column: any) => column.dataField ?? column.name),
     { dependencies: { getInstance } },
   )();
 };