intershell 0.6.1 → 0.6.3

package/src/package-version/package-version.test.ts

@@ -1,11 +1,14 @@
-import { describe, expect, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, mock, test } from "bun:test";
 import type { ParsedCommitData } from "../commit";
 import { EntityPackage } from "../package";
+import { installPackagesShellTestMock } from "../package/package-test-mock";
 import { EntityPackageCommits } from "../package-commits";
 import { EntityPackageTags } from "../package-tags";
 import { EntityPackageVersion } from "./package-version";
 
-// Helper function to create EntityPackageVersion instances
+let restorePackagesShellMock: () => void;
+let originalGetLatestPackageVersionInHistory: typeof EntityPackageTags.prototype.getLatestPackageVersionInHistory;
+
 function createEntityPackageVersion(packageName: string): EntityPackageVersion {
 	const packageInstance = new EntityPackage(packageName);
 	const commitPackage = new EntityPackageCommits(packageInstance);
@@ -14,18 +17,38 @@ function createEntityPackageVersion(packageName: string): EntityPackageVersion {
 }
 
 describe("EntityPackageVersion", () => {
+	beforeEach(() => {
+		restorePackagesShellMock = installPackagesShellTestMock();
+
+		originalGetLatestPackageVersionInHistory =
+			EntityPackageTags.prototype.getLatestPackageVersionInHistory;
+		EntityPackageTags.prototype.getLatestPackageVersionInHistory = mock(function (
+			this: EntityPackageTags,
+		) {
+			const packageName = (this as unknown as { package: EntityPackage }).package.getName();
+			if (packageName === "root") return Promise.resolve("1.0.0");
+			return Promise.resolve("0.1.0");
+		});
+	});
+
+	afterEach(() => {
+		restorePackagesShellMock();
+		EntityPackageTags.prototype.getLatestPackageVersionInHistory =
+			originalGetLatestPackageVersionInHistory;
+	});
+
 	test("should create instance", () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
-		expect(EntityPackageVersion).toBeDefined();
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
+		expect(entityPackageVersion).toBeDefined();
 	});
 
 	test("should create root instance", () => {
-		const EntityPackageVersion = createEntityPackageVersion("root");
-		expect(EntityPackageVersion).toBeDefined();
+		const entityPackageVersion = createEntityPackageVersion("root");
+		expect(entityPackageVersion).toBeDefined();
 	});
 
 	test("should calculate bump type for regular package", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
 		const commits: ParsedCommitData[] = [
 			{
@@ -34,12 +57,12 @@ describe("EntityPackageVersion", () => {
 			} as ParsedCommitData,
 		];
 
-		const versionData = await EntityPackageVersion.calculateVersionData(commits);
+		const versionData = await entityPackageVersion.calculateVersionData(commits);
 		expect(versionData.bumpType).toBe("minor");
 	});
 
 	test("should calculate bump type for breaking changes", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
 		const commits: ParsedCommitData[] = [
 			{
@@ -48,12 +71,12 @@ describe("EntityPackageVersion", () => {
 			} as ParsedCommitData,
 		];
 
-		const versionData = await EntityPackageVersion.calculateVersionData(commits);
+		const versionData = await entityPackageVersion.calculateVersionData(commits);
 		expect(versionData.bumpType).toBe("major");
 	});
 
 	test("should calculate bump type for patch changes", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
 		const commits: ParsedCommitData[] = [
 			{
@@ -62,22 +85,21 @@ describe("EntityPackageVersion", () => {
 			} as ParsedCommitData,
 		];
 
-		const versionData = await EntityPackageVersion.calculateVersionData(commits);
+		const versionData = await entityPackageVersion.calculateVersionData(commits);
 		expect(versionData.bumpType).toBe("patch");
 	});
 
 	test("should return none for empty commits", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
-		const versionData = await EntityPackageVersion.calculateVersionData([]);
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
+		const versionData = await entityPackageVersion.calculateVersionData([]);
 		expect(versionData.bumpType).toBe("none");
 	});
 
 	test("should calculate version data for first version", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
-		const versionData = await EntityPackageVersion.calculateVersionData([]);
+		const versionData = await entityPackageVersion.calculateVersionData([]);
 
-		// The api package already has version 0.1.0, so no bump needed
 		expect(versionData.shouldBump).toBe(false);
 		expect(versionData.targetVersion).toBe("0.1.0");
 		expect(versionData.bumpType).toBe("none");
@@ -85,24 +107,18 @@ describe("EntityPackageVersion", () => {
 	});
 
 	test("should not bump when no commits", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
-		const versionData = await EntityPackageVersion.calculateVersionData([]);
+		const versionData = await entityPackageVersion.calculateVersionData([]);
 
 		expect(versionData.shouldBump).toBe(false);
-		expect(versionData.targetVersion).toBe("0.1.0"); // api package version
+		expect(versionData.targetVersion).toBe("0.1.0");
 		expect(versionData.bumpType).toBe("none");
 		expect(versionData.reason).toBe("No commits in range");
 	});
 
-	test("should throw error when version on disk is higher", async () => {
-		// This test needs to be updated since the method now gets version from package.json internally
-		// We'll need to mock the package version or create a different test
-		expect(true).toBe(true); // Placeholder - this test needs to be redesigned
-	});
-
 	test("should calculate bump type for root package", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("root");
+		const entityPackageVersion = createEntityPackageVersion("root");
 
 		const commits: ParsedCommitData[] = [
 			{
@@ -111,12 +127,12 @@ describe("EntityPackageVersion", () => {
 			} as ParsedCommitData,
 		];
 
-		const versionData = await EntityPackageVersion.calculateVersionData(commits);
+		const versionData = await entityPackageVersion.calculateVersionData(commits);
 		expect(versionData.bumpType).toBe("minor");
 	});
 
 	test("should use override bump type when provided", async () => {
-		const EntityPackageVersion = createEntityPackageVersion("api");
+		const entityPackageVersion = createEntityPackageVersion("@apps/api");
 
 		const commits: ParsedCommitData[] = [
 			{
@@ -125,16 +141,13 @@ describe("EntityPackageVersion", () => {
 			} as ParsedCommitData,
 		];
 
-		// Test with major override (should override the automatic minor bump)
-		const versionDataMajor = await EntityPackageVersion.calculateVersionData(commits, "major");
+		const versionDataMajor = await entityPackageVersion.calculateVersionData(commits, "major");
 		expect(versionDataMajor.bumpType).toBe("major");
 
-		// Test with patch override (should override the automatic minor bump)
-		const versionDataPatch = await EntityPackageVersion.calculateVersionData(commits, "patch");
+		const versionDataPatch = await entityPackageVersion.calculateVersionData(commits, "patch");
 		expect(versionDataPatch.bumpType).toBe("patch");
 
-		// Test with none override (should prevent version bump)
-		const versionDataNone = await EntityPackageVersion.calculateVersionData(commits, "none");
+		const versionDataNone = await entityPackageVersion.calculateVersionData(commits, "none");
 		expect(versionDataNone.bumpType).toBe("none");
 		expect(versionDataNone.shouldBump).toBe(false);
 	});