Bump ESLint dependency to 7.19.0

Author: yassin-kammoun-sonarsource

package.json

@@ -24,19 +24,19 @@
     "prepack": "yarn build"
   },
   "peerDependencies": {
-    "eslint": "^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0"
+    "eslint": "^3.0.0 || ^4.0.0 || ^5.0.0 || ^6.0.0 || ^7.0.0"
   },
   "devDependencies": {
-    "@types/eslint": "6.1.3",
-    "@types/estree": "0.0.39",
+    "@types/eslint": "7.2.6",
+    "@types/estree": "0.0.46",
     "@types/jest": "22.2.2",
     "@types/lodash": "4.14.106",
     "@types/minimist": "1.2.0",
     "@types/node": "12.12.6",
     "@typescript-eslint/experimental-utils": "2.6.1",
     "@typescript-eslint/parser": "2.6.1",
     "babel-eslint": "8.2.2",
-    "eslint": "6.6.0",
+    "eslint": "7.19.0",
     "eslint-config-prettier": "2.9.0",
     "eslint-plugin-import": "2.18.2",
     "eslint-plugin-notice": "0.6.7",

ruling/.eslintignore

@@ -0,0 +1 @@
+!**/node_modules/**

ruling/index.ts

@@ -57,6 +57,7 @@ function run() {
     rules: getEslintRules(rules),
     useEslintrc: false,
     allowInlineConfig: false,
+    ignorePath: path.join(__dirname, ".eslintignore"),
   });
 
   const sourcesPath = path.join(__dirname, "javascript-test-sources/src");

tests/rules/prefer-immediate-return.test.ts

@@ -186,6 +186,11 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 3, column: 19, endColumn: 21 }],
+      output: `
+        function let_returned() {
+          return 42;
+        }
+      `,
     },
     {
       code: `
@@ -195,6 +200,11 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 3, column: 21, endColumn: 23 }],
+      output: `
+        function const_returned() {
+          return 42;
+        }
+      `,
     },
     {
       code: `
@@ -205,6 +215,12 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 4, column: 19, endColumn: 21 }],
+      output: `
+        function code_before_declaration() {
+          foo();
+          return 42;
+        }
+      `,
     },
     {
       code: `
@@ -244,6 +260,21 @@ ruleTester.run("prefer-immediate-return", rule, {
         { line: 12, column: 21, endColumn: 26 },
         { line: 15, column: 21, endColumn: 26 },
       ],
+      output: `
+        function different_blocks() {
+          if (foo) {
+            return foo();
+          }
+          
+          try {
+            return foo();
+          } catch (e) {
+            return foo();
+          } finally {
+            return foo();
+          }
+        }
+      `,
     },
     {
       code: `
@@ -258,6 +289,16 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 4, column: 21, endColumn: 26 }],
+      output: `
+        function two_declarations(a) {
+          if (a) {
+            return foo();
+          } else {
+            let x = bar();
+            return x + 42;
+          }
+        }
+      `,
     },
     {
       code: `
@@ -275,6 +316,19 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 3, column: 23, endLine: 11, endColumn: 12 }],
+      output: `
+        function homonymous_is_used() {
+          return {
+            doSomethingElse(p) {
+              var bar = 2;
+              return p + bar;
+            },
+            doSomething() {
+              return this.doSomethingElse(1);
+            },
+          };
+        }
+      `,
     },
     {
       code: `
@@ -290,6 +344,16 @@ ruleTester.run("prefer-immediate-return", rule, {
         }
       `,
       errors: [{ line: 5, column: 25, endColumn: 26 }, { line: 8, column: 25, endColumn: 26 }],
+      output: `
+        function inside_switch(x) {
+          switch (x) {
+            case 1:
+              return 3;
+            default:
+              return 2;
+          }
+        }
+      `,
     },
     {
       code: `
@@ -318,6 +382,13 @@ ruleTester.run("prefer-immediate-return", rule, {
           message: 'Immediately return this expression instead of assigning it to the temporary variable "x".',
         },
       ],
+      output: `
+      function foo() {
+        if (cond) {
+          return 42;
+      }
+      }
+      `,
     },
   ],
 });