@simplysm/sd-cli 14.0.19 → 14.0.22

package/tests/capacitor/capacitor-android.spec.ts

@@ -0,0 +1,219 @@
+import path from "path";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
+
+//#region Mocks
+
+const mockFsxExists = vi.fn();
+const mockFsxRead = vi.fn();
+const mockFsxWrite = vi.fn().mockResolvedValue(undefined);
+const mockFsxGlob = vi.fn();
+
+vi.mock("@simplysm/core-node", () => ({
+  fsx: {
+    exists: mockFsxExists,
+    read: mockFsxRead,
+    write: mockFsxWrite,
+    glob: mockFsxGlob,
+  },
+  pathx: {
+    posixResolve: (...args: string[]) => path.resolve(...args).replace(/\\/g, "/"),
+    posix: (p: string) => p.replace(/\\/g, "/"),
+  },
+}));
+
+//#endregion
+
+describe("findJava21", () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it("여러 패턴 중 첫 매치를 사용하고 마지막 정렬 결과를 반환한다", async () => {
+    mockFsxGlob.mockImplementation((pattern: string) => {
+      if (pattern.includes("Amazon Corretto")) {
+        return ["C:/Program Files/Amazon Corretto/jdk21.0.1", "C:/Program Files/Amazon Corretto/jdk21.0.3"];
+      }
+      return [];
+    });
+
+    const { findJava21 } = await import("../../src/capacitor/capacitor-android.js");
+    const result = await findJava21();
+    expect(result).toBe("C:/Program Files/Amazon Corretto/jdk21.0.3");
+  });
+
+  it("모든 패턴에 매치가 없으면 undefined를 반환한다", async () => {
+    mockFsxGlob.mockResolvedValue([]);
+
+    const { findJava21 } = await import("../../src/capacitor/capacitor-android.js");
+    const result = await findJava21();
+    expect(result).toBeUndefined();
+  });
+});
+
+describe("findAndroidSdk", () => {
+  let savedEnv: Record<string, string | undefined>;
+  beforeEach(() => {
+    vi.clearAllMocks();
+    savedEnv = { ...process.env };
+  });
+  afterEach(() => {
+    process.env = savedEnv;
+  });
+
+  it("ANDROID_SDK_ROOT로 SDK를 감지한다", async () => {
+    process.env["ANDROID_SDK_ROOT"] = "D:/Android/Sdk";
+    mockFsxExists.mockImplementation((p: string) => p === "D:/Android/Sdk");
+
+    const { findAndroidSdk } = await import("../../src/capacitor/capacitor-android.js");
+    const result = await findAndroidSdk();
+    expect(result).toBe("D:/Android/Sdk");
+  });
+
+  it("환경변수와 후보 경로 모두 없으면 undefined를 반환한다", async () => {
+    delete process.env["ANDROID_SDK_ROOT"];
+    delete process.env["ANDROID_HOME"];
+    mockFsxExists.mockResolvedValue(false);
+
+    const { findAndroidSdk } = await import("../../src/capacitor/capacitor-android.js");
+    const result = await findAndroidSdk();
+    expect(result).toBeUndefined();
+  });
+});
+
+describe("configureAndroid", () => {
+  let savedEnv: Record<string, string | undefined>;
+  beforeEach(() => {
+    vi.clearAllMocks();
+    savedEnv = { ...process.env };
+    process.env["ANDROID_HOME"] = "C:/Android/Sdk";
+  });
+  afterEach(() => {
+    process.env = savedEnv;
+  });
+
+  it("Android 디렉토리가 없으면 에러를 던진다", async () => {
+    mockFsxExists.mockResolvedValue(false);
+
+    const { configureAndroid } = await import("../../src/capacitor/capacitor-android.js");
+    await expect(
+      configureAndroid("/fake/cap", { appId: "com.test.app", appName: "Test" }, { name: "test", version: "1.0.0" }),
+    ).rejects.toThrow("Android 프로젝트 디렉토리를 찾을 수 없습니다");
+  });
+
+  it("모든 Android 설정을 순서대로 수행한다", async () => {
+    mockFsxExists.mockResolvedValue(true);
+    mockFsxGlob.mockResolvedValue(["C:/Program Files/Amazon Corretto/jdk21.0.1"]);
+    mockFsxRead.mockImplementation((p: string) => {
+      if (p.includes("gradle.properties")) return "org.gradle.jvmargs=-Xmx2048m";
+      if (p.includes("local.properties")) return "";
+      if (p.includes("AndroidManifest.xml")) {
+        return '<manifest xmlns:android="http://schemas.android.com/apk/res/android">\n<application>\n<activity android:name=".MainActivity">\n</activity>\n</application>\n</manifest>';
+      }
+      if (p.includes("app/build.gradle")) {
+        return `android {
+    defaultConfig {
+        versionCode 1
+        versionName "1.0"
+        minSdkVersion rootProject.ext.minSdkVersion
+        targetSdkVersion rootProject.ext.targetSdkVersion
+    }
+    buildTypes { release { } }
+}`;
+      }
+      if (p.includes("build.gradle")) {
+        return "buildscript { dependencies { classpath 'com.android.tools.build:gradle:8.2.1' } }";
+      }
+      if (p.includes("styles.xml")) {
+        return `<resources>
+    <style name="AppTheme.NoActionBarLaunch" parent="Theme.SplashScreen">
+        <item name="android:background">@drawable/splash</item>
+    </style>
+</resources>`;
+      }
+      return "";
+    });
+
+    const { configureAndroid } = await import("../../src/capacitor/capacitor-android.js");
+    await configureAndroid(
+      "/fake/cap",
+      { appId: "com.test.app", appName: "Test App", platform: { android: {} } },
+      { name: "test-pkg", version: "2.1.0" },
+    );
+
+    const writeCalls = mockFsxWrite.mock.calls;
+
+    // JAVA_HOME 설정
+    expect(writeCalls.some((c) => typeof c[0] === "string" && c[0].includes("gradle.properties"))).toBe(true);
+
+    // SDK 경로 설정
+    expect(writeCalls.some((c) => typeof c[0] === "string" && c[0].includes("local.properties"))).toBe(true);
+
+    // AndroidManifest.xml usesCleartextTraffic
+    const manifestWrite = writeCalls.find(
+      (c) => typeof c[0] === "string" && c[0].includes("AndroidManifest.xml"),
+    );
+    expect(manifestWrite).toBeDefined();
+    expect(manifestWrite![1]).toContain("usesCleartextTraffic");
+
+    // build.gradle versionCode (2.1.0 → 2001000)
+    const gradleWrite = writeCalls.find(
+      (c) => typeof c[0] === "string" && c[0].includes("app/build.gradle"),
+    );
+    expect(gradleWrite).toBeDefined();
+    expect(gradleWrite![1]).toContain("versionCode 2001000");
+    expect(gradleWrite![1]).toContain('versionName "2.1.0"');
+
+    // styles.xml Theme 변경
+    const stylesWrite = writeCalls.find(
+      (c) => typeof c[0] === "string" && c[0].includes("styles.xml"),
+    );
+    expect(stylesWrite).toBeDefined();
+    expect(stylesWrite![1]).toContain('parent="Theme.AppCompat.DayNight.NoActionBar"');
+    expect(stylesWrite![1]).toContain('"android:windowBackground">@drawable/splash');
+  });
+
+  it("minor/patch >= 100인 버전에서 versionCode가 충돌하지 않는다", async () => {
+    mockFsxExists.mockResolvedValue(true);
+    mockFsxGlob.mockResolvedValue(["C:/Program Files/Amazon Corretto/jdk21.0.1"]);
+    mockFsxRead.mockImplementation((p: string) => {
+      if (p.includes("gradle.properties")) return "";
+      if (p.includes("local.properties")) return "";
+      if (p.includes("AndroidManifest.xml")) {
+        return '<manifest xmlns:android="http://schemas.android.com/apk/res/android">\n<application>\n<activity android:name=".MainActivity">\n</activity>\n</application>\n</manifest>';
+      }
+      if (p.includes("app/build.gradle")) {
+        return "android { defaultConfig { versionCode 1\nversionName \"1.0\"\nminSdkVersion rootProject.ext.minSdkVersion\ntargetSdkVersion rootProject.ext.targetSdkVersion } }";
+      }
+      if (p.includes("build.gradle")) return "";
+      if (p.includes("styles.xml")) {
+        return '<resources><style name="AppTheme.NoActionBarLaunch" parent="Theme.SplashScreen"></style></resources>';
+      }
+      return "";
+    });
+
+    const { configureAndroid } = await import("../../src/capacitor/capacitor-android.js");
+
+    // "1.0.100"과 "1.1.0"은 서로 다른 versionCode를 가져야 한다
+    await configureAndroid(
+      "/fake/cap",
+      { appId: "com.test.app", appName: "Test" },
+      { name: "test", version: "1.0.100" },
+    );
+    const call1 = mockFsxWrite.mock.calls.find(
+      (c) => typeof c[0] === "string" && c[0].includes("app/build.gradle"),
+    );
+    expect(call1![1]).toContain("versionCode 1000100");
+
+    mockFsxWrite.mockClear();
+
+    await configureAndroid(
+      "/fake/cap",
+      { appId: "com.test.app", appName: "Test" },
+      { name: "test", version: "1.1.0" },
+    );
+    const call2 = mockFsxWrite.mock.calls.find(
+      (c) => typeof c[0] === "string" && c[0].includes("app/build.gradle"),
+    );
+    expect(call2![1]).toContain("versionCode 1001000");
+  });
+});

