vrr/openclaw
2
0
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-10-03 01:29:56 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 29

test: route provider integration and wizard recovery tests correctly (#148504)

Browse source 
* test(plugins): keep registered provider auth coverage in integration tests

Move the existing Radius registration and device-code cancellation flow to the
root integration suite so core tests no longer pull the bundled provider into
their type graph. Preserve generic auth adapter coverage and all flow assertions.

Co-authored-by: Ayaan Zaidi <hi@obviy.us>

* test: restore database worker routing for wizard recovery

---------

Co-authored-by: Ayaan Zaidi <hi@obviy.us>
This commit is contained in:
Peter Steinberger 2026-09-14 13:39:07 -07:00 • committed by GitHub
parent cbc5d7e03c
commit 867e655d51
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 63 additions and 51 deletions
Download patch file Download diff file Expand all files Collapse all files

52
src/plugins/provider-auth-method.test.ts
View file

@ -1,18 +1,13 @@
import { afterEach, describe, expect, it, vi } from "vitest";
import radiusPlugin from "../../extensions/radius/index.js";
import { createWizardPrompter } from "../../test/helpers/wizard-prompter.js";
import { createNonExitingRuntime } from "../runtime.js";
import { registerSingleProviderPlugin } from "../test-utils/plugin-registration.js";
import { WizardSession } from "../wizard/session.js";
import { runProviderPluginAuthMethodUnpersisted } from "./provider-auth-method.js";
import type { ProviderAuthMethod } from "./provider-authentication.types.js";
const { openHostBrowser, guardedFetch } = vi.hoisted(() => ({
const { openHostBrowser } = vi.hoisted(() => ({
openHostBrowser: vi.fn(async () => true),
guardedFetch: vi.fn(),
}));
vi.mock("../infra/browser-open.js", () => ({ openUrl: openHostBrowser }));
vi.mock("openclaw/plugin-sdk/ssrf-runtime", () => ({ fetchWithSsrFGuard: guardedFetch }));
afterEach(() => vi.clearAllMocks());
@ -84,49 +79,4 @@ describe("runProviderPluginAuthMethodUnpersisted", () => {
expect(presentUrl).not.toHaveBeenCalled();
expect(openHostBrowser).not.toHaveBeenCalled();
});
it.each([false, true])(
"keeps the registered Radius device destination with its code and cancellation (remote=%s)",
async (isRemote) => {
guardedFetch.mockResolvedValueOnce({
response: Response.json({
device_code: "synthetic-device-secret",
user_code: "ABCD-EFGH",
verification_uri: "https://radius.earendil.com/device",
expires_in: 300,
interval: 5,
}),
release: async () => undefined,
});
const provider = await registerSingleProviderPlugin(radiusPlugin);
const method = provider.auth.find((entry) => entry.id === "oauth");
if (!method) {
throw new Error("Radius did not register its OAuth method");
}
const session = new WizardSession(async (prompter, signal) => {
await runProviderPluginAuthMethodUnpersisted({
...options,
method,
prompter,
signal,
isRemote,
});
});
try {
const pending = await session.next();
expect(pending.step).toMatchObject({
type: "progress",
externalUrl: "https://radius.earendil.com/device",
deviceCode: { code: "ABCD-EFGH", expiresInMinutes: 5 },
});
expect(openHostBrowser).not.toHaveBeenCalled();
session.cancel();
expect(await session.next()).toMatchObject({ done: true, status: "cancelled" });
} finally {
session.cancel();
await session.whenSettled();
}
expect(guardedFetch).toHaveBeenCalledOnce();
},
);
});

61
test/provider-auth-method.radius.integration.test.ts Normal file
View file

@ -0,0 +1,61 @@
import { afterEach, expect, it, vi } from "vitest";
import radiusPlugin from "../extensions/radius/index.js";
import { runProviderPluginAuthMethodUnpersisted } from "../src/plugins/provider-auth-method.js";
import { createNonExitingRuntime } from "../src/runtime.js";
import { registerSingleProviderPlugin } from "../src/test-utils/plugin-registration.js";
import { WizardSession } from "../src/wizard/session.js";
const { openHostBrowser, guardedFetch } = vi.hoisted(() => ({
openHostBrowser: vi.fn(async () => true),
guardedFetch: vi.fn(),
}));
vi.mock("../src/infra/browser-open.js", () => ({ openUrl: openHostBrowser }));
vi.mock("openclaw/plugin-sdk/ssrf-runtime", () => ({ fetchWithSsrFGuard: guardedFetch }));
afterEach(() => vi.clearAllMocks());
it.each([false, true])(
"keeps the registered Radius device destination with its code and cancellation (remote=%s)",
async (isRemote) => {
guardedFetch.mockResolvedValueOnce({
response: Response.json({
device_code: "synthetic-device-secret",
user_code: "ABCD-EFGH",
verification_uri: "https://radius.earendil.com/device",
expires_in: 300,
interval: 5,
}),
release: async () => undefined,
});
const provider = await registerSingleProviderPlugin(radiusPlugin);
const method = provider.auth.find((entry) => entry.id === "oauth");
if (!method) {
throw new Error("Radius did not register its OAuth method");
}
const session = new WizardSession(async (prompter, signal) => {
await runProviderPluginAuthMethodUnpersisted({
config: {},
runtime: createNonExitingRuntime(),
method,
prompter,
signal,
isRemote,
});
});
try {
const pending = await session.next();
expect(pending.step).toMatchObject({
type: "progress",
externalUrl: "https://radius.earendil.com/device",
deviceCode: { code: "ABCD-EFGH", expiresInMinutes: 5 },
});
expect(openHostBrowser).not.toHaveBeenCalled();
session.cancel();
expect(await session.next()).toMatchObject({ done: true, status: "cancelled" });
} finally {
session.cancel();
await session.whenSettled();
}
expect(guardedFetch).toHaveBeenCalledOnce();
},
);

1
test/vitest/vitest.database-worker-core-paths.mjs
View file

@ -7,6 +7,7 @@ export const databaseWorkerCoreTestFiles = [
"src/system-agent/setup-inference.groq-external.integration.test.ts",
"src/system-agent/setup-inference-activate.test.ts",
"src/system-agent/setup-inference-detect.lifecycle.test.ts",
"src/wizard/setup.inference-recovery.integration.test.ts",
"src/wizard/setup.test.ts",
"src/wizard/setup.provenance.integration.test.ts",
"src/flows/search-setup.test.ts",