@flint.fyi/ts 0.15.0 → 0.18.1

package/lib/rules/constructorReturns.test.js

@@ -1,97 +0,0 @@
-import rule from "./constructorReturns.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-class Example {
-    constructor() {
-        return {};
-    }
-}
-`,
-            snapshot: `
-class Example {
-    constructor() {
-        return {};
-        ~~~~~~~~~~
-        Returning a value from a constructor function overrides the newly created instance.
-    }
-}
-`,
-        },
-        {
-            code: `
-class Example {
-    constructor() {
-        return { value: 1 };
-    }
-}
-`,
-            snapshot: `
-class Example {
-    constructor() {
-        return { value: 1 };
-        ~~~~~~~~~~~~~~~~~~~~
-        Returning a value from a constructor function overrides the newly created instance.
-    }
-}
-`,
-        },
-        {
-            code: `
-class Example {
-    constructor() {
-        if (condition) {
-            return new OtherClass();
-        }
-    }
-}
-`,
-            snapshot: `
-class Example {
-    constructor() {
-        if (condition) {
-            return new OtherClass();
-            ~~~~~~~~~~~~~~~~~~~~~~~~
-            Returning a value from a constructor function overrides the newly created instance.
-        }
-    }
-}
-`,
-        },
-        {
-            code: `
-class Example {
-    constructor(value: number) {
-        if (value < 0) {
-            return null;
-        }
-        this.value = value;
-    }
-}
-`,
-            snapshot: `
-class Example {
-    constructor(value: number) {
-        if (value < 0) {
-            return null;
-            ~~~~~~~~~~~~
-            Returning a value from a constructor function overrides the newly created instance.
-        }
-        this.value = value;
-    }
-}
-`,
-        },
-    ],
-    valid: [
-        `class Example { constructor() {} }`,
-        `class Example { constructor() { this.value = 1; } }`,
-        `class Example { constructor() { return; } }`,
-        `class Example { constructor() { if (condition) { return; } this.value = 1; } }`,
-        `class Example { constructor() { const factory = () => { return {}; }; } }`,
-        `class Example { constructor() { function helper() { return 1; } } }`,
-    ],
-});
-//# sourceMappingURL=constructorReturns.test.js.map

package/lib/rules/debuggerStatements.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=debuggerStatements.test.d.ts.map

package/lib/rules/debuggerStatements.test.js

@@ -1,81 +0,0 @@
-import rule from "./debuggerStatements.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-debugger;
-`,
-            snapshot: `
-debugger;
-~~~~~~~~~
-Debugger statements should not be used in production code.
-`,
-            suggestions: [
-                {
-                    id: "removeDebugger",
-                    updated: `
-
-`,
-                },
-            ],
-        },
-        {
-            code: `
-function test() {
-	debugger;
-}
-`,
-            snapshot: `
-function test() {
-	debugger;
-	~~~~~~~~~
-	Debugger statements should not be used in production code.
-}
-`,
-            suggestions: [
-                {
-                    id: "removeDebugger",
-                    updated: `
-function test() {
-	
-}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (condition) {
-	debugger;
-}
-`,
-            snapshot: `
-if (condition) {
-	debugger;
-	~~~~~~~~~
-	Debugger statements should not be used in production code.
-}
-`,
-            suggestions: [
-                {
-                    id: "removeDebugger",
-                    updated: `
-if (condition) {
-	
-}
-`,
-                },
-            ],
-        },
-    ],
-    valid: [
-        `console.log("debugging");`,
-        `function test() { console.log("test"); }`,
-        `for (let i = 0; i < 10; i++) { break; }`,
-        `for (let i = 0; i < 10; i++) { continue; }`,
-        `function test() { return; }`,
-        `throw new Error("test");`,
-    ],
-});
-//# sourceMappingURL=debuggerStatements.test.js.map

package/lib/rules/defaultCaseLast.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=defaultCaseLast.test.d.ts.map

package/lib/rules/defaultCaseLast.test.js

