From f22b69bb927707478a4e61edcc0bf5573ee573fc Mon Sep 17 00:00:00 2001
From: Hare <kei.hiracchi.0928@gmail.com>
Date: Sun, 1 Nov 2026 00:32:06 +0900
Subject: [PATCH] test: reuse routers in workspace config fixtures

---
 .../src/server_workspace_config_wip_tests.rs          | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/crates/workspace-server/src/server_workspace_config_wip_tests.rs b/crates/workspace-server/src/server_workspace_config_wip_tests.rs
index 4650edcf9..d0ba52e17 100644
--- a/crates/workspace-server/src/server_workspace_config_wip_tests.rs
+++ b/crates/workspace-server/src/server_workspace_config_wip_tests.rs
@@ -54,7 +54,8 @@ impl agen::llm_client::LlmClient for ConfigNoopLlmClient {
 // No fake config provider, copied tree, or fabricated attachment ledger is used.
 #[derive(Clone)]
 struct ConfigRouterClient {
-    api: WorkspaceApi,
+    // Reuse routing setup as serve() does; authentication and store reads still run per request.
+    router: Router,
     worker: RuntimeWorkerRef,
     identity: Arc<RuntimeIdentityMaterial>,
     runtime: tokio::runtime::Handle,
@@ -95,7 +96,7 @@ impl worker::WorkspaceClient for ConfigRouterClient {
             .unwrap()
             .push(format!("{method} {}", request.path));
         let is_commit = request.path.ends_with("/commit");
-        let api = self.api.clone();
+        let router = self.router.clone();
         let identity = self.identity.clone();
         let worker = self.worker.clone();
         let runtime = self.runtime.clone();
@@ -111,7 +112,7 @@ impl worker::WorkspaceClient for ConfigRouterClient {
                     &request.path,
                     body,
                 );
-                let response = build_router(api)
+                let response = router
                     .oneshot(signed)
                     .await
                     .map_err(|_| Failure::Request("router dispatch failed".into()))?;
@@ -304,7 +305,7 @@ impl ConfigBatchWipFixture {
         grant(&backend.api, &backend.worker, Access::ReadWrite).await;
         attach(&backend.api, &backend.worker).await;
         let client = Arc::new(ConfigRouterClient {
-            api: backend.api.clone(),
+            router: build_router(backend.api.clone()),
             worker: backend.worker.clone(),
             identity: Arc::new(identity),
             runtime: tokio::runtime::Handle::current(),
@@ -873,7 +874,7 @@ async fn workspace_config_real_selected_invoke_backend_wip_edit_create_delete_im
     configure_runtime_request_auth(&mut fixture.api, &identity, &fixture.worker.runtime_id);
     let granted = grant(&fixture.api, &fixture.worker, Access::ReadWrite).await;
     let client = Arc::new(ConfigRouterClient {
-        api: fixture.api.clone(),
+        router: build_router(fixture.api.clone()),
         worker: fixture.worker.clone(),
         identity: Arc::new(identity),
         runtime: tokio::runtime::Handle::current(),