fix test warnings

Author: claytercek

packages/content/src/__tests__/content-integration.test.ts

@@ -1,4 +1,4 @@
-import path from "node:path";
+import path from "node:path/posix";
 import { createMockLogger } from "@bluecadet/launchpad-testing/test-utils.ts";
 import { vol } from "memfs";
 import { http, HttpResponse } from "msw";

packages/content/src/__tests__/launchpad-content.test.ts

@@ -1,4 +1,4 @@
-import path from "node:path";
+import path from "node:path/posix";
 import { createMockLogger } from "@bluecadet/launchpad-testing/test-utils.ts";
 import { vol } from "memfs";
 import { afterEach, describe, expect, it, vi } from "vitest";

packages/content/src/plugins/__tests__/md-to-html.test.ts

@@ -71,7 +71,7 @@ describe("mdToHtml plugin", () => {
 		await namespace.insert("doc1", Promise.resolve({ content: { foo: "bar" } }));
 
 		const plugin = mdToHtml({ path: "$.content" });
-		expect(plugin.hooks.onContentFetchDone(ctx)).rejects.toThrow(
+		await expect(plugin.hooks.onContentFetchDone(ctx)).rejects.toThrow(
 			"Error applying content transform",
 		);
 	});

packages/content/src/plugins/__tests__/media-downloader.test.ts

@@ -1,4 +1,4 @@
-import path from "node:path";
+import path from "node:path/posix";
 import { vol } from "memfs";
 import { http, HttpResponse } from "msw";
 import { setupServer } from "msw/node";

packages/content/src/sources/__tests__/airtable-source.test.ts

@@ -174,7 +174,7 @@ describe("airtableSource", () => {
 		const result = source.fetch(createFetchContext());
 		expect(result).toHaveLength(1);
 
-		expect(result[0]!.data).rejects.toThrow();
+		await expect(result[0]!.data).rejects.toThrow();
 	});
 
 	it("should handle invalid key-value table data", async () => {
@@ -202,7 +202,7 @@ describe("airtableSource", () => {
 		const result = source.fetch(createFetchContext());
 		expect(result).toHaveLength(1);
 
-		expect(result[0]!.data).rejects.toThrow();
+		await expect(result[0]!.data).rejects.toThrow();
 	});
 
 	it("should handle unauthorized access", async () => {
@@ -222,6 +222,6 @@ describe("airtableSource", () => {
 		const result = source.fetch(createFetchContext());
 		expect(result).toHaveLength(1);
 
-		expect(result[0]!.data).rejects.toThrow();
+		await expect(result[0]!.data).rejects.toThrow();
 	});
 });

packages/content/src/sources/__tests__/contentful-source.test.ts

@@ -28,7 +28,7 @@ function createFetchContext() {
 
 describe("contentfulSource", () => {
 	it("should fail with missing delivery token when not using preview", async () => {
-		expect(
+		await expect(
 			async () =>
 				// @ts-expect-error - testing invalid options
 				await contentfulSource({
@@ -41,7 +41,7 @@ describe("contentfulSource", () => {
 	});
 
 	it("should fail with missing preview token when using preview", async () => {
-		expect(
+		await expect(
 			async () =>
 				// @ts-expect-error - testing invalid options
 				await contentfulSource({
@@ -227,7 +227,7 @@ describe("contentfulSource", () => {
 
 		const result = source.fetch(createFetchContext());
 
-		expect(result.data).rejects.toThrow();
+		await expect(result.data).rejects.toThrow();
 	});
 
 	it("should handle contentful errors array", async () => {
@@ -248,7 +248,7 @@ describe("contentfulSource", () => {
 
 		const result = source.fetch(createFetchContext());
 
-		expect(result.data).rejects.toThrow();
+		await expect(result.data).rejects.toThrow();
 	});
 
 	it("should respect content type filtering", async () => {

packages/content/src/sources/__tests__/json-source.test.ts

@@ -73,7 +73,7 @@ describe("jsonSource", () => {
 		const result = source.fetch(createFetchContext());
 		expect(result).toHaveLength(1);
 
-		expect(async () => {
+		await expect(async () => {
 			await result[0]!.data;
 		}).rejects.toThrow();
 	});
@@ -98,7 +98,7 @@ describe("jsonSource", () => {
 
 		expect(result).toHaveLength(1);
 
-		expect(async () => {
+		await expect(async () => {
 			await result[0]!.data;
 		}).rejects.toThrow();
 	});
@@ -122,14 +122,14 @@ describe("jsonSource", () => {
 
 		const promise = result[0]!.data;
 
-		expect(promise).rejects.toThrow();
-
 		// Need to run the timer and wait for the rejection
-		await vi.runAllTimersAsync();
+		vi.runAllTimersAsync();
+
+		await expect(promise).rejects.toThrow();
 	});
 
 	it("should throw on incomplete config", async () => {
 		// @ts-expect-error - incomplete config
-		expect(() => jsonSource({})).toThrow();
+		await expect(() => jsonSource({})).toThrow();
 	});
 });

packages/content/src/sources/__tests__/strapi-source.test.ts

@@ -28,7 +28,7 @@ function createFetchContext() {
 
 describe("strapiSource", () => {
 	it("should fail with unsupported version", async () => {
-		expect(() =>
+		await expect(() =>
 			strapiSource({
 				id: "test-strapi",
 				baseUrl: "http://localhost:1337",
@@ -278,7 +278,7 @@ describe("strapiSource", () => {
 			}),
 		);
 
-		expect(
+		await expect(
 			strapiSource({
 				id: "test-strapi",
 				version: "4",
@@ -312,6 +312,6 @@ describe("strapiSource", () => {
 		const result = source.fetch(createFetchContext());
 		expect(result).toHaveLength(1);
 
-		expect(async () => (await result[0]!.data.next()).value).rejects.toThrow();
+		await expect(async () => (await result[0]!.data.next()).value).rejects.toThrow();
 	});
 });

packages/content/src/utils/__tests__/content-transform-utils.test.ts

@@ -75,7 +75,7 @@ describe("content-transform-utils", () => {
 				throw new Error("Transform error");
 			};
 
-			expect(
+			await expect(
 				applyTransformToFiles({
 					dataStore,
 					path: "$.content",

packages/content/src/utils/__tests__/data-store.test.ts

@@ -1,4 +1,4 @@
-import path from "node:path";
+import path from "node:path/posix";
 import { vol } from "memfs";
 import { afterEach, beforeEach, describe, expect, it } from "vitest";
 import { DataStore } from "../data-store.js";

packages/utils/src/__tests__/log-manager.test.ts

@@ -1,4 +1,4 @@
-import path from "node:path";
+import path from "node:path/posix";
 import moment from "moment";
 import { afterAll, afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 import winston from "winston";