minitest-reporters 1.3.6 → 1.3.7

data/test/fixtures/progress_detailed_skip_test.rb

@@ -1,8 +1,8 @@
-require 'bundler/setup'
-require 'minitest/autorun'
-require 'minitest/reporters'
-
-Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new(detailed_skip: false)
-
-require_relative 'sample_test'
-
+require 'bundler/setup'
+require 'minitest/autorun'
+require 'minitest/reporters'
+
+Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new(detailed_skip: false)
+
+require_relative 'sample_test'
+

data/test/fixtures/progress_test.rb

@@ -1,8 +1,8 @@
-require 'bundler/setup'
-require 'minitest/autorun'
-require 'minitest/reporters'
-
-Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new
-
-require_relative 'sample_test'
-
+require 'bundler/setup'
+require 'minitest/autorun'
+require 'minitest/reporters'
+
+Minitest::Reporters.use! Minitest::Reporters::ProgressReporter.new
+
+require_relative 'sample_test'
+

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,18 +1,18 @@
-require 'bundler/setup'
-require 'minitest'
-require 'minitest/reporters'
-require 'minitest/autorun'
-
-Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
-
-describe String do
-  describe '#length' do
-    it 'works' do
-      assert_equal 5, 'hello'.length
-    end
-
-    it 'doesn\'t works' do
-      assert_equal 6, 'hello'.length
-    end
-  end
-end
+require 'bundler/setup'
+require 'minitest'
+require 'minitest/reporters'
+require 'minitest/autorun'
+
+Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
+
+describe String do
+  describe '#length' do
+    it 'works' do
+      assert_equal 5, 'hello'.length
+    end
+
+    it 'doesn\'t works' do
+      assert_equal 6, '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,40 +1,40 @@
-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_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'
-    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'
-    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_executable} #{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_executable} #{test_filename} -n /^test_0001_works$/ 2>&1`
-      refute_match '0 out of 0', output, 'Progress should not puts a warning'
-    end
-
-    private
-
-    def ruby_executable
-      defined?(JRUBY_VERSION) ? 'jruby' : 'ruby'
-    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_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'
+    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'
+    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_executable} #{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_executable} #{test_filename} -n /^test_0001_works$/ 2>&1`
+      refute_match '0 out of 0', output, 'Progress should not puts a warning'
+    end
+
+    private
+
+    def ruby_executable
+      defined?(JRUBY_VERSION) ? 'jruby' : 'ruby'
+    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/junit_reporter_test.rb

@@ -1,23 +1,46 @@
-require_relative "../../test_helper"
-
-module MinitestReportersTest
-  class JUnitReporterUnitTest < Minitest::Test
-    def setup
-      @reporter = Minitest::Reporters::JUnitReporter.new(
-          "report",
-          false,
-          :base_apath => Dir.pwd
-      )
-      @result = Minitest::Result.new("test_name")
-    end
-
-    def test_relative_path
-      path = Pathname.new(__FILE__).relative_path_from(Pathname.new(Dir.pwd)).to_s
-      @result.source_location = [path, 10]
-      relative_path = @reporter.get_relative_path(@result)
-      assert_equal path, relative_path.to_s
-    end
-  end
-end
-
-
+require_relative "../../test_helper"
+
+module MinitestReportersTest
+  class JUnitReporterUnitTest < Minitest::Test
+    def setup
+      @reporter = Minitest::Reporters::JUnitReporter.new(
+          "report",
+          false,
+          :base_apath => Dir.pwd
+      )
+      @result = Minitest::Result.new("test_name")
+    end
+
+    def test_relative_path
+      path = Pathname.new(__FILE__).relative_path_from(Pathname.new(Dir.pwd)).to_s
+      @result.source_location = [path, 10]
+      relative_path = @reporter.get_relative_path(@result)
+      assert_equal path, relative_path.to_s
+    end
+
+    def test_defaults_reports_path
+      reporter = Minitest::Reporters::JUnitReporter.new
+      expected_reports_dir = Minitest::Reporters::JUnitReporter::DEFAULT_REPORTS_DIR
+      expected_reports_path = File.absolute_path(expected_reports_dir)
+      assert_equal expected_reports_path, reporter.reports_path
+    end
+
+    def test_accepts_custom_report_dir_using_env
+      expected_reports_dir = "test_reports"
+      expected_reports_path = File.absolute_path(expected_reports_dir)
+      with_env("MINITEST_REPORTERS_REPORTS_DIR" => expected_reports_dir) do
+        reporter = Minitest::Reporters::JUnitReporter.new
+        assert_equal expected_reports_path, reporter.reports_path
+      end
+    end
+
+    private
+
+    def with_env(hash)
+      original_env = ENV.to_hash
+      ENV.update(hash)
+      yield
+      ENV.replace(original_env)
+    end
+  end
+end