minitest-heat 1.1.0 → 1.3.0

data/lib/minitest/heat/output/issue.rb

@@ -69,7 +69,9 @@ module Minitest
         # @return [String] the cleaned up version of the test name
         def test_name(issue)
           test_prefix = 'test_'
-          identifier = issue.test_identifier
+          identifier = issue.test_identifier.to_s
+
+          return 'Unknown test' if identifier.empty?
 
           if identifier.start_with?(test_prefix)
             identifier.delete_prefix(test_prefix).gsub('_', ' ').capitalize
@@ -87,22 +89,24 @@ module Minitest
         end
 
         def test_location_tokens
+          source_line = locations.test_failure.source_code.line
           [
             [:default, locations.test_definition.relative_filename],
             [:muted, ':'],
             [:default, locations.test_definition.line_number],
             arrow_token,
             [:default, locations.test_failure.line_number],
-            [:muted, "\n  #{locations.test_failure.source_code.line.strip}"]
+            [:muted, "\n  #{source_line&.strip || '(source unavailable)'}"]
           ]
         end
 
         def location_tokens
+          source_line = locations.most_relevant.source_code.line
           [
             [:default, locations.most_relevant.relative_filename],
             [:muted, ':'],
             [:default, locations.most_relevant.line_number],
-            [:muted, "\n  #{locations.most_relevant.source_code.line.strip}"]
+            [:muted, "\n  #{source_line&.strip || '(source unavailable)'}"]
           ]
         end
 
@@ -110,12 +114,15 @@ module Minitest
           filename    = locations.project.filename
           line_number = locations.project.line_number
           source = Minitest::Heat::Source.new(filename, line_number: line_number)
+          source_line = source.line
 
-          [[:muted, " #{Output::SYMBOLS[:arrow]} `#{source.line.strip}`"]]
+          [[:muted, " #{Output::SYMBOLS[:arrow]} `#{source_line&.strip || '(source unavailable)'}`"]]
         end
 
         def summary_tokens
-          [[:italicized, issue.summary.delete_suffix('---------------').strip]]
+          summary_text = issue.summary.to_s
+          cleaned = summary_text.delete_suffix('---------------').strip
+          [[:italicized, cleaned.empty? ? '(no details available)' : cleaned]]
         end
 
         def slowness_summary_tokens

data/lib/minitest/heat/output/results.rb

@@ -9,7 +9,12 @@ module Minitest
 
         attr_accessor :results, :timer
 
-        def_delegators :@results, :issues, :errors, :brokens, :failures, :skips, :painfuls, :slows, :problems?
+        # Explicitly define issues to avoid Forwardable warning about Object#issues private method
+        def issues
+          @results.issues
+        end
+
+        def_delegators :@results, :errors, :brokens, :failures, :skips, :painfuls, :slows, :problems?
 
         def initialize(results, timer)
           @results = results

data/lib/minitest/heat/output/token.rb

@@ -17,10 +17,10 @@ module Minitest
           failure: %i[default red],
           skipped: %i[default yellow],
           warning_light: %i[light yellow],
-          italicized: %i[italic gray],
+          italicized: %i[italic default],
           bold: %i[bold default],
           default: %i[default default],
-          muted: %i[light gray]
+          muted: %i[light default]
         }.freeze
 
         attr_accessor :style_key, :content

data/lib/minitest/heat/output.rb

@@ -154,8 +154,9 @@ module Minitest
           tokens.each do |token|
             begin
               print Token.new(*token).to_s(token_format)
-            rescue
-              puts token.inspect
+            rescue ArgumentError => e
+              # Token format error - output debug info and continue
+              puts "Token error (#{e.message}): #{token.inspect}"
             end
           end
           newline

data/lib/minitest/heat/results.rb

@@ -35,15 +35,15 @@ module Minitest
         # numbers if the tests reference a shared method with an assertion in it. So in those cases,
         # the backtrace is simply the test definition
         backtrace = if issue.error?
-          # With errors, we have a backtrace
-          issue.locations.backtrace.project_locations
-        else
-          # With failures, the test definition is the most granular backtrace equivalent
-          location = issue.locations.test_definition
-          location.raw_container = issue.test_identifier
+                      # With errors, we have a backtrace
+                      issue.locations.backtrace.project_locations
+                    else
+                      # With failures, the test definition is the most granular backtrace equivalent
+                      location = issue.locations.test_definition
+                      location.raw_container = issue.test_identifier
 
