@monorepolint/rules 0.6.0-alpha.2 → 0.6.0-alpha.4

package/src/__tests__/bannedDependencies.spec.ts

@@ -4,12 +4,12 @@
  * Licensed under the MIT license. See LICENSE file in the project root for details.
  *
  */
-import { describe, expect, it, beforeEach, vi, afterEach } from "vitest";
 import { WorkspaceContextImpl } from "@monorepolint/core";
 import { SimpleHost } from "@monorepolint/utils";
 import { writeFileSync } from "fs";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { bannedDependencies, Options } from "../bannedDependencies.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
 import { jsonToString } from "./utils.js";
@@ -46,7 +46,7 @@ describe("bannedDependencies", () => {
         verbose: false,
         silent: true,
       },
-      new SimpleHost()
+      new SimpleHost(),
     );
 
     function checkAndSpy(options: Options) {
@@ -77,15 +77,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: ["ccc"] });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: ["ccc"],
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: ["ddd"] });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: ["ddd"],
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: ["ccc", "ddd"] });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: ["ccc", "ddd"],
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -100,15 +106,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: ["c*c"] });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: ["c*c"],
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: ["d*d"] });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: ["d*d"],
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: ["c*c", "d*d"] });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: ["c*c", "d*d"],
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -123,15 +135,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: { glob: ["c*c"] } });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: { glob: ["c*c"] },
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: { glob: ["d*d"] } });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: { glob: ["d*d"] },
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: { glob: ["c*c", "d*d"] } });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: { glob: ["c*c", "d*d"] },
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -146,15 +164,21 @@ describe("bannedDependencies", () => {
     });
     addFile("./package.json", rootPackageJson);
 
-    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({ bannedDependencies: { exact: ["ccc"] } });
+    const { addErrorSpy: addErrorSpy1 } = checkAndSpy({
+      bannedDependencies: { exact: ["ccc"] },
+    });
     expect(addErrorSpy1).toHaveBeenCalledTimes(1);
     addErrorSpy1.mockReset();
 
-    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({ bannedDependencies: { exact: ["ddd"] } });
+    const { addErrorSpy: addErrorSpy2 } = checkAndSpy({
+      bannedDependencies: { exact: ["ddd"] },
+    });
     expect(addErrorSpy2).toHaveBeenCalledTimes(0);
     addErrorSpy2.mockReset();
 
-    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({ bannedDependencies: { exact: ["ccc", "ddd"] } });
+    const { addErrorSpy: addErrorSpy3 } = checkAndSpy({
+      bannedDependencies: { exact: ["ccc", "ddd"] },
+    });
     expect(addErrorSpy3).toHaveBeenCalledTimes(1);
     addErrorSpy3.mockReset();
   });
@@ -181,9 +205,14 @@ describe("bannedDependencies", () => {
       },
     });
     addFile("./node_modules/aaa/node_modules/bbb/package.json", bbbPackageJson);
-    addFile("./node_modules/aaa/node_modules/bbb/node_modules/ddd/package.json", EMPTY_PACKAGE);
+    addFile(
+      "./node_modules/aaa/node_modules/bbb/node_modules/ddd/package.json",
+      EMPTY_PACKAGE,
+    );
     addFile("./node_modules/aaa/node_modules/ccc/package.json", EMPTY_PACKAGE);
 
-    expect(checkAndSpy({ bannedTransitiveDependencies: ["ccc", "ddd"] }).addErrorSpy).toHaveBeenCalledTimes(2);
+    expect(
+      checkAndSpy({ bannedTransitiveDependencies: ["ccc", "ddd"] }).addErrorSpy,
+    ).toHaveBeenCalledTimes(2);
   });
 });

package/src/__tests__/consistentDependencies.spec.ts

@@ -9,10 +9,10 @@ import { SimpleHost } from "@monorepolint/utils";
 import { readFileSync, writeFileSync } from "fs";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, describe, expect, it, vi } from "vitest";
 import { consistentDependencies, Options } from "../consistentDependencies.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
 import { jsonToString } from "./utils.js";
