minitest-reporters 1.3.0 → 1.3.1.beta1

data/test/fixtures/sample_test.rb

@@ -1,15 +1,15 @@
-Class.new(Minitest::Test) do
-  def test_success
-    assert true
-  end
-  def test_failure
-    assert false
-  end
-  def test_skip
-    skip('Skipping rope')
-  end
-  def test_error
-    raise 'An unexpected error'
-  end
-end
-
+Class.new(Minitest::Test) do
+  def test_success
+    assert true
+  end
+  def test_failure
+    assert false
+  end
+  def test_skip
+    skip('Skipping rope')
+  end
+  def test_error
+    raise 'An unexpected error'
+  end
+end
+

data/test/fixtures/spec_test.rb

@@ -1,13 +1,13 @@
-require 'minitest'
-require 'minitest/reporters'
-require 'minitest/autorun'
-
-Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new
-
-describe String do
-  describe '#length' do
-    it 'works' do
-      assert_equal 5, 'hello'.length
-    end
-  end
-end
+require 'minitest'
+require 'minitest/reporters'
+require 'minitest/autorun'
+
+Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new
+
+describe String do
+  describe '#length' do
+    it 'works' do
+      assert_equal 5, 'hello'.length
+    end
+  end
+end

data/test/gallery/bad_test.rb

@@ -1,25 +1,25 @@
-require_relative "../test_helper"
-
-module MinitestReportersTest
-  class BadTest < TestCase
-    def test_a
-      assert_equal 1, 2
-    end
-
-    def test_b
-      assert false # simple failure
-    end
-
-    def test_b
-      assert_equal "ab\nc", "ab\nd" # some nice diff
-    end
-
-    def test_boom
-      raise "A random exception"
-    end
-
-    def test_long_method_name
-      skip
-    end
-  end
-end
+require_relative "../test_helper"
+
+module MinitestReportersTest
+  class BadTest < TestCase
+    def test_a
+      assert_equal 1, 2
+    end
+
+    def test_b
+      assert false # simple failure
+    end
+
+    def test_b
+      assert_equal "ab\nc", "ab\nd" # some nice diff
+    end
+
+    def test_boom
+      raise "A random exception"
+    end
+
+    def test_long_method_name
+      skip
+    end
+  end
+end

data/test/gallery/good_test.rb

@@ -1,14 +1,14 @@
-require_relative "../test_helper"
-
-module MinitestReportersTest
-  class GoodTest < TestCase
-    def test_a
-      assert_equal 1, 1
-      assert 1
-    end
-
-    def test_b
-      assert_equal 2, 2
-    end
-  end
-end
+require_relative "../test_helper"
+
+module MinitestReportersTest
+  class GoodTest < TestCase
+    def test_a
+      assert_equal 1, 1
+      assert 1
+    end
+
+    def test_b
+      assert_equal 2, 2
+    end
+  end
+end

data/test/integration/reporters/junit_reporter_test.rb

@@ -1,12 +1,12 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class JUnitReporterTest < TestCase
-    def test_replaces_special_characters_for_filenames_and_doesnt_crash
-      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
-      test_filename = File.join(fixtures_directory, 'junit_filename_bug_example_test.rb')
-      output = `ruby #{test_filename} 2>&1`
-      refute_match 'No such file or directory', output
-    end
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class JUnitReporterTest < TestCase
+    def test_replaces_special_characters_for_filenames_and_doesnt_crash
+      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
+      test_filename = File.join(fixtures_directory, 'junit_filename_bug_example_test.rb')
+      output = `ruby #{test_filename} 2>&1`
+      refute_match 'No such file or directory', output
+    end
+  end
+end

data/test/integration/reporters/mean_time_reporter_test.rb

@@ -1,7 +1,7 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class MeanTimeReporterTest < TestCase
-
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class MeanTimeReporterTest < TestCase
+
+  end
+end

data/test/integration/reporters/progress_reporter_test.rb

