coverband 6.1.5 → 6.1.7

data/test/integration/mcp_integration_test.rb

@@ -0,0 +1,175 @@
+# frozen_string_literal: true
+
+require File.expand_path("../test_helper", File.dirname(__FILE__))
+
+begin
+  require "coverband/mcp"
+rescue LoadError
+  puts "MCP gem not available, skipping MCP integration tests"
+end
+
+if defined?(Coverband::MCP)
+  class MCPIntegrationTest < Minitest::Test
+    def setup
+      super
+      Coverband.configure do |config|
+        config.store = Coverband::Adapters::RedisStore.new(Redis.new(db: 2))
+        config.mcp_enabled = true  # Enable MCP for testing
+      end
+
+      # Populate some test coverage data
+      store = Coverband.configuration.store
+      coverage_data = {
+        "/app/models/user.rb" => [1, 1, 0, 1, nil, 1, 0],
+        "/app/models/order.rb" => [1, 1, 1, 1, 1]
+      }
+      store.save_report(coverage_data)
+
+      @server = Coverband::MCP::Server.new
+    end
+
+    def teardown
+      super
+      Coverband.configuration.store&.clear!
+    end
+
+    test "MCP integration with dependency checking" do
+      # Test that all components load correctly together
+      refute_nil @server
+      refute_nil @server.mcp_server
+      refute_empty @server.mcp_server.tools
+    end
+
+    test "end-to-end MCP request handling via server" do
+      # Simulate an MCP request for coverage summary
+      json_request = {
+        "jsonrpc" => "2.0",
+        "method" => "tools/call",
+        "params" => {
+          "name" => "Get Coverage Summary",
+          "arguments" => {}
+        },
+        "id" => 1
+      }
+
+      # This should work without throwing exceptions
+      response = @server.handle_json(json_request.to_json)
+
+      # Basic validation that we got a response
+      refute_nil response
+    end
+
+    test "MCP tools integrate correctly with Coverband configuration" do
+      # Test that tools can access Coverband store and configuration
+      assert_respond_to Coverband.configuration, :store
+      assert_respond_to Coverband.configuration.store, :coverage
+
+      # Verify each tool class is properly defined
+      [
+        Coverband::MCP::Tools::GetCoverageSummary,
+        Coverband::MCP::Tools::GetFileCoverage,
+        Coverband::MCP::Tools::GetUncoveredFiles,
+        Coverband::MCP::Tools::GetDeadMethods,
+        Coverband::MCP::Tools::GetViewTrackerData,
+        Coverband::MCP::Tools::GetRouteTrackerData,
+        Coverband::MCP::Tools::GetTranslationTrackerData
+      ].each do |tool_class|
+        assert_respond_to tool_class, :call
+        assert_respond_to tool_class, :title
+        assert_respond_to tool_class, :description
+        assert_respond_to tool_class, :input_schema
+      end
+    end
+
+    test "HTTP handler integrates with MCP server" do
+      handler = Coverband::MCP::HttpHandler.new
+
+      # Verify handler can create and use MCP server
+      server = handler.send(:mcp_server)
+      assert_instance_of Coverband::MCP::Server, server
+
+      # Verify handler responds to rack interface
+      assert_respond_to handler, :call
+    end
+
+    test "bin/coverband-mcp executable dependencies" do
+      # Test that the executable can be loaded
+      executable_path = File.expand_path("../../bin/coverband-mcp", File.dirname(__FILE__))
+      assert File.exist?(executable_path), "Executable should exist"
+      assert File.executable?(executable_path), "File should be executable"
+
+      # Read the content to verify it requires the right modules
+      content = File.read(executable_path)
+      assert_includes content, 'require "coverband/mcp"'
+      assert_includes content, "Coverband::MCP::Server.new"
+    end
+
+    test "MCP module properly handles missing dependencies" do
+      # Temporarily hide the MCP module to test error handling
+      if defined?(::MCP)
+        original_mcp = ::MCP
+        Object.send(:remove_const, :MCP)
+      end
+
+      begin
+        # This should raise a LoadError with helpful message when requiring
+        error = assert_raises(NameError) do
+          # Force re-evaluation of the conditional
+          eval("Coverband::MCP::Server.new", binding, __FILE__, __LINE__)
+        end
+
+        assert_includes error.message, "MCP"
+      ensure
+        # Restore the constant
+        if defined?(original_mcp)
+          Object.const_set(:MCP, original_mcp)
+        end
+      end
+    end
+
+    test "all tools return properly formatted MCP responses" do
+      # Test that each tool returns a valid MCP::Tool::Response
+      tools = [
+        [Coverband::MCP::Tools::GetCoverageSummary, {}],
+        [Coverband::MCP::Tools::GetFileCoverage, {filename: "user.rb"}],
+        [Coverband::MCP::Tools::GetUncoveredFiles, {threshold: 50}],
+        [Coverband::MCP::Tools::GetDeadMethods, {}],
+        [Coverband::MCP::Tools::GetViewTrackerData, {}],
+        [Coverband::MCP::Tools::GetRouteTrackerData, {}],
+        [Coverband::MCP::Tools::GetTranslationTrackerData, {}]
+      ]
+
+      tools.each do |tool_class, params|
+        response = tool_class.call(**params, server_context: {})
+
+        assert_instance_of ::MCP::Tool::Response, response,
+          "#{tool_class} should return MCP::Tool::Response"
+        assert_respond_to response, :content
+        assert response.content.is_a?(Array), "Content should be an array"
+
+        # Check content structure
+        assert response.content.length > 0, "Responses should have content"
+        assert_equal "text", response.content.first[:type],
+          "Content should have text type"
+      rescue => e
+        # Some tools may fail due to missing features/config, but should handle gracefully
+        flunk "#{tool_class} raised unhandled exception: #{e.class}: #{e.message}"
+      end
+    end
+
+    test "MCP server transport configurations" do
+      # Test that server can be configured for different transports
+      server = Coverband::MCP::Server.new
+
+      # STDIO transport
+      transport_mock = mock("stdio_transport")
+      transport_mock.expects(:open).once
+      ::MCP::Server::Transports::StdioTransport.expects(:new).returns(transport_mock)
+
+      server.run_stdio
+
+      # HTTP transport setup (without actually starting server)
+      assert_equal 9023, Coverband::MCP::Server::DEFAULT_HTTP_PORT
+    end
+  end
+end

