Make it impossible to import test files in src

Author: EhabY

.eslintrc.json

@@ -4,7 +4,7 @@
 	"parserOptions": {
 		"ecmaVersion": 6,
 		"sourceType": "module",
-		"project": "./tsconfig.json"
+		"project": true
 	},
 	"plugins": ["@typescript-eslint", "prettier", "import"],
 	"extends": [
@@ -15,7 +15,25 @@
 		"plugin:md/prettier",
 		"prettier"
 	],
+	"ignorePatterns": ["out", "dist", "**/*.d.ts"],
+	"settings": {
+		"import/resolver": {
+			"typescript": { "project": "./tsconfig.json" }
+		},
+		"import/internal-regex": "^@/"
+	},
 	"overrides": [
+		{
+			"files": ["test/**/*.{ts,tsx}", "**/*.{test,spec}.ts?(x)"],
+			"settings": {
+				"import/resolver": {
+					"typescript": {
+						// In tests, resolve using the test tsconfig (has @test/* mapping)
+						"project": "test/tsconfig.json"
+					}
+				}
+			}
+		},
 		{
 			"files": ["*.ts"],
 			"rules": {
@@ -88,12 +106,5 @@
 				}
 			}
 		]
-	},
-	"ignorePatterns": ["out", "dist", "**/*.d.ts"],
-	"settings": {
-		"import/resolver": {
-			"typescript": { "project": "./tsconfig.json" }
-		},
-		"import/internal-regex": "^@/"
 	}
 }

test/tsconfig.json

@@ -0,0 +1,11 @@
+{
+	"extends": "../tsconfig.json",
+	"compilerOptions": {
+		"baseUrl": "..",
+		"paths": {
+			"@/*": ["src/*"],
+			"@test/*": ["test/*"]
+		}
+	},
+	"include": ["**/*", "../src/**/*"]
+}

test/unit/cliManager.test.ts

@@ -19,7 +19,7 @@ import {
 	MockConfigurationProvider,
 	MockProgressReporter,
 	MockUserInteraction,
-} from "@tests/mocks/testHelpers";
+} from "@test/mocks/testHelpers";
 
 vi.mock("os");
 vi.mock("axios");

test/unit/mementoManager.test.ts

@@ -2,7 +2,7 @@ import { beforeEach, describe, expect, it } from "vitest";
 
 import { MementoManager } from "@/core/mementoManager";
 
-import { InMemoryMemento } from "@tests/mocks/testHelpers";
+import { InMemoryMemento } from "@test/mocks/testHelpers";
 
 describe("MementoManager", () => {
 	let memento: InMemoryMemento;

test/unit/secretsManager.test.ts

@@ -2,7 +2,7 @@ import { beforeEach, describe, expect, it } from "vitest";
 
 import { SecretsManager } from "@/core/secretsManager";
 
-import { InMemorySecretStorage } from "@tests/mocks/testHelpers";
+import { InMemorySecretStorage } from "@test/mocks/testHelpers";
 
 describe("SecretsManager", () => {
 	let secretStorage: InMemorySecretStorage;

tsconfig.json

@@ -17,11 +17,9 @@
 			// conflicting types.  For some reason TypeScript is reading both and
 			// throwing errors about AxiosInstance not being compatible with
 			// AxiosInstance.  This ensures we use only index.d.ts.
-			"axios": ["./node_modules/axios/index.d.ts"],
-			"@/*": ["src/*"],
-			"@tests/*": ["test/*"]
+			"axios": ["./node_modules/axios/index.d.ts"]
 		}
 	},
 	"exclude": ["node_modules"],
-	"include": ["src/**/*", "test/**/*"]
+	"include": ["src"]
 }

vitest.config.ts

@@ -21,7 +21,7 @@ export default defineConfig({
 	resolve: {
 		alias: {
 			"@": path.resolve(__dirname, "src"),
-			"@tests": path.resolve(__dirname, "test"),
+			"@test": path.resolve(__dirname, "test"),
 			vscode: path.resolve(__dirname, "test/mocks/vscode.runtime.ts"),
 		},
 	},