tools: improve valid-typeof lint rule

Require that `typeof` comparisons be to string literals. PR-URL: #37924 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Trott

.eslintrc.js

@@ -287,7 +287,7 @@ module.exports ={
 'template-curly-spacing': 'error',
 'unicode-bom': 'error',
 'use-isnan': 'error',
-'valid-typeof': 'error',
+'valid-typeof': ['error',{requireStringLiterals: true}],
 
 // Custom rules from eslint-plugin-node-core
 'node-core/no-unescaped-regexp-dot': 'error',

lib/internal/encoding.js

@@ -63,6 +63,7 @@ function validateDecoder(obj){
 }
 
 functionvalidateArgument(prop,expected,propName,expectedName){
+// eslint-disable-next-line valid-typeof
 if(typeofprop!==expected)
 thrownewERR_INVALID_ARG_TYPE(propName,expectedName,prop);
 }

test/parallel/test-assert-calltracker-report.js

@@ -11,22 +11,16 @@ function foo(){}
 constcallsfoo=tracker.calls(foo,1);
 
 // Ensures that foo was added to the callChecks array.
-if(tracker.report()[0].operator!=='foo'){
-process.exit(1);
-}
+assert.strictEqual(tracker.report()[0].operator,'foo');
 
 callsfoo();
 
 // Ensures that foo was removed from the callChecks array after being called the
 // expected number of times.
-if(typeoftracker.report()[0]==='undefined'){
-process.exit(1);
-}
+assert.strictEqual(typeoftracker.report()[0],'undefined');
 
 callsfoo();
 
 // Ensures that foo was added back to the callChecks array after being called
 // more than the expected number of times.
-if(tracker.report()[0].operator!=='foo'){
-process.exit(1);
-}
+assert.strictEqual(tracker.report()[0].operator,'foo');