@simplysm/sd-cli 14.0.64 → 14.0.66

package/tests/engines/esbuild-client-engine.acc.spec.ts

@@ -1,6 +1,8 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories (vi.mock is hoisted) ---
+import { existsSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import path from "node:path";
+import { Worker } from "@simplysm/core-node";
 
 const mockWorker = {
   build: vi.fn(),
@@ -10,24 +12,9 @@ const mockWorker = {
   on: vi.fn(),
 };
 
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(() => mockWorker),
-  },
-}));
-
-// fs mock for .dev-port deletion in stop()
-const mockUnlinkSync = vi.fn();
-vi.mock("node:fs", () => ({
-  default: {
-    unlinkSync: (...args: any[]) => mockUnlinkSync(...args),
-  },
-  unlinkSync: (...args: any[]) => mockUnlinkSync(...args),
-}));
+vi.spyOn(Worker, "create").mockReturnValue(mockWorker as any);
 
-// --- Dynamic imports after mocking ---
-
-const { EsbuildClientEngine } = await import("../../src/engines/EsbuildClientEngine");
+import { EsbuildClientEngine } from "../../src/engines/EsbuildClientEngine";
 
 import type { ClientPackageInfo } from "../../src/engines/types";
 
@@ -244,26 +231,35 @@ describe("EsbuildClientEngine Acceptance", () => {
 
   // Scenario: 엔진 중지
   it("stop()으로 worker를 종료하고 .dev-port를 삭제한다", async () => {
-    // Given: dev watch 모드가 실행 중이다
-    mockWorker.startWatch.mockResolvedValue({ success: true });
-
-    const engine = new EsbuildClientEngine({
-      cwd: "/root",
-      pkg: createMockPkg({ dir: "/packages/my-client" }),
-    });
-
-    await engine.startWatch({ js: true, dts: false });
-
-    // When: stop()이 호출된다
-    await engine.stop();
-
-    // Then: worker가 종료된다
-    expect(mockWorker.stopWatch).toHaveBeenCalled();
-    expect(mockWorker.terminate).toHaveBeenCalled();
-
-    // And: .dev-port 파일 삭제가 시도된다
-    expect(mockUnlinkSync).toHaveBeenCalledWith(
-      expect.stringContaining(".dev-port"),
-    );
+    // Given: dev watch 모드가 실행 중이고 .dev-port 파일이 존재한다
+    const tmpRoot = mkdtempSync(path.join(tmpdir(), "esbuild-client-engine-"));
+    try {
+      const pkgDir = path.join(tmpRoot, "my-client");
+      const distDir = path.join(pkgDir, "dist");
+      mkdirSync(distDir, { recursive: true });
+      const portFile = path.join(distDir, ".dev-port");
+      writeFileSync(portFile, "4200");
+
+      mockWorker.startWatch.mockResolvedValue({ success: true });
+
+      const engine = new EsbuildClientEngine({
+        cwd: tmpRoot,
+        pkg: createMockPkg({ dir: pkgDir }),
+      });
+
+      await engine.startWatch({ js: true, dts: false });
+
+      // When: stop()이 호출된다
+      await engine.stop();
+
+      // Then: worker가 종료된다
+      expect(mockWorker.stopWatch).toHaveBeenCalled();
+      expect(mockWorker.terminate).toHaveBeenCalled();
+
+      // And: .dev-port 파일이 삭제된다
+      expect(existsSync(portFile)).toBe(false);
+    } finally {
+      rmSync(tmpRoot, { recursive: true, force: true });
+    }
   });
 });

package/tests/engines/esbuild-client-engine.spec.ts