package/tests/capacitor/capacitor-build.spec.ts

@@ -1,5 +1,5 @@
 import path from "path";
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 
 //#region Mocks
 
@@ -36,14 +36,6 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-// env mock
-let mockEnv: Record<string, unknown> = {};
-vi.mock("@simplysm/core-common", () => ({
-  env: new Proxy({} as Record<string, unknown>, {
-    get: (_target, prop) => mockEnv[prop as string],
-  }),
-}));
-
 // cpx mock (was execa)
 const execaCalls: { command: string; args: string[] }[] = [];
 const mockCpxSpawn = vi.fn((...args: unknown[]) => {
@@ -71,19 +63,15 @@ vi.mock("sharp", () => ({
   }),
 }));
 
-// consola mock
-const mockLoggerDebug = vi.fn();
+// consola mock (logger assertion 필요)
 const mockLoggerWarn = vi.fn();
-const mockLoggerSuccess = vi.fn();
+const _mockConsola = {
+  level: 0,
+  withTag: () => ({ debug: vi.fn(), warn: mockLoggerWarn, error: vi.fn(), info: vi.fn(), success: vi.fn() }),
+};
 vi.mock("consola", () => ({
-  consola: {
-    withTag: () => ({
-      debug: mockLoggerDebug,
-      warn: mockLoggerWarn,
-      success: mockLoggerSuccess,
-    }),
-    level: 0,
-  },
+  consola: _mockConsola,
+  default: _mockConsola,
   LogLevels: { debug: 4 },
 }));
 
