2020-08-26 01:57:08 +02:00
|
|
|
"use strict";
|
|
|
|
|
|
|
|
Object.defineProperty(exports, "__esModule", {
|
|
|
|
value: true
|
|
|
|
});
|
|
|
|
exports.default = void 0;
|
2022-11-10 11:43:16 +01:00
|
|
|
var _utils = require("@typescript-eslint/utils");
|
|
|
|
var _utils2 = require("./utils");
|
2020-08-26 01:57:08 +02:00
|
|
|
/*
|
|
|
|
* This implementation is ported from from eslint-plugin-jasmine.
|
|
|
|
* MIT license, Tom Vincent.
|
|
|
|
*/
|
|
|
|
|
|
|
|
/**
|
|
|
|
* Async assertions might be called in Promise
|
|
|
|
* methods like `Promise.x(expect1)` or `Promise.x([expect1, expect2])`.
|
|
|
|
* If that's the case, Promise node have to be awaited or returned.
|
|
|
|
*
|
|
|
|
* @Returns CallExpressionNode
|
|
|
|
*/
|
|
|
|
const getPromiseCallExpressionNode = node => {
|
2022-11-10 11:43:16 +01:00
|
|
|
if (node.type === _utils.AST_NODE_TYPES.ArrayExpression && node.parent && node.parent.type === _utils.AST_NODE_TYPES.CallExpression) {
|
2020-08-26 01:57:08 +02:00
|
|
|
node = node.parent;
|
|
|
|
}
|
2022-11-10 11:43:16 +01:00
|
|
|
if (node.type === _utils.AST_NODE_TYPES.CallExpression && node.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(node.callee.object, 'Promise') && node.parent) {
|
2020-08-26 01:57:08 +02:00
|
|
|
return node;
|
|
|
|
}
|
|
|
|
return null;
|
|
|
|
};
|
2022-11-10 11:43:16 +01:00
|
|
|
const findPromiseCallExpressionNode = node => {
|
|
|
|
var _node$parent;
|
|
|
|
return (_node$parent = node.parent) !== null && _node$parent !== void 0 && _node$parent.parent && [_utils.AST_NODE_TYPES.CallExpression, _utils.AST_NODE_TYPES.ArrayExpression].includes(node.parent.type) ? getPromiseCallExpressionNode(node.parent) : null;
|
|
|
|
};
|
2020-08-26 01:57:08 +02:00
|
|
|
const getParentIfThenified = node => {
|
2022-11-10 11:43:16 +01:00
|
|
|
var _node$parent2;
|
|
|
|
const grandParentNode = (_node$parent2 = node.parent) === null || _node$parent2 === void 0 ? void 0 : _node$parent2.parent;
|
|
|
|
if (grandParentNode && grandParentNode.type === _utils.AST_NODE_TYPES.CallExpression && grandParentNode.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(grandParentNode.callee.property) && ['then', 'catch'].includes((0, _utils2.getAccessorValue)(grandParentNode.callee.property)) && grandParentNode.parent) {
|
2020-08-26 01:57:08 +02:00
|
|
|
// Just in case `then`s are chained look one above.
|
|
|
|
return getParentIfThenified(grandParentNode);
|
|
|
|
}
|
|
|
|
return node;
|
|
|
|
};
|
2022-11-10 11:43:16 +01:00
|
|
|
const isAcceptableReturnNode = (node, allowReturn) => {
|
|
|
|
if (allowReturn && node.type === _utils.AST_NODE_TYPES.ReturnStatement) {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
if (node.type === _utils.AST_NODE_TYPES.ConditionalExpression && node.parent) {
|
|
|
|
return isAcceptableReturnNode(node.parent, allowReturn);
|
|
|
|
}
|
|
|
|
return [_utils.AST_NODE_TYPES.ArrowFunctionExpression, _utils.AST_NODE_TYPES.AwaitExpression].includes(node.type);
|
|
|
|
};
|
2020-08-26 01:57:08 +02:00
|
|
|
const promiseArrayExceptionKey = ({
|
|
|
|
start,
|
|
|
|
end
|
|
|
|
}) => `${start.line}:${start.column}-${end.line}:${end.column}`;
|
2022-11-10 11:43:16 +01:00
|
|
|
const defaultAsyncMatchers = ['toReject', 'toResolve'];
|
|
|
|
var _default = (0, _utils2.createRule)({
|
2020-08-26 01:57:08 +02:00
|
|
|
name: __filename,
|
|
|
|
meta: {
|
|
|
|
docs: {
|
|
|
|
category: 'Best Practices',
|
|
|
|
description: 'Enforce valid `expect()` usage',
|
|
|
|
recommended: 'error'
|
|
|
|
},
|
|
|
|
messages: {
|
|
|
|
tooManyArgs: 'Expect takes at most {{ amount }} argument{{ s }}.',
|
|
|
|
notEnoughArgs: 'Expect requires at least {{ amount }} argument{{ s }}.',
|
2022-11-10 11:43:16 +01:00
|
|
|
modifierUnknown: 'Expect has an unknown modifier.',
|
2020-08-26 01:57:08 +02:00
|
|
|
matcherNotFound: 'Expect must have a corresponding matcher call.',
|
|
|
|
matcherNotCalled: 'Matchers must be called to assert.',
|
|
|
|
asyncMustBeAwaited: 'Async assertions must be awaited{{ orReturned }}.',
|
|
|
|
promisesWithAsyncAssertionsMustBeAwaited: 'Promises which return async assertions must be awaited{{ orReturned }}.'
|
|
|
|
},
|
|
|
|
type: 'suggestion',
|
|
|
|
schema: [{
|
|
|
|
type: 'object',
|
|
|
|
properties: {
|
|
|
|
alwaysAwait: {
|
|
|
|
type: 'boolean',
|
|
|
|
default: false
|
|
|
|
},
|
2022-11-10 11:43:16 +01:00
|
|
|
asyncMatchers: {
|
|
|
|
type: 'array',
|
|
|
|
items: {
|
|
|
|
type: 'string'
|
|
|
|
}
|
|
|
|
},
|
2020-08-26 01:57:08 +02:00
|
|
|
minArgs: {
|
|
|
|
type: 'number',
|
|
|
|
minimum: 1
|
|
|
|
},
|
|
|
|
maxArgs: {
|
|
|
|
type: 'number',
|
|
|
|
minimum: 1
|
|
|
|
}
|
|
|
|
},
|
|
|
|
additionalProperties: false
|
|
|
|
}]
|
|
|
|
},
|
|
|
|
defaultOptions: [{
|
|
|
|
alwaysAwait: false,
|
2022-11-10 11:43:16 +01:00
|
|
|
asyncMatchers: defaultAsyncMatchers,
|
2020-08-26 01:57:08 +02:00
|
|
|
minArgs: 1,
|
|
|
|
maxArgs: 1
|
|
|
|
}],
|
|
|
|
create(context, [{
|
|
|
|
alwaysAwait,
|
2022-11-10 11:43:16 +01:00
|
|
|
asyncMatchers = defaultAsyncMatchers,
|
2020-08-26 01:57:08 +02:00
|
|
|
minArgs = 1,
|
|
|
|
maxArgs = 1
|
|
|
|
}]) {
|
|
|
|
// Context state
|
|
|
|
const arrayExceptions = new Set();
|
|
|
|
const pushPromiseArrayException = loc => arrayExceptions.add(promiseArrayExceptionKey(loc));
|
2022-11-10 11:43:16 +01:00
|
|
|
|
2020-08-26 01:57:08 +02:00
|
|
|
/**
|
|
|
|
* Promise method that accepts an array of promises,
|
|
|
|
* (eg. Promise.all), will throw warnings for the each
|
|
|
|
* unawaited or non-returned promise. To avoid throwing
|
|
|
|
* multiple warnings, we check if there is a warning in
|
|
|
|
* the given location.
|
|
|
|
*/
|
|
|
|
const promiseArrayExceptionExists = loc => arrayExceptions.has(promiseArrayExceptionKey(loc));
|
2022-11-10 11:43:16 +01:00
|
|
|
const findTopMostMemberExpression = node => {
|
|
|
|
let topMostMemberExpression = node;
|
|
|
|
let {
|
|
|
|
parent
|
|
|
|
} = node;
|
|
|
|
while (parent) {
|
|
|
|
if (parent.type !== _utils.AST_NODE_TYPES.MemberExpression) {
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
topMostMemberExpression = parent;
|
|
|
|
parent = parent.parent;
|
|
|
|
}
|
|
|
|
return topMostMemberExpression;
|
|
|
|
};
|
2020-08-26 01:57:08 +02:00
|
|
|
return {
|
|
|
|
CallExpression(node) {
|
2022-11-10 11:43:16 +01:00
|
|
|
const jestFnCall = (0, _utils2.parseJestFnCallWithReason)(node, context);
|
|
|
|
if (typeof jestFnCall === 'string') {
|
|
|
|
var _node$parent3;
|
|
|
|
const reportingNode = ((_node$parent3 = node.parent) === null || _node$parent3 === void 0 ? void 0 : _node$parent3.type) === _utils.AST_NODE_TYPES.MemberExpression ? findTopMostMemberExpression(node.parent).property : node;
|
|
|
|
if (jestFnCall === 'matcher-not-found') {
|
|
|
|
context.report({
|
|
|
|
messageId: 'matcherNotFound',
|
|
|
|
node: reportingNode
|
|
|
|
});
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
if (jestFnCall === 'matcher-not-called') {
|
|
|
|
context.report({
|
|
|
|
messageId: (0, _utils2.isSupportedAccessor)(reportingNode) && _utils2.ModifierName.hasOwnProperty((0, _utils2.getAccessorValue)(reportingNode)) ? 'matcherNotFound' : 'matcherNotCalled',
|
|
|
|
node: reportingNode
|
|
|
|
});
|
|
|
|
}
|
|
|
|
if (jestFnCall === 'modifier-unknown') {
|
|
|
|
context.report({
|
|
|
|
messageId: 'modifierUnknown',
|
|
|
|
node: reportingNode
|
|
|
|
});
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
return;
|
|
|
|
} else if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
|
2020-08-26 01:57:08 +02:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
const {
|
2022-11-10 11:43:16 +01:00
|
|
|
parent: expect
|
|
|
|
} = jestFnCall.head.node;
|
|
|
|
if ((expect === null || expect === void 0 ? void 0 : expect.type) !== _utils.AST_NODE_TYPES.CallExpression) {
|
|
|
|
return;
|
|
|
|
}
|
2020-08-26 01:57:08 +02:00
|
|
|
if (expect.arguments.length < minArgs) {
|
2022-11-10 11:43:16 +01:00
|
|
|
const expectLength = (0, _utils2.getAccessorValue)(jestFnCall.head.node).length;
|
2020-08-26 01:57:08 +02:00
|
|
|
const loc = {
|
|
|
|
start: {
|
2022-11-10 11:43:16 +01:00
|
|
|
column: expect.loc.start.column + expectLength,
|
|
|
|
line: expect.loc.start.line
|
2020-08-26 01:57:08 +02:00
|
|
|
},
|
|
|
|
end: {
|
2022-11-10 11:43:16 +01:00
|
|
|
column: expect.loc.start.column + expectLength + 1,
|
|
|
|
line: expect.loc.start.line
|
2020-08-26 01:57:08 +02:00
|
|
|
}
|
|
|
|
};
|
|
|
|
context.report({
|
|
|
|
messageId: 'notEnoughArgs',
|
|
|
|
data: {
|
|
|
|
amount: minArgs,
|
|
|
|
s: minArgs === 1 ? '' : 's'
|
|
|
|
},
|
2022-11-10 11:43:16 +01:00
|
|
|
node: expect,
|
2020-08-26 01:57:08 +02:00
|
|
|
loc
|
|
|
|
});
|
|
|
|
}
|
|
|
|
if (expect.arguments.length > maxArgs) {
|
|
|
|
const {
|
|
|
|
start
|
|
|
|
} = expect.arguments[maxArgs].loc;
|
|
|
|
const {
|
|
|
|
end
|
2022-11-10 11:43:16 +01:00
|
|
|
} = expect.arguments[expect.arguments.length - 1].loc;
|
2020-08-26 01:57:08 +02:00
|
|
|
const loc = {
|
|
|
|
start,
|
|
|
|
end: {
|
|
|
|
column: end.column - 1,
|
|
|
|
line: end.line
|
|
|
|
}
|
|
|
|
};
|
|
|
|
context.report({
|
|
|
|
messageId: 'tooManyArgs',
|
|
|
|
data: {
|
|
|
|
amount: maxArgs,
|
|
|
|
s: maxArgs === 1 ? '' : 's'
|
|
|
|
},
|
2022-11-10 11:43:16 +01:00
|
|
|
node: expect,
|
2020-08-26 01:57:08 +02:00
|
|
|
loc
|
|
|
|
});
|
|
|
|
}
|
2022-11-10 11:43:16 +01:00
|
|
|
const {
|
|
|
|
matcher
|
|
|
|
} = jestFnCall;
|
|
|
|
const parentNode = matcher.parent.parent;
|
|
|
|
const shouldBeAwaited = jestFnCall.modifiers.some(nod => (0, _utils2.getAccessorValue)(nod) !== 'not') || asyncMatchers.includes((0, _utils2.getAccessorValue)(matcher));
|
|
|
|
if (!(parentNode !== null && parentNode !== void 0 && parentNode.parent) || !shouldBeAwaited) {
|
2020-08-26 01:57:08 +02:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
/**
|
|
|
|
* If parent node is an array expression, we'll report the warning,
|
|
|
|
* for the array object, not for each individual assertion.
|
|
|
|
*/
|
2022-11-10 11:43:16 +01:00
|
|
|
const isParentArrayExpression = parentNode.parent.type === _utils.AST_NODE_TYPES.ArrayExpression;
|
2020-08-26 01:57:08 +02:00
|
|
|
const orReturned = alwaysAwait ? '' : ' or returned';
|
|
|
|
/**
|
|
|
|
* An async assertion can be chained with `then` or `catch` statements.
|
|
|
|
* In that case our target CallExpression node is the one with
|
|
|
|
* the last `then` or `catch` statement.
|
|
|
|
*/
|
|
|
|
const targetNode = getParentIfThenified(parentNode);
|
|
|
|
const finalNode = findPromiseCallExpressionNode(targetNode) || targetNode;
|
2022-11-10 11:43:16 +01:00
|
|
|
if (finalNode.parent &&
|
|
|
|
// If node is not awaited or returned
|
|
|
|
!isAcceptableReturnNode(finalNode.parent, !alwaysAwait) &&
|
|
|
|
// if we didn't warn user already
|
2020-08-26 01:57:08 +02:00
|
|
|
!promiseArrayExceptionExists(finalNode.loc)) {
|
|
|
|
context.report({
|
|
|
|
loc: finalNode.loc,
|
|
|
|
data: {
|
|
|
|
orReturned
|
|
|
|
},
|
|
|
|
messageId: finalNode === targetNode ? 'asyncMustBeAwaited' : 'promisesWithAsyncAssertionsMustBeAwaited',
|
|
|
|
node
|
|
|
|
});
|
|
|
|
if (isParentArrayExpression) {
|
|
|
|
pushPromiseArrayException(finalNode.loc);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
};
|
|
|
|
}
|
|
|
|
});
|
|
|
|
exports.default = _default;
|