Skip to content

Commit 85e7646

Browse files
committed
#153: Updating tests to feed chat params instead of request schema
1 parent 942a9d9 commit 85e7646

File tree

5 files changed

+31
-22
lines changed

5 files changed

+31
-22
lines changed

pkg/providers/azureopenai/chat_stream_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,12 @@ func TestAzureOpenAIClient_ChatStreamRequest(t *testing.T) {
7171
client, err := NewClient(providerCfg, clientCfg, telemetry.NewTelemetryMock())
7272
require.NoError(t, err)
7373

74-
req := schemas.NewChatStreamFromStr("What's the capital of the United Kingdom?")
75-
stream, err := client.ChatStream(ctx, req)
74+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
75+
Role: "user",
76+
Content: "What's the capital of the United Kingdom?",
77+
}}}
78+
79+
stream, err := client.ChatStream(ctx, &chatParams)
7680
require.NoError(t, err)
7781

7882
err = stream.Open()

pkg/providers/bedrock/client_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,12 +61,12 @@ func TestBedrockClient_ChatRequest(t *testing.T) {
6161
client, err := NewClient(providerCfg, clientCfg, telemetry.NewTelemetryMock())
6262
require.NoError(t, err)
6363

64-
request := schemas.ChatRequest{Message: schemas.ChatMessage{
64+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
6565
Role: "user",
6666
Content: "What's the biggest animal?",
67-
}}
67+
}}}
6868

69-
response, err := client.Chat(ctx, &request)
69+
response, err := client.Chat(ctx, &chatParams)
7070

7171
responseString := fmt.Sprintf("%+v", response)
7272
// errString := fmt.Sprintf("%+v", err)

pkg/providers/octoml/client_test.go

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,12 @@ func TestOctoMLClient_ChatRequest(t *testing.T) {
5555
client, err := NewClient(providerCfg, clientCfg, telemetry.NewTelemetryMock())
5656
require.NoError(t, err)
5757

58-
request := schemas.ChatRequest{Message: schemas.ChatMessage{
58+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
5959
Role: "human",
6060
Content: "What's the biggest animal?",
61-
}}
61+
}}}
6262

63-
response, err := client.Chat(ctx, &request)
63+
response, err := client.Chat(ctx, &chatParams)
6464
require.NoError(t, err)
6565

6666
require.Equal(t, providerCfg.ModelName, response.ModelName)
@@ -88,15 +88,13 @@ func TestOctoMLClient_Chat_Error(t *testing.T) {
8888
require.NoError(t, err)
8989

9090
// Create a chat request
91-
request := schemas.ChatRequest{
92-
Message: schemas.ChatMessage{
93-
Role: "human",
94-
Content: "What's the biggest animal?",
95-
},
96-
}
91+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
92+
Role: "human",
93+
Content: "What's the biggest animal?",
94+
}}}
9795

9896
// Call the Chat function
99-
_, err = client.Chat(ctx, &request)
97+
_, err = client.Chat(ctx, &chatParams)
10098

10199
// Check the error
102100
require.Error(t, err)
@@ -122,9 +120,12 @@ func TestDoChatRequest_ErrorResponse(t *testing.T) {
122120
require.NoError(t, err)
123121

124122
// Create a chat request payload
125-
payload := schemas.NewChatFromStr("What's the dealio?")
123+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
124+
Role: "user",
125+
Content: "What's the dealeo?",
126+
}}}
126127

127-
_, err = client.Chat(ctx, payload)
128+
_, err = client.Chat(ctx, &chatParams)
128129

129130
require.Error(t, err)
130131
require.Contains(t, err.Error(), "provider is not available")

pkg/providers/ollama/client_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,12 @@ func TestOllamaClient_ChatRequest(t *testing.T) {
5656
client, err := NewClient(providerCfg, clientCfg, telemetry.NewTelemetryMock())
5757
require.NoError(t, err)
5858

59-
request := schemas.ChatRequest{Message: schemas.ChatMessage{
59+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
6060
Role: "user",
6161
Content: "What's the biggest animal?",
62-
}}
62+
}}}
6363

64-
_, err = client.Chat(ctx, &request)
64+
_, err = client.Chat(ctx, &chatParams)
6565

6666
// require.NoError(t, err)
6767

pkg/providers/openai/chat_stream_test.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,12 @@ func TestOpenAIClient_ChatStreamRequest(t *testing.T) {
7171
client, err := NewClient(providerCfg, clientCfg, telemetry.NewTelemetryMock())
7272
require.NoError(t, err)
7373

74-
req := schemas.NewChatStreamFromStr("What's the capital of the United Kingdom?")
75-
stream, err := client.ChatStream(ctx, req)
74+
chatParams := schemas.ChatParams{Messages: []schemas.ChatMessage{{
75+
Role: "user",
76+
Content: "What's the capital of the United Kingdom?",
77+
}}}
78+
79+
stream, err := client.ChatStream(ctx, &chatParams)
7680
require.NoError(t, err)
7781

7882
err = stream.Open()

0 commit comments

Comments
 (0)