test-unit 3.0.9 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1731a62620afe520b8f3817106f6e9c126f5ea73
-  data.tar.gz: e78724adf56e151021ccb346e17db6fdb1e781af
+  metadata.gz: f0f3d6b7558b8c2dd32e218231b2b826791440da
+  data.tar.gz: 07aabaaba099bf99ad12491d9b747fa121f9ecf3
 SHA512:
-  metadata.gz: 8cc3e2a66dd5fa1366f1442a94b90c0ff7ee98da7732faed76ab536b67afc2e1f7753e5fc8a17bf8301f6927880168cf9248672e93c502d3cd3382eef395e2e6
-  data.tar.gz: 16d07b7d4b54a3826399d0eb9b97457a47cdf6ba945c4deaee1982b4f27809685f32beb8c5dbbb61e416efbc8cfb77922008d39ced1864e47110c749a617cf96
+  metadata.gz: 2f810e4cb4ac200999a8afe12c91557855fefb446ac3f2fb6cffe17e973efb02f4840a4ae6cbf01996a21cc0f64e93c3288c895398f9fdb37e76cdadf25ead0a
+  data.tar.gz: 36725fd11eb881311aa7f8bd67e6a00ff3de6784448ecaa2cdaa5b7041df4045fbf52262fc1aa879e70cef58f9f94cff56fa1d9eb66067108ba3657fb884a1a9

data/doc/text/news.md

@@ -1,5 +1,38 @@
 # News
 
