Skip to content

Commit 9cbe46e

Browse files
Dongni-Yangclaude
andcommitted
test(inference): count the classifier daemon probe in the context-pin case
The context-pinning case asserts every docker diagnostic command carries the persisted-context prefix and pins the command count. The image-pull classifier adds one context-pinned `docker version` probe on the all-empty failure path, so the count is now six. The pinning contract itself is unchanged and the new probe satisfies it. Refs #9308 Signed-off-by: Dongni Yang <dongniy@nvidia.com> Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent d217363 commit 9cbe46e

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

src/lib/inference/local-vllm-auth.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -373,7 +373,9 @@ describe("managed vLLM authentication", () => {
373373
.filter((argv) => argv[0] === "docker");
374374

375375
expect(result.ok).toBe(false);
376-
expect(dockerCommands).toHaveLength(5);
376+
// 3 reachability probes, 2 diagnostic re-probes, and the daemon probe of
377+
// the image-pull classifier (#9308) — every one context-pinned.
378+
expect(dockerCommands).toHaveLength(6);
377379
expect(
378380
dockerCommands.every((argv) => argv.slice(0, 3).join(" ") === "docker --context default"),
379381
).toBe(true);

0 commit comments

Comments
 (0)