chore: migrate to eslint@9

Related to bpmn-io/internal-docs#1042
bpmn-io · Oct 11, 2024 · 93a1bc5 · 93a1bc5
1 parent 36987fb
commit 93a1bc5
Show file tree

Hide file tree

Showing 15 changed files with 18 additions and 60 deletions.
diff --git a/.eslintrc b/.eslintrc
diff --git a/eslint.config.mjs b/eslint.config.mjs
@@ -34,10 +34,14 @@ export default [
   }),
   {
     rules: {
-      'max-len': [ 2, { 'code': 90 } ],
-      'no-restricted-imports': [ 2, {
+      'max-len': [ 'error', { 'code': 90 } ],
+      'no-restricted-imports': [ 'error', {
         'patterns': [ 'dmn-js/src', 'dmn-js-*/src' ]
-      } ]
+      } ],
+      'react/display-name': 'off',
+      'react/no-deprecated': 'off',
+      'react/jsx-key': 'off', // TODO(@barmac): reenable and fix problems
+      'react/no-unknown-property': 'off',
     }
   }
 ];
diff --git a/packages/dmn-js-boxed-expression/test/.eslintrc b/packages/dmn-js-boxed-expression/test/.eslintrc
diff --git a/packages/dmn-js-decision-table/test/.eslintrc b/packages/dmn-js-decision-table/test/.eslintrc
diff --git a/packages/dmn-js-decision-table/test/spec/features/copy-cut-paste/DescriptorUtilSpec.js b/packages/dmn-js-decision-table/test/spec/features/copy-cut-paste/DescriptorUtilSpec.js
@@ -177,7 +177,7 @@ describe('DescriptorUtil', function() {
     }));
 
 
-    it('should create Row descriptor', inject(function(elementRegistry) {
+    it('should create Col descriptor', inject(function(elementRegistry) {
 
       // given
       const col = elementRegistry.get('input1');

diff --git a/packages/dmn-js-decision-table/test/spec/features/modeling/ModelingSpec.js b/packages/dmn-js-decision-table/test/spec/features/modeling/ModelingSpec.js
@@ -153,7 +153,7 @@ describe('Modeling', function() {
       }));
 
 
-      it('should undo', inject(function(modeling, commandStack) {
+      it('should redo', inject(function(modeling, commandStack) {
 
         // given
         var col = modeling.addCol({ type: 'dmn:OutputClause' }, 3);
@@ -250,7 +250,7 @@ describe('Modeling', function() {
       }));
 
 
-      it('should undo', inject(function(modeling, sheet, commandStack) {
+      it('should redo', inject(function(modeling, sheet, commandStack) {
 
         // given
         var table = sheet.getRoot();
@@ -301,7 +301,7 @@ describe('Modeling', function() {
       }));
 
 
-      it('should undo', inject(function(modeling, sheet, commandStack) {
+      it('should redo', inject(function(modeling, sheet, commandStack) {
 
         // given
         var table = sheet.getRoot();
@@ -356,7 +356,7 @@ describe('Modeling', function() {
       }));
 
 
-      it('should undo', inject(function(modeling, sheet, commandStack) {
+      it('should redo', inject(function(modeling, sheet, commandStack) {
 
         // given
         var table = sheet.getRoot();
@@ -407,7 +407,7 @@ describe('Modeling', function() {
       }));
 
 
-      it('should undo', inject(function(modeling, sheet, commandStack) {
+      it('should redo', inject(function(modeling, sheet, commandStack) {
 
         // given
         var table = sheet.getRoot();

diff --git a/packages/dmn-js-decision-table/test/spec/features/modeling/behavior/IdChangeBehaviorSpec.js b/packages/dmn-js-decision-table/test/spec/features/modeling/behavior/IdChangeBehaviorSpec.js
@@ -15,7 +15,7 @@ describe('IdChangeBehavior', function() {
   }));
 
 
-  it('should update IDs on decision ID change', inject(function(modeling, sheet) {
+  it('should requirements refs on decision ID change', inject(function(modeling, sheet) {
 
     // given
     const root = sheet.getRoot(),
@@ -48,7 +48,7 @@ describe('IdChangeBehavior', function() {
   }));
 
 
-  it('should update IDs on decision ID change', inject(function(modeling, sheet) {
+  it('should association refs on decision ID change', inject(function(modeling, sheet) {
 
     // given
     const root = sheet.getRoot(),

diff --git a/packages/dmn-js-decision-table/test/spec/features/simple-string-edit/SimpleStringEditSpec.js b/packages/dmn-js-decision-table/test/spec/features/simple-string-edit/SimpleStringEditSpec.js
@@ -169,7 +169,7 @@ describe('simple string edit', function() {
     }));
 
 
-    it('should override invalid custom value', inject(function(elementRegistry) {
+    it('should override invalid custom value (negate)', inject(function(elementRegistry) {
 
       // given
       const inputEntry3 = elementRegistry.get('inputEntry3');

diff --git a/packages/dmn-js-drd/test/.eslintrc b/packages/dmn-js-drd/test/.eslintrc
diff --git a/packages/dmn-js-drd/test/spec/features/rules/DrdRulesSpec.js b/packages/dmn-js-drd/test/spec/features/rules/DrdRulesSpec.js
@@ -232,7 +232,7 @@ describe('features/rules', function() {
   });
 
 
-  describe('move', function() {
+  describe('move decision', function() {
 
     it('decision -> definitions', inject(
       function(drdRules, elementRegistry) {

diff --git a/packages/dmn-js-literal-expression/test/.eslintrc b/packages/dmn-js-literal-expression/test/.eslintrc
diff --git a/packages/dmn-js-shared/test/.eslintrc b/packages/dmn-js-shared/test/.eslintrc
diff --git a/packages/dmn-js-shared/test/spec/components/InputSelectSpec.js b/packages/dmn-js-shared/test/spec/components/InputSelectSpec.js
@@ -373,7 +373,7 @@ describe('components/InputSelect', function() {
           });
 
 
-          it('should hide options on ENTER', function() {
+          it('should hide options on ESCAPE', function() {
 
             // given
             const renderedTree = renderIntoDocument(

diff --git a/packages/dmn-js/tasks/.eslintrc b/packages/dmn-js/tasks/.eslintrc
diff --git a/packages/dmn-js/test/.eslintrc b/packages/dmn-js/test/.eslintrc