refactor(test): sources assertions from assert/strict to ease strict …

…comparisons

test/api.spec.mjs

@@ -1,7 +1,7 @@
 // eslint disable because import/no-unresolved,node/no-missing-import don't
 // know about (local) module imports yet
 /* eslint-disable import/no-unresolved,node/no-missing-import */
-import { strictEqual } from "node:assert";
+import { equal } from "node:assert/strict";
 import satisfies from "semver/functions/satisfies.js";
 import dependencyCruiser from "dependency-cruiser";
 import extractBabelConfig from "dependency-cruiser/config-utl/extract-babel-config";
@@ -15,27 +15,27 @@ import extractWebpackResolveConfig from "dependency-cruiser/config-utl/extract-w
 if (satisfies(process.versions.node, "^12.19 || >=14.7")) {
  describe("[E] api from esm (on node ^12.19 || >= 14.7)", () => {
  it("exposes dependency-cruiser main with some functions", () => {
- strictEqual(typeof dependencyCruiser, "object");
- strictEqual(typeof dependencyCruiser.cruise, "function");
- strictEqual(typeof dependencyCruiser.format, "function");
- strictEqual(Array.isArray(dependencyCruiser.allExtensions), true);
- strictEqual(typeof dependencyCruiser.getAvailableTranspilers, "function");
+ equal(typeof dependencyCruiser, "object");
+ equal(typeof dependencyCruiser.cruise, "function");
+ equal(typeof dependencyCruiser.format, "function");
+ equal(Array.isArray(dependencyCruiser.allExtensions), true);
+ equal(typeof dependencyCruiser.getAvailableTranspilers, "function");
  });
 
  it("exposes an extract-babel-config function", () => {
- strictEqual(typeof extractBabelConfig, "function");
+ equal(typeof extractBabelConfig, "function");
  });
 
  it("exposes an extract-depcruise-config function", () => {
- strictEqual(typeof extractDepcruiseConfig, "function");
+ equal(typeof extractDepcruiseConfig, "function");
  });
 
  it("exposes an extract-ts-config function", () => {
- strictEqual(typeof extractTsConfig, "function");
+ equal(typeof extractTsConfig, "function");
  });
 
  it("exposes an extract-webpack-resolve-config function", () => {
- strictEqual(typeof extractWebpackResolveConfig, "function");
+ equal(typeof extractWebpackResolveConfig, "function");
  });
  });
 }

test/cache/cache.spec.mjs

@@ -1,4 +1,4 @@
-import { deepStrictEqual, strictEqual } from "node:assert";
+import { deepEqual, equal } from "node:assert/strict";
 import { rmSync } from "node:fs";
 import { join } from "node:path";
 import { isDeepStrictEqual } from "node:util";
