@flint.fyi/plugin-flint 0.2.1 → 0.5.1

package/lib/rules/invalidCodeLines.test.js

@@ -1,198 +0,0 @@
-import rule from "./invalidCodeLines.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: "",
-        snapshot: \`
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            output: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`
-
-\`,
-        snapshot: \`
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            snapshot: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: "",
-              ~~
-              This code block should be formatted across multiple lines for more readable reports.
-        snapshot: \`
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-        },
-        {
-            code: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`console.log(
-);\`,
-        snapshot: \`console.log(
-);
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            output: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`
-console.log(
-);
-\`,
-        snapshot: \`
-console.log(
-);
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            snapshot: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`console.log(
-              ~~~~~~~~~~~~~
-              This code block should be formatted across multiple lines for more readable reports.
-);\`,
-~~~
-        snapshot: \`console.log(
-);
-~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-        },
-        {
-            code: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`console.log();\`,
-        snapshot: \`console.log();
-~~~~~~~~~~~~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            output: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`
-console.log();
-\`,
-        snapshot: \`
-console.log();
-~~~~~~~~~~~~~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-            snapshot: `
-ruleTester.describe(rule, {
-    valid: ['a', 'a'],
-    invalid: [
-      {
-        code: \`console.log();\`,
-              ~~~~~~~~~~~~~~~~
-              This code block should be formatted across multiple lines for more readable reports.
-        snapshot: \`console.log();
-~~~~~~~~~~~~
-Rule report message.
-\`,
-      }
-    ],
-});
-`,
-        },
-    ],
-    valid: [
-        `
-            ruleTester.describe(rule, {
-                valid: [],
-                invalid: []
-            });
-    `,
-        `
-            ruleTester.describe(rule, {
-                valid: [],
-                invalid: [
-                {
-                    code: \`
-\`,
-                    snapshot: \`
-~
-\`,
-    }
-                ],
-            });
-    `,
-        `
-            ruleTester.describe(rule, {
-                valid: [],
-                invalid: [
-                {
-                    code: \`
-console.log();
-\`,
-                    snapshot: \`
-~~~~~~~~~~~~
-Rule report message.
-\`,
-    }
-                ],
-            });
-    `,
-    ],
-});
-//# sourceMappingURL=invalidCodeLines.test.js.map

package/lib/rules/ruleTester.d.ts

@@ -1,3 +0,0 @@
-import { RuleTester } from "@flint.fyi/rule-tester";
-export declare const ruleTester: RuleTester;
-//# sourceMappingURL=ruleTester.d.ts.map

package/lib/rules/ruleTester.js

@@ -1,8 +0,0 @@
-import { RuleTester } from "@flint.fyi/rule-tester";
-import { describe, it } from "vitest";
-export const ruleTester = new RuleTester({
-    defaults: { fileName: "file.test.ts" },
-    describe,
-    it,
-});
-//# sourceMappingURL=ruleTester.js.map

package/lib/rules/testCaseDuplicates.js

@@ -1,56 +0,0 @@
-import { getTSNodeRange, typescriptLanguage } from "@flint.fyi/ts";
-import { getRuleTesterDescribedCases } from "../getRuleTesterDescribedCases.js";
-export default typescriptLanguage.createRule({
-    about: {
-        description: "Reports test cases that are identical to previous test cases.",
-        id: "testCaseDuplicates",
-        preset: "logical",
-    },
-    messages: {
-        duplicateTest: {
-            primary: "This test code already appeared in a previous test.",
-            secondary: [
-                "When writing tests for lint rules, it's possible to accidentally create deeply identical test cases.",
-                "Doing so provides no added benefit for testing and is unnecessary.",
-            ],
-            suggestions: [
-                "Delete this redundant test case.",
-                "Change a property to make the test case unique.",
-            ],
-        },
-    },
-    setup(context) {
-        function checkTestCases(testCases, sourceFile) {
-            const seen = new Set();
-            for (const testCase of testCases) {
-                const key = JSON.stringify({
-                    code: testCase.code,
-                    fileName: testCase.fileName,
-                    options: testCase.options,
-                });
-                if (seen.has(key)) {
-                    context.report({
-                        message: "duplicateTest",
-                        range: getTSNodeRange(testCase.nodes.case, sourceFile),
-                    });
-                }
-                else {
-                    seen.add(key);
-                }
-            }
-        }
-        return {
-            visitors: {
-                CallExpression(node, { sourceFile }) {
-                    const describedCases = getRuleTesterDescribedCases(node);
-                    if (!describedCases) {
-                        return;
-                    }
-                    checkTestCases(describedCases.invalid, sourceFile);
-                    checkTestCases(describedCases.valid, sourceFile);
-                },
-            },
-        };
-    },
-});
-//# sourceMappingURL=testCaseDuplicates.js.map

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

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

package/lib/rules/testCaseDuplicates.test.js

