Fix tests

package.json

@@ -35,7 +35,8 @@
 		"build:ts": "tsc --p ./cfg/tsconfig.types.json",
 		"format": "prettier --write .",
 		"format:check": "prettier --check .",
-		"prepublishOnly": "pnpm run build",
+		"prepublishOnly": "pnpm run build && pnpm run test",
+		"test": "pnpm test:ts&& pnpm test:node --run && pnpm test:browser --run",
 		"test:browser": "vitest --config ./cfg/vitest.browser.config.ts",
 		"test:node": "vitest --config ./cfg/vitest.config.ts",
 		"test:ts": "tsc --project ./cfg/tsconfig.test.json",

src/async_option.ts

@@ -149,7 +149,7 @@ export class AsyncOption<T> implements PromiseLike<Option<T>> {
 	/**
 	 * Async version of `Option#unwrap`.
 	 */
-	async unwrap(): Promise<T> {
+	async unwrap(): Promise<T | undefined> {
 		return (await this).unwrap();
 	}
 

src/async_result.ts

@@ -214,14 +214,14 @@ export class AsyncResult<T, E> implements PromiseLike<Result<T, E>> {
 	/**
 	 * Async version of `Result#unwrap`.
 	 */
-	async unwrap(): Promise<T> {
+	async unwrap(): Promise<T | undefined> {
 		return (await this).unwrap();
 	}
 
 	/**
 	 * Async version of `Result#unwrapErr`.
 	 */
-	async unwrapErr(): Promise<E> {
+	async unwrapErr(): Promise<E | undefined> {
 		return (await this).unwrapErr();
 	}
 

test/async_option.test.ts

@@ -217,9 +217,9 @@ describe.concurrent("unwrap", () => {
 		await expect(option.unwrap()).resolves.toEqual(42);
 	});
 
-	it("throws Panic when called on a None option", async () => {
+	it("returns undefined when called on a None option", async () => {
 		const option = promiseNone();
-		await expect(() => option.unwrap()).rejects.toThrow(Panic);
+		await expect(option.unwrap()).resolves.toEqual(undefined);
 	});
 });
 

test/async_result.test.ts

@@ -360,10 +360,10 @@ describe.concurrent("unwrap", () => {
 		await expect(result.unwrap()).resolves.toEqual(42);
 	});
 
-	it("throws a Panic for an Err result", async () => {
+	it("returns undefined for an Err result", async () => {
 		const error = new Error("Test error");
 		const result = new AsyncResult(Promise.resolve(Err(error)));
-		await expect(result.unwrap()).rejects.toThrow(Panic);
+		await expect(result.unwrap()).resolves.toEqual(undefined);
 	});
 });
 
@@ -374,9 +374,9 @@ describe.concurrent("unwrapErr", () => {
 		await expect(result.unwrapErr()).resolves.toEqual(error);
 	});
 
-	it("throws for an Ok result", async () => {
+	it("returns undefined for an Ok result", async () => {
 		const result = new AsyncResult(Promise.resolve(Ok(42)));
-		await expect(result.unwrapErr()).rejects.toThrow(Panic);
+		await expect(result.unwrapErr()).resolves.toEqual(undefined);
 	});
 });
 

test/option.test.ts

@@ -264,9 +264,9 @@ describe.concurrent("unwrap", () => {
 		expect(option.unwrap()).toEqual(42);
 	});
 
-	it("throws when called on a None option", () => {
+	it("returns undefined when called on a None option", () => {
 		const option = TestNone<string>();
-		expect(() => option.unwrap()).toThrow(Panic);
+		expect(option.unwrap()).toEqual(undefined);
 	});
 });
 

test/result.test.ts

@@ -445,9 +445,9 @@ describe.concurrent("unwrap", () => {
 		expect(result.unwrap()).toEqual(42);
 	});
 
-	it("throws a Panic for an Err result", () => {
+	it("returns undefined for an Err result", () => {
 		const result = TestErr<number, string>("error");
-		expect(() => result.unwrap()).toThrow(Panic);
+		expect(result.unwrap()).toEqual(undefined);
 	});
 });
 
@@ -457,9 +457,9 @@ describe.concurrent("unwrapErr", () => {
 		expect(result.unwrapErr()).toEqual("error");
 	});
 
-	it("throws for an Ok result", () => {
+	it("returns undefined for an Ok result", () => {
 		const result = TestOk<number, string>(42);
-		expect(() => result.unwrapErr()).toThrow(Panic);
+		expect(result.unwrapErr()).toEqual(undefined);
 	});
 });