From 04004c56636bb979a826d15f81a4841223e6b985 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Mon, 16 Feb 2026 00:27:51 +0000
Subject: [PATCH] perf(test): consolidate models-config provider unit tests

---
 .../models-config.providers.minimax.test.ts   | 23 --------
 .../models-config.providers.nvidia.test.ts    | 52 +++++++++++++++++++
 .../models-config.providers.vllm.test.ts      | 33 ------------
 3 files changed, 52 insertions(+), 56 deletions(-)
 delete mode 100644 src/agents/models-config.providers.minimax.test.ts
 delete mode 100644 src/agents/models-config.providers.vllm.test.ts

diff --git a/src/agents/models-config.providers.minimax.test.ts b/src/agents/models-config.providers.minimax.test.ts
deleted file mode 100644
index 94b7994a6cd..00000000000
--- a/src/agents/models-config.providers.minimax.test.ts
+++ /dev/null
@@ -1,23 +0,0 @@
-import { mkdtempSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, expect, it } from "vitest";
-import { captureEnv } from "../test-utils/env.js";
-import { resolveImplicitProviders } from "./models-config.providers.js";
-
-describe("MiniMax implicit provider (#15275)", () => {
-  it("should use anthropic-messages API for API-key provider", async () => {
-    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
-    const envSnapshot = captureEnv(["MINIMAX_API_KEY"]);
-    process.env.MINIMAX_API_KEY = "test-key";
-
-    try {
-      const providers = await resolveImplicitProviders({ agentDir });
-      expect(providers?.minimax).toBeDefined();
-      expect(providers?.minimax?.api).toBe("anthropic-messages");
-      expect(providers?.minimax?.baseUrl).toBe("https://api.minimax.io/anthropic");
-    } finally {
-      envSnapshot.restore();
-    }
-  });
-});
diff --git a/src/agents/models-config.providers.nvidia.test.ts b/src/agents/models-config.providers.nvidia.test.ts
index a9920a3cba2..3a2f86e9829 100644
--- a/src/agents/models-config.providers.nvidia.test.ts
+++ b/src/agents/models-config.providers.nvidia.test.ts
@@ -56,3 +56,55 @@ describe("NVIDIA provider", () => {
     expect(modelIds).toContain("nvidia/mistral-nemo-minitron-8b-8k-instruct");
   });
 });
+
+describe("MiniMax implicit provider (#15275)", () => {
+  it("should use anthropic-messages API for API-key provider", async () => {
+    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
+    const envSnapshot = captureEnv(["MINIMAX_API_KEY"]);
+    process.env.MINIMAX_API_KEY = "test-key";
+
+    try {
+      const providers = await resolveImplicitProviders({ agentDir });
+      expect(providers?.minimax).toBeDefined();
+      expect(providers?.minimax?.api).toBe("anthropic-messages");
+      expect(providers?.minimax?.baseUrl).toBe("https://api.minimax.io/anthropic");
+    } finally {
+      envSnapshot.restore();
+    }
+  });
+});
+
+describe("vLLM provider", () => {
+  it("should not include vllm when no API key is configured", async () => {
+    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
+    const envSnapshot = captureEnv(["VLLM_API_KEY"]);
+    delete process.env.VLLM_API_KEY;
+
+    try {
+      const providers = await resolveImplicitProviders({ agentDir });
+      expect(providers?.vllm).toBeUndefined();
+    } finally {
+      envSnapshot.restore();
+    }
+  });
+
+  it("should include vllm when VLLM_API_KEY is set", async () => {
+    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
+    const envSnapshot = captureEnv(["VLLM_API_KEY"]);
+    process.env.VLLM_API_KEY = "test-key";
+
+    try {
+      const providers = await resolveImplicitProviders({ agentDir });
+
+      expect(providers?.vllm).toBeDefined();
+      expect(providers?.vllm?.apiKey).toBe("VLLM_API_KEY");
+      expect(providers?.vllm?.baseUrl).toBe("http://127.0.0.1:8000/v1");
+      expect(providers?.vllm?.api).toBe("openai-completions");
+
+      // Note: discovery is disabled in test environments (VITEST check)
+      expect(providers?.vllm?.models).toEqual([]);
+    } finally {
+      envSnapshot.restore();
+    }
+  });
+});
diff --git a/src/agents/models-config.providers.vllm.test.ts b/src/agents/models-config.providers.vllm.test.ts
deleted file mode 100644
index 441b4155ec7..00000000000
--- a/src/agents/models-config.providers.vllm.test.ts
+++ /dev/null
@@ -1,33 +0,0 @@
-import { mkdtempSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, expect, it } from "vitest";
-import { resolveImplicitProviders } from "./models-config.providers.js";
-
-describe("vLLM provider", () => {
-  it("should not include vllm when no API key is configured", async () => {
-    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
-    const providers = await resolveImplicitProviders({ agentDir });
-
-    expect(providers?.vllm).toBeUndefined();
-  });
-
-  it("should include vllm when VLLM_API_KEY is set", async () => {
-    const agentDir = mkdtempSync(join(tmpdir(), "openclaw-test-"));
-    process.env.VLLM_API_KEY = "test-key";
-
-    try {
-      const providers = await resolveImplicitProviders({ agentDir });
-
-      expect(providers?.vllm).toBeDefined();
-      expect(providers?.vllm?.apiKey).toBe("VLLM_API_KEY");
-      expect(providers?.vllm?.baseUrl).toBe("http://127.0.0.1:8000/v1");
-      expect(providers?.vllm?.api).toBe("openai-completions");
-
-      // Note: discovery is disabled in test environments (VITEST check)
-      expect(providers?.vllm?.models).toEqual([]);
-    } finally {
-      delete process.env.VLLM_API_KEY;
-    }
-  });
-});