@@ -1,174 +0,0 @@
-import { ruleTester } from "./ruleTester.js";
-import rule from "./testCaseDuplicates.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: ['a', 'a'],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: ['a', 'a'],
-                                 ~~~
-                                 This test code already appeared in a previous test.
-                    invalid: []
-                });
-            `,
-        },
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        'a',
-                        "a",
-                    ],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        'a',
-                        "a",
-                        ~~~
-                        This test code already appeared in a previous test.
-                    ],
-                    invalid: []
-                });
-            `,
-        },
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        \`a\`,
-                        \`a\`,
-                    ],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        \`a\`,
-                        \`a\`,
-                        ~~~
-                        This test code already appeared in a previous test.
-                    ],
-                    invalid: []
-                });
-            `,
-        },
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a" },
-                        { code: "a" },
-                    ],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a" },
-                        { code: "a" },
-                        ~~~~~~~~~~~~~
-                        This test code already appeared in a previous test.
-                    ],
-                    invalid: []
-                });
-            `,
-        },
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a", fileName: "b.ts" },
-                        { code: "a", fileName: "b.ts" },
-                    ],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a", fileName: "b.ts" },
-                        { code: "a", fileName: "b.ts" },
-                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-                        This test code already appeared in a previous test.
-                    ],
-                    invalid: []
-                });
-            `,
-        },
-        {
-            code: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a", fileName: "b.ts", options: { c: "d" } },
-                        { code: "a", fileName: "b.ts", options: { c: "d" } },
-                    ],
-                    invalid: []
-                });
-            `,
-            snapshot: `
-                ruleTester.describe(rule, {
-                    valid: [
-                        { code: "a", fileName: "b.ts", options: { c: "d" } },
-                        { code: "a", fileName: "b.ts", options: { c: "d" } },
-                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-                        This test code already appeared in a previous test.
-                    ],
-                    invalid: []
-                });
-            `,
-        },
-    ],
-    valid: [
-        `
-            describe(rule, {
-                valid: ['a', 'a'],
-                invalid: []
-            });
-        `,
-        `
-            ruleTester.describe(rule, {
-                valid: ['a', 'b'],
-                invalid: []
-            });
-        `,
-        `
-            ruleTester.describe(rule, {
-                valid: [
-                    { code: \`a\`, fileName: "b.ts" },
-                    { code: \`a\`, fileName: "c.ts" },
-                ],
-                invalid: []
-            });
-        `,
-        `
-            ruleTester.describe(rule, {
-                valid: [
-                    { code: \`a\`, fileName: "b.ts", options: { c: "d" } },
-                    { code: \`a\`, fileName: "b.ts", options: { c: "e" } },
-                ],
-                invalid: []
-            });
-        `,
-        `
-            ruleTester.describe(rule, {
-                valid: [
-                    { code: \`a\`, fileName: "b.ts", options: { c: "d" } },
-                    { code: \`a\`, fileName: "c.ts", options: { c: "d" } },
-                ],
-                invalid: []
-            });
-        `,
-    ],
-});
-//# sourceMappingURL=testCaseDuplicates.test.js.map

package/lib/tsAstToLiteral.d.ts

@@ -1,5 +0,0 @@
-import ts from "typescript";
-export declare function tsAstToLiteral(node: ts.ArrayLiteralExpression): unknown[];
-export declare function tsAstToLiteral(node: ts.ObjectLiteralExpression): object;
-export declare function tsAstToLiteral(node: ts.Node): unknown;
-//# sourceMappingURL=tsAstToLiteral.d.ts.map

package/lib/tsAstToLiteral.js

@@ -1,38 +0,0 @@
-// Fork of ts-ast-to-literal pending:
-// https://github.com/dword-design/ts-ast-to-literal/issues/89
-// Changing from the switch to manual ifs is due to:
-// https://github.com/Microsoft/TypeScript/issues/56275
-import ts from "typescript";
-export function tsAstToLiteral(node) {
-    switch (node.kind) {
-        case ts.SyntaxKind.FalseKeyword:
-            return false;
-        case ts.SyntaxKind.NullKeyword:
-            return null;
-        case ts.SyntaxKind.TrueKeyword:
-            return true;
-    }
-    if (ts.isArrayLiteralExpression(node)) {
-        return node.elements
-            .filter((element) => element.kind !== ts.SyntaxKind.SpreadElement)
-            .map((element) => tsAstToLiteral(element));
-    }
-    if (ts.isNumericLiteral(node)) {
-        return parseFloat(node.text);
-    }
-    if (ts.isObjectLiteralExpression(node)) {
-        return Object.fromEntries(node.properties
-            .filter((property) => ts.isPropertyAssignment(property) &&
-            (property.name.kind === ts.SyntaxKind.Identifier ||
-                property.name.kind === ts.SyntaxKind.StringLiteral))
-            .map((property) => [
-            property.name.escapedText || property.name.text,
-            tsAstToLiteral(property.initializer),
-        ]));
-    }
-    if (ts.isStringLiteral(node)) {
-        return node.text;
-    }
-    return undefined;
-}
-//# sourceMappingURL=tsAstToLiteral.js.map

package/lib/types.d.ts

@@ -1,18 +0,0 @@
-import type { InvalidTestCase, TestCase } from "@flint.fyi/rule-tester";
-import * as ts from "typescript";
-export interface ParsedTestCase extends TestCase {
-    nodes: ParsedTestCaseNodes;
-}
-export interface ParsedTestCaseInvalid extends InvalidTestCase {
-    nodes: ParsedTestCaseNodesInvalid;
-}
-export interface ParsedTestCaseNodes {
-    case: ts.Node;
-    code: ts.StringLiteralLike;
-    fileName?: ts.StringLiteralLike;
-    options?: ts.ObjectLiteralExpression;
-}
-export interface ParsedTestCaseNodesInvalid extends ParsedTestCaseNodes {
-    snapshot: ts.Node;
-}
-//# sourceMappingURL=types.d.ts.map

package/lib/types.js

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=types.js.map