nodejs
diff --git a/Collapse file
‎test/parallel/test-eslint-crypto-check.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-eslint-crypto-check.js
+8Lines changed: 8 additions & 0 deletions b/Collapse file
‎test/parallel/test-eslint-crypto-check.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-eslint-crypto-check.js
+8Lines changed: 8 additions & 0 deletions
diff --git a/Collapse file
‎tools/eslint-rules/crypto-check.js‎
Copy file name to clipboardExpand all lines: tools/eslint-rules/crypto-check.js
+16Lines changed: 16 additions & 0 deletions b/Collapse file
‎tools/eslint-rules/crypto-check.js‎
Copy file name to clipboardExpand all lines: tools/eslint-rules/crypto-check.js
+16Lines changed: 16 additions & 0 deletions
@@ -22,6 +22,14 @@ new RuleTester().run('crypto-check', rule, {
     `
   ],
   invalid: [
+    {
+      code: 'require("common")\n' +
+            'require("crypto")\n' +
+            'if (!common.hasCrypto) {\n' +
+            '  common.skip("missing crypto");\n' +
+            '}',
+      errors: [{ message }]
+    },
     {
       code: 'require("common")\n' +
             'require("crypto")',
 
@@ -66,6 +66,22 @@ module.exports = function(context) {
 
   function reportIfMissingCheck() {
     if (hasSkipCall) {
+      // There is a skip, which is good, but verify that the require() calls
+      // in question come after at least one check.
+      if (missingCheckNodes.length > 0) {
+        requireNodes.forEach((requireNode) => {
+          const beforeAllChecks = missingCheckNodes.every((checkNode) => {
+            return requireNode.start < checkNode.start;
+          });
+
+          if (beforeAllChecks) {
+            context.report({
+              node: requireNode,
+              message: msg
+            });
+          }
+        });
+      }
       return;
     }