Skip to content

Commit 63852b5

Browse files
fix: add approveAll to invokes_built-in_tools test and regenerate snapshot
The test reads a file using built-in tools. Now that requestPermission=true is always sent, it needs a permission handler to avoid denial. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 594196b commit 63852b5

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

go/internal/e2e/tools_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,9 @@ func TestTools(t *testing.T) {
2525
t.Fatalf("Failed to write test file: %v", err)
2626
}
2727

28-
session, err := client.CreateSession(t.Context(), nil)
28+
session, err := client.CreateSession(t.Context(), &copilot.SessionConfig{
29+
OnPermissionRequest: copilot.PermissionHandler.ApproveAll,
30+
})
2931
if err != nil {
3032
t.Fatalf("Failed to create session: %v", err)
3133
}

0 commit comments

Comments
 (0)