test-unit 3.2.4 → 3.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 560e12d199e303860aa5410d5b4b7f73e4491970
-  data.tar.gz: b6dbafc77f21fd1b3f179542c4333eb965393cb4
+  metadata.gz: 367cf38cdec937cd00374b17aeddd2cd85271223
+  data.tar.gz: 66d1270f41f63a6d324c419cb91cb8e4a4e66fed
 SHA512:
-  metadata.gz: f1a8fde930effcca2418506776f8142390502033107f8c1ed239d83d854e6255c28cb846bfcf4e76cbec96948f1a74c0d1e9fc0996ca60a548c16f4b9888f40d
-  data.tar.gz: 7fbd95ad36bc11c0cc8efb7610a94f7c0f191a745a5b71723dc4cb5f9711dca4442137c735fd20a8fdb8c38f0afdb4708eca1c95de5c5366ec238bbc375a7f19
+  metadata.gz: 546256249a6ac765e2789a60feec91fd97ec4e66cbcf951897085bf378b435f81a548be56381c22ecdddda0e491b63adcacda90b74d916865f82cb4848e60098
+  data.tar.gz: 4e17e054bd8cdf04b401d7aee9205403be33c7a9a9e0fd7244d468897db61f5aecf9b25f4bdd3a03a75409faa8e56c6eece579b5eebab7accc1a1e48cff9f51b

data/Rakefile

@@ -1,6 +1,6 @@
 # -*- ruby -*-
 #
-# Copyright (C) 2008-2013  Kouhei Sutou <kou@clear-code.com>
+# Copyright (C) 2008-2017  Kouhei Sutou <kou@clear-code.com>
 #
 # This library is free software; you can redistribute it and/or
 # modify it under the terms of the GNU Lesser General Public
@@ -18,6 +18,15 @@
 
 Encoding.default_internal = "UTF-8" if defined?(Encoding.default_internal)
 
+# TODO: Remove me when we drop Ruby 1.9 support.
+unless "".respond_to?(:b)
+  class String
+    def b
+      dup.force_encoding("ASCII-8BIT")
+    end
+  end
+end
+
 require "erb"
 require "yaml"
 require "rubygems"

data/doc/text/news.md

@@ -1,5 +1,16 @@
 # News
 
