monadical/opencode
14
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity

test: fix read test

Browse Source
This commit is contained in:
Aiden Cline
2026-01-12 15:49:31 -06:00
parent 0be37cc2c6
commit 5c4345da4f
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
packages/opencode/test/tool/read.test.ts
View File

@@ -124,7 +124,7 @@ describe("tool.read external_directory permission", () => {
}) })
}) })
describe("tool.read env file blocking", () => { describe("tool.read env file permissions", () => {
const cases: [string, boolean][] = [ const cases: [string, boolean][] = [
[".env", true], [".env", true],
[".env.local", true], [".env.local", true],
@@ -136,7 +136,7 @@ describe("tool.read env file blocking", () => {
] ]
describe.each(["build", "plan"])("agent=%s", (agentName) => { describe.each(["build", "plan"])("agent=%s", (agentName) => {
test.each(cases)("%s blocked=%s", async (filename, blocked) => { test.each(cases)("%s asks=%s", async (filename, shouldAsk) => {
await using tmp = await tmpdir({ await using tmp = await tmpdir({
init: (dir) => Bun.write(path.join(dir, filename), "content"), init: (dir) => Bun.write(path.join(dir, filename), "content"),
}) })
@@ -144,11 +144,15 @@ describe("tool.read env file blocking", () => {
directory: tmp.path, directory: tmp.path,
fn: async () => { fn: async () => {
const agent = await Agent.get(agentName) const agent = await Agent.get(agentName)
let askedForEnv = false
const ctxWithPermissions = { const ctxWithPermissions = {
...ctx, ...ctx,
ask: async (req: Omit<PermissionNext.Request, "id" | "sessionID" | "tool">) => { ask: async (req: Omit<PermissionNext.Request, "id" | "sessionID" | "tool">) => {
for (const pattern of req.patterns) { for (const pattern of req.patterns) {
const rule = PermissionNext.evaluate(req.permission, pattern, agent.permission) const rule = PermissionNext.evaluate(req.permission, pattern, agent.permission)
if (rule.action === "ask" && req.permission === "read") {
askedForEnv = true
}
if (rule.action === "deny") { if (rule.action === "deny") {
throw new PermissionNext.DeniedError(agent.permission) throw new PermissionNext.DeniedError(agent.permission)
} }
@@ -156,12 +160,8 @@ describe("tool.read env file blocking", () => {
}, },
} }
const read = await ReadTool.init() const read = await ReadTool.init()
const promise = read.execute({ filePath: path.join(tmp.path, filename) }, ctxWithPermissions) await read.execute({ filePath: path.join(tmp.path, filename) }, ctxWithPermissions)
if (blocked) { expect(askedForEnv).toBe(shouldAsk)
await expect(promise).rejects.toThrow(PermissionNext.DeniedError)
} else {
expect((await promise).output).toContain("content")
}
}, },
}) })
}) })