@flint.fyi/ts 0.15.0 → 0.18.0

package/lib/rules/exceptionAssignments.test.js

@@ -1,265 +0,0 @@
-import rule from "./exceptionAssignments.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error = new Error("Different error");
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error = new Error("Different error");
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (exception) {
-    exception = null;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (exception) {
-    exception = null;
-    ~~~~~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error++;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error++;
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    ++error;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    ++error;
-      ~~~~~
-      Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error += "additional info";
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error += "additional info";
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error ??= new Error("default error");
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error ??= new Error("default error");
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error &&= false;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error &&= false;
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    error ||= new Error("fallback");
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    error ||= new Error("fallback");
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    function inner() {
-        error = "reassigned in nested function";
-    }
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    function inner() {
-        error = "reassigned in nested function";
-        ~~~~~
-        Exception parameters in catch clauses should not be reassigned.
-    }
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch ({ message }) {
-    message = "reassigned destructured parameter";
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch ({ message }) {
-    message = "reassigned destructured parameter";
-    ~~~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch ({ message, code }) {
-    code = 500;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch ({ message, code }) {
-    code = 500;
-    ~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch ([first, second]) {
-    first = "reassigned array destructured parameter";
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch ([first, second]) {
-    first = "reassigned array destructured parameter";
-    ~~~~~
-    Exception parameters in catch clauses should not be reassigned.
-}
-`,
-        },
-    ],
-    valid: [
-        `try { doSomething(); } catch (error) { console.log(error); }`,
-        `try { doSomething(); } catch (error) { throw error; }`,
-        `try { doSomething(); } catch (error) { const message = error.message; }`,
-        `try { doSomething(); } catch (error) { if (error instanceof TypeError) { handle(); } }`,
-        `try { doSomething(); } catch { handleError(); }`,
-        `const error = new Error("test");`,
-        `
-try {
-    doSomething();
-} catch (error) {
-    const error = "shadowed variable";
-    console.log(error);
-}
-`,
-        `
-try {
-    doSomething();
-} catch (error) {
-    function inner() {
-        const error = "shadowed in function";
-        error = "reassigning shadowed variable is ok";
-    }
-}
-`,
-        `
-let error = "outer";
-try {
-    doSomething();
-} catch (error) {
-    console.log(error);
-}
-error = "reassigning outer is ok";
-`,
-        `try { doSomething(); } catch ({ message }) { console.log(message); }`,
-        `try { doSomething(); } catch ({ message, code }) { if (code === 404) { handle(); } }`,
-        `try { doSomething(); } catch ([first]) { console.log(first); }`,
-    ],
-});
-//# sourceMappingURL=exceptionAssignments.test.js.map

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

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

package/lib/rules/finallyStatementSafety.test.js

@@ -1,307 +0,0 @@
-import rule from "./finallyStatementSafety.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-try {
-    doSomething();
-} finally {
-    return 1;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} finally {
-    return 1;
-    ~~~~~~
-    Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} catch (error) {
-    console.error(error);
-} finally {
-    return;
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} catch (error) {
-    console.error(error);
-} finally {
-    return;
-    ~~~~~~
-    Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} finally {
-    throw new Error("Error");
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} finally {
-    throw new Error("Error");
-    ~~~~~
-    Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-}
-`,
-        },
-        {
-            code: `
-while (condition) {
-    try {
-        doSomething();
-    } finally {
-        break;
-    }
-}
-`,
-            snapshot: `
-while (condition) {
-    try {
-        doSomething();
-    } finally {
-        break;
-        ~~~~~
-        Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-        {
-            code: `
-for (let i = 0; i < 10; i++) {
-    try {
-        doSomething();
-    } finally {
-        continue;
-    }
-}
-`,
-            snapshot: `
-for (let i = 0; i < 10; i++) {
-    try {
-        doSomething();
-    } finally {
-        continue;
-        ~~~~~~~~
-        Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-        {
-            code: `
-function test() {
-    try {
-        doSomething();
-        return "success";
-    } finally {
-        return "override";
-    }
-}
-`,
-            snapshot: `
-function test() {
-    try {
-        doSomething();
-        return "success";
-    } finally {
-        return "override";
-        ~~~~~~
-        Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} finally {
-    if (condition) {
-        return;
-    }
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} finally {
-    if (condition) {
-        return;
-        ~~~~~~
-        Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} finally {
-    switch (value) {
-        case 1:
-            return;
-    }
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} finally {
-    switch (value) {
-        case 1:
-            return;
-            ~~~~~~
-            Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-        {
-            code: `
-try {
-    doSomething();
-} finally {
-    label: {
-        break label;
-    }
-}
-`,
-            snapshot: `
-try {
-    doSomething();
-} finally {
-    label: {
-        break label;
-        ~~~~~
-        Control flow statements in \`finally\` blocks misleadingly override control flow from \`try\`/\`catch\` blocks.
-    }
-}
-`,
-        },
-    ],
-    valid: [
-        `try { doSomething(); } finally { cleanup(); }`,
-        `try { doSomething(); } catch (error) { console.error(error); } finally { cleanup(); }`,
-        `try { return 1; } finally { cleanup(); }`,
-        `try { throw new Error("Error"); } finally { cleanup(); }`,
-        `
-try {
-    doSomething();
-} finally {
-    console.log("cleanup");
-}
-`,
-        `
-function test() {
-    try {
-        return doSomething();
-    } finally {
-        cleanup();
-    }
-}
-`,
-        `
-while (condition) {
-    try {
-        break;
-    } finally {
-        cleanup();
-    }
-}
-`,
-        `
-for (let i = 0; i < 10; i++) {
-    try {
-        continue;
-    } finally {
-        cleanup();
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    if (condition) {
-        cleanup();
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    for (let i = 0; i < 10; i++) {
-        cleanup();
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    while (condition) {
-        cleanup();
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    for (let i = 0; i < 10; i++) {
-        if (i === 5) break;
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    while (condition) {
-        if (shouldExit) break;
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    for (let i = 0; i < 10; i++) {
-        if (i === 0) continue;
-        process(i);
-    }
-}
-`,
-        `
-try {
-    doSomething();
-} finally {
-    try {
-        cleanup();
-    } finally {
-        log();
-    }
-}
-`,
-    ],
-});
-//# sourceMappingURL=finallyStatementSafety.test.js.map

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

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

package/lib/rules/forDirections.test.js

@@ -1,125 +0,0 @@
-import rule from "./forDirections.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-for (let i = 0; i < 10; i--) {}
-`,
-            snapshot: `
-for (let i = 0; i < 10; i--) {}
-                        ~~~
-                        The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 0; index < 10; index--) {}
-`,
-            snapshot: `
-for (let index = 0; index < 10; index--) {}
-                                ~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 10; index >= 0; index++) {}
-`,
-            snapshot: `
-for (let index = 10; index >= 0; index++) {}
-                                 ~~~~~~~
-                                 The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 0; index > 10; index++) {}
-`,
-            snapshot: `
-for (let index = 0; index > 10; index++) {}
-                                ~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 0; 10 > index; index--) {}
-`,
-            snapshot: `
-for (let index = 0; 10 > index; index--) {}
-                                ~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 0; index < 10; index -= 1) {}
-`,
-            snapshot: `
-for (let index = 0; index < 10; index -= 1) {}
-                                ~~~~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 10; index > 0; index += 1) {}
-`,
-            snapshot: `
-for (let index = 10; index > 0; index += 1) {}
-                                ~~~~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-for (let index = 0; index < 10; index += -1) {}
-`,
-            snapshot: `
-for (let index = 0; index < 10; index += -1) {}
-                                ~~~~~~~~~~~
-                                The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-declare context text: string;
-for (let i = 10; i >= 0 && text[i] !== " "; i++) { }
-`,
-            snapshot: `
-declare context text: string;
-for (let i = 10; i >= 0 && text[i] !== " "; i++) { }
-                                            ~~~
-                                            The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-        {
-            code: `
-declare context text: string;
-for (let i = 10; text[i] !== " " && i >= 0; i++) { }
-`,
-            snapshot: `
-declare context text: string;
-for (let i = 10; text[i] !== " " && i >= 0; i++) { }
-                                            ~~~
-                                            The update moves the counter in the wrong direction for this loop condition.
-`,
-        },
-    ],
-    valid: [
-        `for (let i = 0; i < 10; i++) { }`,
-        `for (let index = 0; index < 10; index++) { }`,
-        `for (let index = 10; index >= 0; index--) { }`,
-        `for (let index = 0; 10 > index; index++) { }`,
-        `for (let index = 10; 0 < index; index--) { }`,
-        `for (let index = 10; index >= 0; index += step) { }`,
-        `for (let index = 0; index <= 10; index -= 0) { }`,
-        `for (let index = 0; index < 10; index += 1) { }`,
-        `for (let index = 10; index > 0; index -= 1) { }`,
-        `for (let index = 0; index < 10; index += 2) { }`,
-        `for (let index = 10; index > 0; index -= 2) { }`,
-        `for (let i = 10; i >= 0 && text[i] === "\\"; i--) { }`,
-    ],
-});
-//# sourceMappingURL=forDirections.test.js.map

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

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

package/lib/rules/forInArrays.test.js

@@ -1,25 +0,0 @@
-import rule from "./forInArrays.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-declare const array: string[];
-for (const i in array) {}
-`,
-            snapshot: `
-declare const array: string[];
-for (const i in array) {}
-~~~~~~~~~~~~~~~~~~~~~~
-For-in loops over arrays have surprising behavior that often leads to bugs.
-`,
-        },
-    ],
-    valid: [
-        `
-declare const array: string[];
-for (const i of array) {}
-`,
-    ],
-});
-//# sourceMappingURL=forInArrays.test.js.map