eslint-plugin-sonarjs
Version:
SonarJS rules for ESLint
117 lines (116 loc) • 4.45 kB
JavaScript
/*
* SonarQube JavaScript Plugin
* Copyright (C) 2011-2025 SonarSource SA
* mailto:info AT sonarsource DOT com
*
* This program is free software; you can redistribute it and/or
* modify it under the terms of the Sonar Source-Available License Version 1, as published by SonarSource SA.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
* See the Sonar Source-Available License for more details.
*
* You should have received a copy of the Sonar Source-Available License
* along with this program; if not, see https://sonarsource.com/license/ssal/
*/
// https://sonarsource.github.io/rspec/#/rspec/S3415/javascript
Object.defineProperty(exports, "__esModule", { value: true });
exports.rule = void 0;
const index_js_1 = require("../helpers/index.js");
const meta_js_1 = require("./meta.js");
const ASSERT_FUNCTIONS = [
'equal',
'notEqual',
'strictEqual',
'notStrictEqual',
'deepEqual',
'notDeepEqual',
'closeTo',
'approximately',
];
exports.rule = {
meta: (0, index_js_1.generateMeta)(meta_js_1.meta, { hasSuggestions: true }, true),
create(context) {
const testCases = [];
return {
CallExpression(node) {
if (index_js_1.Mocha.isTestCase(node)) {
testCases.push(node);
return;
}
if (testCases.length > 0) {
checkInvertedArguments(node, context);
}
},
'CallExpression:exit': (node) => {
if (index_js_1.Mocha.isTestCase(node)) {
testCases.pop();
}
},
};
},
};
function checkInvertedArguments(node, context) {
const args = extractAssertionsArguments(node);
if (args) {
const [actual, expected, format] = args;
if ((0, index_js_1.isLiteral)(actual) && !(0, index_js_1.isLiteral)(expected)) {
(0, index_js_1.report)(context, {
node: expected,
message: `Swap these 2 arguments so they are in the correct order: ${format}.`,
suggest: [
{
desc: 'Swap arguments',
fix: fixer => [
fixer.replaceText(actual, context.sourceCode.getText(expected)),
fixer.replaceText(expected, context.sourceCode.getText(actual)),
],
},
],
}, [(0, index_js_1.toSecondaryLocation)(actual, 'Other argument to swap.')]);
}
}
}
function extractAssertionsArguments(node) {
return extractAssertArguments(node) ?? extractExpectArguments(node) ?? extractFailArguments(node);
}
function extractAssertArguments(node) {
if ((0, index_js_1.isMethodCall)(node) && node.arguments.length > 1) {
const { callee: { object, property }, arguments: [actual, expected], } = node;
if ((0, index_js_1.isIdentifier)(object, 'assert') && (0, index_js_1.isIdentifier)(property, ...ASSERT_FUNCTIONS)) {
return [actual, expected, `${object.name}.${property.name}(actual, expected)`];
}
}
return null;
}
function extractExpectArguments(node) {
if (node.callee.type !== 'MemberExpression') {
return null;
}
let { object, property } = node.callee;
if (!(0, index_js_1.isIdentifier)(property, 'equal', 'eql', 'closeTo')) {
return null;
}
while (object.type === 'MemberExpression') {
object = object.object;
}
if (object.type === 'CallExpression' && (0, index_js_1.isIdentifier)(object.callee, 'expect')) {
return [
object.arguments[0],
node.arguments[0],
`${object.callee.name}(actual).to.${property.name}(expected)`,
];
}
return null;
}
function extractFailArguments(node) {
if ((0, index_js_1.isMethodCall)(node) && node.arguments.length > 1) {
const { callee: { object, property }, arguments: [actual, expected], } = node;
if ((0, index_js_1.isIdentifier)(object, 'assert', 'expect', 'should') && (0, index_js_1.isIdentifier)(property, 'fail')) {
return [actual, expected, `${object.name}.${property.name}(actual, expected)`];
}
}
return null;
}
;