@definitelytyped/eslint-plugin 0.0.191 → 0.0.192

package/test/export-just-namespace.test.ts

@@ -1,68 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as exportJustNamespace from "../src/rules/export-just-namespace";
-
-const ruleTester = new RuleTester();
-
-ruleTester.run("@definitelytyped/export-just-namespace", exportJustNamespace, {
-  invalid: [
-    {
-      code: `
-export = Stuff;
-namespace Stuff {}
-`,
-      errors: [
-        {
-          line: 2,
-          messageId: "useTheBody",
-        },
-      ],
-    },
-    {
-      code: `
-namespace Stuff {}
-export = Stuff;
-`,
-      errors: [
-        {
-          line: 3,
-          messageId: "useTheBody",
-        },
-      ],
-    },
-    {
-      code: `
-namespace Stuff {}
-const other = "code";
-export = Stuff;
-`,
-      errors: [
-        {
-          line: 4,
-          messageId: "useTheBody",
-        },
-      ],
-    },
-  ],
-  valid: [
-    `export const value = 3;`,
-    `export default class Hello {}`,
-    'import * as fs from "fs";',
-    "const value = 123;",
-    `export = stuff;`,
-    `export = createStuff();`,
-    `
-class Stuff {}
-namespace Stuff {}
-export = Stuff;
-`,
-    `export = First
-namespace First {}
-declare function First()
-`,
-    `declare namespace Second {}
-export = Second
-declare function Second<U, S>(s: U): S
-`,
-  ],
-});

package/test/fixtureTester.ts

@@ -1,63 +0,0 @@
-import { TSESLint } from "@typescript-eslint/utils";
-import { RuleTester } from "@typescript-eslint/rule-tester";
-import path from "path";
-import fs from "fs";
-
-export const fixtureRoot = path.join(__dirname, "fixtures");
-
-export function getFixturePath(filename: string): string {
-  return path.join(fixtureRoot, filename);
-}
-
-type ValidTestCase<TOptions extends Readonly<unknown[]>> = Omit<
-  TSESLint.ValidTestCase<TOptions>,
-  "code" | "filename"
-> & {
-  filename: string;
-};
-
-type InvalidTestCase<TMessageIds extends string, TOptions extends Readonly<unknown[]>> = Omit<
-  TSESLint.InvalidTestCase<TMessageIds, TOptions>,
-  "code" | "filename"
-> & {
-  filename: string;
-};
-
-interface RunTests<TMessageIds extends string, TOptions extends Readonly<unknown[]>> {
-  readonly valid: readonly ValidTestCase<TOptions>[];
-  readonly invalid: readonly InvalidTestCase<TMessageIds, TOptions>[];
-}
-
-function convertTestCase<TOptions extends Readonly<unknown[]>>(
-  test: ValidTestCase<TOptions>,
-): TSESLint.ValidTestCase<TOptions>;
-function convertTestCase<TMessageIds extends string, TOptions extends Readonly<unknown[]>>(
-  test: InvalidTestCase<TMessageIds, TOptions>,
-): TSESLint.InvalidTestCase<TMessageIds, TOptions>;
-function convertTestCase<
-  TMessageIds extends string,
-  TOptions extends Readonly<unknown[]>,
-  T extends ValidTestCase<TOptions> | InvalidTestCase<TMessageIds, TOptions>,
->(test: T): TSESLint.ValidTestCase<TOptions> | TSESLint.InvalidTestCase<TMessageIds, TOptions> {
-  const fixture = getFixturePath(test.filename);
-  const code = fs.readFileSync(fixture, "utf8");
-  return {
-    name: test.filename,
-    ...test,
-    code,
-    filename: fixture,
-  };
-}
-
-export function runTestsWithFixtures<TMessageIds extends string, TOptions extends Readonly<unknown[]>>(
-  name: string,
-  rule: TSESLint.RuleModule<TMessageIds, TOptions>,
-  tests: RunTests<TMessageIds, TOptions>,
-): void {
-  const ruleTester = new RuleTester();
-
-  return ruleTester.run(name, rule, {
-    valid: tests.valid.map(convertTestCase<TOptions>),
-    invalid: tests.invalid.map(convertTestCase<TMessageIds, TOptions>),
-  });
-}

