@decocms/blocks-cli 7.0.0

package/scripts/migrate/phase-analyze.test.ts

@@ -0,0 +1,63 @@
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import { afterEach, beforeEach, describe, expect, it } from "vitest";
+import { extractPlatform } from "./phase-analyze";
+
+// `extractPlatform` runs four strategies in order:
+//   1. deno.json imports referencing apps/{platform}/
+//   2. apps/{platform}.ts file existence
+//   3. apps/site.ts loose string match against platform name + "platform"
+//   4. .deco/blocks filenames hinting at platform
+// Strategy 3 is the false-positive trap that ate Magento sites before #211:
+// helsinki/granadobr's apps/site.ts imports `apps/vtex/mod.ts` for the color
+// palette, so the content matches `"vtex"` even though the real platform is
+// declared via `apps/magento.ts`. The fix adds "magento" to the platforms
+// list so Strategy 2 fires first and short-circuits before Strategy 3 can
+// over-match.
+
+describe("extractPlatform", () => {
+  let tmp: string;
+
+  beforeEach(() => {
+    tmp = fs.mkdtempSync(path.join(os.tmpdir(), "extract-platform-"));
+    fs.mkdirSync(path.join(tmp, "apps"), { recursive: true });
+  });
+
+  afterEach(() => {
+    fs.rmSync(tmp, { recursive: true, force: true });
+  });
+
+  it("detects magento from apps/magento.ts (granadobr/helsinki shape)", () => {
+    fs.writeFileSync(path.join(tmp, "apps", "magento.ts"), "export default {};\n");
+    // Also write a site.ts that imports vtex for color palettes — this is
+    // what tripped Strategy 3 before #211.
+    fs.writeFileSync(
+      path.join(tmp, "apps", "site.ts"),
+      `import { color as vtex } from "apps/vtex/mod.ts";\n` +
+        `export interface State { platform: string }\n`,
+    );
+    expect(extractPlatform(tmp)).toBe("magento");
+  });
+
+  it("detects vtex from apps/vtex.ts", () => {
+    fs.writeFileSync(path.join(tmp, "apps", "vtex.ts"), "export default { account: \"foo\" };\n");
+    expect(extractPlatform(tmp)).toBe("vtex");
+  });
+
+  it("falls back to custom when no platform signal exists", () => {
+    fs.writeFileSync(path.join(tmp, "apps", "site.ts"), "export default {};\n");
+    expect(extractPlatform(tmp)).toBe("custom");
+  });
+
+  it("prefers magento over vtex when both an apps/magento.ts AND a vtex string match exist", () => {
+    // Reproduce the exact granadobr trap: apps/magento.ts is the real signal,
+    // but apps/site.ts loosely contains "vtex" + "platform".
+    fs.writeFileSync(path.join(tmp, "apps", "magento.ts"), "export default {};\n");
+    fs.writeFileSync(
+      path.join(tmp, "apps", "site.ts"),
+      `import "apps/vtex/mod.ts";\nexport interface State { platform: string }\n`,
+    );
+    expect(extractPlatform(tmp)).toBe("magento");
+  });
+});