Remove unnecessary factory methods for message types with public constructors

Co-authored-by: KrzysztofCwalina <[email protected]>

Author: Copilot

src/Custom/ChatModelFactory.cs

@@ -87,70 +87,7 @@ public static ChatCompletionDeletionResult ChatCompletionDeletionResult(
             additionalBinaryDataProperties: null);
     }
 
-    /// <summary> Initializes a new instance of <see cref="OpenAI.Chat.UserChatMessage"/>. </summary>
-    /// <returns> A new <see cref="OpenAI.Chat.UserChatMessage"/> instance for mocking. </returns>
-    public static UserChatMessage UserChatMessage(
-        string content = null,
-        string participantName = null)
-    {
-        content ??= "User message content";
-        var message = new UserChatMessage(content);
-        if (participantName != null)
-        {
-            message.ParticipantName = participantName;
-        }
-        return message;
-    }
-
-    /// <summary> Initializes a new instance of <see cref="OpenAI.Chat.AssistantChatMessage"/>. </summary>
-    /// <returns> A new <see cref="OpenAI.Chat.AssistantChatMessage"/> instance for mocking. </returns>
-    public static AssistantChatMessage AssistantChatMessage(
-        string content = null,
-        IEnumerable<ChatToolCall> toolCalls = null,
-        string participantName = null)
-    {
-        AssistantChatMessage message;
-        if (toolCalls != null)
-        {
-            message = new AssistantChatMessage(toolCalls);
-        }
-        else
-        {
-            content ??= "Assistant message content";
-            message = new AssistantChatMessage(content);
-        }
-        
-        if (participantName != null)
-        {
-            message.ParticipantName = participantName;
-        }
-        return message;
-    }
 
-    /// <summary> Initializes a new instance of <see cref="OpenAI.Chat.SystemChatMessage"/>. </summary>
-    /// <returns> A new <see cref="OpenAI.Chat.SystemChatMessage"/> instance for mocking. </returns>
-    public static SystemChatMessage SystemChatMessage(
-        string content = null,
-        string participantName = null)
-    {
-        content ??= "System message content";
-        var message = new SystemChatMessage(content);
-        if (participantName != null)
-        {
-            message.ParticipantName = participantName;
-        }
-        return message;
-    }
-
-    /// <summary> Initializes a new instance of <see cref="OpenAI.Chat.ToolChatMessage"/>. </summary>
-    /// <returns> A new <see cref="OpenAI.Chat.ToolChatMessage"/> instance for mocking. </returns>
-    public static ToolChatMessage ToolChatMessage(
-        string toolCallId = "tool_call_id",
-        string content = null)
-    {
-        content ??= "Tool message content";
-        return new ToolChatMessage(toolCallId, content);
-    }
 
     /// <summary> Initializes a new instance of <see cref="OpenAI.Chat.ChatTokenUsage"/>. </summary>
     /// <returns> A new <see cref="OpenAI.Chat.ChatTokenUsage"/> instance for mocking. </returns>

tests/Chat/ChatModelFactoryTests.cs

@@ -146,77 +146,7 @@ public void ChatCompletionDeletionResultWorks()
         Assert.That(result.ChatCompletionId, Is.EqualTo(chatCompletionId));
     }
 
-    [Test]
-    public void UserChatMessageWorks()
-    {
-        string content = "Hello, assistant!";
-        string participantName = "user1";
-
-        UserChatMessage message = ChatModelFactory.UserChatMessage(
-            content: content,
-            participantName: participantName);
-
-        Assert.That(message.Content[0].Text, Is.EqualTo(content));
-        Assert.That(message.ParticipantName, Is.EqualTo(participantName));
-        Assert.That(message.Role, Is.EqualTo(ChatMessageRole.User));
-    }
-
-    [Test]
-    public void AssistantChatMessageWorks()
-    {
-        string content = "Hello, user!";
-        IEnumerable<ChatToolCall> toolCalls = [
-            ChatToolCall.CreateFunctionToolCall("id1", "get_weather", BinaryData.FromString("{}"))
-        ];
-        string participantName = "assistant1";
-
-        AssistantChatMessage message = ChatModelFactory.AssistantChatMessage(
-            content: content,
-            participantName: participantName);
-
-        Assert.That(message.Content[0].Text, Is.EqualTo(content));
-        Assert.That(message.ParticipantName, Is.EqualTo(participantName));
-        Assert.That(message.Role, Is.EqualTo(ChatMessageRole.Assistant));
-
-        // Test with tool calls
-        AssistantChatMessage messageWithToolCalls = ChatModelFactory.AssistantChatMessage(
-            toolCalls: toolCalls,
-            participantName: participantName);
 
-        Assert.That(messageWithToolCalls.ToolCalls.SequenceEqual(toolCalls), Is.True);
-        Assert.That(messageWithToolCalls.ParticipantName, Is.EqualTo(participantName));
-        Assert.That(messageWithToolCalls.Role, Is.EqualTo(ChatMessageRole.Assistant));
-    }
-
-    [Test]
-    public void SystemChatMessageWorks()
-    {
-        string content = "You are a helpful assistant";
-        string participantName = "system";
-
-        SystemChatMessage message = ChatModelFactory.SystemChatMessage(
-            content: content,
-            participantName: participantName);
-
-        Assert.That(message.Content[0].Text, Is.EqualTo(content));
-        Assert.That(message.ParticipantName, Is.EqualTo(participantName));
-        Assert.That(message.Role, Is.EqualTo(ChatMessageRole.System));
-    }
-
-    [Test]
-    public void ToolChatMessageWorks()
-    {
-        string toolCallId = "tool_call_id";
-        string content = "Weather is sunny";
-
-        ToolChatMessage message = ChatModelFactory.ToolChatMessage(
-            toolCallId: toolCallId,
-            content: content);
-
-        Assert.That(message.Content[0].Text, Is.EqualTo(content));
-        Assert.That(message.ToolCallId, Is.EqualTo(toolCallId));
-        Assert.That(message.Role, Is.EqualTo(ChatMessageRole.Tool));
-    }
 
     [Test]
     public void ChatTokenUsageWorks()