Skip to content

Commit 60f4c79

Browse files
committed
fixup
1 parent f9d062b commit 60f4c79

File tree

2 files changed

+1
-17
lines changed

2 files changed

+1
-17
lines changed

src/Custom/Chat/Internal/InternalChatCompletionMessageListDatum.cs renamed to src/Custom/Chat/ChatCompletionMessageListDatum.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@ public partial class ChatCompletionMessageListDatum
55
{
66
// CUSTOM: Ensure enumerated value is used.
77
[CodeGenMember("Role")]
8-
public ChatMessageRole Role { get; set; } = ChatMessageRole.Assistant;
8+
internal ChatMessageRole Role { get; set; } = ChatMessageRole.Assistant;
99
}

tests/Chat/ChatTests.cs

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1716,7 +1716,6 @@ await RetryWithExponentialBackoffAsync(async () =>
17161716
{
17171717
messageCount++;
17181718
Assert.That(message.Id, Is.Not.Null.And.Not.Empty);
1719-
Assert.That(message.Role, Is.Not.EqualTo(default(ChatMessageRole)));
17201719
Assert.AreEqual("Basic messages test: Say 'Hello, this is a test message.'", message.Content);
17211720

17221721
if (messageCount >= 5) break; // Prevent infinite loop
@@ -2018,20 +2017,6 @@ await RetryWithExponentialBackoffAsync(async () =>
20182017
{
20192018
// Validate required properties
20202019
Assert.That(message.Id, Is.Not.Null.And.Not.Empty);
2021-
Assert.That(message.Role, Is.Not.EqualTo(default(ChatMessageRole)));
2022-
2023-
// Check for different message types
2024-
if (message.Role == ChatMessageRole.User)
2025-
{
2026-
foundUserMessage = true;
2027-
Assert.That(message.Content, Is.Not.Null);
2028-
}
2029-
else if (message.Role == ChatMessageRole.Assistant)
2030-
{
2031-
foundAssistantMessage = true;
2032-
// Assistant messages might have content, tool calls, or both
2033-
Assert.That(message.Content != null || message.ToolCalls?.Count > 0, Is.True);
2034-
}
20352020

20362021
// Test collections are properly initialized
20372022
Assert.That(message.ToolCalls, Is.Not.Null);
@@ -2091,7 +2076,6 @@ await RetryWithExponentialBackoffAsync(async () =>
20912076

20922077
// Validate message structure
20932078
Assert.That(message.Id, Is.Not.Null.And.Not.Empty);
2094-
Assert.That(message.Role, Is.Not.EqualTo(default(ChatMessageRole)));
20952079

20962080
if (messages.Count >= 3) break;
20972081
}

0 commit comments

Comments
 (0)