-import { describe, expect, it, afterEach, vi } from "vitest";
 
 const PACKAGE_ROOT = jsonToString({
   workspaces: {
@@ -71,11 +71,13 @@ describe("consistentDependencies", () => {
         verbose: false,
         silent: true,
       },
-      new SimpleHost()
+      new SimpleHost(),
     );
 
     function checkAndSpy(q: string, opts?: Options) {
-      const context = workspaceContext.createChildContext(path.resolve(dir.name, q));
+      const context = workspaceContext.createChildContext(
+        path.resolve(dir.name, q),
+      );
       const addErrorSpy = vi.spyOn(context, "addError");
       consistentDependencies({ options: opts }).check(context);
       return { context, addErrorSpy };
@@ -97,7 +99,9 @@ describe("consistentDependencies", () => {
   }
 
   it("checks correctly", () => {
-    const { addFile, workspaceContext, checkAndSpy } = makeWorkspace({ fix: false });
+    const { addFile, workspaceContext, checkAndSpy } = makeWorkspace({
+      fix: false,
+    });
     addFile("./package.json", PACKAGE_ROOT);
     addFile("./packages/star/package.json", PACKAGE_CHILD_WITH_STAR);
     addFile("./packages/latest/package.json", PACKAGE_CHILD_WITH_LATEST);
@@ -136,7 +140,9 @@ describe("consistentDependencies", () => {
     addFile("./package.json", PACKAGE_ROOT);
     addFile("./packages/wrong/package.json", PACKAGE_CHILD_WITH_WRONG_VERSION);
 
-    const ignored = checkAndSpy("./packages/wrong", { ignoredDependencies: ["foo"] });
+    const ignored = checkAndSpy("./packages/wrong", {
+      ignoredDependencies: ["foo"],
+    });
     expect(ignored.addErrorSpy).toHaveBeenCalledTimes(0);
   });
 });

package/src/__tests__/consistentVersions.spec.ts

@@ -9,9 +9,9 @@ import { WorkspaceContextImpl } from "@monorepolint/core";
 import { Host, PackageJson, SimpleHost } from "@monorepolint/utils";
 import * as path from "path";
 import * as tmp from "tmp";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import { consistentVersions, Options } from "../consistentVersions.js";
 import { makeDirectoryRecursively } from "../util/makeDirectory.js";
-import { describe, expect, it, beforeEach, afterEach, vi } from "vitest";
 
 describe("consistentVersions", () => {
   tmp.setGracefulCleanup();
@@ -45,7 +45,7 @@ describe("consistentVersions", () => {
         verbose: false,
         silent: true,
       },
-      host
+      host,
     );
     const addErrorSpy = vi.spyOn(workspaceContext, "addError");
 
@@ -56,7 +56,11 @@ describe("consistentVersions", () => {
     return { addErrorSpy, check, host };
   }
 
-  function addPackageJson(host: Host, filePath: string, packageJson: PackageJson) {
+  function addPackageJson(
+    host: Host,
+    filePath: string,
+    packageJson: PackageJson,
+  ) {
     const dirPath = path.resolve(cwd!, path.dirname(filePath));
     const resolvedFilePath = path.resolve(cwd!, filePath);
 
@@ -99,15 +103,24 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect dependency version", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredGreatLibVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
       check({
-        matchDependencyVersions: { both: testPackageJson.dependencies!.both, greatLib: requiredGreatLibVersion },
+        matchDependencyVersions: {
+          both: testPackageJson.dependencies!.both,
+          greatLib: requiredGreatLibVersion,
+        },
       });
       expect(addErrorSpy).toHaveBeenCalledTimes(1);
-      expect(readTestPackageJson().dependencies!.greatLib).toEqual(requiredGreatLibVersion);
+      expect(readTestPackageJson().dependencies!.greatLib).toEqual(
+        requiredGreatLibVersion,
+      );
     });
 
     it("Ignores packages that have a correct dependency version", async () => {
@@ -126,13 +139,24 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect devDependency version", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredElseLibVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { both: testPackageJson.dependencies!.both, else: requiredElseLibVersion } });
+      check({
+        matchDependencyVersions: {
+          both: testPackageJson.dependencies!.both,
+          else: requiredElseLibVersion,
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(1);
-      expect(readTestPackageJson().devDependencies!.else).toEqual(requiredElseLibVersion);
+      expect(readTestPackageJson().devDependencies!.else).toEqual(
+        requiredElseLibVersion,
+      );
     });
 
     it("Ignores packages that have a correct devDependency version", async () => {
@@ -151,14 +175,22 @@ describe("consistentVersions", () => {
 
     it("Fixes packages that have an incorrect dependency and devDependency versions", async () => {
       const { addErrorSpy, check, host } = makeWorkspace(true);
-      const readTestPackageJson = addPackageJson(host, "./package.json", testPackageJson);
+      const readTestPackageJson = addPackageJson(
+        host,
+        "./package.json",
+        testPackageJson,
+      );
 
       const requiredBothVersion = "1.2.3";
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
       check({ matchDependencyVersions: { both: requiredBothVersion } });
       expect(addErrorSpy).toHaveBeenCalledTimes(2);
-      expect(readTestPackageJson().dependencies!.both).toEqual(requiredBothVersion);
-      expect(readTestPackageJson().devDependencies!.both).toEqual(requiredBothVersion);
+      expect(readTestPackageJson().dependencies!.both).toEqual(
+        requiredBothVersion,
+      );
+      expect(readTestPackageJson().devDependencies!.both).toEqual(
+        requiredBothVersion,
+      );
     });
   });
 
@@ -187,13 +219,35 @@ describe("consistentVersions", () => {
       addPackageJson(host, "./package.json", testPackageJson);
 
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: [testPackageJson.dependencies!.greatLib] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [testPackageJson.dependencies!.greatLib],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: ["1", "2", testPackageJson.dependencies!.greatLib] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: ["1", "2", testPackageJson.dependencies!.greatLib],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: ["1", "2", testPackageJson.dependencies!.greatLib, "99", "100"] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [
+            "1",
+            "2",
+            testPackageJson.dependencies!.greatLib,
+            "99",
+            "100",
+          ],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
-      check({ matchDependencyVersions: { greatLib: [testPackageJson.dependencies!.greatLib, "99", "100"] } });
+      check({
+        matchDependencyVersions: {
+          greatLib: [testPackageJson.dependencies!.greatLib, "99", "100"],
+        },
+      });
       expect(addErrorSpy).toHaveBeenCalledTimes(0);
     });
 
@@ -212,12 +266,12 @@ describe("consistentVersions", () => {
       expect(addErrorSpy.mock.calls[0][0].message).toEqual(
         `Expected dependency on both to match one of '["99","100"]', got '${
           testPackageJson.dependencies!.both
-        }' instead.`
+        }' instead.`,
       );
       expect(addErrorSpy.mock.calls[1][0].message).toEqual(
         `Expected devDependency on both to match one of '["99","100"]', got '${
           testPackageJson.devDependencies!.both
-        }' instead.`
+        }' instead.`,
       );
     });
   });

package/src/__tests__/fileContents.spec.ts

@@ -6,10 +6,10 @@
  */
 
 // tslint:disable:no-console
-import { createTestingWorkspace, HOST_FACTORIES, TestingWorkspace } from "./utils.js";
 import { AddErrorOptions, Failure } from "@monorepolint/config";
+import { beforeEach, describe, expect, it, MockInstance, vi } from "vitest";
 import { fileContents } from "../fileContents.js";
-import { describe, expect, it, beforeEach, vi, MockInstance } from "vitest";
+import { createTestingWorkspace, HOST_FACTORIES, TestingWorkspace } from "./utils.js";
 
 const EXPECTED_FOO_FILE = "hello world";
 
@@ -44,7 +44,7 @@ describe.each(HOST_FACTORIES)("fileContents ($name)", (hostFactory) => {
           file: "foo.txt",
           hasFixer: true,
           message: "Expect file contents to match",
-        })
+        }),
       );
 
       expect(workspace.readFile("foo.txt")).toEqual(EXPECTED_FOO_FILE);
@@ -68,7 +68,7 @@ describe.each(HOST_FACTORIES)("fileContents ($name)", (hostFactory) => {
           file: "foo.txt",
           hasFixer: true,
           message: "Expect file contents to match",
-        })
+        }),
       );
 
       expect(workspace.readFile("foo.txt")).toEqual(EXPECTED_FOO_FILE);
@@ -92,7 +92,7 @@ describe.each(HOST_FACTORIES)("fileContents ($name)", (hostFactory) => {
           file: "nested/foo.txt",
           hasFixer: true,
           message: "Expect file contents to match",
-        })
+        }),
       );
 
       expect(workspace.readFile("nested/foo.txt")).toEqual(EXPECTED_FOO_FILE);