minitest-reporters 1.3.6 → 1.3.7

data/lib/minitest/reporters/html_reporter.rb

@@ -1,224 +1,224 @@
-require 'builder'
-require 'fileutils'
-require 'erb'
-
-module Minitest
-  module Reporters
-    # A reporter for generating HTML test reports
-    # This is recommended to be used with a CI server, where the report is kept as an artifact and is accessible via
-    # a shared link
-    #
-    # The reporter sorts the results alphabetically and then by results
-    # so that failing and skipped tests are at the top.
-    #
-    # When using Minitest Specs, the number prefix is dropped from the name of the test so that it reads well
-    #
-    # On each test run all files in the reports directory are deleted, this prevents a build up of old reports
-    #
-    # The report is generated using ERB. A custom ERB template can be provided but it is not required
-    # The default ERB template uses JQuery and Bootstrap, both of these are included by referencing the CDN sites
-    class HtmlReporter < BaseReporter
-      # The title of the report
-      attr_reader :title
-
-      # The number of tests that passed
-      def passes
-        count - failures - errors - skips
-      end
-
-      # The percentage of tests that passed, calculated in a way that avoids rounding errors
-      def percent_passes
-        100 - percent_skipps - percent_errors_failures
-      end
-
-      # The percentage of tests that were skipped
-      def percent_skipps
-        (skips / count.to_f * 100).to_i
-      end
-
-      # The percentage of tests that failed
-      def percent_errors_failures
-        ((errors + failures) / count.to_f * 100).to_i
-      end
-
-      # Trims off the number prefix on test names when using Minitest Specs
-      def friendly_name(test)
-        groups = test.name.scan(/(test_\d+_)(.*)/i)
-        return test.name if groups.empty?
-        "it #{groups[0][1]}"
-      end
-
-      # The constructor takes a hash, and uses the following keys:
-      # :title - the title that will be used in the report, defaults to 'Test Results'
-      # :reports_dir - the directory the reports should be written to, defaults to 'test/html_reports'
-      # :erb_template - the path to a custom ERB template, defaults to the supplied ERB template
-      # :mode - Useful for debugging, :terse suppresses errors and is the default, :verbose lets errors bubble up
-      # :output_filename - the report's filename, defaults to 'index.html'
-      def initialize(args = {})
-        super({})
-
-        defaults = {
-          :title           => 'Test Results',
-          :erb_template    => "#{File.dirname(__FILE__)}/../templates/index.html.erb",
-          :reports_dir     => 'test/html_reports',
-          :mode            => :safe,
-          :output_filename => 'index.html',
-        }
-
-        settings = defaults.merge(args)
-
-        @mode = settings[:mode]
-        @title = settings[:title]
-        @erb_template = settings[:erb_template]
-        @output_filename = settings[:output_filename]
-        reports_dir = settings[:reports_dir]
-
-        @reports_path = File.absolute_path(reports_dir)
-      end
-
-      def start
-        super
-
-        puts "Emptying #{@reports_path}"
-        FileUtils.mkdir_p(@reports_path)
-        File.delete(html_file) if File.exist?(html_file)
-      end
-
-      # Called by the framework to generate the report
-      def report
-        super
-
-        begin
-          puts "Writing HTML reports to #{@reports_path}"
-          erb_str = File.read(@erb_template)
-          renderer = ERB.new(erb_str)
-
-          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)
-            suite_summary[:tests] = tests.sort { |a, b| compare_tests(a, b) }
-            suite_summary
-          end
-
-          suites.sort! { |a, b| compare_suites(a, b) }
-
-          result = renderer.result(binding)
-          File.open(html_file, 'w') do |f|
-            f.write(result)
-          end
-
-        # rubocop:disable Lint/RescueException
-        rescue Exception => e
-          puts 'There was an error writing the HTML report'
-          puts 'This may have been caused by cancelling the test run'
-          puts 'Use mode => :verbose in the HTML reporters constructor to see more detail' if @mode == :terse
-          puts 'Use mode => :terse in the HTML reporters constructor to see less detail' if @mode != :terse
-          raise e if @mode != :terse
-        end
-        # rubocop:enable Lint/RescueException
-      end
-
-      private
-
-      def html_file
-        "#{@reports_path}/#{@output_filename}"
-      end
-
-      def compare_suites_by_name(suite_a, suite_b)
-        suite_a[:name] <=> suite_b[:name]
-      end
-
-      def compare_tests_by_name(test_a, test_b)
-        friendly_name(test_a) <=> friendly_name(test_b)
-      end
-
-      # Test suites are first ordered by evaluating the results of the tests, then by test suite name
-      # Test suites which have failing tests are given highest order
-      # Tests suites which have skipped tests are given second highest priority
-      def compare_suites(suite_a, suite_b)
-        return compare_suites_by_name(suite_a, suite_b) if suite_a[:has_errors_or_failures] && suite_b[:has_errors_or_failures]
-        return -1 if suite_a[:has_errors_or_failures] && !suite_b[:has_errors_or_failures]
-        return 1 if !suite_a[:has_errors_or_failures] && suite_b[:has_errors_or_failures]
-
-        return compare_suites_by_name(suite_a, suite_b) if suite_a[:has_skipps] && suite_b[:has_skipps]
-        return -1 if suite_a[:has_skipps] && !suite_b[:has_skipps]
-        return 1 if !suite_a[:has_skipps] && suite_b[:has_skipps]
-
-        compare_suites_by_name(suite_a, suite_b)
-      end
-
-      # Tests are first ordered by evaluating the results of the tests, then by tests names
-      # Tess which fail are given highest order
-      # Tests which are skipped are given second highest priority
-      def compare_tests(test_a, test_b)
-        return compare_tests_by_name(test_a, test_b) if test_fail_or_error?(test_a) && test_fail_or_error?(test_b)
-
-        return -1 if test_fail_or_error?(test_a) && !test_fail_or_error?(test_b)
-        return 1 if !test_fail_or_error?(test_a) && test_fail_or_error?(test_b)
-
-        return compare_tests_by_name(test_a, test_b) if test_a.skipped? && test_b.skipped?
-        return -1 if test_a.skipped? && !test_b.skipped?
-        return 1 if !test_a.skipped? && test_b.skipped?
-
-        compare_tests_by_name(test_a, test_b)
-      end
-
-      def test_fail_or_error?(test)
-        test.error? || test.failure
-      end
-
-      # based on analyze_suite from the JUnit reporter
-      def summarize_suite(suite, tests)
-        summary = Hash.new(0)
-        summary[:name] = suite.to_s
-        tests.each do |test|
-          summary[:"#{result(test)}_count"] += 1
-          summary[:assertion_count] += test.assertions
-          summary[:test_count] += 1
-          summary[:time] += test.time
-        end
-        summary[:has_errors_or_failures] = (summary[:fail_count] + summary[:error_count]) > 0
-        summary[:has_skipps] = summary[:skip_count] > 0
-        summary
-      end
-
-      # based on message_for(test) from the JUnit reporter
-      def message_for(test)
-        suite = test.class
-        name = test.name
-        e = test.failure
-
-        if test.passed?
-          nil
-        elsif test.skipped?
-          "Skipped:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
-        elsif test.failure
-          "Failure:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
-        elsif test.error?
-          "Error:\n#{name}(#{suite}):\n#{e.message}"
-        end
-      end
-
-      # taken from the JUnit reporter
-      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
-        end
-        last_before_assertion.sub(/:in .*$/, '')
-      end
-
-      def total_time_to_hms
-        return ('%.2fs' % total_time) if total_time < 1
-
-        hours = (total_time / (60 * 60)).round
-        minutes = ((total_time / 60) % 60).round.to_s.rjust(2, '0')
-        seconds = (total_time % 60).round.to_s.rjust(2, '0')
-
-        "#{hours}h#{minutes}m#{seconds}s"
-      end
-    end
-  end
-end
+require 'builder'
+require 'fileutils'
+require 'erb'
+
+module Minitest
+  module Reporters
+    # A reporter for generating HTML test reports
+    # This is recommended to be used with a CI server, where the report is kept as an artifact and is accessible via
+    # a shared link
+    #
+    # The reporter sorts the results alphabetically and then by results
+    # so that failing and skipped tests are at the top.
+    #
+    # When using Minitest Specs, the number prefix is dropped from the name of the test so that it reads well
+    #
+    # On each test run all files in the reports directory are deleted, this prevents a build up of old reports
+    #
+    # The report is generated using ERB. A custom ERB template can be provided but it is not required
+    # The default ERB template uses JQuery and Bootstrap, both of these are included by referencing the CDN sites
+    class HtmlReporter < BaseReporter
+      # The title of the report
+      attr_reader :title
+
+      # The number of tests that passed
+      def passes
+        count - failures - errors - skips
+      end
+
+      # The percentage of tests that passed, calculated in a way that avoids rounding errors
+      def percent_passes
+        100 - percent_skipps - percent_errors_failures
+      end
+
+      # The percentage of tests that were skipped
+      def percent_skipps
+        (skips / count.to_f * 100).to_i
+      end
+
+      # The percentage of tests that failed
+      def percent_errors_failures
+        ((errors + failures) / count.to_f * 100).to_i
+      end
+
+      # Trims off the number prefix on test names when using Minitest Specs
+      def friendly_name(test)
+        groups = test.name.scan(/(test_\d+_)(.*)/i)
+        return test.name if groups.empty?
+        "it #{groups[0][1]}"
+      end
+
+      # The constructor takes a hash, and uses the following keys:
+      # :title - the title that will be used in the report, defaults to 'Test Results'
+      # :reports_dir - the directory the reports should be written to, defaults to 'test/html_reports'
+      # :erb_template - the path to a custom ERB template, defaults to the supplied ERB template
+      # :mode - Useful for debugging, :terse suppresses errors and is the default, :verbose lets errors bubble up
+      # :output_filename - the report's filename, defaults to 'index.html'
+      def initialize(args = {})
+        super({})
+
+        defaults = {
+          :title           => 'Test Results',
+          :erb_template    => "#{File.dirname(__FILE__)}/../templates/index.html.erb",
+          :reports_dir     => 'test/html_reports',
+          :mode            => :safe,
+          :output_filename => 'index.html',
+        }
+
+        settings = defaults.merge(args)
+
+        @mode = settings[:mode]
+        @title = settings[:title]
+        @erb_template = settings[:erb_template]
+        @output_filename = settings[:output_filename]
+        reports_dir = settings[:reports_dir]
+
+        @reports_path = File.absolute_path(reports_dir)
+      end
+
+      def start
+        super
+
+        puts "Emptying #{@reports_path}"
+        FileUtils.mkdir_p(@reports_path)
+        File.delete(html_file) if File.exist?(html_file)
+      end
+
+      # Called by the framework to generate the report
+      def report
+        super
+
+        begin
+          puts "Writing HTML reports to #{@reports_path}"
+          erb_str = File.read(@erb_template)
+          renderer = ERB.new(erb_str)
+
+          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)
+            suite_summary[:tests] = tests.sort { |a, b| compare_tests(a, b) }
+            suite_summary
+          end
+
+          suites.sort! { |a, b| compare_suites(a, b) }
+
+          result = renderer.result(binding)
+          File.open(html_file, 'w') do |f|
+            f.write(result)
+          end
+
+        # rubocop:disable Lint/RescueException
+        rescue Exception => e
+          puts 'There was an error writing the HTML report'
+          puts 'This may have been caused by cancelling the test run'
+          puts 'Use mode => :verbose in the HTML reporters constructor to see more detail' if @mode == :terse
+          puts 'Use mode => :terse in the HTML reporters constructor to see less detail' if @mode != :terse
+          raise e if @mode != :terse
+        end
+        # rubocop:enable Lint/RescueException
+      end
+
+      private
+
+      def html_file
+        "#{@reports_path}/#{@output_filename}"
+      end
+
+      def compare_suites_by_name(suite_a, suite_b)
+        suite_a[:name] <=> suite_b[:name]
+      end
+
+      def compare_tests_by_name(test_a, test_b)
+        friendly_name(test_a) <=> friendly_name(test_b)
+      end
+
+      # Test suites are first ordered by evaluating the results of the tests, then by test suite name
+      # Test suites which have failing tests are given highest order
+      # Tests suites which have skipped tests are given second highest priority
+      def compare_suites(suite_a, suite_b)
+        return compare_suites_by_name(suite_a, suite_b) if suite_a[:has_errors_or_failures] && suite_b[:has_errors_or_failures]
+        return -1 if suite_a[:has_errors_or_failures] && !suite_b[:has_errors_or_failures]
+        return 1 if !suite_a[:has_errors_or_failures] && suite_b[:has_errors_or_failures]
+
+        return compare_suites_by_name(suite_a, suite_b) if suite_a[:has_skipps] && suite_b[:has_skipps]
+        return -1 if suite_a[:has_skipps] && !suite_b[:has_skipps]
+        return 1 if !suite_a[:has_skipps] && suite_b[:has_skipps]
+
+        compare_suites_by_name(suite_a, suite_b)
+      end
+
+      # Tests are first ordered by evaluating the results of the tests, then by tests names
+      # Tess which fail are given highest order
+      # Tests which are skipped are given second highest priority
+      def compare_tests(test_a, test_b)
+        return compare_tests_by_name(test_a, test_b) if test_fail_or_error?(test_a) && test_fail_or_error?(test_b)
+
+        return -1 if test_fail_or_error?(test_a) && !test_fail_or_error?(test_b)
+        return 1 if !test_fail_or_error?(test_a) && test_fail_or_error?(test_b)
+
+        return compare_tests_by_name(test_a, test_b) if test_a.skipped? && test_b.skipped?
+        return -1 if test_a.skipped? && !test_b.skipped?
+        return 1 if !test_a.skipped? && test_b.skipped?
+
+        compare_tests_by_name(test_a, test_b)
+      end
+
+      def test_fail_or_error?(test)
+        test.error? || test.failure
+      end
+
+      # based on analyze_suite from the JUnit reporter
+      def summarize_suite(suite, tests)
+        summary = Hash.new(0)
+        summary[:name] = suite.to_s
+        tests.each do |test|
+          summary[:"#{result(test)}_count"] += 1
+          summary[:assertion_count] += test.assertions
+          summary[:test_count] += 1
+          summary[:time] += test.time
+        end
+        summary[:has_errors_or_failures] = (summary[:fail_count] + summary[:error_count]) > 0
+        summary[:has_skipps] = summary[:skip_count] > 0
+        summary
+      end
+
+      # based on message_for(test) from the JUnit reporter
+      def message_for(test)
+        suite = test.class
+        name = test.name
+        e = test.failure
+
+        if test.passed?
+          nil
+        elsif test.skipped?
+          "Skipped:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
+        elsif test.failure
+          "Failure:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
+        elsif test.error?
+          "Error:\n#{name}(#{suite}):\n#{e.message}"
+        end
+      end
+
+      # taken from the JUnit reporter
+      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
+        end
+        last_before_assertion.sub(/:in .*$/, '')
+      end
+
+      def total_time_to_hms
+        return ('%.2fs' % total_time) if total_time < 1
+
+        hours = (total_time / (60 * 60)).round
+        minutes = ((total_time / 60) % 60).round.to_s.rjust(2, '0')
+        seconds = (total_time % 60).round.to_s.rjust(2, '0')
+
+        "#{hours}h#{minutes}m#{seconds}s"
+      end
+    end
+  end
+end