package/test/no-any-union.test.ts

@@ -1,20 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as noAnyUnion from "../src/rules/no-any-union";
-
-const ruleTester = new RuleTester();
-
-ruleTester.run("@definitelytyped/no-any-union", noAnyUnion, {
-  invalid: [
-    {
-      code: `export const y: string | any;`,
-      errors: [
-        {
-          line: 1,
-          messageId: "anyUnion",
-        },
-      ],
-    },
-  ],
-  valid: [`export const x: any;`],
-});

package/test/no-bad-reference.test.ts

@@ -1,127 +0,0 @@
-import { runTestsWithFixtures } from "./fixtureTester";
-import * as rule from "../src/rules/no-bad-reference";
-
-runTestsWithFixtures("@definitelytyped/no-bad-reference", rule, {
-  valid: [
-    {
-      filename: "types/foo/index.d.ts",
-    },
-  ],
-  invalid: [
-    {
-      filename: "types/no-relative-references/index.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../foo/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "./v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../foo" } },
-        { messageId: "importOutside", data: { text: "./v1" } },
-        { messageId: "importOutside", data: { text: "../foo/v1" } },
-        { messageId: "importOutside", data: { text: "no-relative-references/v1" } },
-      ],
-    },
-    {
-      filename: "types/no-relative-references/other/other.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../foo/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../foo" } },
-        { messageId: "importOutside", data: { text: "../v1" } },
-        { messageId: "importOutside", data: { text: "../../foo/v1" } },
-      ],
-    },
-    {
-      filename: "types/no-relative-references/v1/index.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../foo/index.d.ts" } },
-        { messageId: "referenceLeaves", data: { text: "../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../foo/v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../foo" } },
-        { messageId: "importLeaves", data: { text: "../v1" } },
-        { messageId: "importOutside", data: { text: "../../foo/v1" } },
-        { messageId: "importOutside", data: { text: "../index" } },
-        { messageId: "importOutside", data: { text: "../index" } },
-      ],
-    },
-    {
-      filename: "types/no-relative-references/v1/other/other.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../../foo/index.d.ts" } },
-        { messageId: "referenceLeaves", data: { text: "../../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../../foo" } },
-        { messageId: "importLeaves", data: { text: "../../v1" } },
-        { messageId: "importOutside", data: { text: "../../../foo/v1" } },
-      ],
-    },
-    {
-      filename: "types/scoped__no-relative-references/index.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../foo/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "./v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../foo" } },
-        { messageId: "importOutside", data: { text: "./v1" } },
-        { messageId: "importOutside", data: { text: "../foo/v1" } },
-        { messageId: "importOutside", data: { text: "@scoped/no-relative-references/v1" } },
-      ],
-    },
-    {
-      filename: "types/scoped__no-relative-references/other/other.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../foo/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../foo" } },
-        { messageId: "importOutside", data: { text: "../v1" } },
-        { messageId: "importOutside", data: { text: "../../foo/v1" } },
-      ],
-    },
-    {
-      filename: "types/scoped__no-relative-references/v1/index.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../foo/index.d.ts" } },
-        { messageId: "referenceLeaves", data: { text: "../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../foo/v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../foo" } },
-        { messageId: "importLeaves", data: { text: "../v1" } },
-        { messageId: "importOutside", data: { text: "../../foo/v1" } },
-        { messageId: "importOutside", data: { text: "../index" } },
-        { messageId: "importOutside", data: { text: "../index" } },
-      ],
-    },
-    {
-      filename: "types/scoped__no-relative-references/v1/other/other.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../../../foo/index.d.ts" } },
-        { messageId: "referenceLeaves", data: { text: "../../v1/index.d.ts" } },
-        { messageId: "referenceOutside", data: { text: "../../../foo/v1/index.d.ts" } },
-        { messageId: "importOutside", data: { text: "../../../foo" } },
-        { messageId: "importLeaves", data: { text: "../../v1" } },
-        { messageId: "importOutside", data: { text: "../../../foo/v1" } },
-      ],
-    },
-    {
-      filename: "types/no-bad-reference/index.d.ts",
-      errors: [
-        { messageId: "referenceOutside", data: { text: "../other" } },
-        { messageId: "referenceOutside", data: { text: "./v11" } },
-        { messageId: "referenceOutside", data: { text: "./v11/index" } },
-        { messageId: "referenceOutside", data: { text: "./v11/subdir/file" } },
-        { messageId: "referenceOutside", data: { text: "./v0.1" } },
-        { messageId: "referenceOutside", data: { text: "./v0.1/index" } },
-        { messageId: "backslashes", line: 13 },
-      ],
-    },
-    {
-      filename: "types/no-bad-reference/no-bad-reference-tests.ts",
-      errors: [
-        { messageId: "testReference", data: { text: "../other" } },
-        { messageId: "testReference", data: { text: "other" } },
-      ],
-    },
-  ],
-});

