coverband 6.1.6 → 6.1.7

data/test/coverband/user_environment_simulation_test.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+# This test simulates using Coverband without COVERBAND_DISABLE_AUTO_START
+# to see if the issue still occurs
+
+require File.expand_path("../test_helper", File.dirname(__FILE__))
+
+class UserEnvironmentSimulationTest < Minitest::Test
+  def setup
+    super
+    # Clean slate
+    Thread.current[:coverband_instance] = nil
+  end
+
+  def teardown
+    super
+    Thread.current[:coverband_instance] = nil
+    # Restore the configured state
+    Coverband.configure do |config|
+      # Use default config
+    end
+    ENV.delete("COVERBAND_DISABLE_AUTO_START")
+  end
+
+  test "user workflow with auto-start disabled" do
+    # User disables auto-start and configures manually (recommended approach)
+    ENV["COVERBAND_DISABLE_AUTO_START"] = "true"
+
+    begin
+      # Reset state
+      Coverband.class_variable_set(:@@configured, false)
+      Coverband.configuration.instance_variable_set(:@store, nil)
+
+      # User configuration exactly like the GitHub issue
+      Coverband.configure do |config|
+        config.root = Dir.pwd
+        config.background_reporting_enabled = false
+        config.store = Coverband::Adapters::FileStore.new("tmp/coverband_log")
+        config.logger = Logger.new($stdout)
+        config.verbose = false
+      end
+
+      # Start manually
+      Coverband.start
+
+      # Should use FileStore without any Redis errors
+      assert_instance_of Coverband::Adapters::FileStore, Coverband.configuration.store
+
+      # These operations should work fine
+      Coverband.report_coverage
+    ensure
+      ENV.delete("COVERBAND_DISABLE_AUTO_START")
+    end
+  end
+
+  test "multiple reports with FileStore should never try Redis" do
+    Coverband.configure do |config|
+      config.root = Dir.pwd
+      config.background_reporting_enabled = false
+      config.store = Coverband::Adapters::FileStore.new("tmp/test_multiple_reports")
+      config.logger = Logger.new($stdout)
+      config.verbose = false
+    end
+
+    # Mock Redis to ensure it's never called after proper configuration
+    Redis.expects(:new).never
+
+    # Multiple reports should work fine
+    5.times do
+      Coverband.report_coverage
+    end
+
+    assert_instance_of Coverband::Adapters::FileStore, Coverband.configuration.store
+  end
+end

data/test/coverband/utils/lines_classifier_test.rb

@@ -64,7 +64,7 @@ describe Coverband::Utils::LinesClassifier do
 
         it "doesn't mistake interpolation as a comment" do
           classified_lines = subject.classify [
-            'puts "#{var}"'
+            "puts \"\#{var}\""
           ]
 
           assert_equal 1, classified_lines.length

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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coverband
 version: !ruby/object:Gem::Version
-  version: 6.1.6
+  version: 6.1.7
 platform: ruby
 authors:
 - Dan Mayer
@@ -24,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
@@ -84,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
@@ -262,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
@@ -293,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:
@@ -316,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
@@ -345,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
@@ -422,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
@@ -465,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