惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

G
Google Developers Blog
Apple Machine Learning Research
Apple Machine Learning Research
小众软件
小众软件
Recent Announcements
Recent Announcements
阮一峰的网络日志
阮一峰的网络日志
IT之家
IT之家
A
About on SuperTechFans
量子位
Engineering at Meta
Engineering at Meta
B
Blog
The Cloudflare Blog
博客园 - 【当耐特】
Hugging Face - Blog
Hugging Face - Blog
Y
Y Combinator Blog
J
Java Code Geeks
D
DataBreaches.Net
aimingoo的专栏
aimingoo的专栏
T
Tailwind CSS Blog
H
Help Net Security
钛媒体:引领未来商业与生活新知
钛媒体:引领未来商业与生活新知
V
V2EX
Stack Overflow Blog
Stack Overflow Blog
C
Check Point Blog
酷 壳 – CoolShell
酷 壳 – CoolShell

Recent Commits to openclaw:main

test: merge chat side-result checks · openclaw/openclaw@ddd2c2a test: merge cron history checks · openclaw/openclaw@f7eb746 test: merge responsive navigation shell checks · openclaw/openclaw@c2e4b47 docs(changelog): add codex oauth fixes · openclaw/openclaw@628e6cd test: merge navigation routing cases · openclaw/openclaw@5d8cecb Tests: mock channel registry bundled fallback · openclaw/openclaw@2b08233 Secrets: avoid broad web search discovery for single plugin config · openclaw/openclaw@a464f59 test: merge config view browser checks · openclaw/openclaw@20cf511 fix(status): align oauth health with runtime · openclaw/openclaw@eed7116 feat: add macOS screen snapshots for monitor preview (#67954) thanks … · openclaw/openclaw@f377db1 fix: report shared auth scopes in hello-ok (#67810) thanks @BunsDev · openclaw/openclaw@0b6c39b Auto-reply: avoid eager bundled route fallback · openclaw/openclaw@3ea1bf4 Tests: narrow session binding contract setup · openclaw/openclaw@54e4e16 fix(macOS): enable undo/redo in webchat composer text input (#34962) · openclaw/openclaw@00951dc Tests: speed up channel setup promotion · openclaw/openclaw@82b529a Docs: refresh agent instructions · openclaw/openclaw@5775fe2 fix(auth): serialize OAuth refresh across agents to fix #26322 (#67876) · openclaw/openclaw@8e79080 test: allow ollama public surface boundary test · openclaw/openclaw@7d4f1a6 Docs: add test performance guardrails · openclaw/openclaw@89706d3 Tests: restore context-engine usage proof · openclaw/openclaw@e4c4f95 Tests: slim context engine runtime coverage · openclaw/openclaw@74c198f ci: retry failed custom checkouts · openclaw/openclaw@0ee5baf test: trim duplicate provider auth onboarding cases · openclaw/openclaw@1ffc02e matrix: fix sessions_spawn --thread subagent session spawning (#67643) · openclaw/openclaw@1ce2596 test: reduce auth choice fixture churn · openclaw/openclaw@857b9cd test: mock health status config boundaries · openclaw/openclaw@9d5ab4a test: mock onboard config io boundary · openclaw/openclaw@299694d test: mock legacy state plugin boundaries · openclaw/openclaw@2713089 test: mock channel install boundaries · openclaw/openclaw@b945248 test: mock doctor preview channel boundaries · openclaw/openclaw@b1a3ad4
refactor(agents): remove catalog browse timer test hooks ...
vincentkoc · 2026-06-17 · via Recent Commits to openclaw:main

@@ -6,13 +6,11 @@ import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";

66

import type { OpenClawConfig } from "../config/types.openclaw.js";

77

import { MAX_TIMER_TIMEOUT_MS } from "../shared/number-coercion.js";

88

import {

9-

DEFAULT_MODEL_CATALOG_BROWSE_TIMEOUT_MS,

109

loadModelCatalogForBrowse,

11-

restoreModelCatalogBrowseTestDeps,

12-

setModelCatalogBrowseTestDeps,

1310

} from "./model-catalog-browse.js";

1411

import type { ModelCatalogEntry } from "./model-catalog.types.js";

151213+

const DEFAULT_MODEL_CATALOG_BROWSE_TIMEOUT_MS = 750;

1614

const readOnlyCatalog: ModelCatalogEntry[] = [

1715

{ id: "gpt-readonly", name: "GPT Readonly", provider: "openai" },

1816

];

@@ -41,7 +39,6 @@ describe("loadModelCatalogForBrowse", () => {

4139

vi.clearAllTimers();

4240

vi.restoreAllMocks();

4341

vi.useRealTimers();

44-

restoreModelCatalogBrowseTestDeps();

4542

});

46434744

it("uses the read-only catalog for default browse views", async () => {

@@ -97,16 +94,8 @@ describe("loadModelCatalogForBrowse", () => {

9794

});

98959996

it("returns an empty catalog when read-only catalog loading times out with provider wildcards", async () => {

97+

vi.useFakeTimers();

10098

const onTimeout = vi.fn();

101-

const timeoutHandle = { unref: vi.fn() } as unknown as NodeJS.Timeout;

102-

const clearTimeout = vi.fn();

103-

setModelCatalogBrowseTestDeps({

104-

setTimeout: vi.fn((callback: () => void) => {

105-

queueMicrotask(callback);

106-

return timeoutHandle;

107-

}) as unknown as typeof globalThis.setTimeout,

108-

clearTimeout: clearTimeout as unknown as typeof globalThis.clearTimeout,

109-

});

11099

const loadCatalog = vi.fn(() => new Promise<ModelCatalogEntry[]>(() => {}));

111100112101

const resultPromise = loadModelCatalogForBrowse({

@@ -116,21 +105,15 @@ describe("loadModelCatalogForBrowse", () => {

116105

onTimeout,

117106

});

118107108+

await vi.advanceTimersByTimeAsync(5);

119109

await expect(resultPromise).resolves.toEqual([]);

120110

expect(onTimeout).toHaveBeenCalledExactlyOnceWith(5);

121-

expect(timeoutHandle["unref"]).toHaveBeenCalledOnce();

122-

expect(clearTimeout).toHaveBeenCalledExactlyOnceWith(timeoutHandle);

123111

});

124112125113

it("uses the default timeout when timeoutMs is non-finite", async () => {

126114

const onTimeout = vi.fn();

127-

const timeoutHandle = { unref: vi.fn() } as unknown as NodeJS.Timeout;

128-

const setTimeout = vi.fn(() => timeoutHandle);

129-

const clearTimeout = vi.fn();

130-

setModelCatalogBrowseTestDeps({

131-

setTimeout: setTimeout as unknown as typeof globalThis.setTimeout,

132-

clearTimeout: clearTimeout as unknown as typeof globalThis.clearTimeout,

133-

});

115+

const setTimeout = vi.spyOn(globalThis, "setTimeout");

116+

const clearTimeout = vi.spyOn(globalThis, "clearTimeout");

134117

const loadCatalog = vi.fn(async () => readOnlyCatalog);

135118136119

const resultPromise = loadModelCatalogForBrowse({

@@ -145,18 +128,13 @@ describe("loadModelCatalogForBrowse", () => {

145128

expect.any(Function),

146129

DEFAULT_MODEL_CATALOG_BROWSE_TIMEOUT_MS,

147130

);

148-

expect(clearTimeout).toHaveBeenCalledExactlyOnceWith(timeoutHandle);

131+

expect(clearTimeout).toHaveBeenCalledOnce();

149132

expect(onTimeout).not.toHaveBeenCalled();

150133

});

151134152135

it("caps oversized browse timeouts before scheduling the fallback timer", async () => {

153-

const timeoutHandle = { unref: vi.fn() } as unknown as NodeJS.Timeout;

154-

const setTimeout = vi.fn(() => timeoutHandle);

155-

const clearTimeout = vi.fn();

156-

setModelCatalogBrowseTestDeps({

157-

setTimeout: setTimeout as unknown as typeof globalThis.setTimeout,

158-

clearTimeout: clearTimeout as unknown as typeof globalThis.clearTimeout,

159-

});

136+

const setTimeout = vi.spyOn(globalThis, "setTimeout");

137+

const clearTimeout = vi.spyOn(globalThis, "clearTimeout");

160138

const loadCatalog = vi.fn(async () => readOnlyCatalog);

161139162140

const resultPromise = loadModelCatalogForBrowse({

@@ -167,6 +145,6 @@ describe("loadModelCatalogForBrowse", () => {

167145168146

await expect(resultPromise).resolves.toBe(readOnlyCatalog);

169147

expect(setTimeout).toHaveBeenCalledExactlyOnceWith(expect.any(Function), MAX_TIMER_TIMEOUT_MS);

170-

expect(clearTimeout).toHaveBeenCalledExactlyOnceWith(timeoutHandle);

148+

expect(clearTimeout).toHaveBeenCalledOnce();

171149

});

172150

});