package/test/no-const-enum.test.ts

@@ -1,20 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as noConstEnum from "../src/rules/no-const-enum";
-
-const ruleTester = new RuleTester();
-
-ruleTester.run("@definitelytyped/no-const-enum", noConstEnum, {
-  invalid: [
-    {
-      code: ` const enum E { } `,
-      errors: [
-        {
-          line: 1,
-          messageId: "constEnum",
-        },
-      ],
-    },
-  ],
-  valid: [` enum F {}`],
-});

package/test/no-dead-reference.test.ts

@@ -1,64 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as noDeadReference from "../src/rules/no-dead-reference";
-
-const ruleTester = new RuleTester();
-
-ruleTester.run("@definitelytyped/no-dead-reference", noDeadReference, {
-  invalid: [
-    {
-      code: `
-export class C { }
-/// <reference types="terms" />
-`,
-      errors: [
-        {
-          line: 3,
-          messageId: "referenceAtTop",
-        },
-      ],
-    },
-    {
-      code: `
-export class C { }
-/// <reference types="terms" />
-/// <reference types="multiple" />
-`,
-      errors: [
-        {
-          line: 3,
-          messageId: "referenceAtTop",
-        },
-        {
-          line: 4,
-          messageId: "referenceAtTop",
-        },
-      ],
-    },
-    {
-      code: `
-export class C { }
-/// <reference types="terms" />
-export class D { }
-/// <reference types="multiple" />
-export class E { }
-`,
-      errors: [
-        {
-          line: 3,
-          messageId: "referenceAtTop",
-        },
-        {
-          line: 5,
-          messageId: "referenceAtTop",
-        },
-      ],
-    },
-  ],
-  valid: [
-    `
-/// <reference types="tones" />
-export class K {}
-`,
-  ],
-});

package/test/no-declare-current-package.test.ts

@@ -1,59 +0,0 @@
-import * as noDeclareCurrentPackage from "../src/rules/no-declare-current-package";
-import { runTestsWithFixtures } from "./fixtureTester";
-
-runTestsWithFixtures("@definitelytyped/no-declare-current-package", noDeclareCurrentPackage, {
-  valid: [
-    {
-      filename: "types/foo/index.d.ts",
-    },
-    {
-      filename: "types/no-declare-current-package-other/index.d.ts",
-    },
-  ],
-  invalid: [
-    {
-      filename: "types/no-declare-current-package/index.d.ts",
-      errors: [
-        {
-          messageId: "noDeclareCurrentPackage",
-          data: { text: "no-declare-current-package", preferred: '"index.d.ts"' },
-        },
-      ],
-    },
-    {
-      filename: "types/no-declare-current-package/test/deep/import.d.ts",
-      errors: [
-        {
-          messageId: "noDeclareCurrentPackage",
-          data: {
-            text: "no-declare-current-package/deep/import",
-            preferred:
-              '"no-declare-current-package/deep/import.d.ts" or "no-declare-current-package/deep/import/index.d.ts"',
-          },
-        },
-      ],
-    },
-    {
-      filename: "types/scoped__no-declare-current-package/index.d.ts",
-      errors: [
-        {
-          messageId: "noDeclareCurrentPackage",
-          data: { text: "@scoped/no-declare-current-package", preferred: '"index.d.ts"' },
-        },
-      ],
-    },
-    {
-      filename: "types/scoped__no-declare-current-package/test/deep/import.d.ts",
-      errors: [
-        {
-          messageId: "noDeclareCurrentPackage",
-          data: {
-            text: "@scoped/no-declare-current-package/deep/import",
-            preferred:
-              '"@scoped/no-declare-current-package/deep/import.d.ts" or "@scoped/no-declare-current-package/deep/import/index.d.ts"',
-          },
-        },
-      ],
-    },
-  ],
-});

