Merge pull request #94 from andyw8/andyw8/use-valid-syntax-for-uri-examples

Use valid syntax for URIs in examples and tests

Author: atesgoral

examples/http_server.rb

@@ -61,7 +61,7 @@ def template(args, server_context:)
   prompts: [ExamplePrompt],
   resources: [
     MCP::Resource.new(
-      uri: "test_resource",
+      uri: "https://test_resource.invalid",
       name: "Test resource",
       description: "Test resource that echoes back the uri as its content",
       mime_type: "text/plain",

examples/stdio_server.rb

@@ -58,7 +58,7 @@ def template(args, server_context:)
   prompts: [ExamplePrompt],
   resources: [
     MCP::Resource.new(
-      uri: "test_resource",
+      uri: "https://test_resource.invalid",
       name: "Test resource",
       description: "Test resource that echoes back the uri as its content",
       mime_type: "text/plain",

test/mcp/server/transports/stdio_notification_integration_test.rb

@@ -228,7 +228,7 @@ def puts(message)
           # Add a resource and notify
           @server.resources = [
             MCP::Resource.new(
-              uri: "test://resource",
+              uri: "https://test_resource.invalid",
               name: "Test Resource",
               description: "A test resource",
               mime_type: "text/plain",

test/mcp/server_test.rb

@@ -44,14 +44,14 @@ class ServerTest < ActiveSupport::TestCase
       end
 
       @resource = Resource.new(
-        uri: "test_resource",
+        uri: "https://test_resource.invalid",
         name: "Test resource",
         description: "Test resource",
         mime_type: "text/plain",
       )
 
       @resource_template = ResourceTemplate.new(
-        uri_template: "test_resource/{id}",
+        uri_template: "https://test_resource.invalid/{id}",
         name: "Test resource",
         description: "Test resource",
         mime_type: "text/plain",
@@ -532,7 +532,7 @@ def call(message:, server_context: nil)
 
     test "#resources_list_handler sets the resources/list handler" do
       @server.resources_list_handler do
-        [{ uri: "test_resource", name: "Test resource", description: "Test resource" }]
+        [{ uri: "https://test_resource.invalid", name: "Test resource", description: "Test resource" }]
       end
 
       request = {
@@ -543,7 +543,7 @@ def call(message:, server_context: nil)
 
       response = @server.handle(request)
       assert_equal(
-        { resources: [{ uri: "test_resource", name: "Test resource", description: "Test resource" }] },
+        { resources: [{ uri: "https://test_resource.invalid", name: "Test resource", description: "Test resource" }] },
         response[:result],
       )
       assert_instrumentation_data({ method: "resources/list" })
@@ -555,13 +555,13 @@ def call(message:, server_context: nil)
         method: "resources/read",
         id: 1,
         params: {
-          uri: "example.com",
+          uri: "https://test_resource.invalid",
         },
       }
 
       response = @server.handle(request)
       assert_equal({ contents: [] }, response[:result])
-      assert_instrumentation_data({ method: "resources/read", resource_uri: "example.com" })
+      assert_instrumentation_data({ method: "resources/read", resource_uri: "https://test_resource.invalid" })
     end
 
     test "#resources_read_handler sets the resources/read handler" do
@@ -578,13 +578,13 @@ def call(message:, server_context: nil)
         method: "resources/read",
         id: 1,
         params: {
-          uri: "example.com/my_resource",
+          uri: "https://test_resource.invalid/my_resource",
         },
       }
 
       response = @server.handle(request)
       assert_equal(
-        { contents: { uri: "example.com/my_resource", mimeType: "text/plain", text: "Lorem ipsum dolor sit amet" } },
+        { contents: { uri: "https://test_resource.invalid/my_resource", mimeType: "text/plain", text: "Lorem ipsum dolor sit amet" } },
         response[:result],
       )
     end