+## 3.1.0 - 2015-05-28 {#version-3-1-0}
+
+It's a bug fix release.
+
+### Improvements
+
+  * [ui][console] Removed needless new line.
+
+### Fixes
+
+  * Fixed a bug that priority mode can't be used on Windows.
+    [GitHub#95][Reported by Daniel Berger]
+  * Fixed a homepage URL RubyGems spec.
+    [GitHub#96][Patch by Masayoshi Takahashi]
+    supported.) [GitHub#89][Patch by Aaron Stone]
+  * Fixed a bug that shutdown hook isn't called when pass throw
+    exception such as `Interrupt` is raised.
+    [GitHub#98][Reported by jeremiahishere.]
+  * Fixed typos in documents.
+    [GitHub#100][Reported by scivola]
+    [GitHub#102][GitHub#103][Patch by Masafumi Yokoyama]
+  * Fixed a bug that the same name test isn't executed in sub test case.
+    [GitHub#104][Reported by wanabe]
+
+### Thanks
+
+  * Daniel Berger
+  * Masayoshi Takahashi
+  * jeremiahishere
+  * scivola
+  * Masafumi Yokoyama
+  * wanabe
+
 ## 3.0.9 - 2014-12-31 {#version-3-0-9}
 
 It's a release that improves colors.

data/lib/test/unit/assertions.rb

@@ -178,7 +178,7 @@ module Test
       #   refute(false)    # => pass
       #   refute(nil)      # => pass
       #
-      # @example Fialure patterns
+      # @example Failure patterns
       #   refute(true)     # => failure
       #   refute("string") # => failure
       #

data/lib/test/unit/error.rb

@@ -78,8 +78,12 @@ module Test
 
       NOT_PASS_THROUGH_EXCEPTIONS = []
       NOT_PASS_THROUGH_EXCEPTION_NAMES = ["Timeout::Error"]
-      PASS_THROUGH_EXCEPTIONS = [NoMemoryError, SignalException, Interrupt,
-                                 SystemExit]
+      PASS_THROUGH_EXCEPTIONS = [
+        NoMemoryError,
+        SignalException,
+        Interrupt,
+        SystemExit,
+      ]
       PASS_THROUGH_EXCEPTION_NAMES = []
       private
       def handle_all_exception(exception)

data/lib/test/unit/priority.rb

@@ -122,9 +122,11 @@ module Test
         end
 
         def result_dir
-          components = [".test-result",
-                        @test.class.name || "AnonymousTestCase",
-                        escaped_method_name]
+          components = [
+            ".test-result",
+            escape_class_name(@test.class.name || "AnonymousTestCase"),
+            escaped_method_name,
+          ]
           parent_directories = [File.dirname($0), Dir.pwd]
           if Process.respond_to?(:uid)
             parent_directories << File.join(Dir.tmpdir, Process.uid.to_s)
@@ -145,9 +147,17 @@ module Test
           File.join(result_dir, "passed")
         end
 
+        def escape_class_name(class_name)
+          class_name.gsub(/(?:[: \\\/])/, "_")
+        end
+
         def escaped_method_name
-          @test.method_name.to_s.gsub(/[!?=]$/) do |matched|
+          @test.method_name.to_s.gsub(/(?:[: ]|[!?=]$)/) do |matched|
             case matched
+            when ":"
+              "_colon_"
+            when " "
+              "_"
             when "!"
               ".destructive"
             when "?"

data/lib/test/unit/test-suite-creator.rb

@@ -44,9 +44,10 @@ module Test
         methods = @test_case.public_instance_methods(true)
         super_test_case = @test_case.superclass
         methods -= super_test_case.public_instance_methods(true)
+        methods |= @test_case.public_instance_methods(false)
         method_names = methods.collect(&:to_s)
         test_names = method_names.find_all do |method_name|
-          method_name =~ /^test./ or
+          /\Atest./ =~ method_name or
             @test_case.find_attribute(method_name, :test)
         end
         __send__("sort_test_names_in_#{@test_case.test_order}_order", test_names)

data/lib/test/unit/testcase.rb

@@ -320,7 +320,7 @@ module Test
         #     end
         #   end
         #
-        # The diffrence of them are the following:
+        # The difference of them are the following:
         #
         # * Test case created by {sub_test_case} is an anonymous class.
         #   So you can't refer the test case by name.
@@ -331,7 +331,7 @@ module Test
         #
         # @param name [String] The name of newly created sub test case.
         # @yield
-        #   The block is evaludated under the newly created sub test
+        #   The block is evaluated under the newly created sub test
         #   case class context.
         # @return [Test::Unit::TestCase] Created sub test case class.
         def sub_test_case(name, &block)
@@ -346,7 +346,7 @@ module Test
           sub_test_case
         end
 
-        # Checkes whether a test that is mathched the query is
+        # Checks whether a test that is matched the query is
         # defined.
         #
         # @option query [String] :path (nil)

data/lib/test/unit/testsuite.rb

@@ -53,11 +53,14 @@ module Test
           run_test(test, result, &progress_block)
           @passed = false unless test.passed?
         end
-        run_shutdown(result)
       ensure
-        @elapsed_time = Time.now - @start_time
-        yield(FINISHED, name)
-        yield(FINISHED_OBJECT, self)
+        begin
+          run_shutdown(result)
+        ensure
+          @elapsed_time = Time.now - @start_time
+          yield(FINISHED, name)
+          yield(FINISHED_OBJECT, self)
+        end
       end
 
       # Adds the test to the suite.

data/lib/test/unit/ui/console/testrunner.rb

@@ -186,7 +186,7 @@ module Test
             else
               if fault.is_a?(Error)
                 output_single("#{fault.label}: ")
-                output(fault.test_name, fault_color(fault))
+                output_single(fault.test_name, fault_color(fault))
                 output_fault_message(fault)
               else
                 output_single(fault.label, fault_color(fault))

data/lib/test/unit/version.rb

@@ -1,5 +1,5 @@
 module Test
   module Unit
-    VERSION = '3.0.9'
+    VERSION = '3.1.0'
   end
 end

data/test/test-assertions.rb

@@ -124,22 +124,30 @@ module Test
     class TestAssertions < TestCase
       include AssertionCheckable
 
-      def test_assert_block
-        check_nothing_fails {
-          assert_block {true}
-        }
-        check_nothing_fails {
-          assert_block("successful assert_block") {true}
-        }
-        check_nothing_fails {
-          assert_block("successful assert_block") {true}
-        }
-        check_fail("assert_block failed.") {
-          assert_block {false}
-        }
-        check_fail("failed assert_block") {
-          assert_block("failed assert_block") {false}
-        }
+      class TestAssertBlock < self
+        def test_pass_without_message
+          check_nothing_fails {
+            assert_block {true}
+          }
+        end
+
+        def test_pass_with_message
+          check_nothing_fails {
+            assert_block("successful assert_block") {true}
+          }
+        end
+
+        def test_failure_without_message
+          check_fail("assert_block failed.") {
+            assert_block {false}
+          }
+        end
+
+        def test_failure_with_message
+          check_fail("failed assert_block") {
+            assert_block("failed assert_block") {false}
+          }
+        end
       end
 
       class TestAssertEqual < self

data/test/test-priority.rb

@@ -102,33 +102,72 @@ class TestUnitPriority < Test::Unit::TestCase
     assert_in_delta(expected, n_need_to_run.to_f / n, delta)
   end
 
-  class SpecialNameTestCase < Test::Unit::TestCase
-    class << self
-      def suite
-        Test::Unit::TestSuite.new(name)
-      end
+  class TestClassName < self
+    def test_colon
+      assert_escaped_name("Test__Priority", "Test::Priority")
     end
 
-    def test_question?
+    def test_space
+      assert_escaped_name("test_priority", "test priority")
     end
 
-    def test_exclamation!
+    def test_slash
+      assert_escaped_name("test_priority", "test/priority")
     end
 
-    def test_equal=
+    def test_back_slash
+      assert_escaped_name("test_priority", "test\/priority")
     end
-  end
 
-  def test_escaped?
-    assert_escaped_name("test_question.predicate", "test_question?")
-    assert_escaped_name("test_exclamation.destructive", "test_exclamation!")
-    assert_escaped_name("test_equal.equal", "test_equal=")
+    def assert_escaped_name(expected, class_name)
+      checker = Checker.new(nil)
+      escaped_class_name = checker.send(:escape_class_name, class_name)
+      assert_equal(expected, escaped_class_name)
+    end
   end
 
-  def assert_escaped_name(expected, test_method_name)
-    checker = Checker.new(SpecialNameTestCase.new(test_method_name))
-    passed_file = checker.send(:passed_file)
-    method_name_component = File.basename(File.dirname(passed_file))
-    assert_equal(expected, method_name_component)
+  class TestFileName < self
+    class SpecialNameTestCase < Test::Unit::TestCase
+      class << self
+        def suite
+          Test::Unit::TestSuite.new(name)
+        end
+      end
+
+      def test_question?
+      end
+
+      def test_exclamation!
+      end
+
+      def test_equal=
+      end
+
+      test "have space" do
+      end
+    end
+
+    def test_question
+      assert_escaped_name("test_question.predicate", "test_question?")
+    end
+
+    def test_exclamation
+      assert_escaped_name("test_exclamation.destructive", "test_exclamation!")
+    end
+
+    def test_equal
+      assert_escaped_name("test_equal.equal", "test_equal=")
+    end
+
+    def test_colon_and_space
+      assert_escaped_name("test_colon__have_space", "test: have space")
+    end
+
+    def assert_escaped_name(expected, test_method_name)
+      checker = Checker.new(SpecialNameTestCase.new(test_method_name))
+      passed_file = checker.send(:passed_file)
+      method_name_component = File.basename(File.dirname(passed_file))
+      assert_equal(expected, method_name_component)
+    end
   end
 end

data/test/test-test-case.rb

@@ -842,6 +842,33 @@ module Test
           end
           assert_equal(["test_nothing"], test_method_names)
         end
+
+        def test_duplicated_name
+          test_case = Class.new(TestCase) do
+            def test_nothing
+            end
+          end
+          sub_test_case = test_case.sub_test_case("sub test case") do
+            def test_nothing
+            end
+          end
+
+          test_method_names = test_case.suite.tests.collect do |test|
+            test.method_name
+          end
+          sub_test_method_names = sub_test_case.suite.tests.collect do |test|
+            test.method_name
+          end
+
+          assert_equal([
+                         ["test_nothing"],
+                         ["test_nothing"],
+                       ],
+                       [
+                         test_method_names,
+                         sub_test_method_names,
+                       ])
+        end
       end
 
       class TestStartupShutdown < self
@@ -1064,6 +1091,35 @@ module Test
               run_test_case(test_case)
             end
           end
+
+          def test_pass_through_in_test
+            test_case = Class.new(TestCase) do
+              @called = []
+              class << self
+                def called
+                  @called
+                end
+
+                def startup
+                  @called << :startup
+                end
+
+                def shutdown
+                  @called << :shutdown
+                end
+              end
+
+              def test_error
+                raise Interrupt, "from test"
+              end
+            end
+
+            assert_raise(Interrupt) do
+              run_test_case(test_case)
+            end
+            assert_equal([:startup, :shutdown],
+                         test_case.called)
+          end
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test-unit
 version: !ruby/object:Gem::Version
-  version: 3.0.9
+  version: 3.1.0
 platform: ruby
 authors:
 - Kouhei Sutou
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-31 00:00:00.000000000 Z
+date: 2015-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: power_assert
@@ -208,7 +208,7 @@ files:
 - test/util/test_backtracefilter.rb
 - test/util/test_observable.rb
 - test/util/test_procwrapper.rb
-homepage: http://rubygems.org/gems/test-unit
+homepage: http://test-unit.github.io/
 licenses:
 - Ruby
 - PSFL