+## 3.2.5 - 2017-06-24 {#version-3-2-5}
+
+### Improvements
+
+  * Supported `--enable-frozen-string-literal` `ruby` option.
+    [GitHub#149][Reported by Pat Allan]
+
+### Thanks
+
+  * Pat Allan
+
 ## 3.2.4 - 2017-05-23 {#version-3-2-4}
 
 ### Improvements

data/lib/test/unit/assertions.rb

@@ -153,8 +153,8 @@ module Test
               assertion_message = message
             else
               error_message = "assertion message must be String, Proc or "
-              error_message << "#{AssertionMessage}: "
-              error_message << "<#{message.inspect}>(<#{message.class}>)"
+              error_message += "#{AssertionMessage}: "
+              error_message += "<#{message.inspect}>(<#{message.class}>)"
               raise ArgumentError, error_message, filter_backtrace(caller)
             end
             assertion_message ||= build_message(message,
@@ -192,8 +192,8 @@ module Test
             assertion_message = message
           else
             error_message = "assertion message must be String, Proc or "
-            error_message << "#{AssertionMessage}: "
-            error_message << "<#{message.inspect}>(<#{message.class}>)"
+            error_message += "#{AssertionMessage}: "
+            error_message += "<#{message.inspect}>(<#{message.class}>)"
             raise ArgumentError, error_message, filter_backtrace(caller)
           end
           assert_block("refute should not be called with a block.") do
@@ -362,7 +362,7 @@ EOT
           else
             klasses = [klass]
           end
-          assert_block("The first parameter to assert_not_instance_of should be " <<
+          assert_block("The first parameter to assert_not_instance_of should be " +
                        "a Class or an Array of Class.") do
             klasses.all? {|k| k.is_a?(Class)}
           end
@@ -965,7 +965,7 @@ EOT
         end
 
         if relation_format
-          format << <<-EOT
+          format += <<-EOT
 
 Relation:
 #{relation_format}
@@ -1103,7 +1103,7 @@ EOT
         end
 
         if relation_format
-          format << <<-EOT
+          format += <<-EOT
 
 Relation:
 #{relation_format}
@@ -1780,7 +1780,7 @@ EOT
 
               if Diff.need_fold?(diff)
                 folded_diff = Diff.folded_readable(from, to)
-                diff << "\n\nfolded diff:\n#{folded_diff}"
+                diff += "\n\nfolded diff:\n#{folded_diff}"
               end
 
               diff
@@ -1794,9 +1794,9 @@ EOT
             inspector = Inspector.new(object)
             if use_pp
               begin
-                require 'pp' unless defined?(PP)
+                require "pp" unless defined?(PP)
                 begin
-                  return PP.pp(inspector, '').chomp
+                  return PP.pp(inspector, String.new).chomp
                 rescue NameError
                 end
               rescue LoadError
@@ -2045,26 +2045,26 @@ EOT
             expanded_template = ""
             @parts.each do |part|
               if part == '?'
-                encoding_safe_concat(expanded_template, params.shift)
+                param = params.shift
+                if Object.const_defined?(:Encoding)
+                  expanded_template += concatenatable(param,
+                                                      expanded_template.encoding)
+                else
+                  expanded_template += param
+                end
               else
-                expanded_template << part.gsub(/\\\?/m, '?')
+                expanded_template += part.gsub(/\\\?/m, '?')
               end
             end
             expanded_template
           end
 
           private
-          if Object.const_defined?(:Encoding)
-            def encoding_safe_concat(buffer, parameter)
-              if Encoding.compatible?(buffer, parameter)
-                buffer << parameter
-              else
-                buffer << parameter.dup.force_encoding(buffer.encoding)
-              end
-            end
-          else
-            def encoding_safe_concat(buffer, parameter)
-              buffer << parameter
+          def concatenatable(text, encoding)
+            if Encoding.compatible?(text, encoding)
+              text
+            else
+              text.dup.force_encoding(encoding)
             end
           end
         end

data/lib/test/unit/autorunner.rb

@@ -189,7 +189,7 @@ module Test
       def options
         @options ||= OptionParser.new do |o|
           o.banner = "Test::Unit automatic runner."
-          o.banner << "\nUsage: #{$0} [options] [-- untouched arguments]"
+          o.banner += "\nUsage: #{$0} [options] [-- untouched arguments]"
 
           o.on("-r", "--runner=RUNNER", RUNNERS,
                "Use the given RUNNER.",

data/lib/test/unit/diff.rb

@@ -269,7 +269,7 @@ module Test
 
         private
         def tag(mark, contents)
-          contents.collect {|content| "#{mark}#{content}"}
+          contents.collect {|content| mark + content}
         end
       end
 
@@ -450,7 +450,7 @@ module Test
 
         def tag(mark, contents)
           contents.each do |content|
-            @result << "#{mark}#{content}"
+            @result << (mark + content)
           end
         end
 
@@ -577,15 +577,15 @@ module Test
             to_width = compute_width(to_line, to_start, to_end)
             case tag
             when :replace
-              from_tags << "^" * from_width
-              to_tags << "^" * to_width
+              from_tags += "^" * from_width
+              to_tags += "^" * to_width
             when :delete
-              from_tags << "-" * from_width
+              from_tags += "-" * from_width
             when :insert
-              to_tags << "+" * to_width
+              to_tags += "+" * to_width
             when :equal
-              from_tags << " " * from_width
-              to_tags << " " * to_width
+              from_tags += " " * from_width
+              to_tags += " " * to_width
             else
               raise "unknown tag: #{tag}"
             end

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

@@ -30,16 +30,16 @@ module Test
               location_display = "\n" + failure.location.join("\n")
             end
             result = "#{failure.label}:\n"
-            result << "#{failure.test_name}#{location_display}:\n"
-            result << failure.message
+            result += "#{failure.test_name}#{location_display}:\n"
+            result += failure.message
             result
           end
 
           def format_fault_error(error)
             result = "#{error.label}:\n"
-            result << "#{error.test_name}:\n"
-            result << "#{error.message}\n"
-            result << error.backtrace.join("\n")
+            result += "#{error.test_name}:\n"
+            result += "#{error.message}\n"
+            result += error.backtrace.join("\n")
             result
           end
         end

data/lib/test/unit/version.rb

@@ -1,5 +1,5 @@
 module Test
   module Unit
-    VERSION = "3.2.4"
+    VERSION = "3.2.5"
   end
 end

data/test/test-assertions.rb

@@ -819,7 +819,7 @@ EOM
         check_nothing_fails {
           assert_same(thing, thing, "successful assert_same")
         }
-        thing2 = "thing"
+        thing2 = thing.dup
         check_fail(%Q{<"thing">\nwith id <#{thing.__id__}> was expected to be equal? to\n<"thing">\nwith id <#{thing2.__id__}>.}) {
           assert_same(thing, thing2)
         }
@@ -906,7 +906,7 @@ EOM
 
       def test_assert_not_same
         thing = "thing"
-        thing2 = "thing"
+        thing2 = thing.dup
         check_nothing_fails {
           assert_not_same(thing, thing2)
         }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test-unit
 version: !ruby/object:Gem::Version
-  version: 3.2.4
+  version: 3.2.5
 platform: ruby
 authors:
 - Kouhei Sutou
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-23 00:00:00.000000000 Z
+date: 2017-06-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: power_assert