Skip to content

Commit 1bb5dc3

Browse files
committed
Fix: rename variables to improve readability.
1 parent c4bbe25 commit 1bb5dc3

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

lib/rules/no-identical-tests.js

+13-13
Original file line numberDiff line numberDiff line change
@@ -36,33 +36,33 @@ module.exports = {
3636
*compare two test cases despite of properties order.
3737
*@returns {boolean} if eq, return true, else return false.
3838
*/
39-
function eq (ta, tb) {
40-
if (ta.type !== tb.type) {
39+
function eq (testA, testB) {
40+
if (testA.type !== testB.type) {
4141
return false;
4242
}
4343

44-
if (ta.type !== 'ObjectExpression') {
45-
return sourceCode.getText(ta) === sourceCode.getText(tb);
44+
if (testA.type !== 'ObjectExpression') {
45+
return sourceCode.getText(testA) === sourceCode.getText(testB);
4646
}
4747

48-
const pa = ta.properties || [];
49-
const pb = tb.properties || [];
48+
const propertiesA = testA.properties || [];
49+
const propertiesB = testB.properties || [];
5050

5151
// if properties length not eq; return false;
52-
if (pa.length !== pb.length) {
52+
if (propertiesA.length !== propertiesB.length) {
5353
return false;
5454
}
5555

5656
// convert array to object
57-
const paObj = Object.create(null);
58-
pa.forEach(item => {
57+
const propertiesSetA = new Set();
58+
propertiesA.forEach(item => {
5959
const code = sourceCode.getText(item);
60-
paObj[code] = true;
60+
propertiesSetA.add(code);
6161
});
6262

63-
for (let i = 0; i < pb.length; i++) {
64-
const code = sourceCode.getText(pb[i]);
65-
if (!paObj[code]) {
63+
for (let i = 0; i < propertiesB.length; i++) {
64+
const code = sourceCode.getText(propertiesB[i]);
65+
if (!propertiesSetA.has(code)) {
6666
return false;
6767
}
6868
}

0 commit comments

Comments
 (0)