data/lib/minitest/reporters/junit_reporter.rb

@@ -1,164 +1,168 @@
-require 'builder'
-require 'fileutils'
-require 'pathname'
-module Minitest
-  module Reporters
-    # A reporter for writing JUnit test reports
-    # Intended for easy integration with CI servers - tested on JetBrains TeamCity
-    #
-    # Inspired by ci_reporter (see https://github.com/nicksieger/ci_reporter)
-    # 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
-      def initialize(reports_dir = "test/reports", empty = true, options = {})
-        super({})
-        @reports_path = File.absolute_path(reports_dir)
-        @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
-      end
-
-      def report
-        super
-
-        puts "Writing XML reports to #{@reports_path}"
-        suites = tests.group_by { |test|
-          test_class(test)
-        }
-
-        if @single_file
-          xml = Builder::XmlMarkup.new(:indent => 2)
-          xml.instruct!
-          xml.testsuites do
-            suites.each do |suite, tests|
-              parse_xml_for(xml, suite, tests)
-            end
-          end
-          File.open(filename_for('minitest'), "w") { |file| file << xml.target! }
-        else
-          suites.each do |suite, tests|
-            xml = Builder::XmlMarkup.new(:indent => 2)
-            xml.instruct!
-            xml.testsuites do
-              parse_xml_for(xml, suite, tests)
-            end
-            File.open(filename_for(suite), "w") { |file| file << xml.target! }
-          end
-        end
-      end
-
-      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
-      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 = 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?
-            end
-          end
-        end
-      end
-
-      def xml_message_for(test)
-        # This is a trick lifted from ci_reporter
-        xml = Builder::XmlMarkup.new(:indent => 2, :margin => 2)
-
-        def xml.trunc!(txt)
-          txt.sub(/\n.*/m, '...')
-        end
-
-        e = test.failure
-
-        if test.skipped?
-          xml.skipped(:type => test.name)
-        elsif test.error?
-          xml.error(:type => test.name, :message => xml.trunc!(e.message)) do
-            xml.text!(message_for(test))
-          end
-        elsif test.failure
-          xml.failure(:type => test.name, :message => xml.trunc!(e.message)) do
-            xml.text!(message_for(test))
-          end
-        end
-      end
-
-      def message_for(test)
-        suite = test.class
-        name = test.name
-        e = test.failure
-
-        if test.passed?
-          nil
-        elsif test.skipped?
-          "Skipped:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
-        elsif test.failure
-          "Failure:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
-        elsif test.error?
-          "Error:\n#{name}(#{suite}):\n#{e.message}"
-        end
-      end
-
-      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
-        end
-        last_before_assertion.sub(/:in .*$/, '')
-      end
-
-      def analyze_suite(tests)
-        result = Hash.new(0)
-        result[:time] = 0
-        tests.each do |test|
-          result[:"#{result(test)}_count"] += 1
-          result[:assertion_count] += test.assertions
-          result[:test_count] += 1
-          result[:time] += test.time
-        end
-        result
-      end
-
-      def filename_for(suite)
-        file_counter = 0
-        # restrict max filename length, to be kind to filesystems
-        suite_name = suite.to_s[0..240].gsub(/[^a-zA-Z0-9]+/, '-')
-        filename = "TEST-#{suite_name}.xml"
-        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
-        end
-        File.join(@reports_path, filename)
-      end
-    end
-  end
-end
+require 'builder'
+require 'fileutils'
+require 'pathname'
+module Minitest
+  module Reporters
+    # A reporter for writing JUnit test reports
+    # Intended for easy integration with CI servers - tested on JetBrains TeamCity
+    #
+    # Inspired by ci_reporter (see https://github.com/nicksieger/ci_reporter)
+    # 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
+
+      attr_reader :reports_path
+
+      def initialize(reports_dir = DEFAULT_REPORTS_DIR, empty = true, options = {})
+        super({})
+        @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
+
+        if empty
+          puts "Emptying #{@reports_path}"
+          FileUtils.mkdir_p(@reports_path)
+          File.delete(*Dir.glob("#{@reports_path}/TEST-*.xml"))
+        end
+      end
+
+      def report
+        super
+
+        puts "Writing XML reports to #{@reports_path}"
+        suites = tests.group_by { |test|
+          test_class(test)
+        }
+
+        if @single_file
+          xml = Builder::XmlMarkup.new(:indent => 2)
+          xml.instruct!
+          xml.testsuites do
+            suites.each do |suite, tests|
+              parse_xml_for(xml, suite, tests)
+            end
+          end
+          File.open(filename_for('minitest'), "w") { |file| file << xml.target! }
+        else
+          suites.each do |suite, tests|
+            xml = Builder::XmlMarkup.new(:indent => 2)
+            xml.instruct!
+            xml.testsuites do
+              parse_xml_for(xml, suite, tests)
+            end
+            File.open(filename_for(suite), "w") { |file| file << xml.target! }
+          end
+        end
+      end
+
+      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
+      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 = 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?
+            end
+          end
+        end
+      end
+
+      def xml_message_for(test)
+        # This is a trick lifted from ci_reporter
+        xml = Builder::XmlMarkup.new(:indent => 2, :margin => 2)
+
+        def xml.trunc!(txt)
+          txt.sub(/\n.*/m, '...')
+        end
+
+        e = test.failure
+
+        if test.skipped?
+          xml.skipped(:type => test.name)
+        elsif test.error?
+          xml.error(:type => test.name, :message => xml.trunc!(e.message)) do
+            xml.text!(message_for(test))
+          end
+        elsif test.failure
+          xml.failure(:type => test.name, :message => xml.trunc!(e.message)) do
+            xml.text!(message_for(test))
+          end
+        end
+      end
+
+      def message_for(test)
+        suite = test.class
+        name = test.name
+        e = test.failure
+
+        if test.passed?
+          nil
+        elsif test.skipped?
+          "Skipped:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
+        elsif test.failure
+          "Failure:\n#{name}(#{suite}) [#{location(e)}]:\n#{e.message}\n"
+        elsif test.error?
+          "Error:\n#{name}(#{suite}):\n#{e.message}"
+        end
+      end
+
+      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
+        end
+        last_before_assertion.sub(/:in .*$/, '')
+      end
+
+      def analyze_suite(tests)
+        result = Hash.new(0)
+        result[:time] = 0
+        tests.each do |test|
+          result[:"#{result(test)}_count"] += 1
+          result[:assertion_count] += test.assertions
+          result[:test_count] += 1
+          result[:time] += test.time
+        end
+        result
+      end
+
+      def filename_for(suite)
+        file_counter = 0
+        # restrict max filename length, to be kind to filesystems
+        suite_name = suite.to_s[0..240].gsub(/[^a-zA-Z0-9]+/, '-')
+        filename = "TEST-#{suite_name}.xml"
+        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
+        end
+        File.join(@reports_path, filename)
+      end
+    end
+  end
+end