package/test/no-import-default-of-export-equals.test.ts

@@ -1,49 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as noImportDefaultOfExportEquals from "../src/rules/no-import-default-of-export-equals";
-
-const ruleTester = new RuleTester({
-  parser: "@typescript-eslint/parser",
-  parserOptions: {
-    ecmaVersion: 2018,
-    tsconfigRootDir: __dirname,
-    project: "./tsconfig.no-import-default-of-export-equals.json",
-  },
-});
-
-ruleTester.run("@definitelytyped/no-import-default-of-export-equals", noImportDefaultOfExportEquals, {
-  invalid: [
-    {
-      filename: "index.d.ts",
-      code: `declare module "a" {
-    interface I {}
-    export = I;
-}
-
-declare module "b" {
-import a from "a";
-}`,
-      errors: [
-        {
-          line: 7,
-          messageId: "noImportDefaultOfExportEquals",
-          data: { moduleName: "a", importName: "a" },
-        },
-      ],
-    },
-  ],
-  valid: [
-    {
-      filename: "index.d.ts",
-      code: `declare module "a" {
-    interface I {}
-    export default I;
-}
-
-declare module "b" {
-    import a from "a";
-}
-`,
-    },
-  ],
-});

package/test/no-import-of-dev-dependencies.test.ts

@@ -1,27 +0,0 @@
-import * as noImportDefaultOfDevDependencies from "../src/rules/no-import-of-dev-dependencies";
-import { runTestsWithFixtures } from "./fixtureTester";
-
-runTestsWithFixtures("@definitelytyped/no-import-of-dev-dependencies", noImportDefaultOfDevDependencies, {
-  valid: [
-    {
-      filename: "types/foo/index.d.ts",
-    },
-    {
-      filename: "types/no-import-of-dev-dependencies/index.d.ts",
-    },
-    {
-      filename: "types/no-import-of-dev-dependencies/no-import-of-dev-dependencies-tests.ts",
-    },
-  ],
-  invalid: [
-    {
-      filename: "types/no-import-of-dev-dependencies/bad.d.ts",
-      errors: [
-        { messageId: "noReferenceOfDevDependencies" },
-        { messageId: "noReferenceOfDevDependencies" },
-        { messageId: "noImportOfDevDependencies" },
-        { messageId: "noImportOfDevDependencies" },
-      ],
-    },
-  ],
-});

package/test/no-old-dt-header.test.ts

@@ -1,69 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import * as noOldDtHeader from "../src/rules/no-old-dt-header";
-
-const ruleTester = new RuleTester();
-
-ruleTester.run("@definitelytyped/no-old-dt-header", noOldDtHeader, {
-  invalid: [
-    {
-      code: `// Type definitions for AFRAME 1.2
-// Project: https://aframe.io/
-// Definitions by: Paul Shannon <https://github.com/devpaul>
-//                 Roberto Ritger <https://github.com/bertoritger>
-//                 Trygve Wastvedt <https://github.com/twastvedt>
-// Definitions: https://github.com/DefinitelyTyped/DefinitelyTyped
-// TypeScript Version: 4.4
-
-/**
- * Extended tests and examples available at https://github.com/devpaul/aframe-experiments.git
- */
-
-import * as anime from "animejs";
-import * as three from "three";`,
-      errors: [
-        {
-          column: 1,
-          endColumn: 25,
-          line: 1,
-          messageId: "noOldDTHeader",
-        },
-      ],
-      filename: "index.d.ts",
-    },
-    {
-      code: `// Type definitions for AFRAME 1.2
-// Definitions by: Paul Shannon <https://github.com/devpaul>`,
-      errors: [
-        {
-          column: 1,
-          endColumn: 25,
-          line: 1,
-          messageId: "noOldDTHeader",
-        },
-      ],
-      filename: "types.d.ts",
-    },
-  ],
-  valid: [
-    {
-      code: `// Type definitions for AFRAME 1.2
-// Definitions by: Paul Shannon <https://github.com/devpaul>`,
-      filename: "test.ts",
-    },
-    {
-      code: `// Type definitions for AFRAME 1.2`,
-      filename: "types.d.ts",
-    },
-    {
-      code: `// Definitions by: Paul Shannon <https://github.com/devpaul>`,
-      filename: "types.d.ts",
-    },
-    {
-      code: `// A line before the old header
-      // Type definitions for AFRAME 1.2
-// Definitions by: Paul Shannon <https://github.com/devpaul>`,
-      filename: "types.d.ts",
-    },
-  ],
-});