-          [location]
-        end
+                      [location]
+                    end
 
         @heat_map.add(pathname, line_number, issue.type, backtrace: backtrace)
       end
@@ -76,6 +76,39 @@ module Minitest
         @slows ||= select_issues(:slow).sort_by(&:execution_time).reverse
       end
 
+      # Returns count statistics by issue type
+      #
+      # @return [Hash] counts for each issue type
+      def statistics
+        {
+          total: issues.size,
+          errors: errors.size,
+          broken: brokens.size,
+          failures: failures.size,
+          skipped: skips.size,
+          painful: painfuls.size,
+          slow: slows.size
+        }
+      end
+
+      # Returns all issues that are not successes
+      #
+      # @return [Array<Issue>] issues that represent problems (errors, failures, skips, slow)
+      def issues_with_problems
+        issues.select(&:hit?)
+      end
+
+      # Generates a hash representation for JSON serialization
+      #
+      # @return [Hash] results data
+      def to_h
+        {
+          statistics: statistics,
+          heat_map: heat_map.to_h,
+          issues: issues_with_problems.map(&:to_h)
+        }
+      end
+
       private
 
       def select_issues(issue_type)

data/lib/minitest/heat/source.rb

@@ -52,13 +52,14 @@ module Minitest
       # @return [type] [description]
       def file_lines
         @raw_lines ||= File.readlines(filename, chomp: true)
-        @raw_lines.pop while @raw_lines.last.strip.empty?
+        # Remove trailing empty lines, checking for nil/empty safely
+        @raw_lines.pop while @raw_lines.any? && @raw_lines.last&.strip.to_s.empty?
 
         @raw_lines
-      rescue Errno::ENOENT
+      rescue Errno::ENOENT, Errno::EACCES, Errno::EISDIR, IOError, Encoding::UndefinedConversionError
         # Occasionally, for a variety of reasons, a file can't be read. In those cases, it's best to
         # return no source code lines rather than have the test suite raise an error unrelated to
-        # the code being tested becaues that gets confusing.
+        # the code being tested because that gets confusing.
         []
       end
 

data/lib/minitest/heat/timer.rb

@@ -66,6 +66,19 @@ module Minitest
         (assertion_count / total_time).round(2)
       end
 
+      # Generates a hash representation for JSON serialization
+      #
+      # @return [Hash] timing data
+      def to_h
+        {
+          total_seconds: total_time,
+          test_count: test_count,
+          assertion_count: assertion_count,
+          tests_per_second: tests_per_second,
+          assertions_per_second: assertions_per_second
+        }
+      end
+
       private
 
       # The total time the test suite was running.

data/lib/minitest/heat/version.rb

@@ -2,6 +2,6 @@
 
 module Minitest
   module Heat
-    VERSION = '1.1.0'
+    VERSION = '1.3.0'
   end
 end

data/lib/minitest/heat_plugin.rb

@@ -3,6 +3,12 @@
 require_relative 'heat_reporter'
 
 module Minitest # rubocop:disable Style/Documentation
+  def self.plugin_heat_options(opts, options)
+    opts.on '--heat-json', 'Output results as JSON instead of human-readable format' do
+      options[:heat_json] = true
+    end
+  end
+
   def self.plugin_heat_init(options)
     io = options.fetch(:io, $stdout)
 

data/lib/minitest/heat_reporter.rb

@@ -48,7 +48,9 @@ module Minitest
     def start
       timer.start!
 
-      # A couple of blank lines to create some breathing room
+      # A couple of blank lines to create some breathing room (skip for JSON output)
+      return if json_output?
+
       output.newline
       output.newline
     end
@@ -72,8 +74,8 @@ module Minitest
       # Record the issue to show details later
       results.record(issue)
 
-      # Show the marker
-      output.marker(issue.type)
+      # Show the marker (skip for JSON output)
+      output.marker(issue.type) unless json_output?
     rescue StandardError => e
       display_exception_guidance(e)
     end
@@ -93,6 +95,45 @@ module Minitest
     def report
       timer.stop!
 
