minitest-reporters 1.2.0.beta2 → 1.2.0.beta3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 62f787588456a19e8d7f3c259964eb08bb800cb6
-  data.tar.gz: 57f0b830502a46809f639a0000ac6b465c94163f
+  metadata.gz: bf175a4bd5a5eb0b5d1869c69ea3be75678e0eb1
+  data.tar.gz: b54fba7927d3d3847672aa17389edb4a0f13f9c4
 SHA512:
-  metadata.gz: 850a2e0aa57a50ff1603f4d4aab3904aa57a93eef0d126080879ce8fdac468d1c2dc95df58d1844387af3d79d10398c93653212390b2c5731587a7d8a175ebc3
-  data.tar.gz: 7ee01ec3c6f529936d5d762c833980b45758d70b0b34321cb3f8a9a10dd2f48479c2ff674128d7c3cfc81faac330097e5c2eef494d6215b922712a65fd143097
+  metadata.gz: f35339a52b0c8b1eeec7d2eb1d2f5815700889fbc9e6235f5e53cafdf04a8776dbc6edd3587009670630a75ec2933b720d93fd91a53ef3d8653fc7342ccf5b71
+  data.tar.gz: 0c599e9cb7f21434ff2515027f31b473eccfa5acb912e912806b7bd60518e97e3b15c436c5c0cd8769a873cfe3e2585a183129e85a8dcf1e50e517bd4f624856

data/CHANGELOG.md

@@ -1,4 +1,11 @@
-### [v1.2.0.beta2](https://github.com/kern/minitest-reporters/compare/v1.2.0.beta1...v1.2.0.beta2)
+### [dev](https://github.com/kern/minitest-reporters/compare/v1.2.0.beta3...master)
+
+### [1.2.0.beta2](https://github.com/kern/minitest-reporters/compare/v1.2.0.beta2...v1.2.0.beta3)
+
+* junit reporter changed to support mintest >= 5.11 [#252](https://github.com/kern/minitest-reporters/pull/252) (contributed by [Kevinrob](https://github.com/Kevinrob))
+* all reporters changed to be compatible with minitest >= 5.11 (if not - report a bug ;)
+
+### [1.2.0.beta2](https://github.com/kern/minitest-reporters/compare/v1.2.0.beta1...v1.2.0.beta2)
 
 * fixed uninitialized time in junit reporter [#251](https://github.com/kern/minitest-reporters/issues/251)
 * format option added to progress reporter [#240](https://github.com/kern/minitest-reporters/pull/240) (contributed by [jorgesmu](https://github.com/jorgesmu))

data/lib/minitest/reporters/base_reporter.rb

@@ -63,6 +63,14 @@ module Minitest
         end
       end
 
+      def test_class(result)
+        if result.respond_to? :klass
+          result.klass
+        else
+          result.class
+        end
+      end
+
       def print_colored_status(test)
         if test.passed?
           print(green { pad_mark( result(test).to_s.upcase ) })

data/lib/minitest/reporters/default_reporter.rb

@@ -113,7 +113,7 @@ module Minitest
           puts
 
           slow_tests.each do |test|
-            puts "%.6fs %s" % [test.time, "#{test.name}##{test.class}"]
+            puts "%.6fs %s" % [test.time, "#{test.name}##{test_class(test)}"]
           end
         end
 
@@ -125,7 +125,7 @@ module Minitest
           puts
 
           slow_suites.each do |slow_suite|
-            puts "%.6fs %s" % [slow_suite[1], slow_suite[0]]
+            puts "%.6fs %s" % [slow_suite[1], test_class(slow_suite[0])]
           end
         end
 
@@ -201,12 +201,12 @@ module Minitest
 
         if test.skipped?
           if @detailed_skip
-            "Skipped:\n#{test.class}##{test.name} [#{location(e)}]:\n#{e.message}"
+            "Skipped:\n#{test_class(test)}##{test.name} [#{location(e)}]:\n#{e.message}"
           end
         elsif test.error?
-          "Error:\n#{test.class}##{test.name}:\n#{e.message}"
+          "Error:\n#{test_class(test)}##{test.name}:\n#{e.message}"
         else
-          "Failure:\n#{test.class}##{test.name} [#{test.failure.location}]\n#{e.class}: #{e.message}"
+          "Failure:\n#{test_class(test)}##{test.name} [#{test.failure.location}]\n#{e.class}: #{e.message}"
         end
       end
 

data/lib/minitest/reporters/html_reporter.rb

@@ -92,7 +92,7 @@ module Minitest
           erb_str = File.read(@erb_template)
           renderer = ERB.new(erb_str)
 
-          tests_by_suites = tests.group_by(&:class) # taken from the JUnit reporter
+          tests_by_suites = tests.group_by { |test| test_class(test) } # taken from the JUnit reporter
 
           suites = tests_by_suites.map do |suite, tests|
             suite_summary = summarize_suite(suite, tests)

data/lib/minitest/reporters/junit_reporter.rb

@@ -26,7 +26,9 @@ module Minitest
         super
 
         puts "Writing XML reports to #{@reports_path}"
-        suites = tests.group_by(&:class)
+        suites = tests.group_by { |test|
+          test_class(test)
+        }
 
         if @single_file
           xml = Builder::XmlMarkup.new(:indent => 2)
@@ -44,16 +46,24 @@ module Minitest
             xml.test_suites do
               parse_xml_for(xml, suite, tests)
             end
-            File.open(filename_for(suite), "w") { |file| file << @xml.target! }
+            File.open(filename_for(suite), "w") { |file| file << xml.target! }
           end
         end
       end
 
       private
 
+      def get_source_location(result)
+        if result.respond_to? :klass
+          result.source_location
+        else
+          result.method(result.name).source_location
+        end
+      end
+
       def parse_xml_for(xml, suite, tests)
         suite_result = analyze_suite(tests)
-        file_path = Pathname.new(tests.first.method(tests.first.name).source_location.first)
+        file_path = Pathname.new(get_source_location(tests.first).first)
         base_path = Pathname.new(@base_path)
         relative_path = file_path.relative_path_from(base_path)
 
@@ -62,7 +72,7 @@ module Minitest
                       :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 = test.method(test.name).source_location.last
+            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?

data/lib/minitest/reporters/progress_reporter.rb

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

data/lib/minitest/reporters/version.rb

@@ -1,5 +1,5 @@
 module Minitest
   module Reporters
-    VERSION = '1.2.0.beta2'
+    VERSION = '1.2.0.beta3'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: minitest-reporters
 version: !ruby/object:Gem::Version
-  version: 1.2.0.beta2
+  version: 1.2.0.beta3
 platform: ruby
 authors:
 - Alexander Kern
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-03 00:00:00.000000000 Z
+date: 2018-02-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest