@flint.fyi/ts 0.14.6 → 0.18.0

package/lib/rules/negativeZeroComparisons.test.js

@@ -1,209 +0,0 @@
-import rule from "./negativeZeroComparisons.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-if (value === -0) {}
-`,
-            snapshot: `
-if (value === -0) {}
-    ~~~~~~~~~~~~
-    Comparisons with -0 using === do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-if (Object.is(value, -0)) {}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (-0 === value) {}
-`,
-            snapshot: `
-if (-0 === value) {}
-    ~~~~~~~~~~~~
-    Comparisons with -0 using === do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-if (Object.is(-0, value)) {}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (value == -0) {}
-`,
-            snapshot: `
-if (value == -0) {}
-    ~~~~~~~~~~~
-    Comparisons with -0 using == do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-if (Object.is(value, -0)) {}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (value !== -0) {}
-`,
-            snapshot: `
-if (value !== -0) {}
-    ~~~~~~~~~~~~
-    Comparisons with -0 using !== do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-if (!Object.is(value, -0)) {}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (value != -0) {}
-`,
-            snapshot: `
-if (value != -0) {}
-    ~~~~~~~~~~~
-    Comparisons with -0 using != do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-if (!Object.is(value, -0)) {}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-if (value < -0) {}
-`,
-            snapshot: `
-if (value < -0) {}
-    ~~~~~~~~~~
-    Comparisons with -0 using < do not distinguish between -0 and +0.
-`,
-        },
-        {
-            code: `
-if (value <= -0) {}
-`,
-            snapshot: `
-if (value <= -0) {}
-    ~~~~~~~~~~~
-    Comparisons with -0 using <= do not distinguish between -0 and +0.
-`,
-        },
-        {
-            code: `
-if (value > -0) {}
-`,
-            snapshot: `
-if (value > -0) {}
-    ~~~~~~~~~~
-    Comparisons with -0 using > do not distinguish between -0 and +0.
-`,
-        },
-        {
-            code: `
-if (value >= -0) {}
-`,
-            snapshot: `
-if (value >= -0) {}
-    ~~~~~~~~~~~
-    Comparisons with -0 using >= do not distinguish between -0 and +0.
-`,
-        },
-        {
-            code: `
-const result = value === -0 ? "negative zero" : "other";
-`,
-            snapshot: `
-const result = value === -0 ? "negative zero" : "other";
-               ~~~~~~~~~~~~
-               Comparisons with -0 using === do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-const result = Object.is(value, -0) ? "negative zero" : "other";
-`,
-                },
-            ],
-        },
-        {
-            code: `
-while (value !== -0) {
-	value++;
-}
-`,
-            snapshot: `
-while (value !== -0) {
-       ~~~~~~~~~~~~
-       Comparisons with -0 using !== do not distinguish between -0 and +0.
-	value++;
-}
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-while (!Object.is(value, -0)) {
-	value++;
-}
-`,
-                },
-            ],
-        },
-        {
-            code: `
-const isNegativeZero = value === -0;
-`,
-            snapshot: `
-const isNegativeZero = value === -0;
-                       ~~~~~~~~~~~~
-                       Comparisons with -0 using === do not distinguish between -0 and +0.
-`,
-            suggestions: [
-                {
-                    id: "useObjectIs",
-                    updated: `
-const isNegativeZero = Object.is(value, -0);
-`,
-                },
-            ],
-        },
-    ],
-    valid: [
-        `if (value === 0) {}`,
-        `if (value === -1) {}`,
-        `if (value === 1) {}`,
-        `if (Object.is(value, -0)) {}`,
-        `if (Object.is(-0, value)) {}`,
-        `const result = -0;`,
-        `const value = -0 + 1;`,
-        `const value = -0 * 2;`,
-        `const value = Math.abs(-0);`,
-        `const value = [-0, 0, 1];`,
-        `function test() { return -0; }`,
-    ],
-});

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

@@ -1 +0,0 @@
-export {};

package/lib/rules/newExpressions.test.js

@@ -1,79 +0,0 @@
-import rule from "./newExpressions.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-new MyClass();
-`,
-            snapshot: `
-new MyClass();
-~~~
-Constructors should only be called when their return value is used.
-`,
-        },
-        {
-            code: `
-function process() {
-    new EventEmitter();
-}
-`,
-            snapshot: `
-function process() {
-    new EventEmitter();
-    ~~~
-    Constructors should only be called when their return value is used.
-}
-`,
-        },
-        {
-            code: `
-if (condition) {
-    new Logger();
-}
-`,
-            snapshot: `
-if (condition) {
-    new Logger();
-    ~~~
-    Constructors should only be called when their return value is used.
-}
-`,
-        },
-        {
-            code: `
-new DatabaseConnection(), processData();
-`,
-            snapshot: `
-new DatabaseConnection(), processData();
-~~~
-Constructors should only be called when their return value is used.
-`,
-        },
-        {
-            code: `
-someFunction(), new Queue();
-`,
-            snapshot: `
-someFunction(), new Queue();
-                ~~~
-                Constructors should only be called when their return value is used.
-`,
-        },
-    ],
-    valid: [
-        `const instance = new MyClass();`,
-        `let value = new MyClass();`,
-        `const result = condition ? new MyClass() : new OtherClass();`,
-        `return new MyClass();`,
-        `throw new Error("message");`,
-        `function create() { return new MyClass(); }`,
-        `const array = [new MyClass(), new OtherClass()];`,
-        `const object = { value: new MyClass() };`,
-        `myFunction(new MyClass());`,
-        `new MyClass().method();`,
-        `const value = new MyClass() || fallback;`,
-        `const value = new MyClass() && other;`,
-        `const value = new MyClass() ?? fallback;`,
-    ],
-});

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

