minitest-reporters 1.3.8 → 1.4.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4123119ced267253520fc46bb0836603ce932c2285b94fec9be87769de6b6a21
-  data.tar.gz: 93754ce74501aa021c497688b4e1b54c6ab2edbe815a651941a833e252cafb3a
+  metadata.gz: 120b4ce6a1a42386c94c34efee2b65fe1964e75d2532553850f39964a7cbe832
+  data.tar.gz: 7ea83aff68338ad236208a6622b4f3b746cbf330d3189897a4edeae064d05a01
 SHA512:
-  metadata.gz: da75f810d8ab6f3d7dca4bc0ffbcdaa9f58a0e4b107b9e159dadf26327776957c5434eed3aaade73ae468869cd94b32bd5ecca8e416ae4ed2de18aa348aadef0
-  data.tar.gz: 0b935c8e426330064cce777e2ad25c0433a6cd6dd90e4846a1b06354306eaa37d54f0ac5281b389a8af5c76765725a0349ce83a4cf1b992e76e1e0263db72919
+  metadata.gz: a1062e3fac40dc4ad46aa3ee6be1f9996bd9b10877b1610083e40372adc6548396d9b7c83edd3c9034447d244e78c463669754498d7b5dd2d99b8d169deb1eb6
+  data.tar.gz: 0c99ed679c71e933e858faa088fa452351ebc0a1f18663a865e1c59741ce99b9d9a55c05b27fb4bd8a2b184da789c3e20f83e13d1a7cdcb049bd7ec45fcf1c6a

data/.travis.yml

@@ -4,6 +4,7 @@ before_install:
   - gem install bundler -v '< 2'
 rvm:
   - 2
+  - 2.6
   - 2.5
   - 2.4
   - 2.3

data/CHANGELOG.md

@@ -1,20 +1,45 @@
-### [dev](https://github.com/kern/minitest-reporters/compare/v1.3.8...master)
+### [dev](https://github.com/kern/minitest-reporters/compare/v1.4.3...master)
 
