feat: adding bun serve and removing http-server

Topsort · Jul 23, 2024 · a74c48b · a74c48b
1 parent ee38a8d
commit a74c48b
Show file tree

Hide file tree

Showing 10 changed files with 22 additions and 21 deletions.
diff --git a/bun.lockb b/bun.lockb
diff --git a/e2e/auctions.test.ts b/e2e/auctions.test.ts
@@ -1,9 +1,9 @@
 import { expect, test } from "@playwright/test";
 import { apis, baseURL } from "../src/constants/apis.constant";
-import { playwrightConstants } from "./constants";
+import { playwrightConstants } from "./config";
 
 test.describe("Create Auction via Topsort SDK", () => {
-  test.only("should create an auction successfully", async ({ page }) => {
+  test("should create an auction successfully", async ({ page }) => {
     const mockAPIResponse = {
       results: [
         {
@@ -23,7 +23,7 @@ test.describe("Create Auction via Topsort SDK", () => {
       await route.fulfill({ json: mockAPIResponse });
     });
 
-    await page.goto(playwrightConstants.url);
+    await page.goto(playwrightConstants.host);
     const result = await page.evaluate(() => {
       const config = {
         apiKey: "rando-api-key",
@@ -59,7 +59,7 @@ test.describe("Create Auction via Topsort SDK", () => {
 
   test("should fail to call with missing apiKey", async ({ page }) => {
     const expectedError = { status: 401, statusText: "API Key is required.", body: {} };
-    await page.goto(playwrightConstants.url);
+    await page.goto(playwrightConstants.host);
     const result = await page.evaluate(() => {
       const config = {
         apiKey: null,

diff --git a/e2e/config.ts b/e2e/config.ts
@@ -0,0 +1,3 @@
+export const playwrightConstants = {
+  host: `http://localhost:${process.env.SERVER_PORT || 8080}/index.html`,
+};
diff --git a/e2e/constants.ts b/e2e/constants.ts
diff --git a/e2e/events.test.ts b/e2e/events.test.ts
@@ -1,6 +1,6 @@
 import { expect, test } from "@playwright/test";
 import { apis, baseURL } from "../src/constants/apis.constant";
-import { playwrightConstants } from "./constants";
+import { playwrightConstants } from "./config";
 
 test.describe("Report Events via Topsort SDK", () => {
   test("should report an successfully", async ({ page }) => {
@@ -12,7 +12,7 @@ test.describe("Report Events via Topsort SDK", () => {
       await route.fulfill({ json: mockAPIResponse });
     });
 
-    await page.goto(playwrightConstants.url);
+    await page.goto(playwrightConstants.host);
     const result = await page.evaluate(() => {
       const config = {
         apiKey: "rando-api-key",
@@ -45,7 +45,7 @@ test.describe("Report Events via Topsort SDK", () => {
 
   test("should fail to call with missing apiKey", async ({ page }) => {
     const expectedError = { status: 401, statusText: "API Key is required.", body: {} };
-    await page.goto(playwrightConstants.url);
+    await page.goto(playwrightConstants.host);
     const result = await page.evaluate(() => {
       const config = {
         apiKey: null,

diff --git a/e2e/index.ts b/e2e/index.ts
@@ -1,16 +1,18 @@
-import type { MatchedRoute } from "bun";
+import { file } from "bun";
 
 const PORT = process.env.SERVER_PORT || 8080;
-const router = new Bun.FileSystemRouter({
-  style: "nextjs",
-  dir: "./dist/",
-});
 
 Bun.serve({
   fetch(req) {
-    const match = router.match(req) as MatchedRoute;
-    const page = require(match.filePath);
-    return new page(req, match.query, match.params);
+    const url = new URL(req.url);
+    const pathname = url.pathname === "/" ? "/index.html" : url.pathname;
+    const filePath = `./dist${pathname}`;
+
+    try {
+      return new Response(file(filePath));
+    } catch (e) {
+      return new Response("Not Found", { status: 404 });
+    }
   },
   port: PORT,
 });
diff --git a/e2e/public/favicon.ico b/e2e/public/favicon.ico
diff --git a/e2e/index.html → e2e/public/index.html b/e2e/index.html → e2e/public/index.html
@@ -4,7 +4,7 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>Topsort SDK Test Application</title>
-    <script src="../dist/index.global.js"></script>
+    <script src="./index.global.js"></script>
   </head>
   <body>
     <h1>Test Topsort.js Integration</h1>

diff --git a/package.json b/package.json
@@ -24,7 +24,6 @@
     "CHANGELOG.md",
     "LICENSE",
     "README.md",
-    "dist/index.global.js",
     "dist/index.d.mts",
     "dist/index.d.ts",
     "dist/index.js",
@@ -43,7 +42,6 @@
     "@biomejs/biome": "1.8.3",
     "@playwright/test": "^1.45.2",
     "@types/bun": "1.1.6",
-    "http-server": "^14.1.1",
     "lefthook": "1.7.2",
     "msw": "2.3.1",
     "tsup": "8.1.0",

diff --git a/tsup.config.ts b/tsup.config.ts
@@ -10,4 +10,5 @@ export default defineConfig({
     options.keepNames = true;
     options.globalName = "Topsort";
   },
+  onSuccess: "cp -r ./e2e/public/* dist",
 });