@@ -1,34 +1,34 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class ProgressReporterTest < TestCase
-    def test_all_failures_are_displayed
-      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
-      test_filename = File.join(fixtures_directory, 'progress_test.rb')
-      output = `ruby #{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'
-    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 #{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'
-    end
-    def test_progress_works_with_filter_and_specs
-      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
-      test_filename = File.join(fixtures_directory, 'spec_test.rb')
-      output = `ruby #{test_filename} -n /length/ 2>&1`
-      refute_match '0 out of 0', output, 'Progress should not puts a warning'
-    end
-    def test_progress_works_with_strict_filter
-      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
-      test_filename = File.join(fixtures_directory, 'spec_test.rb')
-      output = `ruby #{test_filename} -n /^test_0001_works$/ 2>&1`
-      refute_match '0 out of 0', output, 'Progress should not puts a warning'
-    end
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class ProgressReporterTest < TestCase
+    def test_all_failures_are_displayed
+      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
+      test_filename = File.join(fixtures_directory, 'progress_test.rb')
+      output = `ruby #{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'
+    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 #{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'
+    end
+    def test_progress_works_with_filter_and_specs
+      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
+      test_filename = File.join(fixtures_directory, 'spec_test.rb')
+      output = `ruby #{test_filename} -n /length/ 2>&1`
+      refute_match '0 out of 0', output, 'Progress should not puts a warning'
+    end
+    def test_progress_works_with_strict_filter
+      fixtures_directory = File.expand_path('../../../fixtures', __FILE__)
+      test_filename = File.join(fixtures_directory, 'spec_test.rb')
+      output = `ruby #{test_filename} -n /^test_0001_works$/ 2>&1`
+      refute_match '0 out of 0', output, 'Progress should not puts a warning'
+    end
+  end
+end

data/test/test_helper.rb

@@ -1,22 +1,22 @@
-require "bundler/setup"
-require "minitest/autorun"
-require "minitest/reporters"
-
-ENV['MINITEST_REPORTERS_MONO'] = 'yes'
-module MinitestReportersTest
-  class TestCase < Minitest::Test
-  end
-end
-
-# Testing the built-in reporters using automated unit testing would be extremely
-# brittle. Consequently, there are no unit tests for them.  If you'd like to run
-# all the reporters sequentially on a fake test suite, run `rake gallery`.
-
-if ENV["REPORTER"] == "Pride"
-  require "minitest/pride"
-elsif ENV["REPORTER"]
-  reporter_klass = Minitest::Reporters.const_get(ENV["REPORTER"])
-  Minitest::Reporters.use!(reporter_klass.new)
-else
-  Minitest::Reporters.use!(Minitest::Reporters::DefaultReporter.new)
-end
+require "bundler/setup"
+require "minitest/autorun"
+require "minitest/reporters"
+
+ENV['MINITEST_REPORTERS_MONO'] = 'yes'
+module MinitestReportersTest
+  class TestCase < Minitest::Test
+  end
+end
+
+# Testing the built-in reporters using automated unit testing would be extremely
+# brittle. Consequently, there are no unit tests for them.  If you'd like to run
+# all the reporters sequentially on a fake test suite, run `rake gallery`.
+
+if ENV["REPORTER"] == "Pride"
+  require "minitest/pride"
+elsif ENV["REPORTER"]
+  reporter_klass = Minitest::Reporters.const_get(ENV["REPORTER"])
+  Minitest::Reporters.use!(reporter_klass.new)
+else
+  Minitest::Reporters.use!(Minitest::Reporters::DefaultReporter.new)
+end

data/test/unit/minitest/extensible_backtrace_filter_test.rb

