Skip to content

Commit 729182e

Browse files
committed
Fix CAPI reasoning E2E fixtures
Use gpt-5.4 for tests that explicitly configure high reasoning effort, and advertise that model in the streaming replay fixture. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 9b80f6cb-b851-4598-8294-f4431c6f3143
1 parent 316fe5b commit 729182e

3 files changed

Lines changed: 4 additions & 3 deletions

File tree

dotnet/test/E2E/SessionE2ETests.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -615,10 +615,10 @@ public async Task Should_Set_Model_With_ReasoningEffort()
615615

616616
var modelChangedTask = TestHelper.GetNextEventOfTypeAsync<SessionModelChangeEvent>(session);
617617

618-
await session.SetModelAsync("gpt-4.1", "high");
618+
await session.SetModelAsync("gpt-5.4", "high");
619619

620620
var modelChanged = await modelChangedTask;
621-
Assert.Equal("gpt-4.1", modelChanged.Data.NewModel);
621+
Assert.Equal("gpt-5.4", modelChanged.Data.NewModel);
622622
Assert.Equal("high", modelChanged.Data.ReasoningEffort);
623623
}
624624

dotnet/test/E2E/StreamingFidelityE2ETests.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,7 @@ public async Task Should_Emit_Streaming_Deltas_With_Reasoning_Effort_Configured(
152152
// the streaming pipeline — deltas still arrive and complete successfully.
153153
var session = await CreateSessionAsync(new SessionConfig
154154
{
155+
Model = "gpt-5.4",
155156
Streaming = true,
156157
ReasoningEffort = "high",
157158
});

test/snapshots/streaming_fidelity/should_emit_streaming_deltas_with_reasoning_effort_configured.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
models:
2-
- claude-sonnet-4.5
2+
- gpt-5.4
33
conversations:
44
- messages:
55
- role: system

0 commit comments

Comments
 (0)