tools: auto fix custom eslint rule for crypto-check.js

Review comments + updates test cases Refs : #16636
nodejs · Feb 15, 2018 · 1018b5b · 1018b5b
1 parent 2ecf517
commit 1018b5b
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 20 deletions.
diff --git a/test/parallel/test-eslint-crypto-check.js b/test/parallel/test-eslint-crypto-check.js
@@ -12,32 +12,35 @@ new RuleTester().run('crypto-check', rule, {
   valid: [
     'foo',
     'crypto',
-    `if (!common.hasCrypto) {
-      common.skip();
+    `
+    if (!common.hasCrypto) {
+      common.skip("missing crypto");
     }
     require("crypto");
     `
   ],
   invalid: [
     {
-      code: 'require("crypto")',
+      code: 'require("common")\n' +
+            'require("crypto")',
       errors: [{ message }],
-      output:
-      `if (!common.hasCrypto) {
-        common.skip("missing crypto");
-      }
-      require("crypto");
-      `
+      output: 'require("common")\n' +
+              'if (!common.hasCrypto) {' +
+              ' common.skip("missing crypto");' +
+              '}\n' +
+              'require("crypto")'
     },
     {
-      code: 'if (common.foo) {} require("crypto")',
+      code: 'require("common")\n' + 
+            'if (common.foo) {}\n' +
+            'require("crypto")',
       errors: [{ message }],
-      output:
-      `if (!common.hasCrypto) {
-        common.skip("missing crypto");
-      }
-      require("crypto");
-      `
+      output: 'require("common")\n' +
+              'if (!common.hasCrypto) {' +
+              ' common.skip("missing crypto");' +
+              '}\n' +
+              'if (common.foo) {}\n' +
+              'require("crypto")'
     }
   ]
 });
diff --git a/tools/eslint-rules/crypto-check.js b/tools/eslint-rules/crypto-check.js
@@ -87,9 +87,9 @@ module.exports = function(context) {
           if (commonModuleNode) {
             return fixer.insertTextAfter(
               commonModuleNode,
-              `\nif (!common.hasCrypto) {
-                common.skip("missing crypto");
-              }`
+              '\nif (!common.hasCrypto) {' +
+              ' common.skip("missing crypto");' +
+              '}'
             );
           }
         }

diff --git a/tools/eslint-rules/rules-utils.js b/tools/eslint-rules/rules-utils.js
@@ -12,7 +12,7 @@ module.exports.isRequired = function(node, modules) {
     modules.includes(node.arguments[0].value);
 };
 
-/** 
+/**
 * Return true if common module is required
 * in AST Node under inspection
 */