@@ -1,42 +1,42 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class ExtensibleBacktraceFilterTest < TestCase
-    def setup
-      @default_filter = Minitest::ExtensibleBacktraceFilter.default_filter
-      @filter = Minitest::ExtensibleBacktraceFilter.new
-      @backtrace = ["foo", "bar", "baz"]
-    end
-
-    def test_adding_filters
-      @filter.add_filter(/foo/)
-      assert @filter.filters?("foo")
-      refute @filter.filters?("baz")
-    end
-
-    def test_filter_backtrace_when_first_line_is_filtered
-      @filter.add_filter(/foo/)
-      assert_equal ["bar", "baz"], @filter.filter(@backtrace)
-    end
-
-    def test_filter_backtrace_when_middle_line_is_filtered
-      @filter.add_filter(/bar/)
-      assert_equal ["foo"], @filter.filter(@backtrace)
-    end
-
-    def test_filter_backtrace_when_all_lines_are_filtered
-      @filter.add_filter(/./)
-      assert_equal ["foo", "bar", "baz"], @filter.filter(@backtrace)
-    end
-
-    def test_default_filter
-      assert @default_filter.filters?("lib/minitest")
-      assert @default_filter.filters?("lib/minitest/reporters")
-      refute @default_filter.filters?("lib/my_gem")
-    end
-
-    def test_nil_backtrace
-      assert_equal [], @filter.filter(nil)
-    end
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class ExtensibleBacktraceFilterTest < TestCase
+    def setup
+      @default_filter = Minitest::ExtensibleBacktraceFilter.default_filter
+      @filter = Minitest::ExtensibleBacktraceFilter.new
+      @backtrace = ["foo", "bar", "baz"]
+    end
+
+    def test_adding_filters
+      @filter.add_filter(/foo/)
+      assert @filter.filters?("foo")
+      refute @filter.filters?("baz")
+    end
+
+    def test_filter_backtrace_when_first_line_is_filtered
+      @filter.add_filter(/foo/)
+      assert_equal ["bar", "baz"], @filter.filter(@backtrace)
+    end
+
+    def test_filter_backtrace_when_middle_line_is_filtered
+      @filter.add_filter(/bar/)
+      assert_equal ["foo"], @filter.filter(@backtrace)
+    end
+
+    def test_filter_backtrace_when_all_lines_are_filtered
+      @filter.add_filter(/./)
+      assert_equal ["foo", "bar", "baz"], @filter.filter(@backtrace)
+    end
+
+    def test_default_filter
+      assert @default_filter.filters?("lib/minitest")
+      assert @default_filter.filters?("lib/minitest/reporters")
+      refute @default_filter.filters?("lib/my_gem")
+    end
+
+    def test_nil_backtrace
+      assert_equal [], @filter.filter(nil)
+    end
+  end
+end

data/test/unit/minitest/mean_time_reporter_unit_test.rb

