diff --git a/.rubocop.yml b/.rubocop.yml index 8657204..b0d11dc 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -1,5 +1,10 @@ inherit_gem: rubocop-shopify: rubocop.yml + +plugins: + - rubocop-minitest + - rubocop-rake + Naming/FileName: Exclude: - 'lib/mcp-ruby.rb' diff --git a/Gemfile b/Gemfile index ecf15d1..2d01b7e 100644 --- a/Gemfile +++ b/Gemfile @@ -8,6 +8,8 @@ gemspec # Specify development dependencies below gem "minitest", "~> 5.1", require: false gem "rake", "~> 13.0" +gem "rubocop-minitest" +gem "rubocop-rake" gem "rubocop-shopify", require: false gem "minitest-reporters" diff --git a/test/model_context_protocol/prompt_test.rb b/test/model_context_protocol/prompt_test.rb index b93395f..7200d21 100644 --- a/test/model_context_protocol/prompt_test.rb +++ b/test/model_context_protocol/prompt_test.rb @@ -67,7 +67,7 @@ module MCP assert_equal "a mock prompt for testing", prompt.description assert_equal "test_argument", prompt.arguments.first.name assert_equal "Test argument", prompt.arguments.first.description - assert_equal true, prompt.arguments.first.required + assert prompt.arguments.first.required expected_template_result = { description: "Hello, world!", diff --git a/test/model_context_protocol/server_test.rb b/test/model_context_protocol/server_test.rb index 01b7622..6dc56ee 100644 --- a/test/model_context_protocol/server_test.rb +++ b/test/model_context_protocol/server_test.rb @@ -153,7 +153,7 @@ module MCP assert_kind_of Array, result[:tools] assert_equal "test_tool", result[:tools][0][:name] assert_equal "Test tool", result[:tools][0][:description] - assert_equal({}, result[:tools][0][:inputSchema]) + assert_empty(result[:tools][0][:inputSchema]) assert_instrumentation_data({ method: "tools/list" }) end diff --git a/test/model_context_protocol/tool/input_schema_test.rb b/test/model_context_protocol/tool/input_schema_test.rb index 1ded7af..56463bd 100644 --- a/test/model_context_protocol/tool/input_schema_test.rb +++ b/test/model_context_protocol/tool/input_schema_test.rb @@ -25,7 +25,7 @@ module MCP test "missing_required_arguments returns an empty array if no required arguments are missing" do input_schema = InputSchema.new(properties: { message: { type: "string" } }, required: [:message]) - assert_equal [], input_schema.missing_required_arguments({ message: "Hello, world!" }) + assert_empty input_schema.missing_required_arguments({ message: "Hello, world!" }) end end end diff --git a/test/model_context_protocol/transports/stdio_transport_test.rb b/test/model_context_protocol/transports/stdio_transport_test.rb index 42c76cf..2fe5df2 100644 --- a/test/model_context_protocol/transports/stdio_transport_test.rb +++ b/test/model_context_protocol/transports/stdio_transport_test.rb @@ -46,7 +46,7 @@ module MCP response = JSON.parse(output.string, symbolize_names: true) assert_equal("2.0", response[:jsonrpc]) assert_equal("123", response[:id]) - assert_equal({}, response[:result]) + assert_empty(response[:result]) refute(@transport.instance_variable_get(:@open)) ensure $stdin = original_stdin