Skip to content

Commit 927f6b6

Browse files
committed
Add TODO to handle permission handler errors gracefully in PermissionsTest
1 parent 98a6927 commit 927f6b6

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

src/test/java/com/github/copilot/sdk/PermissionsTest.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,8 @@ void testToolCallIdInPermissionRequests(TestInfo testInfo) throws Exception {
233233
void testShouldHandlePermissionHandlerErrorsGracefully(TestInfo testInfo) throws Exception {
234234
ctx.configureForTest("permissions", "should_handle_permission_handler_errors_gracefully");
235235

236+
// TODO: this test throws this exception and the stacktrace is being logged. We
237+
// need to prevent that.
236238
SessionConfig config = new SessionConfig().setOnPermissionRequest((request, invocation) -> {
237239
// Throw an error in the handler
238240
throw new RuntimeException("Handler error");

0 commit comments

Comments
 (0)