Rubocop

Author: Ginja

test/mcp/server/transports/stdio_notification_integration_test.rb

@@ -30,7 +30,7 @@ def gets
             nil # Simulate end of input
           end
 
-          def set_encoding(encoding)
+          def set_encoding(encoding) # rubocop:disable Naming/AccessorMethodName
             # Mock implementation
           end
 
@@ -131,7 +131,7 @@ def closed?
           @server.define_tool(
             name: "dynamic_tool",
             description: "A dynamically added tool",
-          ) do |**args|
+          ) do |**_args|
             { result: "success" }
           end
 
@@ -213,7 +213,7 @@ def puts(message)
           @server.define_prompt(
             name: "test_prompt",
             description: "Test prompt",
-          ) do |args, server_context:|
+          ) do |_args, _server_context:|
             MCP::PromptResponse.new(messages: [{ role: "user", content: "Test" }])
           end
 

test/mcp/server/transports/streamable_http_notification_integration_test.rb

@@ -181,7 +181,7 @@ class StreamableHTTPNotificationIntegrationTest < ActiveSupport::TestCase
           @server.define_tool(
             name: "dynamic_tool",
             description: "A dynamically added tool",
-          ) do |**args|
+          ) do |**_args|
             { result: "success" }
           end
 

test/mcp/server_notification_test.rb

@@ -29,13 +29,13 @@ def handle_request(request); end
     setup do
       configuration = MCP::Configuration.new
       configuration.instrumentation_callback = instrumentation_helper.callback
-      
+
       @server = Server.new(
         name: "test_server",
         version: "1.0.0",
-        configuration: configuration
+        configuration: configuration,
       )
-      
+
       @mock_transport = MockTransport.new(@server)
       @server.transport = @mock_transport
     end
@@ -69,7 +69,7 @@ def handle_request(request); end
 
     test "notification methods work without transport" do
       server_without_transport = Server.new(name: "test_server")
-      
+
       # Should not raise any errors
       assert_nothing_raised do
         server_without_transport.notify_tools_list_changed
@@ -85,16 +85,16 @@ def send_notification(method, params = nil)
           raise StandardError, "Transport error"
         end
       end.new(@server)
-      
+
       @server.transport = error_transport
-      
+
       # Mock the exception reporter
       expected_contexts = [
         { notification: "tools_list_changed" },
         { notification: "prompts_list_changed" },
-        { notification: "resources_list_changed" }
+        { notification: "resources_list_changed" },
       ]
-      
+
       call_count = 0
       @server.configuration.exception_reporter.expects(:call).times(3).with do |exception, context|
         assert_kind_of StandardError, exception
@@ -103,14 +103,14 @@ def send_notification(method, params = nil)
         call_count += 1
         true
       end
-      
+
       # Should not raise errors to the caller
       assert_nothing_raised do
         @server.notify_tools_list_changed
         @server.notify_prompts_list_changed
         @server.notify_resources_list_changed
       end
-      
+
       assert_equal 3, call_count
     end
 
@@ -120,7 +120,7 @@ def send_notification(method, params = nil)
       @server.notify_resources_list_changed
 
       assert_equal 3, @mock_transport.notifications.size
-      
+
       notifications = @mock_transport.notifications
       assert_equal Methods::NOTIFICATIONS_TOOLS_LIST_CHANGED, notifications[0][:method]
       assert_equal Methods::NOTIFICATIONS_PROMPTS_LIST_CHANGED, notifications[1][:method]
@@ -133,4 +133,4 @@ def send_notification(method, params = nil)
       assert_equal "notifications/resources/list_changed", Methods::NOTIFICATIONS_RESOURCES_LIST_CHANGED
     end
   end
-end 
+end