diff --git a/core/tests/15-plugin-pipeline.test.ts b/core/tests/15-plugin-pipeline.test.ts index 231b8b4..bc3e784 100644 --- a/core/tests/15-plugin-pipeline.test.ts +++ b/core/tests/15-plugin-pipeline.test.ts @@ -342,7 +342,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { }, }; const result = tarResolvePackageEntry(manifest, "", conditions); - expect(result).toBe("/dist/index.mjs"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/dist/index.mjs"); }); test("falls back to legacy main field", () => { @@ -352,7 +353,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { main: "./lib/index.js", }; const result = tarResolvePackageEntry(manifest, "", conditions); - expect(result).toBe("/lib/index.js"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/lib/index.js"); }); test("falls back to module field before main", () => { @@ -364,7 +366,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { }; const result = tarResolvePackageEntry(manifest, "", conditions); // legacy() with browser: true checks browser → module → main - expect(result).toBe("/dist/esm.js"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/dist/esm.js"); }); test("returns /index.js when no fields present", () => { @@ -373,7 +376,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { version: "1.0.0", }; const result = tarResolvePackageEntry(manifest, "", conditions); - expect(result).toBe("/index.js"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/index.js"); }); test("subpath used directly when exports don't match", () => { @@ -384,7 +388,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { }; // Subpath "/custom/file.js" doesn't match any export key const result = tarResolvePackageEntry(manifest, "/custom/file.js", conditions); - expect(result).toBe("/custom/file.js"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/custom/file.js"); }); test("exports subpath resolution: ./utils", () => { @@ -398,7 +403,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { }; const result = tarResolvePackageEntry(manifest, "/utils", conditions); // Subpath "/utils" → exports key "./utils" → "./dist/utils.js" - expect(result).toBe("/dist/utils.js"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/dist/utils.js"); }); test("require fallback when import doesn't match", () => { @@ -411,7 +417,8 @@ describe("tar: resolvePackageEntry (tar.ts)", () => { }; // With import conditions, no "import" key → falls through to require fallback const result = tarResolvePackageEntry(manifest, "", conditions); - expect(result).toBe("/dist/index.cjs"); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/dist/index.cjs"); }); }); diff --git a/core/tests/19-registry-tarballs.test.ts b/core/tests/19-registry-tarballs.test.ts index 3243fd9..d94c249 100644 --- a/core/tests/19-registry-tarballs.test.ts +++ b/core/tests/19-registry-tarballs.test.ts @@ -336,8 +336,9 @@ describe("19.5 — resolvePackageEntry with subpaths", () => { }, }; - const entry = resolvePackageEntry(manifest, "", defaultConditions); - expect(entry).toBe("/dist/index.js"); + const result = resolvePackageEntry(manifest, "", defaultConditions); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/dist/index.js"); }); test("subpath import resolves via exports[\"./expo-sqlite\"]", () => { @@ -351,8 +352,9 @@ describe("19.5 — resolvePackageEntry with subpaths", () => { }, }; - const entry = resolvePackageEntry(manifest, "/expo-sqlite", defaultConditions); - expect(entry).toBe("/expo-sqlite/index.js"); + const result = resolvePackageEntry(manifest, "/expo-sqlite", defaultConditions); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/expo-sqlite/index.js"); }); test("deep subpath import resolves via exports[\"./expo-sqlite/migrator\"]", () => { @@ -366,8 +368,9 @@ describe("19.5 — resolvePackageEntry with subpaths", () => { }, }; - const entry = resolvePackageEntry(manifest, "/expo-sqlite/migrator", defaultConditions); - expect(entry).toBe("/expo-sqlite/migrator.js"); + const result = resolvePackageEntry(manifest, "/expo-sqlite/migrator", defaultConditions); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/expo-sqlite/migrator.js"); }); test("root import falls back to main when no exports", () => { @@ -377,8 +380,9 @@ describe("19.5 — resolvePackageEntry with subpaths", () => { main: "./lib/index.js", }; - const entry = resolvePackageEntry(manifest, "", defaultConditions); - expect(entry).toBe("/lib/index.js"); + const result = resolvePackageEntry(manifest, "", defaultConditions); + expect(result.excluded).toBe(false); + expect(result.entryPath).toBe("/lib/index.js"); }); test("root import falls back to module over main", () => { @@ -389,9 +393,10 @@ describe("19.5 — resolvePackageEntry with subpaths", () => { module: "./lib/index.mjs", }; - const entry = resolvePackageEntry(manifest, "", defaultConditions); + const result = resolvePackageEntry(manifest, "", defaultConditions); + expect(result.excluded).toBe(false); // module is preferred for browser context - expect(entry).toMatch(/\.mjs|\.js/); + expect(result.entryPath).toMatch(/\.mjs|\.js/); }); });