-### [1.3.8](https://github.com/kern/minitest-reporters/compare/v1.3.7...v1.3.8)
+### [1.4.3](https://github.com/kern/minitest-reporters/compare/v1.4.3...v1.4.2)
+
+* fixed rare compatability issue between JUnitReporter and older versions of Minitest [#272](https://github.com/minitest-reporters/minitest-reporters/pull/272) contributed by [chakrit](https://github.com/chakrit)
+* fixed JUnitReporter to use a relative file path if a file path is absolute [#305](https://github.com/minitest-reporters/minitest-reporters/issues/305)
+* fixed MeanTimeReporter to reset by deleting previous run file [#296](https://github.com/kern/minitest-reporters/pull/296) contributed by [AnythonyClark](https://github.com/AnthonyClark)
+* removed debug output from ProgressReporter [#301](https://github.com/kern/minitest-reporters/pull/301) contributed by [wvanbergen](https://github.com/wvanbergen)
+
+### [1.4.2](https://github.com/kern/minitest-reporters/compare/v1.4.1...v1.4.2) (2019-10-26)
+
+* fixed DelegateReporter to delegate prerecord() [#294](https://github.com/kern/minitest-reporters/pull/294) contributed by [mame](https://github.com/mame)
+
+### [1.4.1](https://github.com/kern/minitest-reporters/compare/v1.4....v1.4.1) (2019-10-10)
+
+* fixed Time.current replaced with Time.now in HTML reporter's remplate [#293](https://github.com/kern/minitest-reporters/issues/293)
+
+### [1.4.0](https://github.com/kern/minitest-reporters/compare/v1.4.0.beta1...v1.4.0) (2019-10-06)
+
+* travis updated to include ruby 2.6 ([#292](https://github.com/kern/minitest-reporters/pull/292) contributed by [pvalena](https://github.com/pvalena))
+* location option added to DefaultReporter [#288](https://github.com/kern/minitest-reporters/pull/288) contributed by [bmo](https://github.com/bmo)
+* Date and time added to HTML report [#287](https://github.com/kern/minitest-reporters/pull/287) contributed by [cderche](https://github.com/cderche)
+
+### [1.4.0.beta1](https://github.com/kern/minitest-reporters/compare/v1.3.8...v1.4.0.beta1) (2019-08-28)
+
+* JUnitReporter changed to be compatible with the spec [#286](https://github.com/kern/minitest-reporters/pull/286) contributed by [dylanahsmith](https://github.com/dylanahsmith)
+
+### [1.3.8](https://github.com/kern/minitest-reporters/compare/v1.3.7...v1.3.8) (2019-08-14)
 
 * Fixed default ProgressReporter regression from [#278](https://github.com/kern/minitest-reporters/pull/278); fix [#284](https://github.com/kern/minitest-reporters/pull/284) contributed by [bobmaerten](https://github.com/bobmaerten)
 
-### [1.3.7](https://github.com/kern/minitest-reporters/compare/v1.3.6...v1.3.7)
+### [1.3.7](https://github.com/kern/minitest-reporters/compare/v1.3.6...v1.3.7) (2019-08-14)
 
 * added ability to specify output dir of JUnitReporter through ENV [#277](https://github.com/kern/minitest-reporters/pull/277) countributed by [KevinSjoberg](https://github.com/KevinSjoberg)
 * Added verbose functionality to ProgressReporter [#278](https://github.com/kern/minitest-reporters/pull/278) contributed by [senhalil](https://github.com/senhalil)
 
-### [1.3.6](https://github.com/kern/minitest-reporters/compare/v1.3.5...v1.3.6)
+### [1.3.6](https://github.com/kern/minitest-reporters/compare/v1.3.5...v1.3.6) (2019-01-16)
 
 * fixed possible null pointer in #after_suite [#274](https://github.com/kern/minitest-reporters/pull/274)
   contributed by [casperisfine](https://github.com/casperisfine)
 
-### [1.3.5](https://github.com/kern/minitest-reporters/compare/v1.3.5.beta1...v1.3.5)
+### [1.3.5](https://github.com/kern/minitest-reporters/compare/v1.3.5.beta1...v1.3.5) (2018-09-30)
 
 ### [1.3.5.beta1](https://github.com/kern/minitest-reporters/compare/v1.3.4...v1.3.5.beta1)
 
@@ -98,5 +123,3 @@
 ## [1.1.15](https://github.com/kern/minitest-reporters/compare/v1.1.14...v1.1.15)
 
 * Fixed problem with handling SIGINFO [#231](https://github.com/kern/minitest-reporters/pull/231) (contributed by [joshpencheon](https://github.com/joshpencheon))
-
-

data/README.md

@@ -3,7 +3,6 @@
 
 # minitest-reporters - create customizable Minitest output formats
 
-[![Join the chat at https://gitter.im/kern/minitest-reporters](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/kern/minitest-reporters?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 [![Gem Version](https://badge.fury.io/rb/minitest-reporters.svg)][gem]
 [![Build Status](https://secure.travis-ci.org/kern/minitest-reporters.png)][travis]
 [![Windows build status](https://ci.appveyor.com/api/projects/status/3pugsxatwcldgyjd/branch/master?svg=true)](https://ci.appveyor.com/project/os97673/minitest-reporters/branch/master)
@@ -46,7 +45,7 @@ To override this behavior, you may set the ENV variable MINITEST_REPORTER:
 export MINITEST_REPORTER=JUnitReporter
 ```
 
-Detection of those systems is based on presence of certain ENV variables and are evaulated in the following order:
+Detection of those systems is based on presence of certain ENV variables and are evaluated in the following order:
 
 ```
  MINITEST_REPORTER => use reporter indicated in env variable
@@ -117,6 +116,10 @@ The third parameter to `.use!`, in this case `Minitest.backtrace_filter`, should
 filter object. In the above example, you're telling minitest-reporters to use the filter
 that Rails has already set.
 
+**Test Anything Protocol (TAP)**
+
+The [Test Anything Protocol](https://testanything.org) is a specification for outputting test results in an implementation-agnostic manner so that various tools can read the output. If you need to produce TAP-compliant output for Minitest results, see this [blog post](https://dev.to/davidwessman/rails-minitest-results-output-in-tap-format-for-heroku-ci-46d3) and [gist](https://gist.github.com/davidwessman/09a13840a8a80080e3842ac3051714c7) by [@davidwessman](https://github.com/davidwessman).
+
 ## Note on Patches/Pull Requests ##
 
 * Fork the project.

data/lib/minitest/minitest_reporter_plugin.rb

@@ -16,6 +16,12 @@ module Minitest
         all_reporters.each(&:start)
       end
 
+      def prerecord(klass, name)
+        all_reporters.each do |reporter|
+          reporter.prerecord klass, name
+        end
+      end
+
       def record(result)
         all_reporters.each do |reporter|
           reporter.record result

data/lib/minitest/reporters/default_reporter.rb

@@ -19,6 +19,7 @@ module Minitest
         @suite_times = []
         @suite_start_times = {}
         @fast_fail = options.fetch(:fast_fail, false)
+        @show_test_location = options.fetch(:location, false)
         @options = options
       end
 
@@ -141,11 +142,28 @@ module Minitest
         unless message.nil? || message.strip == ''
           puts
           puts colored_for(result(test), message)
+          if @show_test_location
+            location = get_source_location(test)
+            puts "\n\n#{relative_path(location[0])}:#{location[1]}"
+          end
+
         end
       end
 
       private
 
+      def relative_path(path)
+        Pathname.new(path).relative_path_from(Pathname.new(Dir.getwd))
+      end
+      
+      def get_source_location(result)
+        if result.respond_to? :klass
+          result.source_location
+        else
+          result.method(result.name).source_location
+        end
+      end
+
       def color?
         return @color if defined?(@color)
         @color = @options.fetch(:color) do
@@ -187,7 +205,6 @@ module Minitest
 
       def location(exception)
         last_before_assertion = ''
-
         exception.backtrace.reverse_each do |s|
           break if s =~ /in .(assert|refute|flunk|pass|fail|raise|must|wont)/
           last_before_assertion = s

data/lib/minitest/reporters/junit_reporter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'builder'
 require 'fileutils'
 require 'pathname'
@@ -10,7 +12,7 @@ module Minitest
     # Also inspired by Marc Seeger's attempt at producing a JUnitReporter (see https://github.com/rb2k/minitest-reporters/commit/e13d95b5f884453a9c77f62bc5cba3fa1df30ef5)
     # Also inspired by minitest-ci (see https://github.com/bhenderson/minitest-ci)
     class JUnitReporter < BaseReporter
-      DEFAULT_REPORTS_DIR = "test/reports".freeze
+      DEFAULT_REPORTS_DIR = "test/reports"
 
       attr_reader :reports_path
 
@@ -20,20 +22,20 @@ module Minitest
         @single_file = options[:single_file]
         @base_path = options[:base_path] || Dir.pwd
 
-        if empty
-          puts "Emptying #{@reports_path}"
-          FileUtils.mkdir_p(@reports_path)
-          File.delete(*Dir.glob("#{@reports_path}/TEST-*.xml"))
-        end
+        return unless empty
+
+        puts "Emptying #{@reports_path}"
+        FileUtils.mkdir_p(@reports_path)
+        File.delete(*Dir.glob("#{@reports_path}/TEST-*.xml"))
       end
 
       def report
         super
 
         puts "Writing XML reports to #{@reports_path}"
-        suites = tests.group_by { |test|
+        suites = tests.group_by do |test|
           test_class(test)
-        }
+        end
 
         if @single_file
           xml = Builder::XmlMarkup.new(:indent => 2)
@@ -59,14 +61,18 @@ module Minitest
       def get_relative_path(result)
         file_path = Pathname.new(get_source_location(result).first)
         base_path = Pathname.new(@base_path)
-        file_path.relative_path_from(base_path) if file_path.absolute?
-        file_path
+
+        if file_path.absolute?
+          file_path.relative_path_from(base_path)
+        else
+          file_path
+        end
       end
 
       private
 
       def get_source_location(result)
-        if result.respond_to? :klass
+        if result.respond_to? :source_location
           result.source_location
         else
           result.method(result.name).source_location
@@ -99,16 +105,16 @@ module Minitest
           txt.sub(/\n.*/m, '...')
         end
 
-        e = test.failure
+        failure = test.failure
 
         if test.skipped?
-          xml.skipped(:type => test.name)
+          xml.skipped(:type => failure.error.class.name)
         elsif test.error?
-          xml.error(:type => test.name, :message => xml.trunc!(e.message)) do
+          xml.error(:type => failure.error.class.name, :message => xml.trunc!(failure.message)) do
             xml.text!(message_for(test))
           end
-        elsif test.failure
-          xml.failure(:type => test.name, :message => xml.trunc!(e.message)) do
+        elsif failure
+          xml.failure(:type => failure.error.class.name, :message => xml.trunc!(failure.message)) do
             xml.text!(message_for(test))
           end
         end
@@ -134,6 +140,7 @@ module Minitest
         last_before_assertion = ''
         exception.backtrace.reverse_each do |s|
           break if s =~ /in .(assert|refute|flunk|pass|fail|raise|must|wont)/
+
           last_before_assertion = s
         end
         last_before_assertion.sub(/:in .*$/, '')
@@ -159,7 +166,10 @@ module Minitest
         while File.exist?(File.join(@reports_path, filename)) # restrict number of tries, to avoid infinite loops
           file_counter += 1
           filename = "TEST-#{suite_name}-#{file_counter}.xml"
-          puts "Too many duplicate files, overwriting earlier report #{filename}" and break if file_counter >= 99
+          if file_counter >= 99
+            puts "Too many duplicate files, overwriting earlier report #{filename}"
+            break
+          end
         end
         File.join(@reports_path, filename)
       end

data/lib/minitest/reporters/mean_time_reporter.rb

@@ -100,7 +100,7 @@ module Minitest
       #
       # @return [void]
       def reset_statistics!
-        File.open(previous_runs_filename, 'w+') { |f| f.write('') }
+        File.delete(previous_runs_filename) if File.exist?(previous_runs_filename)
       end
 
       protected

data/lib/minitest/reporters/progress_reporter.rb

@@ -87,8 +87,7 @@ module Minitest
       end
 
       def print_test_with_time(test)
-        puts [test.name, test_class(test), total_time].inspect
-        print(" %s#%s (%.2fs)" % [test.name, test_class(test), total_time])
+        print(" %s#%s (%.2fs)" % [test_class(test), test.name, total_time])
       end
 
       def color

data/lib/minitest/reporters/version.rb

@@ -1,5 +1,5 @@
 module Minitest
   module Reporters
-    VERSION = '1.3.8'.freeze
+    VERSION = '1.4.3'.freeze
   end
 end

data/lib/minitest/templates/index.html.erb

@@ -11,7 +11,9 @@
 <body>
 <div class="container">
     <div class="jumbotron">
-        <h1><%= title %></h1>
+        <h3><small>Generated on <%= Time.now.strftime("%b %d, %Y at %H:%M %Z") %></small></h3>
+        <h1>
+            <%= title %></h1>
         <p>
             Finished in <%= total_time_to_hms %>, <%= '%.2f tests/s' % (count / total_time) %>, <%= '%.2f assertions/s' % (assertions / total_time) %>
         </p>
@@ -80,4 +82,4 @@
     <% end %>
 </div>
 </body>
-</html>
+</html>

data/test/integration/reporters/progress_reporter_test.rb

@@ -6,17 +6,17 @@ module MinitestReportersTest
       fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
       test_filename = File.join(fixtures_directory, 'progress_test.rb')
       output = `#{ruby_executable} #{test_filename} 2>&1`
-      assert_match 'ERROR["test_error"', output, 'Errors should be displayed'
-      assert_match 'FAIL["test_failure"', output, 'Failures should be displayed'
-      assert_match 'SKIP["test_skip', output, 'Skipped tests should be displayed'
+      assert_match 'test_error', output, 'Errors should be displayed'
+      assert_match 'test_failure', output, 'Failures should be displayed'
+      assert_match 'test_skip', output, 'Skipped tests should be displayed'
     end
     def test_skipped_tests_are_not_displayed
       fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
       test_filename = File.join(fixtures_directory, 'progress_detailed_skip_test.rb')
       output = `#{ruby_executable} #{test_filename} 2>&1`
-      assert_match 'ERROR["test_error"', output, 'Errors should be displayed'
-      assert_match 'FAIL["test_failure"', output, 'Failures should be displayed'
-      refute_match 'SKIP["test_skip', output, 'Skipped tests should not be displayed'
+      assert_match 'test_error', output, 'Errors should be displayed'
+      assert_match 'test_failure', output, 'Failures should be displayed'
+      refute_match 'test_skip', output, 'Skipped tests should not be displayed'
     end
     def test_progress_works_with_filter_and_specs
       fixtures_directory = File.expand_path('../../../fixtures', __FILE__)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: minitest-reporters
 version: !ruby/object:Gem::Version
-  version: 1.3.8
+  version: 1.4.3
 platform: ruby
 authors:
 - Alexander Kern
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-08-14 00:00:00.000000000 Z
+date: 2021-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -170,7 +170,7 @@ homepage: https://github.com/CapnKernul/minitest-reporters
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -185,8 +185,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: Create customizable Minitest output formats
 test_files: