From af66e3103ae795dcecf2e162e4e9909539ab4ba6 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Sat, 21 Feb 2026 18:39:45 +0000 Subject: [PATCH] test(agents): cover bundled skills env override and dedupe setup --- src/agents/skills/bundled-dir.e2e.test.ts | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/src/agents/skills/bundled-dir.e2e.test.ts b/src/agents/skills/bundled-dir.e2e.test.ts index 45fad1bcb..0e500e3aa 100644 --- a/src/agents/skills/bundled-dir.e2e.test.ts +++ b/src/agents/skills/bundled-dir.e2e.test.ts @@ -2,7 +2,8 @@ import fs from "node:fs/promises"; import os from "node:os"; import path from "node:path"; import { pathToFileURL } from "node:url"; -import { afterEach, describe, expect, it } from "vitest"; +import { afterEach, beforeEach, describe, expect, it } from "vitest"; +import { captureEnv } from "../../test-utils/env.js"; import { resolveBundledSkillsDir } from "./bundled-dir.js"; async function writeSkill(dir: string, name: string) { @@ -15,14 +16,20 @@ async function writeSkill(dir: string, name: string) { } describe("resolveBundledSkillsDir", () => { - const originalOverride = process.env.OPENCLAW_BUNDLED_SKILLS_DIR; + let envSnapshot: ReturnType; + + beforeEach(() => { + envSnapshot = captureEnv(["OPENCLAW_BUNDLED_SKILLS_DIR"]); + }); afterEach(() => { - if (originalOverride === undefined) { - delete process.env.OPENCLAW_BUNDLED_SKILLS_DIR; - } else { - process.env.OPENCLAW_BUNDLED_SKILLS_DIR = originalOverride; - } + envSnapshot.restore(); + }); + + it("returns OPENCLAW_BUNDLED_SKILLS_DIR override when set", async () => { + const overrideDir = await fs.mkdtemp(path.join(os.tmpdir(), "openclaw-bundled-override-")); + process.env.OPENCLAW_BUNDLED_SKILLS_DIR = ` ${overrideDir} `; + expect(resolveBundledSkillsDir()).toBe(overrideDir); }); it("resolves bundled skills under a flattened dist layout", async () => {