forked from github/copilot-sdk-java
-
Notifications
You must be signed in to change notification settings - Fork 0
Expand file tree
/
Copy pathCopilotSessionTest.java
More file actions
571 lines (439 loc) · 24 KB
/
CopilotSessionTest.java
File metadata and controls
571 lines (439 loc) · 24 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
/*---------------------------------------------------------------------------------------------
* Copyright (c) Microsoft Corporation. All rights reserved.
*--------------------------------------------------------------------------------------------*/
package com.github.copilot.sdk;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.fail;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.CompletableFuture;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicReference;
import org.junit.jupiter.api.AfterAll;
import org.junit.jupiter.api.BeforeAll;
import org.junit.jupiter.api.Test;
import com.github.copilot.sdk.events.AbstractSessionEvent;
import com.github.copilot.sdk.events.AbortEvent;
import com.github.copilot.sdk.events.AssistantMessageDeltaEvent;
import com.github.copilot.sdk.events.AssistantMessageEvent;
import com.github.copilot.sdk.events.SessionIdleEvent;
import com.github.copilot.sdk.events.SessionStartEvent;
import com.github.copilot.sdk.events.ToolExecutionStartEvent;
import com.github.copilot.sdk.events.UserMessageEvent;
import com.github.copilot.sdk.json.MessageOptions;
import com.github.copilot.sdk.json.SessionConfig;
import com.github.copilot.sdk.json.SystemMessageConfig;
/**
* Tests for CopilotSession.
*
* <p>
* These tests use the shared CapiProxy infrastructure for deterministic API
* response replay. Snapshots are stored in test/snapshots/session/.
* </p>
*/
public class CopilotSessionTest {
private static E2ETestContext ctx;
@BeforeAll
static void setup() throws Exception {
ctx = E2ETestContext.create();
}
@AfterAll
static void teardown() throws Exception {
if (ctx != null) {
ctx.close();
}
}
@Test
void testCreateAndDestroySession() throws Exception {
ctx.configureForTest("session", "should_receive_session_events");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession(new SessionConfig().setModel("fake-test-model")).get();
assertNotNull(session.getSessionId());
assertTrue(session.getSessionId().matches("^[a-f0-9-]+$"));
List<AbstractSessionEvent> messages = session.getMessages().get();
assertFalse(messages.isEmpty());
assertTrue(messages.get(0) instanceof SessionStartEvent);
session.close();
// Session should no longer be accessible
try {
session.getMessages().get();
fail("Expected exception for closed session");
} catch (Exception e) {
assertTrue(e.getMessage().toLowerCase().contains("not found")
|| e.getCause().getMessage().toLowerCase().contains("not found"));
}
}
}
@Test
void testStatefulConversation() throws Exception {
ctx.configureForTest("session", "should_have_stateful_conversation");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
AssistantMessageEvent response1 = session.sendAndWait(new MessageOptions().setPrompt("What is 1+1?"), 60000)
.get(90, TimeUnit.SECONDS);
assertNotNull(response1);
assertTrue(response1.getData().getContent().contains("2"),
"Response should contain 2: " + response1.getData().getContent());
AssistantMessageEvent response2 = session
.sendAndWait(new MessageOptions().setPrompt("Now if you double that, what do you get?"), 60000)
.get(90, TimeUnit.SECONDS);
assertNotNull(response2);
assertTrue(response2.getData().getContent().contains("4"),
"Response should contain 4: " + response2.getData().getContent());
session.close();
}
}
@Test
void testReceiveSessionEvents() throws Exception {
ctx.configureForTest("session", "should_receive_session_events");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
List<AbstractSessionEvent> receivedEvents = new ArrayList<>();
CompletableFuture<Void> idleReceived = new CompletableFuture<>();
session.on(evt -> {
receivedEvents.add(evt);
if (evt instanceof SessionIdleEvent) {
idleReceived.complete(null);
}
});
session.send(new MessageOptions().setPrompt("What is 100+200?")).get();
idleReceived.get(60, TimeUnit.SECONDS);
assertFalse(receivedEvents.isEmpty());
assertTrue(receivedEvents.stream().anyMatch(e -> e instanceof UserMessageEvent));
assertTrue(receivedEvents.stream().anyMatch(e -> e instanceof AssistantMessageEvent));
assertTrue(receivedEvents.stream().anyMatch(e -> e instanceof SessionIdleEvent));
// Find the assistant message
AssistantMessageEvent assistantMsg = receivedEvents.stream().filter(e -> e instanceof AssistantMessageEvent)
.map(e -> (AssistantMessageEvent) e).findFirst().orElse(null);
assertNotNull(assistantMsg);
assertTrue(assistantMsg.getData().getContent().contains("300"),
"Response should contain 300: " + assistantMsg.getData().getContent());
session.close();
}
}
@Test
void testSendReturnsImmediately() throws Exception {
ctx.configureForTest("session", "send_returns_immediately_while_events_stream_in_background");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
List<String> events = new ArrayList<>();
AtomicReference<AssistantMessageEvent> lastMessage = new AtomicReference<>();
CompletableFuture<Void> done = new CompletableFuture<>();
session.on(evt -> {
events.add(evt.getType());
if (evt instanceof AssistantMessageEvent msg) {
lastMessage.set(msg);
} else if (evt instanceof SessionIdleEvent) {
done.complete(null);
}
});
// Use a slow command so we can verify send() returns before completion
session.send(new MessageOptions().setPrompt("Run 'sleep 2 && echo done'")).get();
// At this point, we might not have received session.idle yet
// The event handling happens asynchronously
// Wait for completion
done.get(60, TimeUnit.SECONDS);
assertTrue(events.contains("session.idle"));
assertTrue(events.contains("assistant.message"));
assertNotNull(lastMessage.get());
assertTrue(lastMessage.get().getData().getContent().contains("done"),
"Response should contain done: " + lastMessage.get().getData().getContent());
session.close();
}
}
@Test
void testSendAndWaitBlocksUntilIdle() throws Exception {
ctx.configureForTest("session", "sendandwait_blocks_until_session_idle_and_returns_final_assistant_message");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
List<String> events = new ArrayList<>();
session.on(evt -> events.add(evt.getType()));
AssistantMessageEvent response = session.sendAndWait(new MessageOptions().setPrompt("What is 2+2?")).get(60,
TimeUnit.SECONDS);
assertNotNull(response);
assertEquals("assistant.message", response.getType());
assertTrue(response.getData().getContent().contains("4"),
"Response should contain 4: " + response.getData().getContent());
assertTrue(events.contains("session.idle"));
assertTrue(events.contains("assistant.message"));
session.close();
}
}
@Test
void testResumeSessionWithSameClient() throws Exception {
ctx.configureForTest("session", "should_resume_a_session_using_the_same_client");
try (CopilotClient client = ctx.createClient()) {
// Create initial session
CopilotSession session1 = client.createSession().get();
String sessionId = session1.getSessionId();
AssistantMessageEvent answer = session1.sendAndWait(new MessageOptions().setPrompt("What is 1+1?")).get(60,
TimeUnit.SECONDS);
assertNotNull(answer);
assertTrue(answer.getData().getContent().contains("2"),
"Response should contain 2: " + answer.getData().getContent());
// Resume using the same client
CopilotSession session2 = client.resumeSession(sessionId).get();
assertEquals(sessionId, session2.getSessionId());
// Verify resumed session has the previous messages
List<AbstractSessionEvent> messages = session2.getMessages().get(60, TimeUnit.SECONDS);
boolean hasAssistantMessage = messages.stream().filter(m -> m instanceof AssistantMessageEvent)
.map(m -> (AssistantMessageEvent) m).anyMatch(m -> m.getData().getContent().contains("2"));
assertTrue(hasAssistantMessage, "Should find previous assistant message containing 2");
session2.close();
}
}
@Test
void testResumeSessionWithNewClient() throws Exception {
ctx.configureForTest("session", "should_resume_a_session_using_a_new_client");
// Use a single try-with-resources for the first client to keep it alive
// throughout the test, matching the behavior of other SDK implementations
try (CopilotClient client1 = ctx.createClient()) {
// Create initial session
CopilotSession session1 = client1.createSession().get();
String sessionId = session1.getSessionId();
AssistantMessageEvent answer = session1.sendAndWait(new MessageOptions().setPrompt("What is 1+1?")).get(60,
TimeUnit.SECONDS);
assertNotNull(answer);
assertTrue(answer.getData().getContent().contains("2"),
"Response should contain 2: " + answer.getData().getContent());
// Resume using a new client (keeping client1 alive)
try (CopilotClient client2 = ctx.createClient()) {
CopilotSession session2 = client2.resumeSession(sessionId).get();
assertEquals(sessionId, session2.getSessionId());
// When resuming with a new client, validate messages contain expected types
List<AbstractSessionEvent> messages = session2.getMessages().get(60, TimeUnit.SECONDS);
assertTrue(messages.stream().anyMatch(m -> m instanceof UserMessageEvent),
"Should contain user.message event");
assertTrue(messages.stream().anyMatch(m -> "session.resume".equals(m.getType())),
"Should contain session.resume event");
session2.close();
}
}
}
@Test
void testSessionWithAppendedSystemMessage() throws Exception {
ctx.configureForTest("session", "should_create_a_session_with_appended_systemmessage_config");
try (CopilotClient client = ctx.createClient()) {
String systemMessageSuffix = "End each response with the phrase 'Have a nice day!'";
SessionConfig config = new SessionConfig().setSystemMessage(
new SystemMessageConfig().setContent(systemMessageSuffix).setMode(SystemMessageMode.APPEND));
CopilotSession session = client.createSession(config).get();
assertNotNull(session.getSessionId());
AssistantMessageEvent response = session
.sendAndWait(new MessageOptions().setPrompt("What is your full name?")).get(60, TimeUnit.SECONDS);
assertNotNull(response);
assertTrue(response.getData().getContent().contains("GitHub"),
"Response should contain GitHub: " + response.getData().getContent());
assertTrue(response.getData().getContent().contains("Have a nice day!"),
"Response should end with 'Have a nice day!': " + response.getData().getContent());
session.close();
}
}
@Test
void testSessionWithReplacedSystemMessage() throws Exception {
ctx.configureForTest("session", "should_create_a_session_with_replaced_systemmessage_config");
try (CopilotClient client = ctx.createClient()) {
String testSystemMessage = "You are an assistant called Testy McTestface. Reply succinctly.";
SessionConfig config = new SessionConfig().setSystemMessage(
new SystemMessageConfig().setContent(testSystemMessage).setMode(SystemMessageMode.REPLACE));
CopilotSession session = client.createSession(config).get();
assertNotNull(session.getSessionId());
AssistantMessageEvent response = session
.sendAndWait(new MessageOptions().setPrompt("What is your full name?")).get(60, TimeUnit.SECONDS);
assertNotNull(response);
assertTrue(response.getData().getContent().contains("Testy McTestface"),
"Response should contain 'Testy McTestface': " + response.getData().getContent());
session.close();
}
}
@Test
void testSessionWithStreamingEnabled() throws Exception {
ctx.configureForTest("session", "should_receive_streaming_delta_events_when_streaming_is_enabled");
try (CopilotClient client = ctx.createClient()) {
SessionConfig config = new SessionConfig().setStreaming(true);
CopilotSession session = client.createSession(config).get();
List<AbstractSessionEvent> receivedEvents = new ArrayList<>();
CompletableFuture<Void> idleReceived = new CompletableFuture<>();
session.on(evt -> {
receivedEvents.add(evt);
if (evt instanceof SessionIdleEvent) {
idleReceived.complete(null);
}
});
session.send(new MessageOptions().setPrompt("What is 2+2?")).get();
idleReceived.get(60, TimeUnit.SECONDS);
// Should have received delta events when streaming is enabled
boolean hasDeltaEvents = receivedEvents.stream().anyMatch(e -> e instanceof AssistantMessageDeltaEvent);
assertTrue(hasDeltaEvents, "Should receive streaming delta events when streaming is enabled");
session.close();
}
}
@Test
void testAbortSession() throws Exception {
ctx.configureForTest("session", "should_abort_a_session");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
assertNotNull(session.getSessionId());
// Set up wait for tool execution to start BEFORE sending
CompletableFuture<ToolExecutionStartEvent> toolStartFuture = new CompletableFuture<>();
CompletableFuture<SessionIdleEvent> sessionIdleFuture = new CompletableFuture<>();
session.on(evt -> {
if (evt instanceof ToolExecutionStartEvent toolStart && !toolStartFuture.isDone()) {
toolStartFuture.complete(toolStart);
} else if (evt instanceof SessionIdleEvent idle && !sessionIdleFuture.isDone()) {
sessionIdleFuture.complete(idle);
}
});
// Send a message that will trigger a long-running shell command
session.send(new MessageOptions()
.setPrompt("run the shell command 'sleep 100' (note this works on both bash and PowerShell)"))
.get();
// Wait for the tool to start executing
toolStartFuture.get(60, TimeUnit.SECONDS);
// Abort the session while the tool is running
session.abort();
// Wait for session to become idle after abort
sessionIdleFuture.get(30, TimeUnit.SECONDS);
// The session should still be alive and usable after abort
List<AbstractSessionEvent> messages = session.getMessages().get(60, TimeUnit.SECONDS);
assertFalse(messages.isEmpty());
// Verify an abort event exists in messages
assertTrue(messages.stream().anyMatch(m -> m instanceof AbortEvent), "Expected an abort event in messages");
// We should be able to send another message
AssistantMessageEvent answer = session.sendAndWait(new MessageOptions().setPrompt("What is 2+2?")).get(60,
TimeUnit.SECONDS);
assertNotNull(answer);
assertTrue(answer.getData().getContent().contains("4"),
"Response should contain 4: " + answer.getData().getContent());
session.close();
}
}
@Test
void testSessionWithAvailableTools() throws Exception {
ctx.configureForTest("session", "should_create_a_session_with_availabletools");
try (CopilotClient client = ctx.createClient()) {
SessionConfig config = new SessionConfig().setAvailableTools(List.of("view", "edit"));
CopilotSession session = client.createSession(config).get();
assertNotNull(session.getSessionId());
AssistantMessageEvent response = session.sendAndWait(new MessageOptions().setPrompt("What is 1+1?")).get(60,
TimeUnit.SECONDS);
assertNotNull(response);
session.close();
}
}
@Test
void testSessionWithExcludedTools() throws Exception {
ctx.configureForTest("session", "should_create_a_session_with_excludedtools");
try (CopilotClient client = ctx.createClient()) {
SessionConfig config = new SessionConfig().setExcludedTools(List.of("view"));
CopilotSession session = client.createSession(config).get();
assertNotNull(session.getSessionId());
AssistantMessageEvent response = session.sendAndWait(new MessageOptions().setPrompt("What is 1+1?")).get(60,
TimeUnit.SECONDS);
assertNotNull(response);
assertTrue(response.getData().getContent().contains("2"),
"Response should contain 2: " + response.getData().getContent());
session.close();
}
}
@Test
void testThrowErrorWhenResumingNonExistentSession() throws Exception {
ctx.configureForTest("session", "should_receive_session_events");
try (CopilotClient client = ctx.createClient()) {
try {
client.resumeSession("non-existent-session-id").get(30, TimeUnit.SECONDS);
fail("Expected exception when resuming non-existent session");
} catch (Exception e) {
// Should throw an error
assertTrue(e.getMessage() != null || e.getCause() != null, "Exception should have a message or cause");
}
}
}
@Test
void testCreateSessionWithCustomConfigDir() throws Exception {
ctx.configureForTest("session", "should_create_session_with_custom_config_dir");
try (CopilotClient client = ctx.createClient()) {
String customConfigDir = ctx.getWorkDir().resolve("custom-config").toString();
SessionConfig config = new SessionConfig().setConfigDir(customConfigDir);
CopilotSession session = client.createSession(config).get();
assertNotNull(session.getSessionId());
assertTrue(session.getSessionId().matches("^[a-f0-9-]+$"));
// Session should work normally with custom config dir
AssistantMessageEvent response = session.sendAndWait(new MessageOptions().setPrompt("What is 1+1?")).get(60,
TimeUnit.SECONDS);
assertNotNull(response);
assertTrue(response.getData().getContent().contains("2"),
"Response should contain 2: " + response.getData().getContent());
session.close();
}
}
// This test validates client-side timeout behavior. The snapshot has no
// assistant response because the test expects timeout BEFORE completion.
@Test
void testSendAndWaitThrowsOnTimeout() throws Exception {
ctx.configureForTest("session", "sendandwait_throws_on_timeout");
try (CopilotClient client = ctx.createClient()) {
CopilotSession session = client.createSession().get();
// Use a very short timeout that will definitely expire
try {
// Note: We use a command that takes time so timeout triggers before completion
session.sendAndWait(new MessageOptions().setPrompt("Run 'sleep 2 && echo done'"), 100).get(5,
TimeUnit.SECONDS);
fail("Expected timeout exception");
} catch (Exception e) {
// Should throw a timeout-related error
assertTrue(e.getMessage().toLowerCase().contains("timeout")
|| (e.getCause() != null && e.getCause().getMessage().toLowerCase().contains("timeout")),
"Should throw timeout exception: " + e.getMessage());
}
session.close();
}
}
@Test
void testListSessions() throws Exception {
ctx.configureForTest("session", "should_list_sessions");
try (CopilotClient client = ctx.createClient()) {
// Create two sessions and send one message to each (matches snapshot format)
CopilotSession session1 = client.createSession().get();
session1.sendAndWait(new MessageOptions().setPrompt("Say hello")).get(60, TimeUnit.SECONDS);
CopilotSession session2 = client.createSession().get();
session2.sendAndWait(new MessageOptions().setPrompt("Say goodbye")).get(60, TimeUnit.SECONDS);
// Small delay to ensure session files are written to disk
Thread.sleep(200);
// List all sessions
var sessions = client.listSessions().get(30, TimeUnit.SECONDS);
// Should have at least the sessions we created
assertNotNull(sessions);
assertFalse(sessions.isEmpty(), "Should have at least 1 session");
// Our sessions should be in the list
var sessionIds = sessions.stream().map(s -> s.getSessionId()).toList();
assertTrue(sessionIds.contains(session1.getSessionId()), "Session 1 should be in the list");
assertTrue(sessionIds.contains(session2.getSessionId()), "Session 2 should be in the list");
session1.close();
session2.close();
}
}
@Test
void testDeleteSession() throws Exception {
ctx.configureForTest("session", "should_delete_session");
try (CopilotClient client = ctx.createClient()) {
// Create a session
CopilotSession session = client.createSession().get();
String sessionId = session.getSessionId();
session.sendAndWait(new MessageOptions().setPrompt("Hello")).get(60, TimeUnit.SECONDS);
// Delete the session using the client API
client.deleteSession(sessionId).get(30, TimeUnit.SECONDS);
// Trying to resume the deleted session should fail
try {
client.resumeSession(sessionId).get(30, TimeUnit.SECONDS);
fail("Expected exception when resuming deleted session");
} catch (Exception e) {
// Should throw an error indicating session not found
assertTrue(e.getMessage() != null || e.getCause() != null, "Exception should have a message or cause");
}
}
}
}