Skip to content

Commit 73d2f97

Browse files
committed
Fix tests: Update O3 family model tests to expect temperature: undefined
- Updated failing tests in openai.spec.ts to expect temperature: undefined for O3 models - This aligns with the PR changes that remove temperature parameter for Azure OpenAI o1, o3, and o4 models - All 4 previously failing tests now pass
1 parent d5546e5 commit 73d2f97

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/api/providers/__tests__/openai.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,7 @@ describe("OpenAiHandler", () => {
599599
stream: true,
600600
stream_options: { include_usage: true },
601601
reasoning_effort: "medium",
602-
temperature: 0.5,
602+
temperature: undefined,
603603
// O3 models do not support deprecated max_tokens but do support max_completion_tokens
604604
max_completion_tokens: 32000,
605605
}),
@@ -640,7 +640,7 @@ describe("OpenAiHandler", () => {
640640
stream: true,
641641
stream_options: { include_usage: true },
642642
reasoning_effort: "medium",
643-
temperature: 0.7,
643+
temperature: undefined,
644644
}),
645645
{},
646646
)
@@ -682,7 +682,7 @@ describe("OpenAiHandler", () => {
682682
{ role: "user", content: "Hello!" },
683683
],
684684
reasoning_effort: "medium",
685-
temperature: 0.3,
685+
temperature: undefined,
686686
// O3 models do not support deprecated max_tokens but do support max_completion_tokens
687687
max_completion_tokens: 65536, // Using default maxTokens from o3Options
688688
}),
@@ -712,7 +712,7 @@ describe("OpenAiHandler", () => {
712712

713713
expect(mockCreate).toHaveBeenCalledWith(
714714
expect.objectContaining({
715-
temperature: 0, // Default temperature
715+
temperature: undefined, // Temperature is not supported for O3 models
716716
}),
717717
{},
718718
)

0 commit comments

Comments
 (0)