Skip to content

Commit

Permalink
Update model names and remove obsolete test
Browse files Browse the repository at this point in the history
Changed model names to 'Gpt4o' and 'Gpt4o_Mini' across various integration tests. Also, removed an obsolete test for the 'Gpt4_32k' model in AzureOpenAiClientTests.cs.
  • Loading branch information
rodion-m committed Jul 27, 2024
1 parent c937b22 commit 9ff5e09
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ public class ChatGptTranslatorServiceTests : IClassFixture<OpenAiClientFixture>
{
private readonly IAiClient _client;

private const string GtpModel = ChatCompletionModels.Gpt4Turbo;
private const string GtpModel = ChatCompletionModels.Gpt4o;

public ChatGptTranslatorServiceTests(OpenAiClientFixture fixture)
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,17 +12,6 @@ public AzureOpenAiClientTests(ITestOutputHelper outputHelper, AzureOpenAiClientF
_outputHelper = outputHelper;
_client = fixture.Client;
}

[Fact]
public async void Get_response_from_GPT4_32k_model_for_one_message_works()
{
string text = "Who are you? In two words.";
#pragma warning disable CS0618 // Type or member is obsolete
string response = await _client.GetChatCompletions(new UserMessage(text), model: ChatCompletionModels.Gpt4_32k);
#pragma warning restore CS0618 // Type or member is obsolete
_outputHelper.WriteLine(response);
response.Should().NotBeNullOrEmpty();
}

[Fact]
public async void Get_long_response_from_gpt4_Turbo_model()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,8 @@ public OpenAiClientGetStructuredResponseTests(OpenAiClientFixture clientFixture)
[InlineData(ChatCompletionModels.Gpt3_5_Turbo)]
[InlineData(ChatCompletionModels.Gpt4Turbo)]
[InlineData(ChatCompletionModels.Gpt4)]
[InlineData(ChatCompletionModels.Gpt3_5_Turbo_1106)]
[InlineData(ChatCompletionModels.Gpt4o)]
[InlineData(ChatCompletionModels.Gpt4o_Mini)]
public async void Get_simple_structured_response_from_ChatGPT(string model)
{
var message =
Expand All @@ -33,7 +34,7 @@ public async void Get_simple_structured_response_from_ChatGPT(string model)

[Theory]
[InlineData(ChatCompletionModels.Gpt4Turbo)]
[InlineData(ChatCompletionModels.Gpt4)]
[InlineData(ChatCompletionModels.Gpt4o)]
public async void Get_structured_response_with_ARRAY_from_ChatGPT(string model)
{
var message = Dialog
Expand All @@ -56,7 +57,7 @@ public async void Get_structured_response_with_ARRAY_from_ChatGPT(string model)

[Theory]
[InlineData(ChatCompletionModels.Gpt4Turbo)]
[InlineData(ChatCompletionModels.Gpt4)]
[InlineData(ChatCompletionModels.Gpt4o)]
public async void Get_structured_response_with_ENUM_from_ChatGPT(string model)
{
var message = Dialog
Expand All @@ -69,7 +70,7 @@ public async void Get_structured_response_with_ENUM_from_ChatGPT(string model)

[Theory]
[InlineData(ChatCompletionModels.Gpt4Turbo)]
[InlineData(ChatCompletionModels.Gpt4)]
[InlineData(ChatCompletionModels.Gpt4o)]
public async void Get_structured_response_with_extra_data_from_ChatGPT(string model)
{
var message = Dialog
Expand Down

0 comments on commit 9ff5e09

Please sign in to comment.