refactor(tests): factor out test values and add missing assertions

Address PR review feedback:
- Factor out test value objects (toolCall, resultContent, etc.) for clearer tests
- Add missing assertions on receivedRequests[0] for list handlers
- Improve test readability by defining expected values at the top of each test

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: ochafik

src/app-bridge.test.ts

@@ -547,9 +547,9 @@ describe("App <-> AppBridge integration", () => {
 
     it("oncalltool setter registers handler for tools/call requests", async () => {
       const toolCall = { name: "test-tool", arguments: { arg: "value" } };
-      const resultContent = [{ type: "text", text: "result" }];
+      const resultContent = [{ type: "text" as const, text: "result" }];
       const receivedCalls: unknown[] = [];
-      
+
       bridge.oncalltool = async (params) => {
         receivedCalls.push(params);
         return { content: resultContent };
@@ -567,29 +567,34 @@ describe("App <-> AppBridge integration", () => {
     });
 
     it("onlistresources setter registers handler for resources/list requests", async () => {
+      const requestParams = {};
+      const resources = [{ uri: "test://resource", name: "Test" }];
       const receivedRequests: unknown[] = [];
+
       bridge.onlistresources = async (params) => {
         receivedRequests.push(params);
-        return { resources: [{ uri: "test://resource", name: "Test" }] };
+        return { resources };
       };
 
       await bridge.connect(bridgeTransport);
       await app.connect(appTransport);
 
       // App sends resources/list request via the protocol's request method
       const result = await app.request(
-        { method: "resources/list", params: {} },
+        { method: "resources/list", params: requestParams },
         ListResourcesResultSchema,
       );
 
       expect(receivedRequests).toHaveLength(1);
-      expect(result.resources).toEqual([
-        { uri: "test://resource", name: "Test" },
-      ]);
+      expect(receivedRequests[0]).toMatchObject(requestParams);
+      expect(result.resources).toEqual(resources);
     });
 
     it("onreadresource setter registers handler for resources/read requests", async () => {
+      const requestParams = { uri: "test://resource" };
+      const contents = [{ uri: "test://resource", text: "content" }];
       const receivedRequests: unknown[] = [];
+
       bridge.onreadresource = async (params) => {
         receivedRequests.push(params);
         return { contents: [{ uri: params.uri, text: "content" }] };
@@ -599,59 +604,61 @@ describe("App <-> AppBridge integration", () => {
       await app.connect(appTransport);
 
       const result = await app.request(
-        { method: "resources/read", params: { uri: "test://resource" } },
+        { method: "resources/read", params: requestParams },
         ReadResourceResultSchema,
       );
 
       expect(receivedRequests).toHaveLength(1);
-      expect(receivedRequests[0]).toMatchObject({ uri: "test://resource" });
-      expect(result.contents).toEqual([
-        { uri: "test://resource", text: "content" },
-      ]);
+      expect(receivedRequests[0]).toMatchObject(requestParams);
+      expect(result.contents).toEqual(contents);
     });
 
     it("onlistresourcetemplates setter registers handler for resources/templates/list requests", async () => {
+      const requestParams = {};
+      const resourceTemplates = [
+        { uriTemplate: "test://{id}", name: "Test Template" },
+      ];
       const receivedRequests: unknown[] = [];
+
       bridge.onlistresourcetemplates = async (params) => {
         receivedRequests.push(params);
-        return {
-          resourceTemplates: [
-            { uriTemplate: "test://{id}", name: "Test Template" },
-          ],
-        };
+        return { resourceTemplates };
       };
 
       await bridge.connect(bridgeTransport);
       await app.connect(appTransport);
 
       const result = await app.request(
-        { method: "resources/templates/list", params: {} },
+        { method: "resources/templates/list", params: requestParams },
         ListResourceTemplatesResultSchema,
       );
 
       expect(receivedRequests).toHaveLength(1);
-      expect(result.resourceTemplates).toEqual([
-        { uriTemplate: "test://{id}", name: "Test Template" },
-      ]);
+      expect(receivedRequests[0]).toMatchObject(requestParams);
+      expect(result.resourceTemplates).toEqual(resourceTemplates);
     });
 
     it("onlistprompts setter registers handler for prompts/list requests", async () => {
+      const requestParams = {};
+      const prompts = [{ name: "test-prompt" }];
       const receivedRequests: unknown[] = [];
+
       bridge.onlistprompts = async (params) => {
         receivedRequests.push(params);
-        return { prompts: [{ name: "test-prompt" }] };
+        return { prompts };
       };
 
       await bridge.connect(bridgeTransport);
       await app.connect(appTransport);
 
       const result = await app.request(
-        { method: "prompts/list", params: {} },
+        { method: "prompts/list", params: requestParams },
         ListPromptsResultSchema,
       );
 
       expect(receivedRequests).toHaveLength(1);
-      expect(result.prompts).toEqual([{ name: "test-prompt" }]);
+      expect(receivedRequests[0]).toMatchObject(requestParams);
+      expect(result.prompts).toEqual(prompts);
     });
 
     it("sendToolListChanged sends notification to app", async () => {