data/test/test_helper.rb

@@ -5,11 +5,11 @@ require "bigdecimal"
 original_verbosity = $VERBOSE
 $VERBOSE = nil
 
-$SKIP_SIMPLECOV = true
+SKIP_SIMPLECOV = true
 
 require "rubygems"
 
-unless $SKIP_SIMPLECOV
+unless SKIP_SIMPLECOV
   require "simplecov"
   require "coveralls"
 end
@@ -37,7 +37,7 @@ require "spy/integration"
 require_relative "unique_files"
 $VERBOSE = original_verbosity
 
-unless ENV["ONESHOT"] || $SKIP_SIMPLECOV
+unless ENV["ONESHOT"] || SKIP_SIMPLECOV
   SimpleCov.formatter = Coveralls::SimpleCov::Formatter
   SimpleCov.start do
     add_filter "test/forked"
@@ -88,8 +88,7 @@ $LOAD_PATH.unshift(File.dirname(__FILE__))
 
 Mocha.configure do |c|
   c.stubbing_method_unnecessarily = :prevent
-  c.stubbing_method_on_non_mock_object = :allow
-  c.stubbing_method_on_nil = :prevent
+  c.stubbing_non_public_method = :allow
 end
 
 def test(name, &block)

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 6.1.5
+  version: 6.1.7
 platform: ruby
 authors:
 - Dan Mayer
 - Karl Baum
-autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-04-10 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -25,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: benchmark
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: capybara
   requirement: !ruby/object:Gem::Requirement
@@ -85,16 +98,16 @@ dependencies:
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: minitest-fork_executor
   requirement: !ruby/object:Gem::Requirement
@@ -263,6 +276,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: mcp
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -294,7 +321,8 @@ dependencies:
 description: Rack middleware to measure production code usage (LOC runtime usage)
 email:
 - dan@mayerdan.com