@@ -1 +0,0 @@
-export {};

package/lib/rules/newNativeNonConstructors.test.js

@@ -1,94 +0,0 @@
-import rule from "./newNativeNonConstructors.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-new Symbol("description");
-`,
-            output: `
-Symbol("description");
-`,
-            snapshot: `
-new Symbol("description");
-~~~
-Symbol cannot be called with \`new\`.
-`,
-        },
-        {
-            code: `
-new BigInt(42);
-`,
-            output: `
-BigInt(42);
-`,
-            snapshot: `
-new BigInt(42);
-~~~
-BigInt cannot be called with \`new\`.
-`,
-        },
-        {
-            code: `
-const value = new Symbol();
-`,
-            output: `
-const value = Symbol();
-`,
-            snapshot: `
-const value = new Symbol();
-              ~~~
-              Symbol cannot be called with \`new\`.
-`,
-        },
-        {
-            code: `
-function create() {
-    return new BigInt(100);
-}
-`,
-            output: `
-function create() {
-    return BigInt(100);
-}
-`,
-            snapshot: `
-function create() {
-    return new BigInt(100);
-           ~~~
-           BigInt cannot be called with \`new\`.
-}
-`,
-        },
-    ],
-    valid: [
-        `Symbol("description");`,
-        `BigInt(42);`,
-        `const value = Symbol();`,
-        `const number = BigInt(100);`,
-        `function create() { return Symbol("key"); }`,
-        `const array = [Symbol("a"), Symbol("b")];`,
-        `new String("text");`,
-        `new Number(42);`,
-        `new Boolean(true);`,
-        `new Object();`,
-        `new Array();`,
-        `new Date();`,
-        `new Error("message");`,
-        `new Map();`,
-        `new Set();`,
-        `new Promise((resolve) => resolve());`,
-        `new WeakMap();`,
-        `new WeakSet();`,
-        `
-			class BigInt {}
-			new BigInt();
-			export {}
-		`,
-        `
-			class Symbol {}
-			new Symbol();
-			export {}
-		`,
-    ],
-});

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

@@ -1 +0,0 @@
-export {};

package/lib/rules/nonOctalDecimalEscapes.test.js

@@ -1,69 +0,0 @@
-import rule from "./nonOctalDecimalEscapes.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-"\\8"
-`,
-            snapshot: `
-"\\8"
- ~~
- Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-        {
-            code: `
-"\\9"
-`,
-            snapshot: `
-"\\9"
- ~~
- Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-        {
-            code: `
-"w\\8less"
-`,
-            snapshot: `
-"w\\8less"
-  ~~
-  Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-        {
-            code: `
-"December 1\\9"
-`,
-            snapshot: `
-"December 1\\9"
-           ~~
-           Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-        {
-            code: `
-"Don't use \\8 and \\9 escapes."
-`,
-            snapshot: `
-"Don't use \\8 and \\9 escapes."
-           ~~
-           Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-                  ~~
-                  Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-        {
-            code: `
-"\\0\\8"
-`,
-            snapshot: `
-"\\0\\8"
-   ~~
-   Non-octal decimal escape sequences (\\8 and \\9) should not be used in string literals.
-`,
-        },
-    ],
-    valid: ['"8"', '"9"', '"w8less"', '"December 19"', '"\\\\8"', '"\\0\\u0038"'],
-});

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

@@ -1 +0,0 @@
-export {};

package/lib/rules/numericLiteralParsing.test.js

