test: add more tests

Author: guenhter

frontend_omni/tests/Caddyfile

@@ -0,0 +1,5 @@
+:80 {
+    root * /usr/share/caddy
+    try_files {path} {path}/ /index.html
+    file_server
+}

frontend_omni/tests/container.ts

@@ -0,0 +1,58 @@
+import { execSync } from "child_process";
+import * as net from "net";
+
+const TEST_PORT = 8088;
+
+async function waitForPort(
+    port: number,
+    host: string = "localhost",
+    timeout: number = 10000,
+): Promise<void> {
+    const startTime = Date.now();
+    while (Date.now() - startTime < timeout) {
+        try {
+            await new Promise<void>((resolve, reject) => {
+                const socket = net.createConnection({ port, host }, () => {
+                    socket.destroy();
+                    resolve();
+                });
+                socket.on("error", () => {
+                    reject();
+                });
+            });
+            return;
+        } catch {
+            await new Promise((resolve) => setTimeout(resolve, 500));
+        }
+    }
+    throw new Error(`Port ${port} on ${host} not available after ${timeout}ms`);
+}
+
+export async function buildApp() {
+    console.log("Building the app...");
+    execSync("pnpm build", { stdio: "inherit" });
+}
+
+export async function startContainer() {
+    console.log(`Starting Caddy container (In dir ${process.cwd()})...`);
+    execSync(`docker container rm -f caddy-uitest || true`, {
+        stdio: "ignore",
+    });
+    execSync(
+        `docker run --rm -d --name caddy-uitest -p ${TEST_PORT}:80 -v ${process.cwd()}/dist:/usr/share/caddy -v ${process.cwd()}/tests/Caddyfile:/etc/caddy/Caddyfile caddy`,
+        { stdio: "inherit" },
+    );
+
+    await waitForPort(TEST_PORT);
+}
+
+export async function stopContainer() {
+    try {
+        console.log("Stopping Caddy container...");
+        execSync("docker container rm -f caddy-uitest || true", {
+            stdio: "ignore",
+        });
+    } catch (error) {
+        console.warn("Failed to stop Caddy container:", error);
+    }
+}

frontend_omni/tests/features/app_load.feature

@@ -0,0 +1,15 @@
+Feature: App loads successfully
+
+  Scenario: User opens the app
+    When I visit the homepage
+    Then I should see the page title contains "modAI"
+
+  Scenario Outline: User gets redirected to /chat as fallback
+    When I visit the url path "<url>"
+    Then the url path should be "/chat"
+
+    Examples:
+      | url   |
+      | /     |
+      | /chat |
+      | /foo  |

frontend_omni/tests/features/app_loading.feature

@@ -1,14 +1,42 @@
-import { Given, When, Then, After } from "@cucumber/cucumber";
+import {
+    When,
+    Then,
+    BeforeAll,
+    AfterAll,
+    Before,
+    After,
+} from "@cucumber/cucumber";
 import { CustomWorld, TEST_PORT } from "./world.ts";
+import { buildApp, startContainer, stopContainer } from "./container.ts";
 
-Given("the app is running", async function (this: CustomWorld) {
+BeforeAll(async () => {
+    await buildApp();
+    await startContainer();
+});
+
+AfterAll(async () => {
+    await stopContainer();
+});
+
+Before(async function (this: CustomWorld) {
     await this.init();
 });
 
+After(async function (this: CustomWorld) {
+    await this.close();
+});
+
 When("I visit the homepage", async function (this: CustomWorld) {
     await this.page.goto(`http://localhost:${TEST_PORT}`);
 });
 
+When(
+    "I visit the url path {string}",
+    async function (this: CustomWorld, path: string) {
+        await this.page.goto(`http://localhost:${TEST_PORT}${path}`);
+    },
+);
+
 Then(
     "I should see the page title contains {string}",
     async function (this: CustomWorld, expectedTitle: string) {
@@ -21,6 +49,10 @@ Then(
     },
 );
 
-After(async function (this: CustomWorld) {
-    await this.close();
-});
+Then(
+    "the url path should be {string}",
+    async function (this: CustomWorld, expectedPath: string) {
+        const expectedUrl = `http://localhost:${TEST_PORT}${expectedPath}`;
+        await this.page.waitForURL(expectedUrl);
+    },
+);

frontend_omni/tests/steps.ts

@@ -1,69 +1,21 @@
 import { setWorldConstructor } from "@cucumber/cucumber";
 import * as playwright from "playwright";
-import { execSync } from "child_process";
-import * as net from "net";
 
 const TEST_PORT = 8088;
 
 export { TEST_PORT };
 
-async function waitForPort(
-    port: number,
-    host: string = "localhost",
-    timeout: number = 30000,
-): Promise<void> {
-    const startTime = Date.now();
-    while (Date.now() - startTime < timeout) {
-        try {
-            await new Promise<void>((resolve, reject) => {
-                const socket = net.createConnection({ port, host }, () => {
-                    socket.destroy();
-                    resolve();
-                });
-                socket.on("error", () => {
-                    reject();
-                });
-            });
-            return;
-        } catch {
-            await new Promise((resolve) => setTimeout(resolve, 500));
-        }
-    }
-    throw new Error(`Port ${port} on ${host} not available after ${timeout}ms`);
-}
-
 export class CustomWorld {
     browser!: playwright.Browser;
     context!: playwright.BrowserContext;
     page!: playwright.Page;
 
     async init() {
-        // Build the app
-        console.log("Building the app...");
-        execSync("pnpm build", { stdio: "inherit" });
-
-        // Start Caddy container
-        console.log("Starting Caddy container...");
-        execSync(`docker container rm -f caddy-uitest || true`, {
-            stdio: "ignore",
-        });
-        execSync(
-            `docker run --rm -d --name caddy-uitest -p ${TEST_PORT}:80 -v ${process.cwd()}/dist:/usr/share/caddy caddy`,
-            { stdio: "inherit" },
-        );
-
-        // Wait for container to start
-        console.log("Waiting for Caddy to be ready...");
-        await waitForPort(TEST_PORT);
-
-        console.log("Launching browser...");
         this.browser = await playwright.chromium.launch({
             headless: process.env.HEADED !== "true",
         });
         this.context = await this.browser.newContext();
         this.page = await this.context.newPage();
-
-        console.log("Init done");
     }
 
     async close() {
@@ -78,15 +30,6 @@ export class CustomWorld {
         } catch (error) {
             console.warn("Failed to close browser:", error);
         }
-
-        try {
-            console.log("Stopping Caddy container...");
-            execSync("docker container rm -f caddy-uitest || true", {
-                stdio: "ignore",
-            });
-        } catch (error) {
-            console.warn("Failed to stop Caddy container:", error);
-        }
     }
 }