@@ -167,7 +155,7 @@ function setupDefaultMocks() {
     return [];
   });
 
-  mockEnv = { ANDROID_HOME: "C:/Android/Sdk" };
+  process.env["ANDROID_HOME"] = "C:/Android/Sdk";
 
   execaCalls.length = 0;
   mockFsWriteFile.mockReset();
@@ -176,6 +164,14 @@ function setupDefaultMocks() {
 
 //#endregion
 
+let savedEnv: Record<string, string | undefined>;
+beforeEach(() => {
+  savedEnv = { ...process.env };
+});
+afterEach(() => {
+  process.env = savedEnv;
+});
+
 describe("Capacitor 빌드", () => {
   beforeEach(() => {
     vi.clearAllMocks();

package/tests/capacitor/capacitor-icon.spec.ts

@@ -1,5 +1,5 @@
 import path from "path";
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 
 //#region Mocks
 
@@ -34,12 +34,6 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-// env mock
-let mockEnv: Record<string, string | undefined> = {};
-vi.mock("@simplysm/core-common", () => ({
-  env: (key: string) => mockEnv[key],
-}));
-
 // cpx mock (was execa)
 const execaCalls: { command: string; args: string[] }[] = [];
 const mockCpxSpawn = vi.fn((...args: unknown[]) => {
@@ -69,19 +63,15 @@ const mockSharpInstance = {
 const mockSharp = vi.fn().mockReturnValue(mockSharpInstance);
 vi.mock("sharp", () => ({ default: mockSharp }));
 
-// consola mock
-const mockLoggerDebug = vi.fn();
+// consola mock (logger assertion 필요)
 const mockLoggerWarn = vi.fn();
-const mockLoggerSuccess = vi.fn();
+const _mockConsola = {
+  level: 0,
+  withTag: () => ({ debug: vi.fn(), warn: mockLoggerWarn, error: vi.fn(), info: vi.fn(), success: vi.fn() }),
+};
 vi.mock("consola", () => ({
-  consola: {
-    withTag: () => ({
-      debug: mockLoggerDebug,
-      warn: mockLoggerWarn,
-      success: mockLoggerSuccess,
-    }),
-    level: 0,
-  },
+  consola: _mockConsola,
+  default: _mockConsola,
   LogLevels: { debug: 4 },
 }));
 
@@ -149,7 +139,7 @@ function setupDefaultMocks() {
   mockFsxGlob.mockResolvedValue(["C:/Program Files/Amazon Corretto/jdk21.0.1"]);
 
   // env: Android SDK
-  mockEnv = { ANDROID_HOME: "C:/Android/Sdk" };
+  process.env["ANDROID_HOME"] = "C:/Android/Sdk";
 
   // execa 호출 기록 초기화
   execaCalls.length = 0;
@@ -157,6 +147,14 @@ function setupDefaultMocks() {
 
 //#endregion
 
+let savedEnv: Record<string, string | undefined>;
+beforeEach(() => {
+  savedEnv = { ...process.env };
+});
+afterEach(() => {
+  process.env = savedEnv;
+});
+
 describe("Capacitor 아이콘 처리", () => {
   beforeEach(() => {
     vi.clearAllMocks();

package/tests/capacitor/capacitor-init.spec.ts

@@ -1,5 +1,5 @@
 import path from "path";
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 
 //#region Mocks
 
@@ -35,11 +35,6 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-let mockEnv: Record<string, string | undefined> = {};
-vi.mock("@simplysm/core-common", () => ({
-  env: (key: string) => mockEnv[key],
-}));
-
 const execaCalls: { command: string; args: string[] }[] = [];
 const mockCpxSpawn = vi.fn((...args: unknown[]) => {
   execaCalls.push({ command: args[0] as string, args: (args[1] as string[] | undefined) ?? [] });
@@ -65,13 +60,14 @@ vi.mock("sharp", () => ({
   }),
 }));
 
-const mockLoggerDebug = vi.fn();
 const mockLoggerWarn = vi.fn();
+const _mockConsola = {
+  level: 0,
+  withTag: () => ({ debug: vi.fn(), warn: mockLoggerWarn, error: vi.fn(), info: vi.fn(), success: vi.fn() }),
+};
 vi.mock("consola", () => ({
-  consola: {
-    level: 0,
-    withTag: () => ({ debug: mockLoggerDebug, warn: mockLoggerWarn }),
-  },
+  consola: _mockConsola,
+  default: _mockConsola,
   LogLevels: { debug: 4 },
 }));
 
@@ -152,7 +148,7 @@ function setupDefaultMocks() {
 
   mockFsxGlob.mockResolvedValue(["C:/Program Files/Amazon Corretto/jdk21.0.1"]);
 
-  mockEnv = { ANDROID_HOME: "C:/Android/Sdk" };
+  process.env["ANDROID_HOME"] = "C:/Android/Sdk";
 
   execaCalls.length = 0;
   mockFsWriteFile.mockReset();
@@ -161,6 +157,14 @@ function setupDefaultMocks() {
 
 //#endregion
 
+let savedEnv: Record<string, string | undefined>;
+beforeEach(() => {
+  savedEnv = { ...process.env };
+});
+afterEach(() => {
+  process.env = savedEnv;
+});
+
 describe("Capacitor 설정 검증", () => {
   beforeEach(() => {
     vi.clearAllMocks();
@@ -398,7 +402,7 @@ describe("Android 개발 도구 감지", () => {
   });
 
   it("Android SDK 미설치 시 에러가 발생한다", async () => {
-    mockEnv = {};
+    delete process.env["ANDROID_HOME"];
     mockFsxExists.mockImplementation((p: string) => {
       const n = p.replace(/\\/g, "/");
       if (n.includes(".capacitor.lock")) return false;
@@ -518,7 +522,7 @@ describe("Android 네이티브 설정", () => {
         typeof call[0] === "string" &&
         call[0].includes("build.gradle") &&
         typeof call[1] === "string" &&
-        call[1].includes("versionCode 10203"),
+        call[1].includes("versionCode 1002003"),
     );
     expect(gradleWrite).toBeDefined();
   });

package/tests/capacitor/capacitor-run.spec.ts

@@ -1,5 +1,5 @@
 import path from "path";
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 
 //#region Mocks
 
@@ -36,14 +36,6 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-// env mock
-let mockEnv: Record<string, unknown> = {};
-vi.mock("@simplysm/core-common", () => ({
-  env: new Proxy({} as Record<string, unknown>, {
-    get: (_target, prop) => mockEnv[prop as string],
-  }),
-}));
-
 // cpx mock (was execa) — tracks commands and resolves immediately
 const execaCalls: { command: string; args: string[] }[] = [];
 let execaFactory: (...args: unknown[]) => Promise<{ stdout: string; stderr: string; exitCode: number }> = () =>
@@ -74,20 +66,6 @@ vi.mock("sharp", () => ({
   }),
 }));
 
-// consola mock
-const mockLoggerDebug = vi.fn();
-const mockLoggerWarn = vi.fn();
-vi.mock("consola", () => ({
-  consola: {
-    withTag: () => ({
-      debug: mockLoggerDebug,
-      warn: mockLoggerWarn,
-      info: vi.fn(),
-    }),
-    level: 0,
-  },
-  LogLevels: { debug: 4 },
-}));
 
 //#endregion
 
@@ -141,7 +119,7 @@ export default config;`;
     return [];
   });
 
-  mockEnv = { ANDROID_HOME: "C:/Android/Sdk" };
+  process.env["ANDROID_HOME"] = "C:/Android/Sdk";
 
   execaCalls.length = 0;
   execaFactory = () => Promise.resolve({ stdout: "", stderr: "", exitCode: 0 });
@@ -151,6 +129,14 @@ export default config;`;
 
 //#endregion
 
+let savedEnv: Record<string, string | undefined>;
+beforeEach(() => {
+  savedEnv = { ...process.env };
+});
+afterEach(() => {
+  process.env = savedEnv;
+});
+
 describe("Capacitor.run()", () => {
   beforeEach(() => {
     vi.clearAllMocks();

package/tests/capacitor/capacitor-workspace.spec.ts

@@ -1,5 +1,5 @@
 import path from "path";
-import { describe, it, expect, vi, beforeEach } from "vitest";
+import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
 
 //#region Mocks
 
@@ -35,11 +35,6 @@ vi.mock("@simplysm/core-node", () => ({
   },
 }));
 
-let mockEnv: Record<string, string | undefined> = {};
-vi.mock("@simplysm/core-common", () => ({
-  env: (key: string) => mockEnv[key],
-}));
-
 const execaCalls: { command: string; args: string[] }[] = [];
 const mockCpxSpawn = vi.fn((...args: unknown[]) => {
   execaCalls.push({ command: args[0] as string, args: (args[1] as string[] | undefined) ?? [] });
@@ -81,14 +76,6 @@ vi.mock("module", () => ({
   }),
 }));
 
-vi.mock("consola", () => ({
-  consola: {
-    withTag: () => ({ debug: vi.fn(), warn: vi.fn() }),
-    level: 0,
-  },
-  LogLevels: { debug: 4 },
-}));
-
 //#endregion
 
 //#region Helpers
@@ -154,12 +141,20 @@ function setupDefaultMocks() {
   });
 
   mockFsxGlob.mockResolvedValue(["C:/Program Files/Amazon Corretto/jdk21.0.1"]);
-  mockEnv = { ANDROID_HOME: "C:/Android/Sdk" };
+  process.env["ANDROID_HOME"] = "C:/Android/Sdk";
   execaCalls.length = 0;
 }
 
 //#endregion
 
+let savedEnv: Record<string, string | undefined>;
+beforeEach(() => {
+  savedEnv = { ...process.env };
+});
+afterEach(() => {
+  process.env = savedEnv;
+});
+
 describe("workspace:* 플러그인 해석", () => {
   beforeEach(() => {
     vi.clearAllMocks();

package/tests/commands/check.spec.ts

@@ -9,11 +9,11 @@ const mocks = vi.hoisted(() => ({
   discoverWorkspacePackages: vi.fn(),
 }));
 
-vi.mock("../../src/commands/typecheck", () => ({
+vi.mock("../../src/orchestrators/TypecheckOrchestrator", () => ({
   executeTypecheck: mocks.executeTypecheck,
 }));
 
-vi.mock("../../src/commands/lint", () => ({
+vi.mock("../../src/utils/lint-core", () => ({
   executeLint: mocks.executeLint,
 }));