@@ -1,196 +0,0 @@
-import rule from "./defaultCaseLast.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-switch (value) {
-    default:
-        break;
-    case 1:
-        break;
-}
-`,
-            snapshot: `
-switch (value) {
-    default:
-    ~~~~~~~
-    Default clauses in switch statements should be last.
-        break;
-    case 1:
-        break;
-}
-`,
-        },
-        {
-            code: `
-switch (value) {
-    case 1:
-        break;
-    default:
-        break;
-    case 2:
-        break;
-}
-`,
-            snapshot: `
-switch (value) {
-    case 1:
-        break;
-    default:
-    ~~~~~~~
-    Default clauses in switch statements should be last.
-        break;
-    case 2:
-        break;
-}
-`,
-        },
-        {
-            code: `
-switch (value) {
-    case 1:
-    case 2:
-        break;
-    default:
-        break;
-    case 3:
-        break;
-}
-`,
-            snapshot: `
-switch (value) {
-    case 1:
-    case 2:
-        break;
-    default:
-    ~~~~~~~
-    Default clauses in switch statements should be last.
-        break;
-    case 3:
-        break;
-}
-`,
-        },
-        {
-            code: `
-switch (value) {
-    default:
-        console.log("default");
-        break;
-    case 1:
-        console.log("one");
-        break;
-}
-`,
-            snapshot: `
-switch (value) {
-    default:
-    ~~~~~~~
-    Default clauses in switch statements should be last.
-        console.log("default");
-        break;
-    case 1:
-        console.log("one");
-        break;
-}
-`,
-        },
-        {
-            code: `
-const result = (value: number) => {
-    switch (value) {
-        case 1:
-            return "one";
-        default:
-            return "default";
-        case 2:
-            return "two";
-    }
-};
-`,
-            snapshot: `
-const result = (value: number) => {
-    switch (value) {
-        case 1:
-            return "one";
-        default:
-        ~~~~~~~
-        Default clauses in switch statements should be last.
-            return "default";
-        case 2:
-            return "two";
-    }
-};
-`,
-        },
-        {
-            code: `
-switch (value) {
-    default:
-    case 1:
-        break;
-}
-`,
-            snapshot: `
-switch (value) {
-    default:
-    ~~~~~~~
-    Default clauses in switch statements should be last.
-    case 1:
-        break;
-}
-`,
-        },
-    ],
-    valid: [
-        `switch (value) { case 1: break; default: break; }`,
-        `switch (value) { case 1: break; case 2: break; default: break; }`,
-        `switch (value) { case 1: break; case 2: break; }`,
-        `switch (value) { default: break; }`,
-        `switch (value) { case 1: case 2: break; default: break; }`,
-        `
-switch (value) {
-    case 1:
-        break;
-    default:
-        break;
-}
-`,
-        `
-switch (value) {
-    case 1:
-        console.log("one");
-        break;
-    case 2:
-        console.log("two");
-        break;
-    default:
-        console.log("default");
-        break;
-}
-`,
-        `
-const result = (value: number) => {
-    switch (value) {
-        case 1:
-            return "one";
-        case 2:
-            return "two";
-        default:
-            return "default";
-    }
-};
-`,
-        `
-switch (value) {
-    case 1:
-    case 2:
-        break;
-    default:
-        break;
-}
-`,
-    ],
-});
-//# sourceMappingURL=defaultCaseLast.test.js.map

package/lib/rules/duplicateArguments.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=duplicateArguments.test.d.ts.map

package/lib/rules/duplicateArguments.test.js

@@ -1,140 +0,0 @@
-import rule from "./duplicateArguments.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-function test(value, value) {
-    return value;
-}
-`,
-            snapshot: `
-function test(value, value) {
-                     ~~~~~
-                     This parameter overrides the previous parameter of the same name.
-    return value;
-}
-`,
-        },
-        {
-            code: `
-function test(first, second, first) {
-    return first + second;
-}
-`,
-            snapshot: `
-function test(first, second, first) {
-                             ~~~~~
-                             This parameter overrides the previous parameter of the same name.
-    return first + second;
-}
-`,
-        },
-        {
-            code: `
-const arrow = (value, value) => value;
-`,
-            snapshot: `
-const arrow = (value, value) => value;
-                      ~~~~~
-                      This parameter overrides the previous parameter of the same name.
-`,
-        },
-        {
-            code: `
-const func = function (value, value) {
-    return value;
-};
-`,
-            snapshot: `
-const func = function (value, value) {
-                              ~~~~~
-                              This parameter overrides the previous parameter of the same name.
-    return value;
-};
-`,
-        },
-        {
-            code: `
-class Example {
-    method(value, value) {
-        return value;
-    }
-}
-`,
-            snapshot: `
-class Example {
-    method(value, value) {
-                  ~~~~~
-                  This parameter overrides the previous parameter of the same name.
-        return value;
-    }
-}
-`,
-        },
-        {
-            code: `
-class Example {
-    constructor(value, value) {
-        this.value = value;
-    }
-}
-`,
-            snapshot: `
-class Example {
-    constructor(value, value) {
-                       ~~~~~
-                       This parameter overrides the previous parameter of the same name.
-        this.value = value;
-    }
-}
-`,
-        },
-        {
-            code: `
-function test(first, second, third, second) {
-    return first + second + third;
-}
-`,
-            snapshot: `
-function test(first, second, third, second) {
-                                    ~~~~~~
-                                    This parameter overrides the previous parameter of the same name.
-    return first + second + third;
-}
-`,
-        },
-        {
-            code: `
-function test(value, value, value) {
-    return value;
-}
-`,
-            snapshot: `
-function test(value, value, value) {
-                     ~~~~~
-                     This parameter overrides the previous parameter of the same name.
-                            ~~~~~
-                            This parameter overrides the previous parameter of the same name.
-    return value;
-}
-`,
-        },
-    ],
-    valid: [
-        `function test(value) { return value; }`,
-        `function test(first, second) { return first + second; }`,
-        `const arrow = (value) => value;`,
-        `const arrow = (first, second) => first + second;`,
-        `const func = function (value) { return value; };`,
-        `const func = function (first, second) { return first + second; };`,
-        `class Example { method(value) { return value; } }`,
-        `class Example { method(first, second) { return first + second; } }`,
-        `class Example { constructor(value) { this.value = value; } }`,
-        `class Example { constructor(first, second) { this.first = first; this.second = second; } }`,
-        `function test({ value }) { return value; }`,
-        `function test({ first, second }) { return first + second; }`,
-        `function test([first, second]) { return first + second; }`,
-    ],
-});
-//# sourceMappingURL=duplicateArguments.test.js.map

