test: refactor test setup for dynamic redis client imports

Co-authored-by: ramvignesh-b <ramvignesh-b@github.com>
Reviewed-on: #5

src/constants.ts

@@ -0,0 +1,6 @@
+import { API_VERSION, APP_VERSION } from "./version";
+
+export { API_VERSION, APP_VERSION };
+export const API_PREFIX = `/api/${API_VERSION}`;
+export const AUTH_PREFIX = `/${API_VERSION}/auth`;
+export const DOCS_PREFIX = `/docs/${API_VERSION}`;

src/core/ConfigManager.ts

@@ -40,4 +40,13 @@ export class ConfigManager {
 
 		return result;
 	}
+
+	async deleteProviderConfig(provider: string): Promise<void> {
+		await this.redis.del(`config:${provider}`);
+		// Also clean up tokens
+		const tokenKeys = await this.redis.keys(`provider:${provider}:*`);
+		if (tokenKeys.length > 0) {
+			await this.redis.del(...tokenKeys);
+		}
+	}
 }

src/index.ts

@@ -4,7 +4,9 @@ import { serveStatic } from "hono/bun";
 import { logger } from "hono/logger";
 import { prettyJSON } from "hono/pretty-json";
 import { config } from "./config";
+import { API_PREFIX, AUTH_PREFIX, DOCS_PREFIX } from "./constants";
 import { redis } from "./core/RedisClient";
+import { openApiSpec, securityScheme } from "./openapi";
 import { apiRoutes } from "./routes/api";
 import { authRoutes } from "./routes/auth";
 import { configRoutes } from "./routes/config";
@@ -13,29 +15,19 @@ import { dashboardRoutes } from "./routes/dashboard";
 const app = new OpenAPIHono({ strict: false });
 
 // OpenAPI specs
-app.doc("/doc", {
+app.doc(`${DOCS_PREFIX}/openapi.json`, openApiSpec);
-	openapi: "3.0.0",
+app.openAPIRegistry.registerComponent("securitySchemes", "API_KEY", securityScheme);
-	info: {
-		version: "1.0.0",
-		title: "toknd — Auth Broker API",
-		description: "Centralized token management and OAuth2 broker service.",
-	},
-});
-
-app.openAPIRegistry.registerComponent("securitySchemes", "API_KEY", {
-	type: "http",
-	scheme: "bearer",
-});
 
 // Scalar API Reference
 app.get(
-	"/api",
+	DOCS_PREFIX,
 	Scalar({
 		theme: "solarized",
-		url: "/doc",
+		url: `${DOCS_PREFIX}/openapi.json`,
 	}),
 );
-app.get("/docs", (c) => c.redirect("/api"));
+app.get("/docs", (c) => c.redirect(DOCS_PREFIX));
+app.get("/api", (c) => c.redirect(DOCS_PREFIX));
 
 app.use("*", logger());
 app.use("*", prettyJSON());
@@ -43,9 +35,9 @@ app.use("*", prettyJSON());
 app.get("/", (c) => c.redirect("/app"));
 
 app.get("/app/dashboard.js", serveStatic({ path: "./src/views/dashboard.js" }));