-executables: []
+executables:
+- coverband-mcp
 extensions: []
 extra_rdoc_files: []
 files:
@@ -317,9 +345,12 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- agents.md
+- bin/coverband-mcp
 - changes.md
 - config.ru
 - coverband.gemspec
+- coverband/log.272267
 - diagram.svg
 - lib/alternative_coverband_patch.rb
 - lib/coverband.rb
@@ -346,6 +377,16 @@ files:
 - lib/coverband/integrations/report_middleware.rb
 - lib/coverband/integrations/resque.rb
 - lib/coverband/integrations/sidekiq_swarm.rb
+- lib/coverband/mcp.rb
+- lib/coverband/mcp/http_handler.rb
+- lib/coverband/mcp/server.rb
+- lib/coverband/mcp/tools/get_coverage_summary.rb
+- lib/coverband/mcp/tools/get_dead_methods.rb
+- lib/coverband/mcp/tools/get_file_coverage.rb
+- lib/coverband/mcp/tools/get_route_tracker_data.rb
+- lib/coverband/mcp/tools/get_translation_tracker_data.rb
+- lib/coverband/mcp/tools/get_uncovered_files.rb
+- lib/coverband/mcp/tools/get_view_tracker_data.rb
 - lib/coverband/reporters/base.rb
 - lib/coverband/reporters/console_report.rb
 - lib/coverband/reporters/html_report.rb
@@ -423,18 +464,34 @@ files:
 - test/coverband/collectors/view_tracker_test.rb
 - test/coverband/configuration_test.rb
 - test/coverband/coverband_test.rb
+- test/coverband/file_store_integration_test.rb
+- test/coverband/file_store_redis_error_test.rb
+- test/coverband/github_issue_586_test.rb
+- test/coverband/initialization_timing_test.rb
 - test/coverband/integrations/background_middleware_test.rb
 - test/coverband/integrations/background_test.rb
 - test/coverband/integrations/rack_server_check_test.rb
 - test/coverband/integrations/report_middleware_test.rb
 - test/coverband/integrations/resque_worker_test.rb
 - test/coverband/integrations/test_resque_job.rb
+- test/coverband/mcp/http_handler_test.rb
+- test/coverband/mcp/security_test.rb
+- test/coverband/mcp/server_test.rb
+- test/coverband/mcp/tools/get_coverage_summary_test.rb
+- test/coverband/mcp/tools/get_dead_methods_test.rb
+- test/coverband/mcp/tools/get_file_coverage_test.rb
+- test/coverband/mcp/tools/get_route_tracker_data_test.rb
+- test/coverband/mcp/tools/get_translation_tracker_data_test.rb
+- test/coverband/mcp/tools/get_uncovered_files_test.rb
+- test/coverband/mcp/tools/get_view_tracker_data_test.rb
 - test/coverband/reporters/base_test.rb
 - test/coverband/reporters/console_test.rb
 - test/coverband/reporters/html_test.rb
 - test/coverband/reporters/json_test.rb
 - test/coverband/reporters/web_test.rb
 - test/coverband/track_key_test.rb
+- test/coverband/tracker_initialization_test.rb
+- test/coverband/user_environment_simulation_test.rb
 - test/coverband/utils/absolute_file_converter_test.rb
 - test/coverband/utils/dead_methods_test.rb
 - test/coverband/utils/file_hasher_test.rb
@@ -466,6 +523,7 @@ files:
 - test/integration/full_stack_deferred_eager_test.rb
 - test/integration/full_stack_send_deferred_eager_test.rb
 - test/integration/full_stack_test.rb
+- test/integration/mcp_integration_test.rb
 - test/jruby_check.rb
 - test/rails7_dummy/Rakefile
 - test/rails7_dummy/app/controllers/dummy_controller.rb
@@ -517,7 +575,6 @@ metadata:
   documentation_uri: https://github.com/danmayer/coverband
   changelog_uri: https://github.com/danmayer/coverband/blob/main/changes.md
   source_code_uri: https://github.com/danmayer/coverband
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -532,8 +589,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
-signing_key: 
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Rack middleware to measure production code usage (LOC runtime usage)
 test_files: []