Skip to content

Commit 9017d85

Browse files
github-actions[bot]stephentoubCopilot
authored
Update @github/copilot to 1.0.55-5 (#1456)
* Update @github/copilot to 1.0.55-5 - Updated nodejs and test harness dependencies - Re-ran code generators - Formatted generated code * Fix flaky E2E CI tests Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Stephen Toub <stoub@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent b40a6f2 commit 9017d85

16 files changed

Lines changed: 580 additions & 481 deletions

File tree

dotnet/src/Generated/Rpc.cs

Lines changed: 70 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dotnet/test/E2E/SessionE2ETests.cs

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -110,16 +110,19 @@ public async Task Should_Create_A_Session_With_Customized_SystemMessage_Config()
110110
}
111111
});
112112

113-
await session.SendAsync(new MessageOptions { Prompt = "Who are you?" });
114-
var assistantMessage = await TestHelper.GetFinalAssistantMessageAsync(session);
115-
Assert.NotNull(assistantMessage);
116-
117-
var traffic = await Ctx.GetExchangesAsync();
118-
Assert.NotEmpty(traffic);
119-
var systemMessage = GetSystemMessage(traffic[0]);
120-
Assert.Contains(customTone, systemMessage);
121-
Assert.Contains(appendedContent, systemMessage);
122-
Assert.DoesNotContain("<code_change_instructions>", systemMessage);
113+
try
114+
{
115+
await session.SendAsync(new MessageOptions { Prompt = "Who are you?" });
116+
var traffic = await WaitForExchangesAsync();
117+
var systemMessage = GetSystemMessage(traffic[0]);
118+
Assert.Contains(customTone, systemMessage);
119+
Assert.Contains(appendedContent, systemMessage);
120+
Assert.DoesNotContain("<code_change_instructions>", systemMessage);
121+
}
122+
finally
123+
{
124+
await session.DisposeAsync();
125+
}
123126
}
124127

125128
[Fact]

go/internal/e2e/session_e2e_test.go

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -209,20 +209,15 @@ func TestSessionE2E(t *testing.T) {
209209
if err != nil {
210210
t.Fatalf("Failed to create session: %v", err)
211211
}
212+
t.Cleanup(func() { _ = session.Disconnect() })
212213

213-
_, err = session.SendAndWait(t.Context(), copilot.MessageOptions{Prompt: "Who are you?"})
214+
_, err = session.Send(t.Context(), copilot.MessageOptions{Prompt: "Who are you?"})
214215
if err != nil {
215216
t.Fatalf("Failed to send message: %v", err)
216217
}
217218

218219
// Validate the system message sent to the model
219-
traffic, err := ctx.GetExchanges()
220-
if err != nil {
221-
t.Fatalf("Failed to get exchanges: %v", err)
222-
}
223-
if len(traffic) == 0 {
224-
t.Fatal("Expected at least one exchange")
225-
}
220+
traffic := ctx.WaitForExchanges(t, 1)
226221
systemMessage := getSystemMessage(traffic[0])
227222
if !strings.Contains(systemMessage, customTone) {
228223
t.Errorf("Expected system message to contain custom tone, got %q", systemMessage)

0 commit comments

Comments
 (0)