@@ -1,152 +1,152 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class MeanTimeReporterUnitTest < Minitest::Test
-    def setup
-      configure_report_paths
-      previous_test_run
-    end
-
-    def teardown
-      File.delete(@previous_run_path) if File.exist?(@previous_run_path)
-      File.delete(@report_file_path) if File.exist?(@report_file_path)
-    end
-
-    def test_sorts_avg_numerically
-      report_output = generate_report(:avg)
-
-      expected_order = [
-        'AVG_SLOW',
-        'MAX_SLOW',
-        'MIN_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'MAX_FAST',
-        'AVG_FAST'
-      ]
-      verify_result_order(report_output, expected_order)
-    end
-
-    def test_sorts_min_numerically
-      report_output = generate_report(:min)
-
-      expected_order = [
-        'MIN_SLOW',
-        'AVG_SLOW',
-        'MAX_SLOW',
-        'MIDDLE',
-        'MAX_FAST',
-        'AVG_FAST',
-        'MIN_FAST'
-      ]
-      verify_result_order(report_output, expected_order)
-    end
-
-    def test_sorts_max_numerically
-      report_output = generate_report(:max)
-
-      expected_order = [
-        'MAX_SLOW',
-        'AVG_SLOW',
-        'MIN_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'AVG_FAST',
-        'MAX_FAST'
-      ]
-      verify_result_order(report_output, expected_order)
-    end
-
-    def test_sorts_last_numerically
-      report_output = generate_report(:last)
-
-      expected_order = [
-        'AVG_SLOW',
-        'MIN_SLOW',
-        'MAX_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'MAX_FAST',
-        'AVG_FAST'
-      ]
-      verify_result_order(report_output, expected_order)
-    end
-
-    private
-
-    def simulate_suite_runtime(suite_name, run_time)
-      test_suite = Minitest::Test.new(suite_name)
-      base_clock_time = Minitest::Reporters.clock_time
-      Minitest::Reporters.stub(:clock_time, base_clock_time - run_time) do
-        @reporter.before_suite(test_suite)
-      end
-      @reporter.after_suite(test_suite)
-    end
-
-    def previous_test_run
-      @reporter = Minitest::Reporters::MeanTimeReporter.new(
-        previous_runs_filename: @previous_run_path,
-        report_filename: @report_file_path
-      )
-
-      simulate_suite_runtime('MIDDLE', 5.0)
-      simulate_suite_runtime('MIN_FAST', 0.5)
-      simulate_suite_runtime('MIN_SLOW', 10.5)
-      simulate_suite_runtime('MAX_FAST', 1.2)
-      simulate_suite_runtime('MAX_SLOW', 16.3)
-      simulate_suite_runtime('AVG_FAST', 1.3)
-      simulate_suite_runtime('AVG_SLOW', 10.2)
-      @reporter.tests << Minitest::Test.new('Final')
-      # Generate a "previous" run
-      @reporter.io = StringIO.new
-      @reporter.start
-      @reporter.report
-    end
-
-    def configure_report_paths
-      previous_runs_file = Tempfile.new('minitest-mean-time-previous-runs')
-      previous_runs_file.close
-      @previous_run_path = previous_runs_file.path
-      previous_runs_file.delete
-      report_file = Tempfile.new('minitest-mean-time-report')
-      report_file.close
-      @report_file_path = report_file.path
-      report_file.delete
-    end
-
-    def generate_report(sort_column)
-      # Reset the reporter for the test run
-      @reporter = Minitest::Reporters::MeanTimeReporter.new(
-        previous_runs_filename: @previous_run_path,
-        report_filename: @report_file_path,
-        sort_column: sort_column
-      )
-      simulate_suite_runtime('MIDDLE', 5.0)
-      simulate_suite_runtime('MIN_FAST', 3.5)
-      simulate_suite_runtime('MIN_SLOW', 10.5)
-      simulate_suite_runtime('MAX_FAST', 0.9)
-      simulate_suite_runtime('MAX_SLOW', 6.3)
-      simulate_suite_runtime('AVG_FAST', 0.65)
-      simulate_suite_runtime('AVG_SLOW', 14.2)
-      @reporter.tests << Minitest::Test.new('Final')
-
-      report_output = StringIO.new
-      @reporter.io = report_output
-      @reporter.start
-      @reporter.report
-      report_output
-    end
-
-    def verify_result_order(report_output, expected_order)
-      report_output.rewind
-      test_lines = report_output.read.split("\n")
-      test_lines.select! { |line| line.start_with?('Avg:') }
-
-      # Exclude the final placeholder 0 second test from assertions
-      test_lines.reject! { |line| line.end_with?('Minitest::Test') }
-      actual_order = test_lines.map { |line| line.gsub(/.*Description: /, '') }
-
-      assert_equal(expected_order, actual_order, "\n#{test_lines.join("\n")}")
-    end
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class MeanTimeReporterUnitTest < Minitest::Test
+    def setup
+      configure_report_paths
+      previous_test_run
+    end
+
+    def teardown
+      File.delete(@previous_run_path) if File.exist?(@previous_run_path)
+      File.delete(@report_file_path) if File.exist?(@report_file_path)
+    end
+
+    def test_sorts_avg_numerically
+      report_output = generate_report(:avg)
+
+      expected_order = [
+        'AVG_SLOW',
+        'MAX_SLOW',
+        'MIN_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'MAX_FAST',
+        'AVG_FAST'
+      ]
+      verify_result_order(report_output, expected_order)
+    end
+
+    def test_sorts_min_numerically
+      report_output = generate_report(:min)
+
+      expected_order = [
+        'MIN_SLOW',
+        'AVG_SLOW',
+        'MAX_SLOW',
+        'MIDDLE',
+        'MAX_FAST',
+        'AVG_FAST',
+        'MIN_FAST'
+      ]
+      verify_result_order(report_output, expected_order)
+    end
+
+    def test_sorts_max_numerically
+      report_output = generate_report(:max)
+
+      expected_order = [
+        'MAX_SLOW',
+        'AVG_SLOW',
+        'MIN_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'AVG_FAST',
+        'MAX_FAST'
+      ]
+      verify_result_order(report_output, expected_order)
+    end
+
+    def test_sorts_last_numerically
+      report_output = generate_report(:last)
+
+      expected_order = [
+        'AVG_SLOW',
+        'MIN_SLOW',
+        'MAX_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'MAX_FAST',
+        'AVG_FAST'
+      ]
+      verify_result_order(report_output, expected_order)
+    end
+
+    private
+
+    def simulate_suite_runtime(suite_name, run_time)
+      test_suite = Minitest::Test.new(suite_name)
+      base_clock_time = Minitest::Reporters.clock_time
+      Minitest::Reporters.stub(:clock_time, base_clock_time - run_time) do
+        @reporter.before_suite(test_suite)
+      end
+      @reporter.after_suite(test_suite)
+    end
+
+    def previous_test_run
+      @reporter = Minitest::Reporters::MeanTimeReporter.new(
+        previous_runs_filename: @previous_run_path,
+        report_filename: @report_file_path
+      )
+
+      simulate_suite_runtime('MIDDLE', 5.0)
+      simulate_suite_runtime('MIN_FAST', 0.5)
+      simulate_suite_runtime('MIN_SLOW', 10.5)
+      simulate_suite_runtime('MAX_FAST', 1.2)
+      simulate_suite_runtime('MAX_SLOW', 16.3)
+      simulate_suite_runtime('AVG_FAST', 1.3)
+      simulate_suite_runtime('AVG_SLOW', 10.2)
+      @reporter.tests << Minitest::Test.new('Final')
+      # Generate a "previous" run
+      @reporter.io = StringIO.new
+      @reporter.start
+      @reporter.report
+    end
+
+    def configure_report_paths
+      previous_runs_file = Tempfile.new('minitest-mean-time-previous-runs')
+      previous_runs_file.close
+      @previous_run_path = previous_runs_file.path
+      previous_runs_file.delete
+      report_file = Tempfile.new('minitest-mean-time-report')
+      report_file.close
+      @report_file_path = report_file.path
+      report_file.delete
+    end
+
+    def generate_report(sort_column)
+      # Reset the reporter for the test run
+      @reporter = Minitest::Reporters::MeanTimeReporter.new(
+        previous_runs_filename: @previous_run_path,
+        report_filename: @report_file_path,
+        sort_column: sort_column
+      )
+      simulate_suite_runtime('MIDDLE', 5.0)
+      simulate_suite_runtime('MIN_FAST', 3.5)
+      simulate_suite_runtime('MIN_SLOW', 10.5)
+      simulate_suite_runtime('MAX_FAST', 0.9)
+      simulate_suite_runtime('MAX_SLOW', 6.3)
+      simulate_suite_runtime('AVG_FAST', 0.65)
+      simulate_suite_runtime('AVG_SLOW', 14.2)
+      @reporter.tests << Minitest::Test.new('Final')
+
+      report_output = StringIO.new
+      @reporter.io = report_output
+      @reporter.start
+      @reporter.report
+      report_output
+    end
+
+    def verify_result_order(report_output, expected_order)
+      report_output.rewind
+      test_lines = report_output.read.split("\n")
+      test_lines.select! { |line| line.start_with?('Avg:') }
+
+      # Exclude the final placeholder 0 second test from assertions
+      test_lines.reject! { |line| line.end_with?('Minitest::Test') }
+      actual_order = test_lines.map { |line| line.gsub(/.*Description: /, '') }
+
+      assert_equal(expected_order, actual_order, "\n#{test_lines.join("\n")}")
+    end
+  end
+end