minitest-reporters 1.4.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4a0d617bd5555dc8aa2251bb8e6351700ac826b81c76a4146aa46c4de04bf37c
-  data.tar.gz: e1b2ab8a11ae00a613b75134008c6be55a0efb53c42c186346e40a90f76546e3
+  metadata.gz: 2e6b7a770ddd1f72abb68531badb27cb13fc38189c0f638d91b7244c5a2856c9
+  data.tar.gz: 44b074935fd563c3be78833b59a5aba4c0faad662b5adb6d531f4adc7180bd62
 SHA512:
-  metadata.gz: 89af1791568eb12157ee3d438a27f97dbecad07d5c92a9bc307c21b1079b584444384107acac7c8b972962c3fe3ba66b77a749aa1e836c2e8c924c1fb79f9a76
-  data.tar.gz: 1a5698902d3237adc9711b68732fe3b6029a33ab2e79dd720261b5bec9689fc83767c4d91b3f4f71280b19abdd626b6429dfb44a389eab36892f66f9ab9e43e2
+  metadata.gz: 9cee23ccb7b443ef4fefea6a8420e3452d171d0c544b6835c1442658616a5db513182ba1dc77731259050fc2431b47e318ff0c116d4a20d7a592bba100b9837e
+  data.tar.gz: 216c4f9cabcb7d43304266c3bbb83bf033ad84686bc7258f8c6b6f5dd9970fdc6fc66655bf76e94f27728bd0afb924ef030e29fc8a9a75753f0f879487f9a47a

data/CHANGELOG.md

@@ -1,30 +1,52 @@
-### [dev](https://github.com/kern/minitest-reporters/compare/v1.4.0...master)
+### [dev](https://github.com/kern/minitest-reporters/compare/v1.5.0...master)
 