-app.route("/auth", authRoutes);
+app.route(AUTH_PREFIX, authRoutes);
-app.route("/api/config", configRoutes);
+app.route(`${API_PREFIX}/config`, configRoutes);
-app.route("/api", apiRoutes);
+app.route(API_PREFIX, apiRoutes);
 app.route("/app", dashboardRoutes);
 
 app.notFound((c) => {

src/openapi.ts

@@ -0,0 +1,31 @@
+import { API_VERSION, APP_VERSION } from "./constants";
+
+export const openApiSpec = {
+	openapi: "3.0.0",
+	info: {
+		version: `${API_VERSION}.${APP_VERSION}`,
+		title: "toknd Auth Broker API",
+		description:
+			"A high-performance OAuth2 broker and token management service. Designed to centralize provider configurations and automate token lifecycle management across distributed systems.",
+	},
+	tags: [
+		{
+			name: "Tokens",
+			description: "Endpoint operations for accessing and force-refreshing active provider tokens.",
+		},
+		{
+			name: "Management",
+			description: "Administrative operations for provider lifecycle and configuration.",
+		},
+		{
+			name: "Auth (Internal)",
+			description: "System-level OAuth2 handshake and callback processing.",
+		},
+	],
+	security: [{ API_KEY: [] }],
+};
+
+export const securityScheme = {
+	type: "http",
+	scheme: "bearer",
+} as const;

src/routes/api.ts

@@ -47,6 +47,7 @@ const statusRoute = createRoute({
 	method: "get",
 	path: "/status",
 	security: [{ API_KEY: [] }],
+	tags: ["Tokens"],
 	responses: {
 		200: {
 			content: { "application/json": { schema: StatusResponseSchema } },
@@ -59,6 +60,7 @@ const tokenRoute = createRoute({
 	method: "get",
 	path: "/token/{provider}",
 	security: [{ API_KEY: [] }],
+	tags: ["Tokens"],
 	request: {
 		params: z.object({
 			provider: z.string().openapi({ example: "trakt" }),
@@ -80,6 +82,7 @@ const refreshRoute = createRoute({
 	method: "post",
 	path: "/refresh/{provider}",
 	security: [{ API_KEY: [] }],
+	tags: ["Tokens"],
 	request: {
 		params: z.object({
 			provider: z.string().openapi({ example: "trakt" }),

src/routes/auth.ts

@@ -17,6 +17,8 @@ const AuthErrorResponse = z
 const loginRoute = createRoute({
 	method: "get",
 	path: "/{provider}/login",
+	tags: ["Auth (Internal)"],
+	summary: "Start OAuth2 flow (Managed by System)",
 	request: {
 		params: z.object({
 			provider: z.string().openapi({ example: "trakt" }),
@@ -36,6 +38,8 @@ const loginRoute = createRoute({
 const callbackRoute = createRoute({
 	method: "get",
 	path: "/callback",
+	tags: ["Auth (Internal)"],
+	summary: "OAuth2 callback handler (Managed by System)",
 	request: {
 		query: z.object({
 			state: z

src/routes/config.ts

@@ -38,6 +38,7 @@ const listConfigRoute = createRoute({
 	method: "get",
 	path: "/",
 	security: [{ API_KEY: [] }],
+	tags: ["Management"],
 	responses: {
 		200: {
 			content: { "application/json": { schema: AllProvidersResponse } },
@@ -50,6 +51,7 @@ const setConfigRoute = createRoute({
 	method: "post",
 	path: "/{provider}",
 	security: [{ API_KEY: [] }],
+	tags: ["Management"],
 	request: {
 		params: z.object({
 			provider: z.string().openapi({ example: "trakt" }),
@@ -70,6 +72,24 @@ const setConfigRoute = createRoute({
 	},
 });
 
+const deleteConfigRoute = createRoute({
+	method: "delete",
+	path: "/{provider}",
+	security: [{ API_KEY: [] }],
+	tags: ["Management"],
+	request: {
+		params: z.object({
+			provider: z.string().openapi({ example: "trakt" }),
+		}),
+	},
+	responses: {
+		200: {
+			content: { "application/json": { schema: SuccessMessage } },
+			description: "Delete a provider configuration and its tokens",
+		},
+	},
+});
+
 // Implementations
 configRoutes.use("*", authMiddleware);
 
@@ -88,4 +108,12 @@ configRoutes.openapi(setConfigRoute, async (c) => {
 	return c.json({ message: `Config for ${provider} saved successfully` }, 200);
 });
 
+configRoutes.openapi(deleteConfigRoute, async (c) => {
+	const provider = c.req.valid("param").provider;
+	const configManager = new ConfigManager(redis);
+
+	await configManager.deleteProviderConfig(provider);
+	return c.json({ message: `Config for ${provider} deleted successfully` }, 200);
+});
+
 export { configRoutes };

src/routes/dashboard.tsx

@@ -4,6 +4,7 @@ import { deleteCookie, getCookie, setCookie } from "hono/cookie";
 import { html } from "hono/html";
 import type { Child } from "hono/jsx";
 import { config } from "../config";
+import { API_PREFIX, API_VERSION, APP_VERSION, AUTH_PREFIX, DOCS_PREFIX } from "../constants";
 
 const dashboardRoutes = new Hono({ strict: false });
 
@@ -39,7 +40,14 @@ export const Layout = (props: { title: string; children: Child; isUnlocked?: boo
 			</head>
 			<body
 				class="bg-base-200/50 min-h-screen font-['DM_Sans',sans-serif] antialiased text-base-content tracking-tight"
-				x-data={`dashboard({ initialIsUnlocked: ${props.isUnlocked || false} })`}
+				x-data={`dashboard({ 
+					initialIsUnlocked: ${props.isUnlocked || false},
+					apiVersion: '${API_VERSION}',
+					appVersion: '${APP_VERSION}',
+					apiPrefix: '${API_PREFIX}',
+					authPrefix: '${AUTH_PREFIX}',
+					docsPrefix: '${DOCS_PREFIX}'
+				})`}
 			>
 				{props.children}
 				<script src="/app/dashboard.js"></script>
@@ -52,7 +60,7 @@ export const Layout = (props: { title: string; children: Child; isUnlocked?: boo
 export const Dashboard = (props: { isUnlocked: boolean }) => (
 	<Layout title="toknd — Auth Broker Dashboard" isUnlocked={props.isUnlocked}>
 		<div class="navbar bg-base-100 shadow-sm px-4 md:px-8 border-b border-base-300">
-			<div class="flex-1">
+			<div class="flex-1 flex items-center gap-6">
 				<div class="flex items-center gap-2">
 					<div class="w-8 h-8 bg-primary rounded-lg flex items-center justify-center text-primary-content font-semibold text-lg">
 						<i class="ph-duotone ph-fingerprint"></i>
@@ -61,49 +69,75 @@ export const Dashboard = (props: { isUnlocked: boolean }) => (
 						toknd <span class="text-xs font-normal opacity-50 ml-1">auth broker</span>
 					</div>
 				</div>
+
+				<nav class="hidden md:flex items-center gap-1">
+					<a
+						href={DOCS_PREFIX}
+						target="_blank"
+						class="btn btn-ghost btn-sm text-base-content/60 hover:text-primary gap-2 px-3"
+						rel="noopener"
+					>
+						<i class="ph-duotone ph-book-open text-lg"></i>
+						<span class="font-bold uppercase tracking-widest text-xs">
+							API Reference{" "}
+							<sup class="text-[8px] opacity-50 ml-0.5">
+								{API_VERSION}.{APP_VERSION}
+							</sup>
+						</span>
+					</a>
+				</nav>
 			</div>
 			<div class="flex-none hidden sm:flex">
-				<div class="join border border-base-200/50 bg-base-200/50 rounded-xl overflow-hidden focus-within:border-primary transition-colors">
+				<template x-if="!isUnlocked">
-					<div class="join-item flex items-center px-4 bg-base-200">
+					<div class="join border border-base-200/50 bg-base-200/50 rounded-xl overflow-hidden focus-within:border-primary transition-colors">
-						<i class="ph-duotone ph-key text-secondary text-lg"></i>
+						<div class="join-item flex items-center px-4 bg-base-200">
-					</div>
+							<i class="ph-duotone ph-key text-secondary text-lg"></i>
-					<div class="relative flex-1" x-data="{ show: false }">
+						</div>
-						<input
+						<div class="relative flex-1" x-data="{ show: false }">
-							x-bind:type="show ? 'text' : 'password'"
+							<input
-							id="apiKey"
+								x-bind:type="show ? 'text' : 'password'"
-							name="apiKey"
+								id="apiKey"
-							x-model="apiKey"
+								name="apiKey"
-							aria-label="Master API Key"
+								x-model="apiKey"
-							placeholder="API_KEY"
+								aria-label="Master API Key"
-							class="input join-item input-sm bg-transparent border-none focus:outline-none w-48 lg:w-64 text-xs pr-10 font-mono"
+								placeholder="API_KEY"
-						/>
+								class="input join-item input-sm bg-transparent border-none focus:outline-none w-48 lg:w-64 text-xs pr-10 font-mono"
+							/>
+							<button
+								type="button"
+								x-on:click="show = !show"
+								class="absolute right-2 top-1/2 -translate-y-1/2 btn btn-ghost btn-xs btn-square"
+							>
+								<i x-bind:class="show ? 'ph-duotone ph-eye-slash text-base opacity-50' : 'ph-duotone ph-eye text-base opacity-50'"></i>
+							</button>
+						</div>
 						<button
-							type="button"
+							x-on:click="unlock()"
-							x-on:click="show = !show"
+							type="submit"
-							class="absolute right-2 top-1/2 -translate-y-1/2 btn btn-ghost btn-xs btn-square"
+							class="btn btn-primary btn-sm join-item px-6"
+							x-bind:disabled="loading"
 						>
-							<i x-bind:class="show ? 'ph-duotone ph-eye-slash text-base opacity-50' : 'ph-duotone ph-eye text-base opacity-50'"></i>
+							<i class="ph-duotone ph-lock-key-open text-lg" x-show="!loading"></i>
+							<span class="loading loading-spinner loading-xs" x-show="loading"></span>
+							<span
+								class="ml-1 hidden md:inline"
+								x-text="loading ? 'Unlocking...' : 'Unlock'"
+							></span>
 						</button>
 					</div>
+				</template>
+
+				<template x-if="isUnlocked">
 					<button
-						x-on:click="unlock()"
-						type="submit"
-						class="btn btn-primary btn-sm join-item px-6"
-						x-bind:disabled="loading"
-					>
-						<i class="ph-duotone ph-lock-key-open text-lg" x-show="!loading"></i>
-						<span class="loading loading-spinner loading-xs" x-show="loading"></span>
-						<span class="ml-1 hidden md:inline" x-text="loading ? 'Unlocking...' : 'Unlock'"></span>
-					</button>
-					<button
-						x-show="isUnlocked"
 						x-on:click="logout()"
 						type="button"
-						class="btn btn-ghost btn-sm join-item text-error hover:bg-error/10"
+						class="btn btn-ghost btn-sm text-error hover:bg-error/10 gap-2 px-4"
+						x-bind:disabled="loading"
 					>
 						<i class="ph-bold ph-power text-lg"></i>
+						<span class="font-bold uppercase tracking-wider text-xs">Logout</span>
 					</button>
-				</div>
+				</template>
 			</div>
 		</div>
 
@@ -308,10 +342,20 @@ export const Dashboard = (props: { isUnlocked: boolean }) => (
 									<div class="card bg-base-200/50 border border-base-300 shadow-sm hover:shadow-md transition-all group">
 										<div class="card-body p-5">
 											<div class="flex flex-col mb-4">
-												<span
+												<div class="flex justify-between items-start">
-													x-text="provider.name"
+													<span
-													class="text-lg font-black text-base-content/90 uppercase"
+														x-text="provider.name"
-												></span>
+														class="text-lg font-black text-base-content/90 uppercase"
+													></span>
+													<button
+														type="button"
+														x-on:click="deleteProvider(provider.name)"
+														class="btn btn-error btn-xs mt-1 opacity-0 group-hover:opacity-100 transition-all duration-300"
+														title="Delete Provider"
+													>
+														<i class="ph-bold ph-trash text-lg"></i>
+													</button>
+												</div>
 												<span
 													x-text="provider.config.clientId"
 													x-bind:title="provider.config.clientId"
@@ -412,7 +456,7 @@ export const Dashboard = (props: { isUnlocked: boolean }) => (
 												<div class="grid grid-cols-2 gap-2">
 													<button
 														type="button"
-														x-on:click="window.open('/auth/' + provider.name + '/login', '_blank')"
+														x-on:click={`window.open('${AUTH_PREFIX}/' + provider.name + '/login', '_blank')`}
 														class="btn btn-primary btn-sm"
 													>
 														<i class="ph-bold ph-link"></i> Connect

src/version.ts

@@ -0,0 +1,2 @@
+export const API_VERSION = "v1";
+export const APP_VERSION = "1.0";

src/views/dashboard.js

@@ -10,194 +10,230 @@ document.addEventListener("alpine:init", () => {
 		return date.toLocaleDateString();
 	};
 
-	window.Alpine.data("dashboard", ({ initialIsUnlocked }) => ({
+	window.Alpine.data(
-		apiKey: "",
+		"dashboard",
-		isUnlocked: initialIsUnlocked,
+		({ initialIsUnlocked, apiPrefix, authPrefix, docsPrefix, apiVersion, appVersion }) => ({
-		loading: false,
+			apiKey: "",
-		providers: [],
+			isUnlocked: initialIsUnlocked,
-		form: {
+			apiPrefix,
-			providerName: "",
+			authPrefix,
-			clientId: "",
+			docsPrefix,
-			clientSecret: "",
+			apiVersion,
-			authUrl: "",
+			appVersion,
-			tokenUrl: "",
+			loading: false,
-			scope: "public",
+			providers: [],
-		},
+			form: {
-		notification: {
+				providerName: "",
-			show: false,
+				clientId: "",
-			message: "",
+				clientSecret: "",
-			type: "success",
+				authUrl: "",
-		},
+				tokenUrl: "",
+				scope: "public",
+			},
+			notification: {
+				show: false,
+				message: "",
+				type: "success",
+			},
 
-		init() {
+			init() {
-			if (this.isUnlocked) {
+				if (this.isUnlocked) {
-				this.fetchProviders();
+					this.fetchProviders();
-			}
+				}
-		},
+			},
 
-		async unlock() {
+			async unlock() {
-			if (!this.apiKey) return;
+				if (!this.apiKey) return;
-			this.loading = true;
+				this.loading = true;
-			try {
+				try {
-				const res = await fetch("/app/unlock", {
+					const res = await fetch("/app/unlock", {
-					method: "POST",
+						method: "POST",
-					headers: { "Content-Type": "application/json" },
+						headers: { "Content-Type": "application/json" },
-					body: JSON.stringify({ apiKey: this.apiKey }),
+						body: JSON.stringify({ apiKey: this.apiKey }),
-				});
+					});
 
-				if (!res.ok) throw new Error("Invalid API Key");
+					if (!res.ok) throw new Error("Invalid API Key");
 
-				this.isUnlocked = true;
+					this.isUnlocked = true;
-				await this.fetchProviders();
+					await this.fetchProviders();
-				this.apiKey = ""; // Clear after success
+					this.apiKey = ""; // Clear after success
-			} catch (err) {
+				} catch (err) {
-				this.showNotification(err.message, "error");
+					this.showNotification(err.message, "error");
-				this.isUnlocked = false;
+					this.isUnlocked = false;
-			} finally {
+				} finally {
-				this.loading = false;
+					this.loading = false;
-			}
+				}
-		},
+			},
 
-		async logout() {
+			async logout() {
-			this.loading = true;
+				this.loading = true;
-			try {
+				try {
-				await fetch("/app/logout", { method: "POST" });
+					await fetch("/app/logout", { method: "POST" });
+					this.isUnlocked = false;
+					this.providers = [];
+					this.showNotification("Logged out successfully");
+				} catch (err) {
+					this.showNotification(`Logout failed: ${err.message}`, "error");
+				} finally {
+					this.loading = false;
+				}
+			},
+
+			async fetchProviders() {
+				this.loading = true;
+				try {
+					const [configRes, statusRes] = await Promise.all([
+						fetch(`${this.apiPrefix}/config`),
+						fetch(`${this.apiPrefix}/status`),
+					]);
+
+					if (configRes.status === 401 || statusRes.status === 401) {
+						return this.handleSessionExpired();
+					}
+
+					if (!configRes.ok || !statusRes.ok) throw new Error("Failed to fetch data");
+
+					const [config, status] = await Promise.all([configRes.json(), statusRes.json()]);
+					this.providers = this.mapProviders(config, status);
+				} catch (err) {
+					this.showNotification(err.message, "error");
+				} finally {
+					this.loading = false;
+				}
+			},
+
+			mapProviders(config, status) {
+				return Object.entries(config).map(([name, cfg]) => ({
+					name,
+					config: cfg,
+					status: status[name] || { accessToken: null, refreshToken: null, lastUpdated: null },
+				}));
+			},
+
+			handleSessionExpired() {
 				this.isUnlocked = false;
 				this.providers = [];
-				this.showNotification("Logged out successfully");
+				this.showNotification("Session expired", "error");
-			} catch (err) {
+			},
-				this.showNotification(`Logout failed: ${err.message}`, "error");
-			} finally {
-				this.loading = false;
-			}
-		},
 
-		async fetchProviders() {
+			async saveConfig() {
-			this.loading = true;
+				this.loading = true;
-			try {
+				try {
-				const [configRes, statusRes] = await Promise.all([
+					const res = await fetch(`${this.apiPrefix}/config/${this.form.providerName}`, {
-					fetch("/api/config"),
+						method: "POST",
-					fetch("/api/status"),
+						headers: { "Content-Type": "application/json" },
-				]);
+						body: JSON.stringify(this.form),
+					});
 
-				if (configRes.status === 401 || statusRes.status === 401) {
+					if (res.status === 401) {
-					return this.handleSessionExpired();
+						this.isUnlocked = false;
+						throw new Error("Session expired");
+					}
+
+					if (!res.ok) throw new Error("Failed to save");
+
+					this.showNotification("Saved successfully");
+					await this.fetchProviders();
+
+					this.form = {
+						providerName: "",
+						clientId: "",
+						clientSecret: "",
+						authUrl: "",
+						tokenUrl: "",
+						scope: "public",
+					};
+				} catch (err) {
+					this.showNotification(err.message, "error");
+				} finally {
+					this.loading = false;
 				}
+			},
 
-				if (!configRes.ok || !statusRes.ok) throw new Error("Failed to fetch data");
+			async forceRefresh(name) {
+				this.loading = true;
+				try {
+					const res = await fetch(`${this.apiPrefix}/refresh/${name}`, {
+						method: "POST",
+					});
 
-				const [config, status] = await Promise.all([configRes.json(), statusRes.json()]);
+					if (res.status === 401) {
-				this.providers = this.mapProviders(config, status);
+						return this.handleSessionExpired();
-			} catch (err) {
+					}
-				this.showNotification(err.message, "error");
-			} finally {
-				this.loading = false;
-			}
-		},
 
-		mapProviders(config, status) {
+					if (!res.ok) throw new Error("Refresh failed");
-			return Object.entries(config).map(([name, cfg]) => ({
-				name,
-				config: cfg,
-				status: status[name] || { accessToken: null, refreshToken: null, lastUpdated: null },
-			}));
-		},
 
-		handleSessionExpired() {
+					this.showNotification(`Refreshed ${name}`);
-			this.isUnlocked = false;
+					await this.fetchProviders();
-			this.providers = [];
+				} catch (err) {
-			this.showNotification("Session expired", "error");
+					this.showNotification(err.message, "error");
-		},
+				} finally {
-
+					this.loading = false;
-		async saveConfig() {
-			this.loading = true;
-			try {
-				const res = await fetch(`/api/config/${this.form.providerName}`, {
-					method: "POST",
-					headers: { "Content-Type": "application/json" },
-					body: JSON.stringify(this.form),
-				});
-
-				if (res.status === 401) {
-					this.isUnlocked = false;
-					throw new Error("Session expired");
 				}
+			},
 
-				if (!res.ok) throw new Error("Failed to save");
+			async deleteProvider(name) {
+				if (
+					!confirm(
+						`Are you sure you want to delete ${name}? This will also remove all associated tokens.`,
+					)
+				)
+					return;
 
-				this.showNotification("Saved successfully");
+				this.loading = true;
-				await this.fetchProviders();
+				try {
+					const res = await fetch(`${this.apiPrefix}/config/${name}`, {
+						method: "DELETE",
+					});
 
+					if (res.status === 401) {
+						return this.handleSessionExpired();
+					}
+
+					if (!res.ok) throw new Error("Delete failed");
+
+					this.showNotification(`Deleted ${name}`);
+					await this.fetchProviders();
+				} catch (err) {
+					this.showNotification(err.message, "error");
+				} finally {
+					this.loading = false;
+				}
+			},
+
+			editProvider(provider) {
 				this.form = {
-					providerName: "",
+					providerName: provider.name,
-					clientId: "",
+					clientId: provider.config.clientId,
-					clientSecret: "",
+					clientSecret: provider.config.clientSecret,
-					authUrl: "",
+					authUrl: provider.config.authUrl,
-					tokenUrl: "",
+					tokenUrl: provider.config.tokenUrl,
-					scope: "public",
+					scope: provider.config.scope,
 				};
-			} catch (err) {
+				window.scrollTo({ top: 0, behavior: "smooth" });
-				this.showNotification(err.message, "error");
+			},
-			} finally {
-				this.loading = false;
-			}
-		},
 
-		async forceRefresh(name) {
+			getRedirectUri() {
-			this.loading = true;
+				return `${window.location.origin}${this.authPrefix}/${this.form.providerName || "{provider}"}/callback`;
-			try {
+			},
-				const res = await fetch(`/api/refresh/${name}`, {
+
-					method: "POST",
+			copyToClipboard(text) {
+				if (!text) return;
+				navigator.clipboard.writeText(text).then(() => {
+					this.showNotification("Copied");
 				});
+			},
 
-				if (res.status === 401) {
+			showNotification(message, type = "success") {
-					this.isUnlocked = false;
+				this.notification = { show: true, message, type };
-					throw new Error("Session expired");
+				setTimeout(() => {
-				}
+					this.notification.show = false;
+				}, 3000);
+			},
 
-				if (!res.ok) throw new Error("Refresh failed");
+			formatTime(timestamp) {
-
+				return formatTime(timestamp);
-				this.showNotification(`Refreshed ${name}`);
+			},
-				await this.fetchProviders();
+		}),
-			} catch (err) {
+	);
-				this.showNotification(err.message, "error");
-			} finally {
-				this.loading = false;
-			}
-		},
-
-		editProvider(provider) {
-			this.form = {
-				providerName: provider.name,
-				clientId: provider.config.clientId,
-				clientSecret: provider.config.clientSecret,
-				authUrl: provider.config.authUrl,
-				tokenUrl: provider.config.tokenUrl,
-				scope: provider.config.scope,
-			};
-			window.scrollTo({ top: 0, behavior: "smooth" });
-		},
-
-		getRedirectUri() {
-			return `${window.location.origin}/auth/${this.form.providerName || "{provider}"}/callback`;
-		},
-
-		copyToClipboard(text) {
-			if (!text) return;
-			navigator.clipboard.writeText(text).then(() => {
-				this.showNotification("Copied");
-			});
-		},
-
-		showNotification(message, type = "success") {
-			this.notification = { show: true, message, type };
-			setTimeout(() => {
-				this.notification.show = false;
-			}, 3000);
-		},
-
-		formatTime(timestamp) {
-			return formatTime(timestamp);
-		},
-	}));
 });

tests/integration/api.test.ts

@@ -1,16 +1,10 @@
 // @ts-nocheck
-import { afterEach, describe, expect, it, mock, spyOn } from "bun:test";
+import { describe, expect, it, spyOn } from "bun:test";
+import { API_PREFIX } from "../../src/constants";
 import { redis } from "../../src/core/RedisClient";
 import { app } from "../../src/index";
 
 describe("API Integration", () => {
-	afterEach(() => {
-		mock.restore();
-		redis.get.mockImplementation(() => Promise.resolve(null));
-		redis.set.mockImplementation(() => Promise.resolve());
-		redis.keys.mockImplementation(() => Promise.resolve([]));
-	});
-
 	const mockTraktConfig = JSON.stringify({
 		clientId: "trakt-client-id",
 		clientSecret: "trakt-client-secret",
@@ -20,7 +14,7 @@ describe("API Integration", () => {
 	});
 
 	it("should return 401 if API Key is missing", async () => {
-		const res = await app.request("/api/status");
+		const res = await app.request(`${API_PREFIX}/status`);
 
 		expect(res.status).toBe(401);
 		const body = await res.json();
@@ -55,7 +49,7 @@ describe("API Integration", () => {
 			return Promise.resolve(null);
 		});
 
-		const res = await app.request("/api/status", {
+		const res = await app.request(`${API_PREFIX}/status`, {
 			headers: {
 				Authorization: "Bearer test-api-key",
 			},
@@ -75,7 +69,7 @@ describe("API Integration", () => {
 			return Promise.resolve(null);
 		});
 
-		const res = await app.request("/api/status", {
+		const res = await app.request(`${API_PREFIX}/status`, {
 			headers: {
 				Cookie: "toknd_api_key=test-api-key",
 			},
@@ -87,7 +81,7 @@ describe("API Integration", () => {
 	});
 
 	it("should return 404 for unknown provider token", async () => {
-		const res = await app.request("/api/token/unconfigured-provider", {
+		const res = await app.request(`${API_PREFIX}/token/unconfigured-provider`, {
 			headers: {
 				Authorization: "Bearer test-api-key",
 			},
@@ -103,7 +97,7 @@ describe("API Integration", () => {
 			return Promise.resolve(null);
 		});
 
-		const res = await app.request("/api/token/trakt", {
+		const res = await app.request(`${API_PREFIX}/token/trakt`, {
 			headers: {
 				Authorization: "Bearer test-api-key",
 			},
@@ -120,7 +114,7 @@ describe("API Integration", () => {
 			return Promise.resolve(null);
 		});
 
-		const res = await app.request("/api/token/trakt", {
+		const res = await app.request(`${API_PREFIX}/token/trakt`, {
 			headers: {
 				Authorization: "Bearer test-api-key",
 			},
@@ -150,7 +144,7 @@ describe("API Integration", () => {
 			}),
 		);
 
-		const res = await app.request("/api/refresh/trakt", {
+		const res = await app.request(`${API_PREFIX}/refresh/trakt`, {
 			method: "POST",
 			headers: {
 				Authorization: "Bearer test-api-key",

tests/integration/auth.test.ts

@@ -1,14 +1,10 @@
 // @ts-nocheck
-import { afterEach, describe, expect, it, mock, spyOn } from "bun:test";
+import { describe, expect, it, spyOn } from "bun:test";
+import { AUTH_PREFIX } from "../../src/constants";
 import { redis } from "../../src/core/RedisClient";
 import { app } from "../../src/index";
 
 describe("Auth Integration", () => {
-	afterEach(() => {
-		mock.restore();
-		redis.get.mockImplementation(() => Promise.resolve(null));
-	});
-
 	const mockProviderConfig = JSON.stringify({
 		clientId: "trakt-client-id",
 		clientSecret: "trakt-client-secret",
@@ -23,7 +19,7 @@ describe("Auth Integration", () => {
 			return Promise.resolve(null);
 		});
 
-		const res = await app.request("/auth/trakt/login");
+		const res = await app.request(`${AUTH_PREFIX}/trakt/login`);
 
 		expect(res.status).toBe(302);
 		expect(res.headers.get("Location")).toContain("trakt.tv/oauth/authorize");
@@ -48,7 +44,7 @@ describe("Auth Integration", () => {
 			}),
 		);
 
-		const res = await app.request("/auth/callback?state=trakt&code=temporary-auth-code");
+		const res = await app.request(`${AUTH_PREFIX}/callback?state=trakt&code=temporary-auth-code`);
 
 		expect(res.status).toBe(302);
 		expect(res.headers.get("Location")).toBe("/app/success?provider=trakt");
@@ -57,13 +53,13 @@ describe("Auth Integration", () => {
 	});
 
 	it("should return 404 if provider not configured during login", async () => {
-		const res = await app.request("/auth/unknown-provider/login");
+		const res = await app.request(`${AUTH_PREFIX}/unknown-provider/login`);
 
 		expect(res.status).toBe(404);
 	});
 
 	it("should return 400 if callback is missing state or code", async () => {
-		const res = await app.request("/auth/callback?code=some-code");
+		const res = await app.request(`${AUTH_PREFIX}/callback?code=some-code`);
 
 		expect(res.status).toBe(400);
 	});

tests/integration/config.test.ts

@@ -1,16 +1,10 @@
 // @ts-nocheck
-import { afterEach, describe, expect, it, mock } from "bun:test";
+import { describe, expect, it } from "bun:test";
+import { API_PREFIX } from "../../src/constants";
 import { redis } from "../../src/core/RedisClient";
 import { app } from "../../src/index";
 
 describe("Config Integration", () => {
-	afterEach(() => {
-		mock.restore();
-		redis.get.mockImplementation(() => Promise.resolve(null));
-		redis.set.mockImplementation(() => Promise.resolve());
-		redis.keys.mockImplementation(() => Promise.resolve([]));
-	});
-
 	it("should list all configured providers", async () => {
 		redis.keys.mockReturnValue(Promise.resolve(["config:trakt"]));
 		redis.get.mockImplementation(() =>
@@ -25,7 +19,7 @@ describe("Config Integration", () => {
 			),
 		);
 
-		const res = await app.request("/api/config", {
+		const res = await app.request(`${API_PREFIX}/config`, {
 			headers: {
 				Authorization: "Bearer test-api-key",
 			},
@@ -46,7 +40,7 @@ describe("Config Integration", () => {
 			scope: "user:email",
 		};
 
-		const res = await app.request("/api/config/github", {
+		const res = await app.request(`${API_PREFIX}/config/github`, {
 			method: "POST",
 			headers: {
 				Authorization: "Bearer test-api-key",
@@ -67,7 +61,7 @@ describe("Config Integration", () => {
 			clientId: "missing-other-required-fields",
 		};
 
-		const res = await app.request("/api/config/invalid", {
+		const res = await app.request(`${API_PREFIX}/config/invalid`, {
 			method: "POST",
 			headers: {
 				Authorization: "Bearer test-api-key",
@@ -78,4 +72,16 @@ describe("Config Integration", () => {
 
 		expect(res.status).toBe(400);
 	});
+
+	it("should delete a provider configuration", async () => {
+		const res = await app.request(`${API_PREFIX}/config/trakt`, {
+			method: "DELETE",
+			headers: {
+				Authorization: "Bearer test-api-key",
+			},
+		});
+
+		expect(res.status).toBe(200);
+		expect(redis.del).toHaveBeenCalled();
+	});
 });

tests/integration/dashboard.test.ts

@@ -1,5 +1,6 @@
 // @ts-nocheck
 import { describe, expect, it } from "bun:test";
+import { API_PREFIX } from "../../src/constants";
 import { redis } from "../../src/core/RedisClient";
 import { app } from "../../src/index";
 
@@ -26,7 +27,7 @@ describe("Dashboard & Common Integration", () => {
 			throw new Error("Redis Crash");
 		});
 
-		const res = await app.request("/api/status", {
+		const res = await app.request(`${API_PREFIX}/status`, {
 			headers: { Authorization: "Bearer test-api-key" },
 		});
 

tests/setup.ts

@@ -1,18 +1,38 @@
-import { mock } from "bun:test";
+// @ts-nocheck
-
-// Global test setup to stub environment variables
 process.env.API_KEY = "test-api-key";
 process.env.REDIS_HOST = "localhost";
 process.env.REDIS_PORT = "6379";
 process.env.APP_PORT = "3000";
 
+import { afterEach, mock } from "bun:test";
+
+// Global config mock
+mock.module("../src/config", () => ({
+	config: {
+		API_KEY: "test-api-key",
+		REDIS_HOST: "localhost",
+		REDIS_PORT: 6379,
+		APP_PORT: "3000",
+	},
+}));
+
 // Global Redis mock
 mock.module("../src/core/RedisClient", () => ({
 	redis: {
 		status: "ready",
 		get: mock(() => Promise.resolve(null)),
 		set: mock(() => Promise.resolve()),
+		del: mock(() => Promise.resolve(1)),
 		keys: mock(() => Promise.resolve([])),
 		on: mock(() => {}),
 	},
 }));
+
+afterEach(async () => {
+	const { redis } = await import("../src/core/RedisClient");
+	mock.restore();
+	redis.get.mockImplementation(() => Promise.resolve(null));
+	redis.set.mockImplementation(() => Promise.resolve());
+	redis.del.mockImplementation(() => Promise.resolve(1));
+	redis.keys.mockImplementation(() => Promise.resolve([]));
+});