@@ -1,99 +0,0 @@
-import rule from "./numericLiteralParsing.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-const binary = parseInt("111110111", 2);
-`,
-            snapshot: `
-const binary = parseInt("111110111", 2);
-               ~~~~~~~~~~~~~~~~~~~~~~~~
-               Use 0b111110111 instead of parseInt with radix 2.
-`,
-        },
-        {
-            code: `
-const octal = parseInt("767", 8);
-`,
-            snapshot: `
-const octal = parseInt("767", 8);
-              ~~~~~~~~~~~~~~~~~~
-              Use 0o767 instead of parseInt with radix 8.
-`,
-        },
-        {
-            code: `
-const hex = parseInt("1F7", 16);
-`,
-            snapshot: `
-const hex = parseInt("1F7", 16);
-            ~~~~~~~~~~~~~~~~~~~
-            Use 0x1F7 instead of parseInt with radix 16.
-`,
-        },
-        {
-            code: `
-const value = Number.parseInt("111110111", 2);
-`,
-            snapshot: `
-const value = Number.parseInt("111110111", 2);
-              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-              Use 0b111110111 instead of parseInt with radix 2.
-`,
-        },
-        {
-            code: `
-const result = Number.parseInt("767", 8);
-`,
-            snapshot: `
-const result = Number.parseInt("767", 8);
-               ~~~~~~~~~~~~~~~~~~~~~~~~~
-               Use 0o767 instead of parseInt with radix 8.
-`,
-        },
-        {
-            code: `
-const num = Number.parseInt("1F7", 16);
-`,
-            snapshot: `
-const num = Number.parseInt("1F7", 16);
-            ~~~~~~~~~~~~~~~~~~~~~~~~~~
-            Use 0x1F7 instead of parseInt with radix 16.
-`,
-        },
-        {
-            code: `
-const value = parseInt(\`111110111\`, 2);
-`,
-            snapshot: `
-const value = parseInt(\`111110111\`, 2);
-              ~~~~~~~~~~~~~~~~~~~~~~~~
-              Use 0b111110111 instead of parseInt with radix 2.
-`,
-        },
-    ],
-    valid: [
-        `const decimal = parseInt("123");`,
-        `const value = parseInt("123", 10);`,
-        `const ternary = parseInt("102", 3);`,
-        `const binary = 0b111110111;`,
-        `const octal = 0o767;`,
-        `const hex = 0x1F7;`,
-        `const dynamic = parseInt(value, 2);`,
-        `const variable = parseInt("111", radix);`,
-        `const single = parseInt("111");`,
-        `Number.parseInt("123");`,
-        `Number.parseInt("123", 10);`,
-        `
-			function parseInt(...values: unknown[]) {}
-			parseInt("1", 2);
-			export {};
-		`,
-        `
-			const Number = { parseInt(...values: unknown[]) {} };
-			Number.parseInt("1", 2);
-			export {};
-		`,
-    ],
-});

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

@@ -1 +0,0 @@
-export {};

package/lib/rules/objectProto.test.js

@@ -1,96 +0,0 @@
-import rule from "./objectProto.js";
-import { ruleTester } from "./ruleTester.js";
-ruleTester.describe(rule, {
-    invalid: [
-        {
-            code: `
-const proto = obj.__proto__;
-`,
-            snapshot: `
-const proto = obj.__proto__;
-                  ~~~~~~~~~
-                  Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-`,
-        },
-        {
-            code: `
-obj.__proto__ = prototype;
-`,
-            snapshot: `
-obj.__proto__ = prototype;
-    ~~~~~~~~~
-    Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-`,
-        },
-        {
-            code: `
-const descriptor = Object.getOwnPropertyDescriptor(obj, "__proto__");
-const proto = obj.__proto__;
-`,
-            snapshot: `
-const descriptor = Object.getOwnPropertyDescriptor(obj, "__proto__");
-const proto = obj.__proto__;
-                  ~~~~~~~~~
-                  Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-`,
-        },
-        {
-            code: `
-const value = obj["__proto__"];
-`,
-            snapshot: `
-const value = obj["__proto__"];
-                  ~~~~~~~~~~~
-                  Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-`,
-        },
-        {
-            code: `
-obj["__proto__"] = prototype;
-`,
-            snapshot: `
-obj["__proto__"] = prototype;
-    ~~~~~~~~~~~
-    Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-`,
-        },
-        {
-            code: `
-if (obj.__proto__ === null) {
-    console.log("No prototype");
-}
-`,
-            snapshot: `
-if (obj.__proto__ === null) {
-        ~~~~~~~~~
-        Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-    console.log("No prototype");
-}
-`,
-        },
-        {
-            code: `
-function getProto(obj: object) {
-    return obj.__proto__;
-}
-`,
-            snapshot: `
-function getProto(obj: object) {
-    return obj.__proto__;
-               ~~~~~~~~~
-               Use Object.getPrototypeOf or Object.setPrototypeOf instead of the deprecated __proto__ property.
-}
-`,
-        },
-    ],
-    valid: [
-        `const proto = Object.getPrototypeOf(obj);`,
-        `Object.setPrototypeOf(obj, prototype);`,
-        `const obj = Object.create(prototype);`,
-        `const descriptor = Object.getOwnPropertyDescriptor(obj, "__proto__");`,
-        `const key = "__proto__";`,
-        `const string = "obj.__proto__";`,
-        `const obj = { "__proto__": null };`,
-        `const obj = { __proto__: null };`,
-    ],
-});

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

@@ -1 +0,0 @@
-export {};