-### [1.4.0](https://github.com/kern/minitest-reporters/compare/v1.4.0.beta1...v1.4.0)
+### [1.5.0](https://github.com/kern/minitest-reporters/compare/v1.4.3...v1.5.0)
+
+* Added support for environment variables to define the output location of HTML reports. [#311](https://github.com/minitest-reporters/minitest-reporters/pull/311) contributed by [estebanbouza](https://github.com/estebanbouza)
+* Fixed ProgressReporter accuracy on skipped tests while `detailed_skip` is disabled [#312](https://github.com/minitest-reporters/minitest-reporters/pull/312) contributed by [seven1m](https://github.com/seven1m)
+* Added `file` attribute to `<testcase>` tags in JUnitReporter for CircleCI compatibility [#313](https://github.com/minitest-reporters/minitest-reporters/pull/313) contributed by [nbudin](https://github.com/nbudin)
+* Added timestamp option to JUnitReporter [#316] (https://github.com/minitest-reporters/minitest-reporters/pull/316) contributed by [sipani909](https://github.com/sipani909)
+
+### [1.4.3](https://github.com/kern/minitest-reporters/compare/v1.4.2...v1.4.3)
+
+* 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)
+### [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)
+### [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)
 
@@ -49,7 +71,7 @@
 
 * SpecReporter do not print exception name any more (unless it is an test error) [#264](https://github.com/kern/minitest-reporters/issues/264)
 * Fixed loading error caused by fix for [#265](https://github.com/kern/minitest-reporters/pull/265)
-  see [#267](https://github.com/kern/minitest-reporters/issues/267) and 
+  see [#267](https://github.com/kern/minitest-reporters/issues/267) and
   [#268](https://github.com/kern/minitest-reporters/pull/268) for more details.
 
 ### [1.3.1](https://github.com/kern/minitest-reporters/compare/v1.3.1.beta1...v1.3.1)
@@ -108,5 +130,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)
@@ -37,7 +36,7 @@ Want to use multiple reporters?
 Minitest::Reporters.use! [Minitest::Reporters::SpecReporter.new, Minitest::Reporters::JUnitReporter.new]
 ```
 
-If RubyMate, TeamCity, RubyMine or VIM presence is detected, the reporter will be automatically chosen,
+If TextMate, TeamCity, RubyMine or VIM presence is detected, the reporter will be automatically chosen,
 regardless of any reporters passed to the `use!` method.
 
 To override this behavior, you may set the ENV variable MINITEST_REPORTER:
@@ -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/html_reporter.rb

@@ -60,9 +60,9 @@ module Minitest
         defaults = {
           :title           => 'Test Results',
           :erb_template    => "#{File.dirname(__FILE__)}/../templates/index.html.erb",
-          :reports_dir     => 'test/html_reports',
+          :reports_dir     => ENV['MINITEST_HTML_REPORTS_DIR'] || 'test/html_reports',
           :mode            => :safe,
-          :output_filename => 'index.html',
+          :output_filename => ENV['MINITEST_HTML_REPORTS_FILENAME'] || 'index.html',
         }
 
         settings = defaults.merge(args)

data/lib/minitest/reporters/junit_reporter.rb

@@ -3,6 +3,8 @@
 require 'builder'
 require 'fileutils'
 require 'pathname'
+require 'time'
+
 module Minitest
   module Reporters
     # A reporter for writing JUnit test reports
@@ -21,6 +23,7 @@ module Minitest
         @reports_path = File.absolute_path(ENV.fetch("MINITEST_REPORTERS_REPORTS_DIR", reports_dir))
         @single_file = options[:single_file]
         @base_path = options[:base_path] || Dir.pwd
+        @timestamp_report = options[:include_timestamp]
 
         return unless empty
 
@@ -61,14 +64,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
@@ -79,15 +86,31 @@ module Minitest
         suite_result = analyze_suite(tests)
         file_path = get_relative_path(tests.first)
 
-        xml.testsuite(:name => suite, :filepath => file_path,
-                      :skipped => suite_result[:skip_count], :failures => suite_result[:fail_count],
-                      :errors => suite_result[:error_count], :tests => suite_result[:test_count],
-                      :assertions => suite_result[:assertion_count], :time => suite_result[:time]) do
-          tests.each do |test|
-            lineno = get_source_location(test).last
-            xml.testcase(:name => test.name, :lineno => lineno, :classname => suite, :assertions => test.assertions,
-                         :time => test.time) do
-              xml << xml_message_for(test) unless test.passed?
+        if @timestamp_report
+          xml.testsuite(:name => suite, :filepath => file_path,
+                                    :skipped => suite_result[:skip_count], :failures => suite_result[:fail_count],
+                                    :errors => suite_result[:error_count], :tests => suite_result[:test_count],
+                                    :assertions => suite_result[:assertion_count], :time => suite_result[:time],
+                                    :timestamp => suite_result[:timestamp]) do
+            tests.each do |test|
+              lineno = get_source_location(test).last
+              xml.testcase(:name => test.name, :lineno => lineno, :classname => suite, :assertions => test.assertions,
+                           :time => test.time, :file => file_path) do
+                xml << xml_message_for(test) unless test.passed?
+              end
+            end
+          end
+        else
+         xml.testsuite(:name => suite, :filepath => file_path,
+                                    :skipped => suite_result[:skip_count], :failures => suite_result[:fail_count],
+                                    :errors => suite_result[:error_count], :tests => suite_result[:test_count],
+                                    :assertions => suite_result[:assertion_count], :time => suite_result[:time]) do
+            tests.each do |test|
+              lineno = get_source_location(test).last
+              xml.testcase(:name => test.name, :lineno => lineno, :classname => suite, :assertions => test.assertions,
+                           :time => test.time, :file => file_path) do
+                xml << xml_message_for(test) unless test.passed?
+              end
             end
           end
         end
@@ -150,6 +173,7 @@ module Minitest
           result[:assertion_count] += test.assertions
           result[:test_count] += 1
           result[:time] += test.time
+          result[:timestamp] = Time.now.iso8601 if @timestamp_report
         end
         result
       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

@@ -48,7 +48,7 @@ module Minitest
 
       def record(test)
         super
-        return if test.skipped? && !@detailed_skip
+        return show if test.skipped? && !@detailed_skip
         if test.failure
           print "\e[0m\e[1000D\e[K"
           print_colored_status(test)
@@ -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.4.0'.freeze
+	VERSION = '1.5.0'.freeze
   end
 end

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

@@ -11,7 +11,7 @@
 <body>
 <div class="container">
     <div class="jumbotron">
-        <h3><small>Generated on <%= Time.current.strftime("%b %d, %Y at %H:%M %Z") %></small></h3>
+        <h3><small>Generated on <%= Time.now.strftime("%b %d, %Y at %H:%M %Z") %></small></h3>
         <h1>
             <%= title %></h1>
         <p>

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.4.0
+  version: 1.5.0
 platform: ruby
 authors:
 - Alexander Kern
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-10-06 00:00:00.000000000 Z
+date: 2022-01-15 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,26 +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.3.3
+signing_key:
 specification_version: 4
 summary: Create customizable Minitest output formats
-test_files:
-- test/fixtures/junit_filename_bug_example_test.rb
-- test/fixtures/mean_time_test.rb
-- test/fixtures/progress_detailed_skip_test.rb
-- test/fixtures/progress_test.rb
-- test/fixtures/sample_test.rb
-- test/fixtures/spec_test.rb
-- test/gallery/bad_test.rb
-- test/gallery/good_test.rb
-- test/integration/reporters/junit_reporter_test.rb
-- test/integration/reporters/mean_time_reporter_test.rb
-- test/integration/reporters/progress_reporter_test.rb
-- test/test_helper.rb
-- test/unit/minitest/extensible_backtrace_filter_test.rb
-- test/unit/minitest/junit_reporter_test.rb
-- test/unit/minitest/mean_time_reporter_unit_test.rb
-- test/unit/minitest/minitest_reporter_plugin_test.rb
-- test/unit/minitest/reporters_test.rb
-- test/unit/minitest/spec_reporter_test.rb
+test_files: []