refactor: uses node imports aliasses in package.json for validate

sverweij · Oct 8, 2023 · 3315882 · 3315882
1 parent 76e1f9b
commit 3315882
Show file tree

Hide file tree

Showing 28 changed files with 32 additions and 31 deletions.
diff --git a/package.json b/package.json
@@ -55,7 +55,8 @@
     "#main/*": "./src/main/*",
     "#meta": "./src/meta.js",
     "#report/*": "./src/report/*",
-    "#utl/*": "./src/utl/*"
+    "#utl/*": "./src/utl/*",
+    "#validate/*": "./src/validate/*"
   },
   "exports": {
     ".": {

diff --git a/src/enrich/add-validations.mjs b/src/enrich/add-validations.mjs
@@ -1,4 +1,4 @@
-import validate from "../validate/index.mjs";
+import validate from "#validate/index.mjs";
 
 function addDependencyViolations(pModule, pDependency, pRuleSet, pValidate) {
   return {
@@ -26,7 +26,7 @@ export default function addValidations(pModules, pRuleSet, pValidate) {
     ...pModule,
     ...(pValidate ? validate.module(pRuleSet, pModule) : { valid: true }),
     dependencies: pModule.dependencies.map((pDependency) =>
-      addDependencyViolations(pModule, pDependency, pRuleSet, pValidate)
+      addDependencyViolations(pModule, pDependency, pRuleSet, pValidate),
     ),
   }));
 }
diff --git a/src/enrich/derive/folders/index.mjs b/src/enrich/derive/folders/index.mjs
@@ -1,5 +1,5 @@
-import validate from "../../../validate/index.mjs";
 import aggregateToFolders from "./aggregate-to-folders.mjs";
+import validate from "#validate/index.mjs";
 
 /**
  * @param {import("../../../../types/dependency-cruiser.js").IFolder} pFolder
@@ -18,7 +18,7 @@ function addFolderDependencyViolations(pOptions) {
     ...pFolder,
 
     dependencies: pFolder.dependencies.map(
-      validateFolderDependency(pFolder, pOptions)
+      validateFolderDependency(pFolder, pOptions),
     ),
   });
 }
@@ -34,7 +34,7 @@ export default function deriveFolderMetrics(pModules, pOptions) {
   if (pOptions.metrics) {
     lReturnValue = {
       folders: aggregateToFolders(pModules).map(
-        addFolderDependencyViolations(pOptions)
+        addFolderDependencyViolations(pOptions),
       ),
     };
   }

diff --git a/src/enrich/derive/reachable.mjs b/src/enrich/derive/reachable.mjs
@@ -1,6 +1,6 @@
 /* eslint-disable security/detect-object-injection, no-inline-comments */
 import has from "lodash/has.js";
-import matchers from "../../validate/matchers.mjs";
+import matchers from "#validate/matchers.mjs";
 import IndexedModuleGraph from "#graph-utl/indexed-module-graph.mjs";
 import { extractGroups } from "#utl/regex-util.mjs";
 

diff --git a/test/configs/no-orphans.spec.mjs b/test/configs/no-orphans.spec.mjs
@@ -1,6 +1,6 @@
 import { equal } from "node:assert/strict";
 import noOrphansRule from "../../configs/rules/no-orphans.js";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 describe("[I] configs/rules/no-orphans", () => {
   it("flags non-excepted orphans as orphan rule transgression", () => {

diff --git a/test/validate/index.core.spec.mjs b/test/validate/index.core.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - core", () => {
   const lNotToCoreRuleSet = parseRuleSet({

diff --git a/test/validate/index.could-not-resolve.spec.mjs b/test/validate/index.could-not-resolve.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - couldNotResolve", () => {
   const lNotToUnresolvableRuleSet = parseRuleSet({

diff --git a/test/validate/index.cycle-via-not.spec.mjs b/test/validate/index.cycle-via-not.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index dependency - cycle viaNot", () => {
   const lCycleViaNotRuleSet = parseRuleSet({

diff --git a/test/validate/index.cycle-via-some-not.spec.mjs b/test/validate/index.cycle-via-some-not.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index dependency - cycle viaSomeNot - with group matching", () => {
   const lCycleButNotViaGroupMatchRuleSet = {

diff --git a/test/validate/index.cycle-via.spec.mjs b/test/validate/index.cycle-via.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index dependency - cycle via", () => {
   const lCycleViaRuleSet = parseRuleSet({

diff --git a/test/validate/index.exotic-require.spec.mjs b/test/validate/index.exotic-require.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - exoticallyRequired", () => {
   const lExoticallyRequiredRuleSet = parseRuleSet({

diff --git a/test/validate/index.groupmatching.spec.mjs b/test/validate/index.groupmatching.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index group matching - path group matched in a pathnot", () => {
   const lGroupToPathNotRuleSet = {

diff --git a/test/validate/index.license.spec.mjs b/test/validate/index.license.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - license", () => {
   const lLicenseRuleSet = parseRuleSet({

diff --git a/test/validate/index.more-than-one-dependency-type.spec.mjs b/test/validate/index.more-than-one-dependency-type.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] index/validate - moreThanOneDependencyType", () => {
   it(`no relations with modules of > 1 dep type (e.g. specified 2x in package.json)`, () => {

diff --git a/test/validate/index.more-unstable.spec.mjs b/test/validate/index.more-unstable.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - stability checks", () => {
   const lForbiddenRuleSet = parseRuleSet({

diff --git a/test/validate/index.orphans.spec.mjs b/test/validate/index.orphans.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - orphans", () => {
   const lOrphanRuleSet = parseRuleSet({

diff --git a/test/validate/index.pre-compilation-only.spec.mjs b/test/validate/index.pre-compilation-only.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - preCompilationOnly", () => {
   const lPreCompilationOnlyRuleSet = parseRuleSet({

diff --git a/test/validate/index.reachable.spec.mjs b/test/validate/index.reachable.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - reachable (in forbidden set)", () => {
   const lReachableFalseRuleSet = parseRuleSet({

diff --git a/test/validate/index.required-rules.spec.mjs b/test/validate/index.required-rules.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index - required rules", () => {
   const lRequiredRuleSet = parseRuleSet({

diff --git a/test/validate/index.spec.mjs b/test/validate/index.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] validate/index dependency - generic tests", () => {
   it("is ok with the empty validation", () => {

diff --git a/test/validate/index.type-only.spec.mjs b/test/validate/index.type-only.spec.mjs
@@ -1,6 +1,6 @@
 import { deepEqual } from "node:assert/strict";
-import validate from "../../src/validate/index.mjs";
 import parseRuleSet from "./parse-ruleset.utl.mjs";
+import validate from "#validate/index.mjs";
 
 describe("[I] [I] validate/index - type-only", () => {
   const lTypeOnlyRuleSet = parseRuleSet({

diff --git a/test/validate/match-folder-dependency-rule.spec.mjs b/test/validate/match-folder-dependency-rule.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchFolderRule from "../../src/validate/match-folder-dependency-rule.mjs";
+import matchFolderRule from "#validate/match-folder-dependency-rule.mjs";
 
 describe("[I] validate/match-folder-dependency-rule - match generic", () => {
   const lEmptyRule = { scope: "folder", from: {}, to: {} };

diff --git a/test/validate/match-module-rule.dependents.spec.mjs b/test/validate/match-module-rule.dependents.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 const EMPTY_RULE = { from: {}, module: {} };
 const ANY_DEPENDENTS = {

diff --git a/test/validate/match-module-rule.orphan.spec.mjs b/test/validate/match-module-rule.orphan.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 const EMPTY_RULE = { from: {}, to: {} };
 const ANY_ORPHAN = { from: { orphan: true }, to: {} };

diff --git a/test/validate/match-module-rule.reachable.spec.mjs b/test/validate/match-module-rule.reachable.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 const EMPTY_RULE = { from: {}, to: {} };
 const ANY_UNREACHABLE = {

diff --git a/test/validate/match-module-rule.reaches.spec.mjs b/test/validate/match-module-rule.reaches.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 const EMPTY_RULE = { from: {}, to: {} };
 const ANY_REACHABLE = {

diff --git a/test/validate/match-module-rule.spec.mjs b/test/validate/match-module-rule.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import matchModuleRule from "../../src/validate/match-module-rule.mjs";
+import matchModuleRule from "#validate/match-module-rule.mjs";
 
 const EMPTY_RULE = { from: {}, to: {} };
 const ORPHAN_RULE = { from: { orphan: true }, to: {} };

diff --git a/test/validate/violates-required-rule.spec.mjs b/test/validate/violates-required-rule.spec.mjs
@@ -1,5 +1,5 @@
 import { equal } from "node:assert/strict";
-import violatesRequiredRule from "../../src/validate/violates-required-rule.mjs";
+import violatesRequiredRule from "#validate/violates-required-rule.mjs";
 
 const SIMPLE_RULE = {
   module: {