package/test/no-relative-import-in-test.test.ts

@@ -1,27 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import rule from "../src/rules/no-relative-import-in-test";
-
-const ruleTester = new RuleTester({
-  parser: "@typescript-eslint/parser",
-  parserOptions: {
-    tsconfigRootDir: __dirname,
-    project: "./tsconfig.no-relative-import-in-test.json",
-  },
-});
-
-ruleTester.run("no-relative-import-in-test", rule, {
-  invalid: [
-    {
-      code: `import abc from "./no-relative-import-in-test/abc.d.ts";`,
-      errors: [
-        {
-          line: 1,
-          messageId: "useGlobalImport",
-        },
-      ],
-      filename: "file.ts",
-    },
-  ],
-  valid: [`import ts from "does-not-exist";`, `import ts from "typescript";`, `import other from "./does-not-exit";`],
-});

package/test/no-self-import.test.ts

@@ -1,30 +0,0 @@
-import * as dtHeader from "../src/rules/no-self-import";
-import { runTestsWithFixtures } from "./fixtureTester";
-
-runTestsWithFixtures("@definitelytyped/no-self-import", dtHeader, {
-  valid: [
-    {
-      filename: "types/foo/index.d.ts",
-    },
-    {
-      filename: "types/no-self-import/index.d.ts",
-    },
-    {
-      filename: "types/no-self-import/no-self-import-tests.ts",
-    },
-  ],
-  invalid: [
-    {
-      filename: "types/no-self-import/bad.d.ts",
-      errors: [
-        { messageId: "useRelativeImport" },
-        { messageId: "useRelativeImport" },
-        { messageId: "useOnlyCurrentVersion" },
-        { messageId: "useOnlyCurrentVersion" },
-        { messageId: "useOnlyCurrentVersion" },
-        { messageId: "useOnlyCurrentVersion" },
-        { messageId: "useOnlyCurrentVersion" },
-      ],
-    },
-  ],
-});

package/test/no-single-declare-module.test.ts

@@ -1,49 +0,0 @@
-import { RuleTester } from "@typescript-eslint/rule-tester";
-
-import rule from "../src/rules/no-single-declare-module";
-
-const ruleTester = new RuleTester({
-  parser: "@typescript-eslint/parser",
-  parserOptions: {
-    tsconfigRootDir: __dirname,
-    project: "./tsconfig.no-single-declare-module.json",
-  },
-});
-
-ruleTester.run("no-single-declare-module", rule, {
-  invalid: [
-    {
-      code: `
-        declare module "foo" {}
-
-        // Other global declarations don't affect this. They should go in "declare global".
-        interface I {}
-      `,
-      errors: [
-        {
-          line: 2,
-          messageId: "oneModuleDeclaration",
-        },
-      ],
-    },
-  ],
-  valid: [
-    {
-      code: `
-        import x from "x";
-        declare module "foo" {}
-      `,
-    },
-    {
-      code: `
-        declare module "foo" {}
-        declare module "bar" {}      
-      `,
-    },
-    {
-      code: `
-        declare module "*.svg" {}
-      `,
-    },
-  ],
-});