c588582acc
## Motivation and Context The MCP specification constrains `Annotations.priority` to the 0–1 range (`@minimum 0`, `@maximum 1`), but the Ruby SDK accepts any value. The other official SDKs already enforce this (TypeScript, Python, and PHP). This follows the existing pattern in `MCP::Icon`, which validates its constrained `theme` value in the constructor and raises `ArgumentError`. ## How Has This Been Tested? Added tests in `test/mcp/annotations_test.rb`: - valid `priority` at the lower (0) and upper (1) bounds is accepted - out-of-range `priority` (above 1, below 0) raises `ArgumentError` The full unit suite and RuboCop pass locally. ## Breaking Changes Strictly speaking yes: a caller that currently passes an out-of-range `priority` (e.g. `priority: 99`) will now get an `ArgumentError`. Such values already violate the MCP specification, and `nil` / in-range values are unaffected.
85 lines
2.6 KiB
Ruby
85 lines
2.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require "test_helper"
|
|
|
|
module MCP
|
|
class AnnotationsTest < ActiveSupport::TestCase
|
|
def test_initialization
|
|
annotations = Annotations.new(audience: ["developers"], priority: 0.8)
|
|
|
|
assert_equal(["developers"], annotations.audience)
|
|
assert_equal(0.8, annotations.priority)
|
|
assert_nil(annotations.last_modified)
|
|
|
|
assert_equal({ audience: ["developers"], priority: 0.8 }, annotations.to_h)
|
|
end
|
|
|
|
def test_initialization_with_all_attributes
|
|
timestamp = Time.utc(2025, 1, 12, 15, 0, 58).iso8601
|
|
annotations = Annotations.new(audience: ["developers"], priority: 0.8, last_modified: timestamp)
|
|
|
|
assert_equal(["developers"], annotations.audience)
|
|
assert_equal(0.8, annotations.priority)
|
|
assert_equal(timestamp, annotations.last_modified)
|
|
|
|
assert_equal({ audience: ["developers"], priority: 0.8, lastModified: timestamp }, annotations.to_h)
|
|
end
|
|
|
|
def test_initialization_by_default
|
|
annotations = Annotations.new
|
|
|
|
assert_nil(annotations.audience)
|
|
assert_nil(annotations.priority)
|
|
assert_nil(annotations.last_modified)
|
|
|
|
assert_equal({}, annotations.to_h)
|
|
end
|
|
|
|
def test_initialization_with_partial_attributes
|
|
annotations = Annotations.new(audience: ["developers"])
|
|
|
|
assert_equal(["developers"], annotations.audience)
|
|
assert_nil(annotations.priority)
|
|
assert_nil(annotations.last_modified)
|
|
|
|
assert_equal({ audience: ["developers"] }, annotations.to_h)
|
|
end
|
|
|
|
def test_initialization_with_last_modified_only
|
|
timestamp = Time.utc(2025, 1, 12, 15, 0, 58).iso8601
|
|
annotations = Annotations.new(last_modified: timestamp)
|
|
|
|
assert_nil(annotations.audience)
|
|
assert_nil(annotations.priority)
|
|
assert_equal(timestamp, annotations.last_modified)
|
|
|
|
assert_equal({ lastModified: timestamp }, annotations.to_h)
|
|
end
|
|
|
|
def test_valid_priority_at_lower_bound
|
|
assert_nothing_raised do
|
|
Annotations.new(priority: 0)
|
|
end
|
|
end
|
|
|
|
def test_valid_priority_at_upper_bound
|
|
assert_nothing_raised do
|
|
Annotations.new(priority: 1)
|
|
end
|
|
end
|
|
|
|
def test_invalid_priority_above_upper_bound
|
|
exception = assert_raises(ArgumentError) do
|
|
Annotations.new(priority: 1.5)
|
|
end
|
|
assert_equal("The value of priority must be between 0 and 1.", exception.message)
|
|
end
|
|
|
|
def test_invalid_priority_below_lower_bound
|
|
exception = assert_raises(ArgumentError) do
|
|
Annotations.new(priority: -0.1)
|
|
end
|
|
assert_equal("The value of priority must be between 0 and 1.", exception.message)
|
|
end
|
|
end
|
|
end
|