fix: remove model id validation
This commit is contained in:
@@ -126,7 +126,6 @@ public abstract class BaseMessageRequest
|
|||||||
/// <param name="stream">A value indicating whether the message should be streamed.</param>
|
/// <param name="stream">A value indicating whether the message should be streamed.</param>
|
||||||
/// <param name="stopSequences">The prompt stop sequences.</param>
|
/// <param name="stopSequences">The prompt stop sequences.</param>
|
||||||
/// <param name="systemMessages">The system messages to use for the request.</param>
|
/// <param name="systemMessages">The system messages to use for the request.</param>
|
||||||
/// <exception cref="ArgumentException">Thrown when the model ID is invalid.</exception>
|
|
||||||
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
||||||
@@ -151,11 +150,6 @@ public abstract class BaseMessageRequest
|
|||||||
ArgumentValidator.ThrowIfNull(model, nameof(model));
|
ArgumentValidator.ThrowIfNull(model, nameof(model));
|
||||||
ArgumentValidator.ThrowIfNull(messages, nameof(messages));
|
ArgumentValidator.ThrowIfNull(messages, nameof(messages));
|
||||||
|
|
||||||
if (AnthropicModels.IsValidModel(model) is false)
|
|
||||||
{
|
|
||||||
throw new ArgumentException($"Invalid model ID: {model}");
|
|
||||||
}
|
|
||||||
|
|
||||||
if (messages.Count < 1)
|
if (messages.Count < 1)
|
||||||
{
|
{
|
||||||
throw new ArgumentException("Messages must contain at least one message");
|
throw new ArgumentException("Messages must contain at least one message");
|
||||||
|
|||||||
@@ -25,7 +25,6 @@ public class MessageRequest : BaseMessageRequest
|
|||||||
/// <param name="tools">The tools to use for the request.</param>
|
/// <param name="tools">The tools to use for the request.</param>
|
||||||
/// <param name="stopSequences">The prompt stop sequences.</param>
|
/// <param name="stopSequences">The prompt stop sequences.</param>
|
||||||
/// <param name="systemMessages">The system messages to include with the request.</param>
|
/// <param name="systemMessages">The system messages to include with the request.</param>
|
||||||
/// <exception cref="ArgumentException">Thrown when the model ID is invalid.</exception>
|
|
||||||
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
||||||
|
|||||||
@@ -25,7 +25,6 @@ public class StreamMessageRequest : BaseMessageRequest
|
|||||||
/// <param name="tools">The tools to use for the request.</param>
|
/// <param name="tools">The tools to use for the request.</param>
|
||||||
/// <param name="stopSequences">The prompt stop sequences.</param>
|
/// <param name="stopSequences">The prompt stop sequences.</param>
|
||||||
/// <param name="systemMessages">The system messages to include with the request.</param>
|
/// <param name="systemMessages">The system messages to include with the request.</param>
|
||||||
/// <exception cref="ArgumentException">Thrown when the model ID is invalid.</exception>
|
|
||||||
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
/// <exception cref="ArgumentNullException">Thrown when the model or messages is null.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
/// <exception cref="ArgumentException">Thrown when the messages contain no messages.</exception>
|
||||||
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
/// <exception cref="ArgumentException">Thrown when the max tokens is less than one.</exception>
|
||||||
|
|||||||
@@ -232,14 +232,14 @@ public class MessageRequestTests : SerializationTest
|
|||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public void Constructor_WhenCalledAndModelIsInvalid_ItShouldThrowArgumentException()
|
public void Constructor_WhenCalledAndModelIsInvalid_ItShouldNotThrowException()
|
||||||
{
|
{
|
||||||
var action = () => new MessageRequest(
|
var action = () => new MessageRequest(
|
||||||
model: "invalid-model",
|
model: "invalid-model",
|
||||||
messages: [new()]
|
messages: [new()]
|
||||||
);
|
);
|
||||||
|
|
||||||
action.Should().Throw<ArgumentException>();
|
action.Should().NotThrow();
|
||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
|
|||||||
@@ -85,14 +85,14 @@ public class StreamMessageRequestTests : SerializationTest
|
|||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
public void Constructor_WhenCalledAndModelIsInvalid_ItShouldThrowArgumentException()
|
public void Constructor_WhenCalledAndModelIsInvalid_ItShouldNotThrowException()
|
||||||
{
|
{
|
||||||
var action = () => new StreamMessageRequest(
|
var action = () => new StreamMessageRequest(
|
||||||
model: "invalid-model",
|
model: "invalid-model",
|
||||||
messages: [new()]
|
messages: [new()]
|
||||||
);
|
);
|
||||||
|
|
||||||
action.Should().Throw<ArgumentException>();
|
action.Should().NotThrow();
|
||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
[Fact]
|
||||||
|
|||||||
Reference in New Issue
Block a user