vultr/openclaw
1
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-04-15 03:01:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(test): isolate provider auth env marker mocks

Browse Source
This commit is contained in:
Peter Steinberger
2026-04-07 16:44:41 +01:00
parent d366b13ec9
commit 8cee6f96e6
3 changed files with 35 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/agents/model-auth-markers.test.ts
View File

@@ -2,12 +2,14 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
async function loadMarkerModules() {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../secrets/provider-env-vars.js");
vi.resetModules();
return Promise.all([import("./model-auth-env-vars.js"), import("./model-auth-markers.js")]);
}
beforeEach(() => {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../secrets/provider-env-vars.js");
});
describe("model auth markers", () => {

39
src/agents/models-config.providers.auth-provenance.test.ts
View File

@@ -1,10 +1,22 @@
import { describe, expect, it } from "vitest";
import { beforeEach, describe, expect, it, vi } from "vitest";
import { captureEnv } from "../test-utils/env.js";
import { MINIMAX_OAUTH_MARKER, NON_ENV_SECRETREF_MARKER } from "./model-auth-markers.js";
import {
createProviderAuthResolver,
resolveApiKeyFromCredential,
} from "./models-config.providers.secrets.js";
async function loadProviderAuthModules() {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../plugins/provider-runtime.js");
vi.doUnmock("../secrets/provider-env-vars.js");
vi.resetModules();
return Promise.all([
import("./model-auth-markers.js"),
import("./models-config.providers.secrets.js"),
]);
}
beforeEach(() => {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../plugins/provider-runtime.js");
vi.doUnmock("../secrets/provider-env-vars.js");
});
function buildPairedApiKeyProviders(apiKey: string) {
return {
@@ -14,7 +26,8 @@ function buildPairedApiKeyProviders(apiKey: string) {
}
describe("models-config provider auth provenance", () => {
it("persists env keyRef and tokenRef auth profiles as env var markers", () => {
it("persists env keyRef and tokenRef auth profiles as env var markers", async () => {
const [, { resolveApiKeyFromCredential }] = await loadProviderAuthModules();
const envSnapshot = captureEnv(["VOLCANO_ENGINE_API_KEY", "TOGETHER_API_KEY"]);
delete process.env.VOLCANO_ENGINE_API_KEY;
delete process.env.TOGETHER_API_KEY;
@@ -39,7 +52,9 @@ describe("models-config provider auth provenance", () => {
}
});
it("uses non-env marker for ref-managed profiles even when runtime plaintext is present", () => {
it("uses non-env marker for ref-managed profiles even when runtime plaintext is present", async () => {
const [{ NON_ENV_SECRETREF_MARKER }, { resolveApiKeyFromCredential }] =
await loadProviderAuthModules();
const byteplusApiKey = resolveApiKeyFromCredential({
type: "api_key",
provider: "byteplus",
@@ -59,7 +74,8 @@ describe("models-config provider auth provenance", () => {
expect(togetherApiKey).toBe(NON_ENV_SECRETREF_MARKER);
});
it("keeps oauth compatibility markers for minimax-portal", () => {
it("keeps oauth compatibility markers for minimax-portal", async () => {
const [{ MINIMAX_OAUTH_MARKER }] = await loadProviderAuthModules();
const providers = {
"minimax-portal": {
apiKey: MINIMAX_OAUTH_MARKER,
@@ -69,6 +85,7 @@ describe("models-config provider auth provenance", () => {
});
it("prefers profile auth over env auth in provider summaries to match runtime resolution", async () => {
const [, { createProviderAuthResolver }] = await loadProviderAuthModules();
const auth = createProviderAuthResolver(
{
OPENAI_API_KEY: "env-openai-key",
@@ -94,7 +111,9 @@ describe("models-config provider auth provenance", () => {
});
});
it("resolves plugin-owned synthetic auth through the provider hook", () => {
it("resolves plugin-owned synthetic auth through the provider hook", async () => {
const [{ NON_ENV_SECRETREF_MARKER }, { createProviderAuthResolver }] =
await loadProviderAuthModules();
const auth = createProviderAuthResolver(
{} as NodeJS.ProcessEnv,
{

4
src/agents/models-config.providers.vercel-ai-gateway.test.ts
View File

@@ -2,6 +2,8 @@ import { beforeEach, describe, expect, it, vi } from "vitest";
async function loadModules() {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../plugins/provider-runtime.js");
vi.doUnmock("../secrets/provider-env-vars.js");
vi.resetModules();
return Promise.all([
import("./model-auth-markers.js"),
@@ -11,6 +13,8 @@ async function loadModules() {
beforeEach(() => {
vi.doUnmock("../plugins/manifest-registry.js");
vi.doUnmock("../plugins/provider-runtime.js");
vi.doUnmock("../secrets/provider-env-vars.js");
});
describe("vercel-ai-gateway provider resolution", () => {