minitest-reporters 1.3.6 → 1.3.7

data/test/unit/minitest/mean_time_reporter_unit_test.rb

@@ -1,149 +1,149 @@
-require_relative '../../test_helper'
-
-module MinitestReportersTest
-  class MeanTimeReporterUnitTest < Minitest::Test
-    def setup
-      @test_data = []
-      @test_data << { name: 'MIDDLE',  prev_time: 5.0,  cur_time: 5.0 }
-      @test_data << { name: 'MIN_FAST', prev_time: 0.5,  cur_time: 3.5 }
-      @test_data << { name: 'MIN_SLOW', prev_time: 10.5, cur_time: 10.5 }
-      @test_data << { name: 'MAX_FAST', prev_time: 1.2,  cur_time: 0.9 }
-      @test_data << { name: 'MAX_SLOW', prev_time: 16.3, cur_time: 6.3 }
-      @test_data << { name: 'AVG_FAST', prev_time: 1.3,  cur_time: 0.65 }
-      @test_data << { name: 'AVG_SLOW', prev_time: 10.2, cur_time: 14.2 }
-      configure_report_paths
-    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_defaults
-      subject = Minitest::Reporters::MeanTimeReporter.new.send(:defaults)
-
-      expected_prefix = "#{Dir.tmpdir}#{File::Separator}"
-      assert_match expected_prefix, subject[:previous_runs_filename]
-      assert_match expected_prefix, subject[:report_filename]
-    end
-
-    def test_sorts_avg_numerically
-      prev_output = generate_report(:avg, :prev_time)
-      report_output = generate_report(:avg, :cur_time)
-
-      expected_order = [
-        'AVG_SLOW',
-        'MAX_SLOW',
-        'MIN_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'MAX_FAST',
-        'AVG_FAST'
-      ]
-      verify_result_order(report_output, expected_order, prev_output)
-    end
-
-    def test_sorts_min_numerically
-      prev_output = generate_report(:min, :prev_time)
-      report_output = generate_report(:min, :cur_time)
-
-      expected_order = [
-        'MIN_SLOW',
-        'AVG_SLOW',
-        'MAX_SLOW',
-        'MIDDLE',
-        'MAX_FAST',
-        'AVG_FAST',
-        'MIN_FAST'
-      ]
-      verify_result_order(report_output, expected_order, prev_output)
-    end
-
-    def test_sorts_max_numerically
-      prev_output = generate_report(:max, :prev_time)
-      report_output = generate_report(:max, :cur_time)
-
-      expected_order = [
-        'MAX_SLOW',
-        'AVG_SLOW',
-        'MIN_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'AVG_FAST',
-        'MAX_FAST'
-      ]
-      verify_result_order(report_output, expected_order, prev_output)
-    end
-
-    def test_sorts_last_numerically
-      prev_output = generate_report(:last, :prev_time)
-      report_output = generate_report(:last, :cur_time)
-
-      expected_order = [
-        'AVG_SLOW',
-        'MIN_SLOW',
-        'MAX_SLOW',
-        'MIDDLE',
-        'MIN_FAST',
-        'MAX_FAST',
-        'AVG_FAST'
-      ]
-      verify_result_order(report_output, expected_order, prev_output)
-    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
-      Minitest::Reporters.stub(:clock_time, base_clock_time) do
-        @reporter.after_suite(test_suite)
-      end
-    end
-
-    def configure_report_paths
-      @previous_run_path = File.expand_path("../minitest-mean-time-previous-runs",  File.realpath(__FILE__))
-      File.delete(@previous_run_path) if File.exist?(@previous_run_path)
-      @report_file_path = File.expand_path("../minitest-mean-time-report",  File.realpath(__FILE__))
-      File.delete(@report_file_path) if File.exist?(@report_file_path)
-    end
-
-    def generate_report(sort_column, time_name)
-      # 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
-      )
-      @test_data.each { |hash| simulate_suite_runtime(hash[:name], hash[time_name])}
-      @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, prev_output)
-      prev_lines = extract_test_lines(prev_output)
-      test_lines = extract_test_lines(report_output)
-
-      actual_order = test_lines.map { |line| line.gsub(/.*Description: /, '') }
-
-      assert_equal(expected_order, actual_order, "\nCurrent report:\n#{test_lines.join("\n")}\n\nPrevious report:\n#{prev_lines.join("\n")}")
-    end
-
-    def extract_test_lines(report_output)
-      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')}
-      test_lines
-    end
-  end
-end
+require_relative '../../test_helper'
+
+module MinitestReportersTest
+  class MeanTimeReporterUnitTest < Minitest::Test
+    def setup
+      @test_data = []
+      @test_data << { name: 'MIDDLE',  prev_time: 5.0,  cur_time: 5.0 }
+      @test_data << { name: 'MIN_FAST', prev_time: 0.5,  cur_time: 3.5 }
+      @test_data << { name: 'MIN_SLOW', prev_time: 10.5, cur_time: 10.5 }
+      @test_data << { name: 'MAX_FAST', prev_time: 1.2,  cur_time: 0.9 }
+      @test_data << { name: 'MAX_SLOW', prev_time: 16.3, cur_time: 6.3 }
+      @test_data << { name: 'AVG_FAST', prev_time: 1.3,  cur_time: 0.65 }
+      @test_data << { name: 'AVG_SLOW', prev_time: 10.2, cur_time: 14.2 }
+      configure_report_paths
+    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_defaults
+      subject = Minitest::Reporters::MeanTimeReporter.new.send(:defaults)
+
+      expected_prefix = "#{Dir.tmpdir}#{File::Separator}"
+      assert_match expected_prefix, subject[:previous_runs_filename]
+      assert_match expected_prefix, subject[:report_filename]
+    end
+
+    def test_sorts_avg_numerically
+      prev_output = generate_report(:avg, :prev_time)
+      report_output = generate_report(:avg, :cur_time)
+
+      expected_order = [
+        'AVG_SLOW',
+        'MAX_SLOW',
+        'MIN_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'MAX_FAST',
+        'AVG_FAST'
+      ]
+      verify_result_order(report_output, expected_order, prev_output)
+    end
+
+    def test_sorts_min_numerically
+      prev_output = generate_report(:min, :prev_time)
+      report_output = generate_report(:min, :cur_time)
+
+      expected_order = [
+        'MIN_SLOW',
+        'AVG_SLOW',
+        'MAX_SLOW',
+        'MIDDLE',
+        'MAX_FAST',
+        'AVG_FAST',
+        'MIN_FAST'
+      ]
+      verify_result_order(report_output, expected_order, prev_output)
+    end
+
+    def test_sorts_max_numerically
+      prev_output = generate_report(:max, :prev_time)
+      report_output = generate_report(:max, :cur_time)
+
+      expected_order = [
+        'MAX_SLOW',
+        'AVG_SLOW',
+        'MIN_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'AVG_FAST',
+        'MAX_FAST'
+      ]
+      verify_result_order(report_output, expected_order, prev_output)
+    end
+
+    def test_sorts_last_numerically
+      prev_output = generate_report(:last, :prev_time)
+      report_output = generate_report(:last, :cur_time)
+
+      expected_order = [
+        'AVG_SLOW',
+        'MIN_SLOW',
+        'MAX_SLOW',
+        'MIDDLE',
+        'MIN_FAST',
+        'MAX_FAST',
+        'AVG_FAST'
+      ]
+      verify_result_order(report_output, expected_order, prev_output)
+    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
+      Minitest::Reporters.stub(:clock_time, base_clock_time) do
+        @reporter.after_suite(test_suite)
+      end
+    end
+
+    def configure_report_paths
+      @previous_run_path = File.expand_path("../minitest-mean-time-previous-runs",  File.realpath(__FILE__))
+      File.delete(@previous_run_path) if File.exist?(@previous_run_path)
+      @report_file_path = File.expand_path("../minitest-mean-time-report",  File.realpath(__FILE__))
+      File.delete(@report_file_path) if File.exist?(@report_file_path)
+    end
+
+    def generate_report(sort_column, time_name)
+      # 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
+      )
+      @test_data.each { |hash| simulate_suite_runtime(hash[:name], hash[time_name])}
+      @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, prev_output)
+      prev_lines = extract_test_lines(prev_output)
+      test_lines = extract_test_lines(report_output)
+
+      actual_order = test_lines.map { |line| line.gsub(/.*Description: /, '') }
+
+      assert_equal(expected_order, actual_order, "\nCurrent report:\n#{test_lines.join("\n")}\n\nPrevious report:\n#{prev_lines.join("\n")}")
+    end
+
+    def extract_test_lines(report_output)
+      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')}
+      test_lines
+    end
+  end
+end

