mirror of
https://github.com/joelwmale/webhook-action.git
synced 2024-11-26 01:16:56 +01:00
128 lines
4 KiB
JavaScript
128 lines
4 KiB
JavaScript
|
"use strict";
|
||
|
|
||
|
Object.defineProperty(exports, "__esModule", {
|
||
|
value: true
|
||
|
});
|
||
|
exports.default = void 0;
|
||
|
|
||
|
var _experimentalUtils = require("@typescript-eslint/experimental-utils");
|
||
|
|
||
|
var _utils = require("./utils");
|
||
|
|
||
|
var _default = (0, _utils.createRule)({
|
||
|
name: __filename,
|
||
|
meta: {
|
||
|
docs: {
|
||
|
category: 'Best Practices',
|
||
|
description: 'Avoid using a callback in asynchronous tests',
|
||
|
recommended: 'error',
|
||
|
suggestion: true
|
||
|
},
|
||
|
messages: {
|
||
|
illegalTestCallback: 'Illegal usage of test callback',
|
||
|
suggestWrappingInPromise: 'Wrap in `new Promise({{ callback }} => ...`',
|
||
|
useAwaitInsteadOfCallback: 'Use await instead of callback in async functions'
|
||
|
},
|
||
|
schema: [],
|
||
|
type: 'suggestion'
|
||
|
},
|
||
|
defaultOptions: [],
|
||
|
|
||
|
create(context) {
|
||
|
return {
|
||
|
CallExpression(node) {
|
||
|
if (!(0, _utils.isTestCase)(node) || node.arguments.length !== 2) {
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
const [, callback] = node.arguments;
|
||
|
|
||
|
if (!(0, _utils.isFunction)(callback) || callback.params.length !== 1) {
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
const [argument] = callback.params;
|
||
|
|
||
|
if (argument.type !== _experimentalUtils.AST_NODE_TYPES.Identifier) {
|
||
|
context.report({
|
||
|
node: argument,
|
||
|
messageId: 'illegalTestCallback'
|
||
|
});
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
if (callback.async) {
|
||
|
context.report({
|
||
|
node: argument,
|
||
|
messageId: 'useAwaitInsteadOfCallback'
|
||
|
});
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
context.report({
|
||
|
node: argument,
|
||
|
messageId: 'illegalTestCallback',
|
||
|
suggest: [{
|
||
|
messageId: 'suggestWrappingInPromise',
|
||
|
data: {
|
||
|
callback: argument.name
|
||
|
},
|
||
|
|
||
|
fix(fixer) {
|
||
|
const {
|
||
|
body
|
||
|
} = callback;
|
||
|
/* istanbul ignore if https://github.com/typescript-eslint/typescript-eslint/issues/734 */
|
||
|
|
||
|
if (!body) {
|
||
|
throw new Error(`Unexpected null when attempting to fix ${context.getFilename()} - please file a github issue at https://github.com/jest-community/eslint-plugin-jest`);
|
||
|
}
|
||
|
|
||
|
const sourceCode = context.getSourceCode();
|
||
|
const firstBodyToken = sourceCode.getFirstToken(body);
|
||
|
const lastBodyToken = sourceCode.getLastToken(body);
|
||
|
const tokenBeforeArgument = sourceCode.getTokenBefore(argument);
|
||
|
const tokenAfterArgument = sourceCode.getTokenAfter(argument);
|
||
|
/* istanbul ignore if */
|
||
|
|
||
|
if (!firstBodyToken || !lastBodyToken || !tokenBeforeArgument || !tokenAfterArgument) {
|
||
|
throw new Error(`Unexpected null when attempting to fix ${context.getFilename()} - please file a github issue at https://github.com/jest-community/eslint-plugin-jest`);
|
||
|
}
|
||
|
|
||
|
const argumentInParens = tokenBeforeArgument.value === '(' && tokenAfterArgument.value === ')';
|
||
|
let argumentFix = fixer.replaceText(argument, '()');
|
||
|
|
||
|
if (argumentInParens) {
|
||
|
argumentFix = fixer.remove(argument);
|
||
|
}
|
||
|
|
||
|
let newCallback = argument.name;
|
||
|
|
||
|
if (argumentInParens) {
|
||
|
newCallback = `(${newCallback})`;
|
||
|
}
|
||
|
|
||
|
let beforeReplacement = `new Promise(${newCallback} => `;
|
||
|
let afterReplacement = ')';
|
||
|
let replaceBefore = true;
|
||
|
|
||
|
if (body.type === _experimentalUtils.AST_NODE_TYPES.BlockStatement) {
|
||
|
const keyword = 'return';
|
||
|
beforeReplacement = `${keyword} ${beforeReplacement}{`;
|
||
|
afterReplacement += '}';
|
||
|
replaceBefore = false;
|
||
|
}
|
||
|
|
||
|
return [argumentFix, replaceBefore ? fixer.insertTextBefore(firstBodyToken, beforeReplacement) : fixer.insertTextAfter(firstBodyToken, beforeReplacement), fixer.insertTextAfter(lastBodyToken, afterReplacement)];
|
||
|
}
|
||
|
|
||
|
}]
|
||
|
});
|
||
|
}
|
||
|
|
||
|
};
|
||
|
}
|
||
|
|
||
|
});
|
||
|
|
||
|
exports.default = _default;
|