test(cli): makes integration test case easier to manually veriry

sverweij · Dec 9, 2023 · a995d26 · a995d26
1 parent 709e520
commit a995d26
Show file tree

Hide file tree

Showing 10 changed files with 132 additions and 63 deletions.
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/direct-relative-import.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/direct-relative-import.ts
@@ -0,0 +1,3 @@
+import aap from "../../libs/aap";
+
+console.log(aap);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-subpath-import.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-subpath-import.ts
@@ -0,0 +1,3 @@
+import subPathImport from "#mies/index.js";
+
+console.log(subPathImport);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-base-url.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-base-url.ts
@@ -0,0 +1,3 @@
+import tsConfigBasePath from "libs/wim";
+
+console.log(tsConfigBasePath);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-path-star.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-path-star.ts
@@ -0,0 +1,3 @@
+import tsConfigPathStar from "@modules/zus/index.js";
+
+console.log(tsConfigPathStar);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-path.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-tsconfig-path.ts
@@ -0,0 +1,3 @@
+import tsConfigPath from "@sister";
+
+console.log(tsConfigPath);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-workspace.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/from-workspace.ts
@@ -0,0 +1,3 @@
+import workspace from "noot/index.js";
+
+console.log(workspace);
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/typescript-though.ts b/test/cli/__fixtures__/workspaces-mono-repo-aliases/apps/admin/typescript-though.ts
diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/expected.json b/test/cli/__fixtures__/workspaces-mono-repo-aliases/expected.json
@@ -1,46 +1,37 @@
 {
   "modules": [
     {
-      "source": "apps/admin/typescript-though.ts",
+      "source": "apps/admin/direct-relative-import.ts",
       "dependencies": [
         {
           "dynamic": false,
-          "module": "@schwester/index.js",
+          "module": "../../libs/aap",
           "moduleSystem": "es6",
           "exoticallyRequired": false,
-          "resolved": "libs/zus/index.js",
-          "coreModule": false,
-          "followable": true,
-          "couldNotResolve": false,
-          "dependencyTypes": [
-            "aliased",
-            "aliased-tsconfig",
-            "aliased-tsconfig-paths",
-            "local"
-          ],
-          "matchesDoNotFollow": false,
-          "circular": false,
-          "valid": true
-        },
-        {
-          "dynamic": false,
-          "module": "@sister",
-          "moduleSystem": "es6",
-          "exoticallyRequired": false,
-          "resolved": "libs/zus/index.js",
+          "resolved": "libs/aap/index.js",
           "coreModule": false,
           "followable": true,
           "couldNotResolve": false,
-          "dependencyTypes": [
-            "aliased",
-            "aliased-tsconfig",
-            "aliased-tsconfig-paths",
-            "local"
-          ],
+          "dependencyTypes": ["local"],
           "matchesDoNotFollow": false,
           "circular": false,
           "valid": true
-        },
+        }
+      ],
+      "dependents": [],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "libs/aap/index.js",
+      "dependencies": [],
+      "dependents": ["apps/admin/direct-relative-import.ts"],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "apps/admin/from-subpath-import.ts",
+      "dependencies": [
         {
           "dynamic": false,
           "module": "#mies/index.js",
@@ -54,13 +45,28 @@
           "matchesDoNotFollow": false,
           "circular": false,
           "valid": true
-        },
+        }
+      ],
+      "dependents": [],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "libs/mies/index.js",
+      "dependencies": [],
+      "dependents": ["apps/admin/from-subpath-import.ts"],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "apps/admin/from-tsconfig-base-url.ts",
+      "dependencies": [
         {
           "dynamic": false,
-          "module": "libs/aap",
+          "module": "libs/wim",
           "moduleSystem": "es6",
           "exoticallyRequired": false,
-          "resolved": "libs/aap/index.js",
+          "resolved": "libs/wim/index.js",
           "coreModule": false,
           "followable": true,
           "couldNotResolve": false,
@@ -73,17 +79,37 @@
           "matchesDoNotFollow": false,
           "circular": false,
           "valid": true
-        },
+        }
+      ],
+      "dependents": [],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "libs/wim/index.js",
+      "dependencies": [],
+      "dependents": ["apps/admin/from-tsconfig-base-url.ts"],
+      "orphan": false,
+      "valid": true
+    },
+    {
+      "source": "apps/admin/from-tsconfig-path-star.ts",
+      "dependencies": [
         {
           "dynamic": false,
-          "module": "noot/index.js",
+          "module": "@modules/zus/index.js",
           "moduleSystem": "es6",
           "exoticallyRequired": false,
-          "resolved": "libs/noot/index.js",
+          "resolved": "libs/zus/index.js",
           "coreModule": false,
           "followable": true,
           "couldNotResolve": false,
-          "dependencyTypes": ["aliased", "aliased-workspace", "local"],
+          "dependencyTypes": [
+            "aliased",
+            "aliased-tsconfig",
+            "aliased-tsconfig-paths",
+            "local"
+          ],
           "matchesDoNotFollow": false,
           "circular": false,
           "valid": true
@@ -96,28 +122,66 @@
     {
       "source": "libs/zus/index.js",
       "dependencies": [],
-      "dependents": ["apps/admin/typescript-though.ts"],
+      "dependents": [
+        "apps/admin/from-tsconfig-path-star.ts",
+        "apps/admin/from-tsconfig-path.ts"
+      ],
       "orphan": false,
       "valid": true
     },
     {
-      "source": "libs/mies/index.js",
-      "dependencies": [],
-      "dependents": ["apps/admin/typescript-though.ts"],
+      "source": "apps/admin/from-tsconfig-path.ts",
+      "dependencies": [
+        {
+          "dynamic": false,
+          "module": "@sister",
+          "moduleSystem": "es6",
+          "exoticallyRequired": false,
+          "resolved": "libs/zus/index.js",
+          "coreModule": false,
+          "followable": true,
+          "couldNotResolve": false,
+          "dependencyTypes": [
+            "aliased",
+            "aliased-tsconfig",
+            "aliased-tsconfig-paths",
+            "local"
+          ],
+          "matchesDoNotFollow": false,
+          "circular": false,
+          "valid": true
+        }
+      ],
+      "dependents": [],
       "orphan": false,
       "valid": true
     },
     {
-      "source": "libs/aap/index.js",
-      "dependencies": [],
-      "dependents": ["apps/admin/typescript-though.ts"],
+      "source": "apps/admin/from-workspace.ts",
+      "dependencies": [
+        {
+          "dynamic": false,
+          "module": "noot/index.js",
+          "moduleSystem": "es6",
+          "exoticallyRequired": false,
+          "resolved": "libs/noot/index.js",
+          "coreModule": false,
+          "followable": true,
+          "couldNotResolve": false,
+          "dependencyTypes": ["aliased", "aliased-workspace", "local"],
+          "matchesDoNotFollow": false,
+          "circular": false,
+          "valid": true
+        }
+      ],
+      "dependents": [],
       "orphan": false,
       "valid": true
     },
     {
       "source": "libs/noot/index.js",
       "dependencies": [],
-      "dependents": ["apps/admin/typescript-though.ts"],
+      "dependents": ["apps/admin/from-workspace.ts"],
       "orphan": false,
       "valid": true
     }
@@ -128,8 +192,8 @@
     "warn": 0,
     "info": 0,
     "ignore": 0,
-    "totalCruised": 5,
-    "totalDependenciesCruised": 5,
+    "totalCruised": 11,
+    "totalDependenciesCruised": 6,
     "optionsUsed": {
       "baseDir": ".",
       "cache": false,
@@ -149,7 +213,7 @@
         "fileName": "tsconfig.json"
       },
       "tsPreCompilationDeps": true,
-      "args": "apps/admin/typescript-though.ts"
+      "args": "apps/admin/"
     },
     "ruleSetUsed": {
       "forbidden": []

diff --git a/test/cli/__fixtures__/workspaces-mono-repo-aliases/tsconfig.json b/test/cli/__fixtures__/workspaces-mono-repo-aliases/tsconfig.json
@@ -4,7 +4,7 @@
     "module": "commonjs",
     "baseUrl": "./",
     "paths": {
-      "@schwester/*": ["./libs/zus/*"],
+      "@modules/*": ["./libs/*"],
       "@sister": ["./libs/zus/*"]
     },
     "noEmit": true,

diff --git a/test/cli/index.spec.mjs b/test/cli/index.spec.mjs
@@ -1,7 +1,7 @@
 import { readFileSync, unlinkSync } from "node:fs";
+import { doesNotThrow, equal, throws } from "node:assert/strict";
 // path.posix instead of path because otherwise on win32 the resulting
 // outputTo would contain \\ instead of / which for this unit test doesn't matter
-import { doesNotThrow, equal, throws } from "node:assert/strict";
 import { join, posix as path } from "node:path";
 import chalk from "chalk";
 import intercept from "intercept-stdout";
@@ -506,7 +506,7 @@ describe("[E] cli/index", () => {
     const lOutputTo = path.join("../../__output__", lOutputFileName);
     process.chdir("./test/cli/__fixtures__/workspaces-mono-repo-aliases");
 
-    const lExitCode = await cli(["apps/admin/typescript-though.ts"], {
+    const lExitCode = await cli(["apps/admin/"], {
       config: ".dependency-cruiser.js",
       outputType: "json",
       outputTo: lOutputTo,