package/lib/rules/elseIfDuplicates.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=elseIfDuplicates.test.d.ts.map

package/lib/rules/elseIfDuplicates.test.js

@@ -1,177 +0,0 @@
-import rule from "./elseIfDuplicates.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-if (isSomething(value)) {
-    doFirst();
-} else if (isSomething(value)) {
-    doSecond();
-}
-`,
-            snapshot: `
-if (isSomething(value)) {
-    doFirst();
-} else if (isSomething(value)) {
-           ~~~~~~~~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    doSecond();
-}
-`,
-        },
-        {
-            code: `
-if (isSomething /* ... */ (value)) {
-    doFirst();
-} else if (isSomething(value /* ... */)) {
-    doSecond();
-}
-`,
-            snapshot: `
-if (isSomething /* ... */ (value)) {
-    doFirst();
-} else if (isSomething(value /* ... */)) {
-           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    doSecond();
-}
-`,
-        },
-        {
-            code: `
-if (first) {
-    handleFirst();
-} else if (second) {
-    handleSecond();
-} else if (third && fourth) {
-    handleThird();
-} else if (third && fourth) {
-    handleFourth();
-}
-`,
-            snapshot: `
-if (first) {
-    handleFirst();
-} else if (second) {
-    handleSecond();
-} else if (third && fourth) {
-    handleThird();
-} else if (third && fourth) {
-           ~~~~~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    handleFourth();
-}
-`,
-        },
-        {
-            code: `
-if (count === 1) {
-    handleOne();
-} else if (count === 2) {
-    handleTwo();
-} else if (count === 3) {
-    handleThree();
-} else if (count === 2) {
-    handleFour();
-} else if (count === 5) {
-    handleFive();
-}
-`,
-            snapshot: `
-if (count === 1) {
-    handleOne();
-} else if (count === 2) {
-    handleTwo();
-} else if (count === 3) {
-    handleThree();
-} else if (count === 2) {
-           ~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    handleFour();
-} else if (count === 5) {
-    handleFive();
-}
-`,
-        },
-        {
-            code: `
-if (value === "first") {
-    process();
-} else if (value === "second") {
-    process();
-} else if (value === "first") {
-    process();
-}
-`,
-            snapshot: `
-if (value === "first") {
-    process();
-} else if (value === "second") {
-    process();
-} else if (value === "first") {
-           ~~~~~~~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    process();
-}
-`,
-        },
-        {
-            code: `
-if (isValid && isActive) {
-    execute();
-} else if (isPending) {
-    wait();
-} else if (isValid && isActive) {
-    execute();
-}
-`,
-            snapshot: `
-if (isValid && isActive) {
-    execute();
-} else if (isPending) {
-    wait();
-} else if (isValid && isActive) {
-           ~~~~~~~~~~~~~~~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    execute();
-}
-`,
-        },
-        {
-            code: `
-if (value) {
-    first();
-} else if (value) {
-    second();
-} else if (value) {
-    third();
-}
-`,
-            snapshot: `
-if (value) {
-    first();
-} else if (value) {
-           ~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    second();
-} else if (value) {
-           ~~~~~
-           This condition is identical to a previous condition in the if-else-if chain.
-    third();
-}
-`,
-        },
-    ],
-    valid: [
-        `if (isSomething(value)) { doFirst(); } else if (isSomethingElse(value)) { doSecond(); }`,
-        `if (first) { handleFirst(); } else if (second) { handleSecond(); } else if (third && fourth) { handleThird(); } else if (third && fifth) { handleFourth(); }`,
-        `if (count === 1) { handleOne(); } else if (count === 2) { handleTwo(); } else if (count === 3) { handleThree(); } else if (count === 4) { handleFour(); }`,
-        `if (value === "first") { process(); } else if (value === "second") { process(); } else if (value === "third") { process(); }`,
-        `if (isValid) { execute(); } else if (isPending) { wait(); } else if (isInvalid) { reject(); }`,
-        `if (value) { process(); }`,
-        `if (first) { doFirst(); } else { doSecond(); }`,
-        `if (first) { doFirst(); } else if (second) { doSecond(); } else { doDefault(); }`,
-    ],
-});
-//# sourceMappingURL=elseIfDuplicates.test.js.map

package/lib/rules/emptyBlocks.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=emptyBlocks.test.d.ts.map