test(plugin): cover remaining manifest summary branches

Author: guillaumegay13

packages/backend/src/routing/resolve.controller.spec.ts

@@ -256,6 +256,32 @@ describe('ResolveController', () => {
       ]);
       expect(mockRoutingService.getEffectiveModel).toHaveBeenCalledTimes(1);
     });
+
+    it('sorts same-provider entries by auth type and defaults missing auth type to api_key', async () => {
+      const req = {
+        ingestionContext: {
+          userId: 'user-42',
+          tenantId: 't1',
+          agentId: 'a1',
+          agentName: 'test-agent',
+        },
+      } as never;
+
+      mockRoutingService.getProviders.mockResolvedValue([
+        { provider: 'openai', auth_type: 'subscription', is_active: true },
+        { provider: 'openai', auth_type: 'api_key', is_active: true },
+        { provider: 'mistral', is_active: true },
+      ]);
+      mockRoutingService.getTiers.mockResolvedValue([]);
+
+      const result = await controller.getSummary(req);
+
+      expect(result.providers).toEqual([
+        { provider: 'mistral', auth_type: 'api_key' },
+        { provider: 'openai', auth_type: 'api_key' },
+        { provider: 'openai', auth_type: 'subscription' },
+      ]);
+    });
   });
 
   describe('RegisterSubscriptionsDto', () => {

packages/openclaw-plugin/__tests__/command.test.ts

@@ -130,6 +130,46 @@ describe("registerCommand", () => {
     );
   });
 
+  it("formats empty providers, unknown tiers, and api-key auth headers", async () => {
+    mockVerify.mockResolvedValueOnce({
+      endpointReachable: true,
+      authValid: true,
+      agentName: null,
+      error: null,
+    });
+    mockFetch.mockResolvedValueOnce({
+      ok: true,
+      json: async () => ({
+        agentName: "custom-agent",
+        providers: [],
+        tiers: [
+          { tier: "custom", model: null, source: "auto", fallback_models: [] },
+        ],
+      }),
+    });
+
+    const api = { registerCommand: jest.fn() };
+    registerCommand(
+      api,
+      { ...config, apiKey: "mnfst_test_key", devMode: false },
+      mockLogger,
+    );
+
+    const cmd = api.registerCommand.mock.calls[0][0];
+    const result = await cmd.execute();
+
+    expect(result).toContain("Dev mode: no");
+    expect(result).toContain("Agent: custom-agent");
+    expect(result).toContain("Providers: none");
+    expect(result).toContain("custom -> unassigned (auto)");
+    expect(mockFetch).toHaveBeenCalledWith(
+      "http://localhost:38238/api/v1/routing/summary",
+      expect.objectContaining({
+        headers: { Authorization: "Bearer mnfst_test_key" },
+      }),
+    );
+  });
+
   it("falls back to status output when the routing summary request fails", async () => {
     mockVerify.mockResolvedValueOnce({
       endpointReachable: true,
@@ -153,6 +193,26 @@ describe("registerCommand", () => {
     );
   });
 
+  it("stringifies non-Error routing summary failures", async () => {
+    mockVerify.mockResolvedValueOnce({
+      endpointReachable: true,
+      authValid: true,
+      agentName: "test-agent",
+      error: null,
+    });
+    mockFetch.mockRejectedValueOnce("summary-string-error");
+
+    const api = { registerCommand: jest.fn() };
+    registerCommand(api, config, mockLogger);
+
+    const cmd = api.registerCommand.mock.calls[0][0];
+    await cmd.execute();
+
+    expect(mockLogger.debug).toHaveBeenCalledWith(
+      "[manifest] Routing summary failed (summary-string-error)",
+    );
+  });
+
   it("includes error in status text when verify reports one", async () => {
     mockVerify.mockResolvedValueOnce({
       endpointReachable: false,