@@ -1,6 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories (vi.mock is hoisted) ---
+import { Worker } from "@simplysm/core-node";
 
 const mockWorker = {
   build: vi.fn(),
@@ -10,24 +9,9 @@ const mockWorker = {
   on: vi.fn(),
 };
 
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(() => mockWorker),
-  },
-}));
-
-// fs mock
-const mockUnlinkSync = vi.fn();
-vi.mock("node:fs", () => ({
-  default: {
-    unlinkSync: (...args: any[]) => mockUnlinkSync(...args),
-  },
-  unlinkSync: (...args: any[]) => mockUnlinkSync(...args),
-}));
-
-// --- Dynamic imports after mocking ---
+vi.spyOn(Worker, "create").mockReturnValue(mockWorker as any);
 
-const { EsbuildClientEngine } = await import("../../src/engines/EsbuildClientEngine");
+import { EsbuildClientEngine } from "../../src/engines/EsbuildClientEngine";
 
 import type { ClientPackageInfo } from "../../src/engines/types";
 
@@ -400,12 +384,9 @@ describe("EsbuildClientEngine", () => {
     });
 
     it(".dev-port 파일 삭제 실패를 무시한다", async () => {
-      mockUnlinkSync.mockImplementation(() => {
-        throw new Error("ENOENT");
-      });
-
       mockWorker.build.mockResolvedValue({ success: true });
 
+      // pkg.dir이 실재하지 않는 경로이므로 unlinkSync는 ENOENT를 throw한다
       const engine = new EsbuildClientEngine({ cwd: "/root", pkg: createMockPkg() });
       await engine.run({ js: true, dts: false });
 

package/tests/engines/ngtsc-engine.spec.ts

@@ -1,6 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories (vi.mock is hoisted) ---
+import { Worker } from "@simplysm/core-node";
 
 const mockWorker = {
   build: vi.fn(),
@@ -10,15 +9,9 @@ const mockWorker = {
   on: vi.fn(),
 };
 
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(() => mockWorker),
-  },
-}));
-
-// --- Dynamic imports after mocking ---
+vi.spyOn(Worker, "create").mockReturnValue(mockWorker as any);
 
-const { NgtscEngine } = await import("../../src/engines/NgtscEngine");
+import { NgtscEngine } from "../../src/engines/NgtscEngine";
 
 import type { BuildPackageInfo } from "../../src/engines/types";
 

package/tests/engines/server-esbuild-engine.spec.ts

@@ -1,6 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories ---
+import { Worker } from "@simplysm/core-node";
 
 const mockWorker = {
   build: vi.fn(),
@@ -10,15 +9,9 @@ const mockWorker = {
   on: vi.fn(),
 };
 
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(() => mockWorker),
-  },
-}));
-
-// --- Dynamic imports ---
+vi.spyOn(Worker, "create").mockReturnValue(mockWorker as any);
 
-const { ServerEsbuildEngine } = await import("../../src/engines/ServerEsbuildEngine");
+import { ServerEsbuildEngine } from "../../src/engines/ServerEsbuildEngine";
 
 import type { ServerPackageInfo } from "../../src/engines/types";
 

package/tests/engines/tsc-engine.spec.ts

@@ -1,6 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories (vi.mock is hoisted) ---
+import { Worker } from "@simplysm/core-node";
 
 const mockWorker = {
   build: vi.fn(),
@@ -10,15 +9,9 @@ const mockWorker = {
   on: vi.fn(),
 };
 
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(() => mockWorker),
-  },
-}));
-
-// --- Dynamic imports after mocking ---
+vi.spyOn(Worker, "create").mockReturnValue(mockWorker as any);
 
-const { TscEngine } = await import("../../src/engines/TscEngine");
+import { TscEngine } from "../../src/engines/TscEngine";
 
 import type { BuildPackageInfo } from "../../src/engines/types";
 

package/tests/esbuild/esbuild-tsc-plugin.acc.spec.ts

@@ -1,21 +1,16 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import type esbuild from "esbuild";
 import type { ISdTsCompilerResult } from "../../src/ts-compiler/sd-ts-compiler-result";
-
-//#region Mocks
+import * as sdTsCompilerMod from "../../src/ts-compiler/SdTsCompiler";
 
 const mockCompileAsync = vi.fn<() => Promise<ISdTsCompilerResult>>();
 const MockSdTsCompiler = vi.fn().mockImplementation(function () {
   return { compileAsync: mockCompileAsync };
 });
 
