mirror of
https://github.com/nodejs/node.git
synced 2024-12-01 16:10:02 +01:00
870ae72227
Add a custom eslint rule to check for `common.skipIfEslintMissing()` to allow tests to run from source tarballs that do not include eslint. Fix up rule tests that were failing the new check. Refs: https://github.com/nodejs/node/issues/20336 PR-URL: https://github.com/nodejs/node/pull/20372 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
62 lines
1.7 KiB
JavaScript
62 lines
1.7 KiB
JavaScript
/**
|
|
* @fileoverview Check that common.skipIfInspectorDisabled is used if
|
|
* the inspector module is required.
|
|
* @author Daniel Bevenius <daniel.bevenius@gmail.com>
|
|
*/
|
|
'use strict';
|
|
|
|
const utils = require('./rules-utils.js');
|
|
|
|
//------------------------------------------------------------------------------
|
|
// Rule Definition
|
|
//------------------------------------------------------------------------------
|
|
const msg = 'Please add a skipIfInspectorDisabled() call to allow this ' +
|
|
'test to be skipped when Node is built \'--without-inspector\'.';
|
|
|
|
module.exports = function(context) {
|
|
const missingCheckNodes = [];
|
|
var commonModuleNode = null;
|
|
var hasInspectorCheck = false;
|
|
|
|
function testInspectorUsage(context, node) {
|
|
if (utils.isRequired(node, ['inspector'])) {
|
|
missingCheckNodes.push(node);
|
|
}
|
|
|
|
if (utils.isCommonModule(node)) {
|
|
commonModuleNode = node;
|
|
}
|
|
}
|
|
|
|
function checkMemberExpression(context, node) {
|
|
if (utils.usesCommonProperty(node, ['skipIfInspectorDisabled'])) {
|
|
hasInspectorCheck = true;
|
|
}
|
|
}
|
|
|
|
function reportIfMissing(context) {
|
|
if (!hasInspectorCheck) {
|
|
missingCheckNodes.forEach((node) => {
|
|
context.report({
|
|
node,
|
|
message: msg,
|
|
fix: (fixer) => {
|
|
if (commonModuleNode) {
|
|
return fixer.insertTextAfter(
|
|
commonModuleNode,
|
|
'\ncommon.skipIfInspectorDisabled();'
|
|
);
|
|
}
|
|
}
|
|
});
|
|
});
|
|
}
|
|
}
|
|
|
|
return {
|
|
'CallExpression': (node) => testInspectorUsage(context, node),
|
|
'MemberExpression': (node) => checkMemberExpression(context, node),
|
|
'Program:exit': (node) => reportIfMissing(context, node)
|
|
};
|
|
};
|