Skip to content

Commit 107d545

Browse files
committed
fix race
1 parent d00a614 commit 107d545

1 file changed

Lines changed: 3 additions & 18 deletions

File tree

go/internal/e2e/mcp_and_agents_test.go

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -37,18 +37,13 @@ func TestMCPServers(t *testing.T) {
3737
}
3838

3939
// Simple interaction to verify session works
40-
_, err = session.Send(t.Context(), copilot.MessageOptions{
40+
message, err := session.SendAndWait(t.Context(), copilot.MessageOptions{
4141
Prompt: "What is 2+2?",
4242
})
4343
if err != nil {
4444
t.Fatalf("Failed to send message: %v", err)
4545
}
4646

47-
message, err := testharness.GetFinalAssistantMessage(t.Context(), session)
48-
if err != nil {
49-
t.Fatalf("Failed to get final message: %v", err)
50-
}
51-
5247
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "4") {
5348
t.Errorf("Expected message to contain '4', got: %v", message.Data.Content)
5449
}
@@ -168,18 +163,13 @@ func TestCustomAgents(t *testing.T) {
168163
}
169164

170165
// Simple interaction to verify session works
171-
_, err = session.Send(t.Context(), copilot.MessageOptions{
166+
message, err := session.SendAndWait(t.Context(), copilot.MessageOptions{
172167
Prompt: "What is 5+5?",
173168
})
174169
if err != nil {
175170
t.Fatalf("Failed to send message: %v", err)
176171
}
177172

178-
message, err := testharness.GetFinalAssistantMessage(t.Context(), session)
179-
if err != nil {
180-
t.Fatalf("Failed to get final message: %v", err)
181-
}
182-
183173
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "10") {
184174
t.Errorf("Expected message to contain '10', got: %v", message.Data.Content)
185175
}
@@ -373,18 +363,13 @@ func TestCombinedConfiguration(t *testing.T) {
373363
t.Error("Expected non-empty session ID")
374364
}
375365

376-
_, err = session.Send(t.Context(), copilot.MessageOptions{
366+
message, err := session.SendAndWait(t.Context(), copilot.MessageOptions{
377367
Prompt: "What is 7+7?",
378368
})
379369
if err != nil {
380370
t.Fatalf("Failed to send message: %v", err)
381371
}
382372

383-
message, err := testharness.GetFinalAssistantMessage(t.Context(), session)
384-
if err != nil {
385-
t.Fatalf("Failed to get final message: %v", err)
386-
}
387-
388373
if message.Data.Content == nil || !strings.Contains(*message.Data.Content, "14") {
389374
t.Errorf("Expected message to contain '14', got: %v", message.Data.Content)
390375
}

0 commit comments

Comments
 (0)