-vi.mock("../../src/ts-compiler/SdTsCompiler", () => ({
-  SdTsCompiler: MockSdTsCompiler,
-}));
-
-//#endregion
+vi.spyOn(sdTsCompilerMod, "SdTsCompiler" as any).mockImplementation(MockSdTsCompiler as any);
 
-const { createTscPlugin } = await import("../../src/esbuild/esbuild-tsc-plugin");
+import { createTscPlugin } from "../../src/esbuild/esbuild-tsc-plugin";
 
 /** esbuild 플러그인 lifecycle을 시뮬레이션하는 헬퍼 */
 function setupPlugin(plugin: esbuild.Plugin) {

package/tests/esbuild/esbuild-tsc-plugin.spec.ts

@@ -1,21 +1,16 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
 import type esbuild from "esbuild";
 import type { ISdTsCompilerResult } from "../../src/ts-compiler/sd-ts-compiler-result";
-
-//#region Mocks
+import * as sdTsCompilerMod from "../../src/ts-compiler/SdTsCompiler";
 
 const mockCompileAsync = vi.fn<() => Promise<ISdTsCompilerResult>>();
 const MockSdTsCompiler = vi.fn().mockImplementation(function () {
   return { compileAsync: mockCompileAsync };
 });
 
-vi.mock("../../src/ts-compiler/SdTsCompiler", () => ({
-  SdTsCompiler: MockSdTsCompiler,
-}));
-
-//#endregion
+vi.spyOn(sdTsCompilerMod, "SdTsCompiler" as any).mockImplementation(MockSdTsCompiler as any);
 
-const { createTscPlugin } = await import("../../src/esbuild/esbuild-tsc-plugin");
+import { createTscPlugin } from "../../src/esbuild/esbuild-tsc-plugin";
 
 /** esbuild 플러그인 lifecycle을 시뮬레이션하는 헬퍼 */
 function setupPlugin(plugin: esbuild.Plugin) {

package/tests/orchestrators/build-orchestrator.spec.ts

@@ -1,9 +1,29 @@
 import { describe, it, expect, vi, beforeEach } from "vitest";
-
-// --- Mock factories (vi.mock is hoisted) ---
-
 import { consola } from "consola";
 
+import * as sdConfig from "../../src/utils/sd-config";
+import * as buildEnv from "../../src/utils/build-env";
+import * as copySrc from "../../src/utils/copy-src";
+import * as lintUtils from "../../src/lint/lint-utils";
+import * as outputUtils from "../../src/utils/output-utils";
+import * as typecheckSerialization from "../../src/typecheck/typecheck-serialization";
+import * as coreNode from "@simplysm/core-node";
+import * as engineFactory from "../../src/engines/engine-factory";
+import * as capacitorMod from "../../src/capacitor/capacitor";
+import * as electronMod from "../../src/electron/electron";
+
+import { loadSdConfig } from "../../src/utils/sd-config";
+import { getVersion } from "../../src/utils/build-env";
+import { copySrcFiles } from "../../src/utils/copy-src";
+import { runLintInWorker } from "../../src/lint/lint-utils";
+import { Worker, fsx } from "@simplysm/core-node";
+import { createBuildEngine } from "../../src/engines/engine-factory";
+import { Capacitor } from "../../src/capacitor/capacitor";
+import { Electron } from "../../src/electron/electron";
+
+import { BuildOrchestrator, classifyPackages } from "../../src/orchestrators/BuildOrchestrator";
+import type { SdConfig } from "../../src/sd-config.types";
+
 const mockLogger = {
   debug: vi.fn(),
   info: vi.fn(),
@@ -13,106 +33,24 @@ const mockLogger = {
   success: vi.fn(),
 };
 
-vi.spyOn(consola, "withTag").mockReturnValue(mockLogger as any);
-
-vi.mock("../../src/utils/sd-config", () => ({
-  loadSdConfig: vi.fn(),
-}));
-
-vi.mock("../../src/utils/build-env", () => ({
-  getVersion: vi.fn(),
-}));
-
-vi.mock("../../src/utils/copy-src", () => ({
-  copySrcFiles: vi.fn().mockResolvedValue(undefined),
-}));
-
-vi.mock("../../src/lint/lint-utils", () => ({
-  runLintInWorker: vi.fn().mockResolvedValue({ success: true, errorCount: 0, warningCount: 0, formattedOutput: "" }),
-}));
-
-vi.mock("../../src/utils/output-utils", () => ({
-  formatBuildMessages: vi.fn(
-    (name: string, target: string, msgs: string[]) => `${name} (${target}): ${msgs.join(", ")}`,
-  ),
-}));
-
-vi.mock("../../src/typecheck/typecheck-serialization", () => ({
-  deserializeDiagnostic: vi.fn((d: any) => d),
-}));
-
-vi.mock("@simplysm/core-node", () => ({
-  Worker: {
-    create: vi.fn(),
-  },
-  fsx: {
-    rm: vi.fn().mockResolvedValue(undefined),
-  },
-  pathx: {
-    posixResolve: vi.fn((...args: string[]) => args.join("/").replace(/\\/g, "/")),
-  },
-}));
-
 const mockEngines: Array<{
   run: ReturnType<typeof vi.fn>;
   startWatch: ReturnType<typeof vi.fn>;
   stop: ReturnType<typeof vi.fn>;
 }> = [];
 
-vi.mock("../../src/engines/engine-factory", () => ({
-  createBuildEngine: vi.fn(() => {
-    const engine = {
-      run: vi.fn().mockResolvedValue({
-        build: { success: true, errors: [], warnings: [], diagnostics: [] },
-      }),
-      startWatch: vi.fn().mockResolvedValue(undefined),
-      stop: vi.fn().mockResolvedValue(undefined),
-    };
-    mockEngines.push(engine);
-    return engine;
-  }),
-}));
-
-// Capacitor mock
 const mockCapacitorInstance = {
   initialize: vi.fn().mockResolvedValue(undefined),
   build: vi.fn().mockResolvedValue(undefined),
   run: vi.fn().mockResolvedValue(undefined),
 };
 
-vi.mock("../../src/capacitor/capacitor", () => ({
-  Capacitor: {
-    create: vi.fn().mockResolvedValue(mockCapacitorInstance),
-  },
-}));
-
-// Electron mock
 const mockElectronInstance = {
   initialize: vi.fn().mockResolvedValue(undefined),
   build: vi.fn().mockResolvedValue(undefined),
   run: vi.fn().mockResolvedValue(undefined),
 };
 
-vi.mock("../../src/electron/electron", () => ({
-  Electron: {
-    create: vi.fn().mockResolvedValue(mockElectronInstance),
-  },
-}));
-
-// --- Dynamic imports after mocking ---
-
-const { BuildOrchestrator, classifyPackages } = await import("../../src/orchestrators/BuildOrchestrator");
-const { loadSdConfig } = await import("../../src/utils/sd-config");
-const { getVersion } = await import("../../src/utils/build-env");
-const { copySrcFiles } = await import("../../src/utils/copy-src");
-const { Worker, fsx } = await import("@simplysm/core-node");
-const { createBuildEngine } = await import("../../src/engines/engine-factory");
-
-const { Capacitor } = await import("../../src/capacitor/capacitor");
-const { Electron } = await import("../../src/electron/electron");
-
-import type { SdConfig } from "../../src/sd-config.types";
-
 // --- Helpers ---
 
 interface MockWorkerProxy {
@@ -136,27 +74,46 @@ function setupDefaults(config: Partial<SdConfig> = {}): void {
 // --- Tests ---
 
 beforeEach(() => {
-  vi.clearAllMocks();
+  vi.restoreAllMocks();
   mockEngines.length = 0;
-  // Reset process.exitCode
   process.exitCode = undefined;
-  // Mock process.stdout.write
+
+  vi.spyOn(consola, "withTag").mockReturnValue(mockLogger as any);
+  Object.values(mockLogger).forEach((m) => m.mockReset());
+
+  mockCapacitorInstance.initialize.mockReset().mockResolvedValue(undefined);
+  mockCapacitorInstance.build.mockReset().mockResolvedValue(undefined);
+  mockCapacitorInstance.run.mockReset().mockResolvedValue(undefined);
+  mockElectronInstance.initialize.mockReset().mockResolvedValue(undefined);
+  mockElectronInstance.build.mockReset().mockResolvedValue(undefined);
+  mockElectronInstance.run.mockReset().mockResolvedValue(undefined);
+
   vi.spyOn(process.stdout, "write").mockReturnValue(true);
-  // Reset Capacitor/Electron instance mocks
-  mockCapacitorInstance.initialize.mockResolvedValue(undefined);
-  mockCapacitorInstance.build.mockResolvedValue(undefined);
-  mockCapacitorInstance.run.mockResolvedValue(undefined);
-  mockElectronInstance.initialize.mockResolvedValue(undefined);
-  mockElectronInstance.build.mockResolvedValue(undefined);
-  mockElectronInstance.run.mockResolvedValue(undefined);
-  vi.mocked(Capacitor.create).mockResolvedValue(mockCapacitorInstance as any);
-  vi.mocked(Electron.create).mockResolvedValue(mockElectronInstance as any);
-  // Restore default createBuildEngine mock implementation
-  vi.mocked(createBuildEngine).mockImplementation(() => {
+
+  vi.spyOn(sdConfig, "loadSdConfig").mockResolvedValue({ packages: {} });
+  vi.spyOn(buildEnv, "getVersion").mockResolvedValue("1.0.0");
+  vi.spyOn(copySrc, "copySrcFiles").mockResolvedValue(undefined);
+  vi.spyOn(lintUtils, "runLintInWorker").mockResolvedValue({
+    success: true, errorCount: 0, warningCount: 0, formattedOutput: "",
+  });
+  vi.spyOn(outputUtils, "formatBuildMessages").mockImplementation(
+    (name: string, target: string, msgs: string[]) => `${name} (${target}): ${msgs.join(", ")}`,
+  );
+  vi.spyOn(typecheckSerialization, "deserializeDiagnostic").mockImplementation((d: any) => d);
+
+  vi.spyOn(coreNode.fsx, "rm").mockResolvedValue(undefined);
+  vi.spyOn(coreNode.pathx, "posixResolve").mockImplementation(
+    (...args: string[]) => args.join("/").replace(/\\/g, "/") as coreNode.pathx.PosixPath,
+  );
+  vi.spyOn(coreNode.Worker, "create").mockReturnValue(undefined as any);
+
+  vi.spyOn(capacitorMod.Capacitor, "create").mockResolvedValue(mockCapacitorInstance as any);
+  vi.spyOn(electronMod.Electron, "create").mockResolvedValue(mockElectronInstance as any);
+
+  vi.spyOn(engineFactory, "createBuildEngine").mockImplementation(() => {
     const engine = {
       run: vi.fn().mockResolvedValue({
         build: { success: true, errors: [], warnings: [], diagnostics: [] },
-
       }),
       startWatch: vi.fn().mockResolvedValue(undefined),
       stop: vi.fn().mockResolvedValue(undefined),
@@ -969,8 +926,6 @@ describe("BuildOrchestrator native build integration (Slice 1)", () => {
 
 //#region Slice 3: build 명령어 lint 통합 (Feature 3.2)
 
-const { runLintInWorker } = await import("../../src/lint/lint-utils");
-
 describe("BuildOrchestrator lint integration", () => {
   // build에서 lint를 실행하지 않는다 (lint는 check에서만 실행)
   it("passes lint:false to engine.run for all package types", async () => {