data/test/unit/minitest/minitest_reporter_plugin_test.rb

@@ -1,14 +1,14 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class MinitestReporterPluginTest < Minitest::Test
-    def test_delegates_io
-      reporter = Minitest::Reporters::DefaultReporter.new
-      io_handle = STDOUT
-      dr = Minitest::Reporters::DelegateReporter.new([ reporter ], :io => io_handle)
-      assert_equal io_handle, dr.io
-      dr.send :all_reporters
-      assert_equal io_handle, reporter.io
-    end
-  end
-end
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class MinitestReporterPluginTest < Minitest::Test
+    def test_delegates_io
+      reporter = Minitest::Reporters::DefaultReporter.new
+      io_handle = STDOUT
+      dr = Minitest::Reporters::DelegateReporter.new([ reporter ], :io => io_handle)
+      assert_equal io_handle, dr.io
+      dr.send :all_reporters
+      assert_equal io_handle, reporter.io
+    end
+  end
+end

data/test/unit/minitest/reporters_test.rb

@@ -1,65 +1,65 @@
-require_relative "../../test_helper"
-require "minitest/mock"
-
-module MinitestReportersTest
-  class ReportersTest < Minitest::Test
-    def test_chooses_the_rubymine_reporter_when_necessary
-      # Rubymine reporter complains when RubyMine libs are not available, so
-      # stub its #puts method out.
-      $stdout.stub :puts, nil do
-        reporters = Minitest::Reporters.choose_reporters [], { "RM_INFO" => "x" }
-        assert_instance_of Minitest::Reporters::RubyMineReporter, reporters[0]
-
-        reporters = Minitest::Reporters.choose_reporters [], { "TEAMCITY_VERSION" => "x" }
-        assert_instance_of Minitest::Reporters::RubyMineReporter, reporters[0]
-      end
-    end
-
-    def test_chooses_the_textmate_reporter_when_necessary
-      reporters = Minitest::Reporters.choose_reporters [], {"TM_PID" => "x"}
-      assert_instance_of Minitest::Reporters::RubyMateReporter, reporters[0]
-    end
-
-    def test_chooses_the_console_reporters_when_necessary
-      reporters = Minitest::Reporters.choose_reporters [Minitest::Reporters::SpecReporter.new], {}
-      assert_instance_of Minitest::Reporters::SpecReporter, reporters[0]
-    end
-
-    def test_chooses_no_reporters_when_running_under_vim
-      reporters = Minitest::Reporters.choose_reporters(
-        [Minitest::Reporters::DefaultReporter.new], { "VIM" => "/usr/share/vim" })
-      assert_nil reporters
-    end
-
-    def test_chooses_given_reporter_when_MINITEST_REPORTERS_env_set
-      env = {
-        "MINITEST_REPORTER" => "JUnitReporter", 
-        "RM_INFO" => "x", 
-        "TEAMCITY_VERSION" => "x", 
-        "TM_PID" => "x" }
-      # JUnit reporter init has stdout messages... capture them to keep test output clean
-      $stdout.stub :puts, nil do
-        reporters = Minitest::Reporters.choose_reporters [], env
-        assert_instance_of Minitest::Reporters::JUnitReporter, reporters[0]
-      end
-    end
-
-    def test_uses_minitest_clock_time_when_minitest_version_greater_than_561
-      Minitest::Reporters.stub :minitest_version, 583 do
-        Minitest.stub :clock_time, 6765.378751009 do
-          clock_time = Minitest::Reporters.clock_time
-          assert_equal 6765.378751009, clock_time
-        end
-      end
-    end
-
-    def test_uses_minitest_clock_time_when_minitest_version_less_than_561
-      Minitest::Reporters.stub :minitest_version, 431 do
-        Time.stub :now, Time.new(2015, 11, 20, 17, 35) do
-          clock_time = Minitest::Reporters.clock_time
-          assert_equal Time.new(2015, 11, 20, 17, 35), clock_time
-        end
-      end
-    end
-  end
-end
+require_relative "../../test_helper"
+require "minitest/mock"
+
+module MinitestReportersTest
+  class ReportersTest < Minitest::Test
+    def test_chooses_the_rubymine_reporter_when_necessary
+      # Rubymine reporter complains when RubyMine libs are not available, so
+      # stub its #puts method out.
+      $stdout.stub :puts, nil do
+        reporters = Minitest::Reporters.choose_reporters [], { "RM_INFO" => "x" }
+        assert_instance_of Minitest::Reporters::RubyMineReporter, reporters[0]
+
+        reporters = Minitest::Reporters.choose_reporters [], { "TEAMCITY_VERSION" => "x" }
+        assert_instance_of Minitest::Reporters::RubyMineReporter, reporters[0]
+      end
+    end
+
+    def test_chooses_the_textmate_reporter_when_necessary
+      reporters = Minitest::Reporters.choose_reporters [], {"TM_PID" => "x"}
+      assert_instance_of Minitest::Reporters::RubyMateReporter, reporters[0]
+    end
+
+    def test_chooses_the_console_reporters_when_necessary
+      reporters = Minitest::Reporters.choose_reporters [Minitest::Reporters::SpecReporter.new], {}
+      assert_instance_of Minitest::Reporters::SpecReporter, reporters[0]
+    end
+
+    def test_chooses_no_reporters_when_running_under_vim
+      reporters = Minitest::Reporters.choose_reporters(
+        [Minitest::Reporters::DefaultReporter.new], { "VIM" => "/usr/share/vim" })
+      assert_nil reporters
+    end
+
+    def test_chooses_given_reporter_when_MINITEST_REPORTERS_env_set
+      env = {
+        "MINITEST_REPORTER" => "JUnitReporter", 
+        "RM_INFO" => "x", 
+        "TEAMCITY_VERSION" => "x", 
+        "TM_PID" => "x" }
+      # JUnit reporter init has stdout messages... capture them to keep test output clean
+      $stdout.stub :puts, nil do
+        reporters = Minitest::Reporters.choose_reporters [], env
+        assert_instance_of Minitest::Reporters::JUnitReporter, reporters[0]
+      end
+    end
+
+    def test_uses_minitest_clock_time_when_minitest_version_greater_than_561
+      Minitest::Reporters.stub :minitest_version, 583 do
+        Minitest.stub :clock_time, 6765.378751009 do
+          clock_time = Minitest::Reporters.clock_time
+          assert_equal 6765.378751009, clock_time
+        end
+      end
+    end
+
+    def test_uses_minitest_clock_time_when_minitest_version_less_than_561
+      Minitest::Reporters.stub :minitest_version, 431 do
+        Time.stub :now, Time.new(2015, 11, 20, 17, 35) do
+          clock_time = Minitest::Reporters.clock_time
+          assert_equal Time.new(2015, 11, 20, 17, 35), clock_time
+        end
+      end
+    end
+  end
+end