tools: fix error in custom ESLint rule

Trott · MylesBorins · commit 910fa50e0eba · 2017-07-30T23:11:11.000-05:00
Fix previously-unnoticed typo in `required-modules.js`. Backport-PR-URL: #14360 Refs: #13758 (comment) PR-URL: #13758 Reviewed-By: Teddy Katz <teddy.katz@gmail.com> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
diff --git a/tools/eslint-rules/required-modules.js b/tools/eslint-rules/required-modules.js
@@ -75,7 +75,7 @@ module.exports = function(context) {
       if (foundModules.length < requiredModules.length) {
         var missingModules = requiredModules.filter(
           function(module) {
-            return foundModules.indexOf(module === -1);
+            return foundModules.indexOf(module) === -1;
           }
         );
         missingModules.forEach(function(moduleName) {

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ module.exports = function(context) {`
`75`	`75`	`if (foundModules.length < requiredModules.length) {`
`76`	`76`	`var missingModules = requiredModules.filter(`
`77`	`77`	`function(module) {`
`78`		`- return foundModules.indexOf(module === -1);`
	`78`	`+ return foundModules.indexOf(module) === -1;`
`79`	`79`	`}`
`80`	`80`	`);`
`81`	`81`	`missingModules.forEach(function(moduleName) {`