+      if json_output?
+        output_json
+      else
+        output_text
+      end
+    end
+
+    # Whether to output JSON instead of human-readable text
+    #
+    # @return [Boolean] true if --heat-json flag was passed
+    def json_output?
+      options[:heat_json]
+    end
+
+    # Did this run pass?
+    def passed?
+      results.errors.empty? && results.failures.empty?
+    end
+
+    private
+
+    def output_json
+      require 'json'
+      output.stream.puts JSON.pretty_generate(json_results)
+    end
+
+    def json_results
+      {
+        version: '1.0',
+        status: results.problems? ? 'failed' : 'passed',
+        timestamp: Time.now.iso8601,
+        statistics: results.statistics,
+        timing: timer.to_h,
+        heat_map: results.heat_map.to_h,
+        issues: results.issues_with_problems.map(&:to_h)
+      }
+    end
+
+    def output_text
       # The list of individual issues and their associated details
       output.issues_list(results)
 
@@ -107,10 +148,5 @@ module Minitest
       output.newline
       output.newline
     end
-
-    # Did this run pass?
-    def passed?
-      results.errors.empty? && results.failures.empty?
-    end
   end
 end

data/minitest-heat.gemspec

@@ -12,7 +12,7 @@ Gem::Specification.new do |spec|
   spec.description   = 'Presents test results in a visual manner to guide you to where to look first.'
   spec.homepage      = 'https://github.com/garrettdimon/minitest-heat'
   spec.license       = 'MIT'
-  spec.required_ruby_version = Gem::Requirement.new('>= 2.5.9')
+  spec.required_ruby_version = Gem::Requirement.new('>= 3.1')
 
   spec.metadata['homepage_uri'] = spec.homepage
   spec.metadata['bug_tracker_uri'] = 'https://github.com/garrettdimon/minitest-heat/issues'
@@ -20,6 +20,7 @@ Gem::Specification.new do |spec|
   spec.metadata['documentation_uri'] = 'https://www.rubydoc.info/gems/minitest-heat'
   spec.metadata['source_code_uri'] = 'https://github.com/garrettdimon/minitest-heat'
   spec.metadata['wiki_uri'] = 'https://github.com/garrettdimon/minitest-heat/wiki'
+  spec.metadata['rubygems_mfa_required'] = 'true'
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
@@ -33,10 +34,10 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'minitest'
 
   spec.add_development_dependency 'awesome_print'
-  spec.add_development_dependency 'dead_end'
-  spec.add_development_dependency 'pry'
+  spec.add_development_dependency 'debug'
   spec.add_development_dependency 'rubocop'
   spec.add_development_dependency 'rubocop-minitest'
   spec.add_development_dependency 'rubocop-rake'
   spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'simplecov_json_formatter'
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: minitest-heat
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Garrett Dimon
-autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-12-09 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -39,7 +38,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dead_end
+  name: debug
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -53,7 +52,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
+  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -67,7 +66,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
+  name: rubocop-minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -81,7 +80,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-minitest
+  name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -95,7 +94,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop-rake
+  name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +108,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: simplecov_json_formatter
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -132,20 +131,22 @@ extra_rdoc_files: []
 files:
 - ".github/FUNDING.yml"
 - ".github/workflows/main.yml"
+- ".github/workflows/release.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
 - README.md
+- RELEASING.md
 - Rakefile
 - bin/console
 - bin/setup
 - lib/minitest/heat.rb
 - lib/minitest/heat/backtrace.rb
+- lib/minitest/heat/backtrace/line_count.rb
 - lib/minitest/heat/backtrace/line_parser.rb
 - lib/minitest/heat/configuration.rb
 - lib/minitest/heat/hit.rb
@@ -178,7 +179,7 @@ metadata:
   documentation_uri: https://www.rubydoc.info/gems/minitest-heat
   source_code_uri: https://github.com/garrettdimon/minitest-heat
   wiki_uri: https://github.com/garrettdimon/minitest-heat/wiki
-post_install_message:
+  rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
 - lib
@@ -186,15 +187,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.9
+      version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Presents test results in a visual manner to guide you to where to look first.
 test_files: []

data/.travis.yml

@@ -1,6 +0,0 @@
----
-language: ruby
-cache: bundler
-rvm:
-  - 2.7.2
-before_install: gem install bundler -v 2.1.4