From 16f58af0c22a441aa422de16c3d51faac2450d1e Mon Sep 17 00:00:00 2001
From: Riya <riya@openclaw.local>
Date: Sat, 7 Feb 2026 15:53:07 +0000
Subject: [PATCH] test(frontend): add /activity auth-boundary regression test

---
 frontend/src/app/activity/page.test.tsx | 55 +++++++++++++++++++++++++
 frontend/vitest.config.ts               |  7 ++++
 2 files changed, 62 insertions(+)
 create mode 100644 frontend/src/app/activity/page.test.tsx

diff --git a/frontend/src/app/activity/page.test.tsx b/frontend/src/app/activity/page.test.tsx
new file mode 100644
index 00000000..af5cec2b
--- /dev/null
+++ b/frontend/src/app/activity/page.test.tsx
@@ -0,0 +1,55 @@
+import React from "react";
+import { describe, expect, it, vi } from "vitest";
+import { render, screen } from "@testing-library/react";
+
+import ActivityPage from "./page";
+import { AuthProvider } from "@/components/providers/AuthProvider";
+import { QueryProvider } from "@/components/providers/QueryProvider";
+
+vi.mock("next/link", () => {
+  return {
+    default: ({ href, children, ...props }: any) => (
+      <a href={typeof href === "string" ? href : "#"} {...props}>
+        {children}
+      </a>
+    ),
+  };
+});
+
+// Make Clerk components explode if we ever try to render them without the provider.
+// The regression we want to catch is: AuthProvider skips <ClerkProvider/>, but the
+// wrappers still render <SignedOut/> from @clerk/nextjs (which crashes in real builds).
+vi.mock("@clerk/nextjs", () => {
+  return {
+    ClerkProvider: ({ children }: { children: React.ReactNode }) => <>{children}</>,
+    SignedIn: () => {
+      throw new Error("@clerk/nextjs SignedIn rendered (unexpected in secretless mode)");
+    },
+    SignedOut: () => {
+      throw new Error("@clerk/nextjs SignedOut rendered without ClerkProvider");
+    },
+    SignInButton: ({ children }: { children: React.ReactNode }) => <>{children}</>,
+    SignOutButton: ({ children }: { children: React.ReactNode }) => <>{children}</>,
+    useAuth: () => ({ isLoaded: true, isSignedIn: false }),
+    useUser: () => ({ isLoaded: true, isSignedIn: false, user: null }),
+  };
+});
+
+describe("/activity auth boundary", () => {
+  it("renders without ClerkProvider runtime errors when publishable key is a placeholder", () => {
+    // Simulate CI/secretless env where an arbitrary placeholder value may be present.
+    // AuthProvider should treat this as disabled, and the auth wrappers must not render
+    // Clerk SignedOut/SignedIn components.
+    process.env.NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY = "placeholder";
+
+    render(
+      <AuthProvider>
+        <QueryProvider>
+          <ActivityPage />
+        </QueryProvider>
+      </AuthProvider>,
+    );
+
+    expect(screen.getByText(/sign in to view the feed/i)).toBeInTheDocument();
+  });
+});
diff --git a/frontend/vitest.config.ts b/frontend/vitest.config.ts
index 197150a6..880f1c3c 100644
--- a/frontend/vitest.config.ts
+++ b/frontend/vitest.config.ts
@@ -1,6 +1,13 @@
+import path from "node:path";
+
 import { defineConfig } from "vitest/config";
 
 export default defineConfig({
+  resolve: {
+    alias: {
+      "@": path.resolve(__dirname, "./src"),
+    },
+  },
   test: {
     environment: "jsdom",
     setupFiles: ["./src/setupTests.ts"],