@@ -10,26 +10,23 @@ const OUTPUTS_FOLDER = "test/cache/__outputs__/";
 describe("[I] cache/cache - readCache", () => {
  it("returns an empty cache when trying to read from a non-existing one", async () => {
  const lCache = new Cache();
- deepStrictEqual(await lCache.read("this/folder/does/not-exist"), {
+ deepEqual(await lCache.read("this/folder/does/not-exist"), {
  modules: [],
  summary: {},
  });
  });
 
  it("returns an empty cache when trying to read from a file that is invalid JSON", async () => {
  const lCache = new Cache();
- deepStrictEqual(
- await lCache.read("test/cache/__mocks__/cache/invalid-json"),
- {
- modules: [],
- summary: {},
- },
- );
+ deepEqual(await lCache.read("test/cache/__mocks__/cache/invalid-json"), {
+ modules: [],
+ summary: {},
+ });
  });
 
  it("returns the contents of the cache when trying to read from an existing, valid json", async () => {
  const lCache = new Cache();
- deepStrictEqual(
+ deepEqual(
  await lCache.read("test/cache/__mocks__/cache/valid-minimal-cache"),
  {
  modules: [],
@@ -57,7 +54,7 @@ describe("[I] cache/cache - writeCache", () => {
  const lCache = new Cache();
 
  await lCache.write(lCacheFolder, lDummyCacheContents);
- deepStrictEqual(await lCache.read(lCacheFolder), lDummyCacheContents);
+ deepEqual(await lCache.read(lCacheFolder), lDummyCacheContents);
  });
 
  it("writes the passed cruise options to the cache folder (folder already exists -> overwrite)", async () => {
@@ -72,7 +69,7 @@ describe("[I] cache/cache - writeCache", () => {
 
  await lCache.write(lCacheFolder, lDummyCacheContents);
  await lCache.write(lCacheFolder, lSecondDummyCacheContents);
- deepStrictEqual(await lCache.read(lCacheFolder), lSecondDummyCacheContents);
+ deepEqual(await lCache.read(lCacheFolder), lSecondDummyCacheContents);
  });
 
  it("writes the passed cruise options to the cache folder (which is created when it doesn't exist yet) - content based cached strategy", async () => {
@@ -124,7 +121,7 @@ describe("[I] cache/cache - canServeFromCache", () => {
  changes: [],
  },
  );
- strictEqual(lResult, false);
+ equal(lResult, false);
  });
 
  it("returns false when the base SHA differs", async () => {
@@ -142,7 +139,7 @@ describe("[I] cache/cache - canServeFromCache", () => {
  },
  );
 
- strictEqual(lFound, false);
+ equal(lFound, false);
  });
 
  it("returns false when a file was added", async () => {
@@ -166,7 +163,7 @@ describe("[I] cache/cache - canServeFromCache", () => {
  },
  );
 
- strictEqual(lFound, false);
+ equal(lFound, false);
  });
 
  it("returns false when cache written & revision data equal & options incompatible", async () => {
@@ -184,7 +181,7 @@ describe("[I] cache/cache - canServeFromCache", () => {
  { SHA1: "dummy-sha", changes: [] },
  );
 
- strictEqual(lFound, false);
+ equal(lFound, false);
  });
 
  it("returns true when cache written & revision data equal & options compatible", async () => {
@@ -198,6 +195,6 @@ describe("[I] cache/cache - canServeFromCache", () => {
  { SHA1: "dummy-sha", changes: [] },
  );
 
- strictEqual(lFound, true);
+ equal(lFound, true);
  });
 });

test/cache/content-strategy.spec.mjs

@@ -1,4 +1,4 @@
-import { deepStrictEqual, strictEqual } from "node:assert";
+import { deepEqual, equal } from "node:assert/strict";
 import ContentStrategy from "../../src/cache/content-strategy.mjs";
 
 const INTERESTING_EXTENSIONS = new Set([".aap", ".noot", ".mies"]);
@@ -37,7 +37,7 @@ describe("[U] cache/content-strategy - getRevisionData", () => {
  ],
  };
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().getRevisionData(
  null,
  null,
@@ -54,7 +54,7 @@ describe("[U] cache/content-strategy - getRevisionData", () => {
  });
 
  it("if there's no changes the change set contains the passed sha & an empty array", () => {
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().getRevisionData(
  null,
  null,
@@ -76,7 +76,7 @@ describe("[U] cache/content-strategy - getRevisionData", () => {
  it("returns only the extensions passed", () => {
  const lLimitedExtensions = new Set([".wim", ".noot"]);
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().getRevisionData(
  ".",
  { modules: [] },
@@ -131,7 +131,7 @@ describe("[U] cache/content-strategy - getRevisionData", () => {
  },
  ];
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().getRevisionData(
  null,
  null,
@@ -186,11 +186,11 @@ describe("[U] cache/content-strategy - revisionDataEqual", () => {
  ];
 
  it("returns false when revision data objects don't exist", () => {
- strictEqual(new ContentStrategy().revisionDataEqual(null, null), false);
+ equal(new ContentStrategy().revisionDataEqual(null, null), false);
  });
 
  it("returns false when old revision data object doesn't exist", () => {
- strictEqual(
+ equal(
  new ContentStrategy().revisionDataEqual(null, {
  SHA1: "some-sha",
  changes: [],
@@ -200,7 +200,7 @@ describe("[U] cache/content-strategy - revisionDataEqual", () => {
  });
 
  it("returns false when new revision data object doesn't exist", () => {
- strictEqual(
+ equal(
  new ContentStrategy().revisionDataEqual(
  { SHA1: "some-sha", changes: [] },
  null,
@@ -210,7 +210,7 @@ describe("[U] cache/content-strategy - revisionDataEqual", () => {
  });
 
  it("returns false when changes are not equal", () => {
- strictEqual(
+ equal(
  new ContentStrategy().revisionDataEqual(
  { SHA1: "some-sha", changes: [] },
  { SHA1: "some-sha", changes: lChanges },
@@ -220,7 +220,7 @@ describe("[U] cache/content-strategy - revisionDataEqual", () => {
  });
 
  it("returns true when changes are equal", () => {
- strictEqual(
+ equal(
  new ContentStrategy().revisionDataEqual(
  { SHA1: "some-sha", changes: lChanges },
  { SHA1: "some-sha", changes: lChanges },
@@ -230,7 +230,7 @@ describe("[U] cache/content-strategy - revisionDataEqual", () => {
  });
 
  it("returns true when changes are equal (even when neither contain changes)", () => {
- strictEqual(
+ equal(
  new ContentStrategy().revisionDataEqual(
  { SHA1: "some-sha", changes: [] },
  { SHA1: "some-sha", changes: [] },
@@ -274,7 +274,7 @@ describe("[U] cache/content-strategy - prepareRevisionDataForSaving", () => {
  revisionData: lEmptyRevisionData,
  };
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().prepareRevisionDataForSaving(
  lEmptyCruiseResult,
  lEmptyRevisionData,
@@ -333,7 +333,7 @@ describe("[U] cache/content-strategy - prepareRevisionDataForSaving", () => {
  revisionData: lEmptyRevisionData,
  };
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().prepareRevisionDataForSaving(
  lEmptyCruiseResult,
  lEmptyRevisionData,
@@ -422,7 +422,7 @@ describe("[U] cache/content-strategy - prepareRevisionDataForSaving", () => {
  },
  };
 
- deepStrictEqual(
+ deepEqual(
  new ContentStrategy().prepareRevisionDataForSaving(
  lCruiseResult,
  lRevisionData,

test/cache/find-content-changes.spec.mjs

@@ -1,9 +1,9 @@
-import { deepStrictEqual } from "node:assert";
+import { deepEqual } from "node:assert/strict";
 import findContentChanges from "../../src/cache/find-content-changes.mjs";
 
 describe("[U] cache/find-content-changes - cached vs new", () => {
  it("returns files not in directory but in cache as 'ignored' when they're not interesting for diffing", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  {
@@ -55,7 +55,7 @@ describe("[U] cache/find-content-changes - cached vs new", () => {
  });
 
  it("returns files not in directory but in cache as 'deleted'", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [{ source: "only-in-cache-ends-up-as-deleted.js" }] },
@@ -76,7 +76,7 @@ describe("[U] cache/find-content-changes - cached vs new", () => {
  });
 
  it("returns files that have been earmarked as not followable as 'ignored'", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  {
@@ -105,7 +105,7 @@ describe("[U] cache/find-content-changes - cached vs new", () => {
  });
 
  it("returns files both in directory and in cache that are different as 'modified'", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  {
@@ -142,7 +142,7 @@ describe("[U] cache/find-content-changes - cached vs new", () => {
  });
 
  it("returns files both in directory and in cache that are the same as 'unmodified'", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  {
@@ -173,7 +173,7 @@ describe("[U] cache/find-content-changes - cached vs new", () => {
 
 describe("[U] cache/find-content-changes - new vs cached", () => {
  it("returns an empty set when the directory and modules are empty", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [] },
@@ -189,7 +189,7 @@ describe("[U] cache/find-content-changes - new vs cached", () => {
  });
 
  it("returns changes when there's file in the directory and modules is empty (extensions only)", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [] },
@@ -211,7 +211,7 @@ describe("[U] cache/find-content-changes - new vs cached", () => {
  });
 
  it("returns changes when there's file in the directory and modules is empty (missing includeOnly filter)", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [] },
@@ -232,7 +232,7 @@ describe("[U] cache/find-content-changes - new vs cached", () => {
  });
 
  it("returns changes when there's file in the directory and modules is empty (exclude filter)", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [] },
@@ -254,7 +254,7 @@ describe("[U] cache/find-content-changes - new vs cached", () => {
  });
 
  it("returns changes when there's file in the directory and modules is empty (includeOnly filter)", () => {
- deepStrictEqual(
+ deepEqual(
  findContentChanges(
  ".",
  { modules: [] },