Merge pull request #111 from KubrickCode/develop/shlee/90

test: standardize test file location and improve Jest configuration

Author: KubrickCode

package.json

@@ -271,8 +271,8 @@
     "install-package": "code --install-extension quick-command-buttons-$npm_package_version.vsix",
     "lint": "pnpm lint:format && pnpm lint:extension && pnpm lint:view",
     "lint:config": "npx prettier --write \"**/*.{json,yml,yaml,md}\"",
-    "lint:extension": "cd src && npx eslint --fix extension/main.ts internal pkg shared tests",
-    "lint:format": "npx prettier --write \"src/{extension,internal,pkg,shared,tests}/**/*.ts\" \"src/view/src/**/*.{ts,tsx}\" \"**/*.{json,yml,yaml,md}\"",
+    "lint:extension": "cd src && npx eslint --fix extension/main.ts internal pkg shared",
+    "lint:format": "npx prettier --write \"src/{extension,internal,pkg,shared}/**/*.ts\" \"src/view/src/**/*.{ts,tsx}\" \"**/*.{json,yml,yaml,md}\"",
     "lint:view": "cd src/view && npx eslint --fix src/**/*.{ts,tsx}",
     "ovsx-publish": "ovsx publish",
     "package": "vsce package --no-dependencies --allow-star-activation && find . -maxdepth 1 -name '*.vsix' ! -name '*'$npm_package_version'.vsix' -delete",

src/eslint.config.js

@@ -27,7 +27,7 @@ export default [
     },
   },
   {
-    files: ["tests/**/*.ts"],
+    files: ["**/*.spec.ts"],
     languageOptions: {
       parser: tseslint.parser,
       parserOptions: {

src/extension/__mocks__/zod.js

@@ -0,0 +1,17 @@
+// Mock zod for Jest
+const mockZodObject = () => ({
+  optional: () => mockZodObject(),
+  parse: (value) => value,
+  safeParse: (value) => ({ success: true, data: value }),
+});
+
+const z = {
+  string: () => mockZodObject(),
+  boolean: () => mockZodObject(),
+  array: () => mockZodObject(),
+  object: () => mockZodObject(),
+  lazy: () => mockZodObject(),
+  ZodType: class ZodType {},
+};
+
+module.exports = { z };

src/extension/jest.config.cjs

@@ -1,27 +1,40 @@
 /** @type {import('jest').Config} */
 module.exports = {
-  preset: "ts-jest",
-  testEnvironment: "node",
-  roots: ["<rootDir>/../tests"],
-  testMatch: ["**/*.spec.ts"],
-  transform: {
-    "^.+\\.ts$": "ts-jest",
-  },
+  clearMocks: true,
   collectCoverageFrom: [
-    "../extension/**/*.ts",
-    "../internal/**/*.ts",
-    "../pkg/**/*.ts",
+    "<rootDir>/extension/**/*.ts",
+    "<rootDir>/internal/**/*.ts",
+    "<rootDir>/pkg/**/*.ts",
     "!**/*.d.ts",
     "!**/*.spec.ts",
+    "!**/__mocks__/**",
   ],
-  coverageDirectory: "coverage",
+  coverageDirectory: "<rootDir>/extension/coverage",
   coverageReporters: ["text", "lcov", "html"],
+  moduleDirectories: [
+    "node_modules",
+    "<rootDir>/extension/node_modules",
+    "<rootDir>/../node_modules",
+  ],
   moduleFileExtensions: ["ts", "js", "json"],
-  moduleDirectories: ["node_modules", "<rootDir>/node_modules"],
-  verbose: true,
-  clearMocks: true,
-  restoreMocks: true,
   moduleNameMapper: {
-    "^vscode$": "<rootDir>/__mocks__/vscode.js",
+    "^vscode$": "<rootDir>/extension/__mocks__/vscode.js",
+    "^zod$": "<rootDir>/extension/__mocks__/zod.js",
+  },
+  preset: "ts-jest",
+  restoreMocks: true,
+  rootDir: "..",
+  roots: ["<rootDir>/extension", "<rootDir>/internal", "<rootDir>/pkg"],
+  testEnvironment: "node",
+  testMatch: ["**/*.spec.ts"],
+  transform: {
+    "^.+\\.ts$": [
+      "ts-jest",
+      {
+        tsconfig: "<rootDir>/extension/tsconfig.json",
+      },
+    ],
   },
+  transformIgnorePatterns: ["node_modules/(?!(zod)/)"],
+  verbose: true,
 };

src/tests/main.spec.ts renamed to src/extension/main.spec.ts

@@ -1,5 +1,5 @@
 import * as vscode from "vscode";
-import { registerCommands } from "../extension/main";
+import { registerCommands } from "./main";
 import { ConfigManager } from "../internal/managers/config-manager";
 import { StatusBarManager } from "../internal/managers/status-bar-manager";
 import { TerminalManager } from "../internal/managers/terminal-manager";

src/extension/package.json

@@ -2,7 +2,7 @@
   "name": "quick-command-buttons-extension",
   "private": true,
   "scripts": {
-    "lint": "cd ../.. && npx prettier --write \"src/{extension,internal,pkg,shared,tests}/**/*.ts\" && cd src && npx eslint --fix extension/main.ts internal pkg shared tests"
+    "lint": "cd ../.. && npx prettier --write \"src/{extension,internal,pkg,shared}/**/*.ts\" && cd src && npx eslint --fix extension/main.ts internal pkg shared"
   },
   "dependencies": {
     "@indic-transliteration/sanscript": "1.3.3",

src/tests/adapters.spec.ts renamed to src/internal/adapters.spec.ts

@@ -1,5 +1,5 @@
 import * as vscode from "vscode";
-import { createVSCodeConfigReader } from "../internal/adapters";
+import { createVSCodeConfigReader } from "./adapters";
 
 describe("adapters", () => {
   beforeEach(() => {

src/tests/command-executor.spec.ts renamed to src/internal/command-executor.spec.ts

@@ -1,12 +1,12 @@
+import { ButtonConfig } from "../pkg/types";
 import {
   determineButtonExecutionType,
   executeCommandsRecursively,
   executeTerminalCommand,
   findShortcutItem,
   validateShortcuts,
-} from "../internal/command-executor";
-import { createQuickPickItems } from "../internal/utils/ui-items";
-import { ButtonConfig } from "../pkg/types";
+} from "./command-executor";
+import { createQuickPickItems } from "./utils/ui-items";
 
 describe("command-executor", () => {
   describe("validateShortcuts", () => {

src/tests/keyboard-layout-converter.spec.ts renamed to src/internal/keyboard-layout-converter.spec.ts

@@ -4,7 +4,7 @@ import {
   exactMatcher,
   caseInsensitiveMatcher,
   layoutAwareMatcher,
-} from "../internal/keyboard-layout-converter";
+} from "./keyboard-layout-converter";
 
 describe("generateKeyVariants", () => {
   it("should return input key with case variations for single character", async () => {

src/tests/config-manager.spec.ts renamed to src/internal/managers/config-manager.spec.ts

@@ -1,6 +1,6 @@
 import * as vscode from "vscode";
-import { ConfigManager } from "../internal/managers/config-manager";
-import { CONFIGURATION_TARGETS } from "../pkg/config-constants";
+import { CONFIGURATION_TARGETS } from "../../pkg/config-constants";
+import { ConfigManager } from "./config-manager";
 
 describe("ConfigManager", () => {
   const createMockConfig = () => ({