warning 0.10.0 → 0.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c0a00f58401bfb0c3b182b5e472de7d417e5e9c7
-  data.tar.gz: 6e4a3377be9ee85d5de9848e4eaaa20b367035a4
+  metadata.gz: '08cf870b5689f1fc79b3a2c1c9bd787c10802ed0'
+  data.tar.gz: 43f33015143cf420dd92f3db2ad0c5b91f05335d
 SHA512:
-  metadata.gz: 1e3c7971f5ec2a5b28ec263aa6847778f45790b3554cdfee4fc8b4c08af21e3459dcf148fd8618b4d632cbd285e6afd6518ededc69a01469ea676666df564a5c
-  data.tar.gz: 674820530875fa42a1920f06673add77b66210fee9b50bd150cbafdce603e275c30c422cf985838e6f5500096aeb815c40d2a5d7cb5766b430a7912bf7f2238b
+  metadata.gz: 2f35fc384e25902773a66ec191ccdf522a762a3c6006d339b66ce4b16c11e1e67df9454d12be4a941c2ad0458b978707b1fc98bfde052c0918c48304d9bea30e
+  data.tar.gz: 6cd8864633508b1bcf4928d4d25c4d8e19763a48f58b1160eb9f03aa3a178413f4b42270bb92b9bf7f7d8941de2c5f02c53f05463ea5f003ab0eff769f535bea

data/CHANGELOG

@@ -1,3 +1,7 @@
+=== 0.10.1 (2017-11-16)
+
+* Correctly handle Warning.freeze (jeremyevans)
+
 === 0.10.0 (2016-11-16)
 
 * Add support for :arg_prefix and :shadow as regexp argument to Warning.ignore (jeremyevans)

data/Rakefile

@@ -15,11 +15,19 @@ end
 desc "Run test"
 Rake::TestTask.new do |t|
   t.libs.push "lib"
-  t.test_files = FileList['test/test_*.rb']
+  t.test_files = FileList['test/test_warning.rb']
   t.verbose = true
 end
 
-task :default=>:test
+desc "Run test"
+Rake::TestTask.new(:test_freeze) do |t|
+  t.libs.push "lib"
+  t.test_files = FileList['test/test_freeze_warning.rb']
+  t.verbose = true
+end
+
+desc "Run all tests"
+task :default=>[:test, :test_freeze]
 
 ### RDoc
 

data/lib/warning.rb

@@ -24,6 +24,12 @@ module Warning
         @process.clear
       end
     end
+
+    def freeze
+      @ignore.freeze
+      @process.freeze
+      super
+    end
     
     # Ignore any warning messages matching the given regexp, if they
     # start with the given path.
@@ -117,11 +123,17 @@ module Warning
 
       super
     end
+
+    private
+
+    def synchronize(&block)
+      @monitor.synchronize(&block)
+    end
   end
 
   @ignore = []
   @process = []
+  @monitor = Monitor.new
 
-  extend MonitorMixin
   extend Processor
 end

data/test/test_freeze_warning.rb

@@ -0,0 +1,69 @@
+require 'minitest/autorun'
+require 'warning'
+
+class WarningFreezeTest < Minitest::Test
+  module EnvUtil
+    def verbose_warning
+      class << (stderr = "")
+        alias write <<
+      end
+      stderr, $stderr, verbose, $VERBOSE = $stderr, stderr, $VERBOSE, true
+      yield stderr
+      return $stderr
+    ensure
+      stderr, $stderr, $VERBOSE = $stderr, stderr, verbose
+    end
+    module_function :verbose_warning
+
+    def with_default_internal(enc)
+      verbose, $VERBOSE = $VERBOSE, nil
+      origenc, Encoding.default_internal = Encoding.default_internal, enc
+      $VERBOSE = verbose
+      yield
+    ensure
+      verbose, $VERBOSE = $VERBOSE, nil
+      Encoding.default_internal = origenc
+      $VERBOSE = verbose
+    end
+    module_function :with_default_internal
+  end
+
+  def assert_warning(pat, msg = nil)
+    stderr = EnvUtil.verbose_warning {
+      EnvUtil.with_default_internal(pat.encoding) {
+        yield
+      }
+    }
+    msg = message(msg) {diff pat, stderr}
+    assert(pat === stderr, msg)
+  end
+
+  def test_warning_ignore
+    obj = Object.new
+    w = nil
+
+    Warning.ignore(/instance variable @ivar not initialized/)
+    Warning.process do |warning|
+      w = [4, warning]
+    end
+    Warning.freeze
+
+    assert_raises RuntimeError do
+      Warning.ignore(/instance variable @ivar not initialized/)
+    end
+    assert_raises RuntimeError do
+      Warning.process{|warning| w = [4, warning]}
+    end
+
+    assert_warning '' do
+      assert_nil(obj.instance_variable_get(:@ivar))
+    end
+    assert_nil w
+
+    assert_warning '' do
+      assert_nil(obj.instance_variable_get(:@ivar6))
+    end
+    assert_equal(4, w.first)
+    assert_match(/instance variable @ivar6 not initialized/, w.last)
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: warning
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.10.1
 platform: ruby
 authors:
 - Jeremy Evans
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-16 00:00:00.000000000 Z
+date: 2017-11-16 00:00:00.000000000 Z
 dependencies: []
 description: |
   ruby-warning adds custom processing for warnings, including the
@@ -28,6 +28,7 @@ files:
 - README.rdoc
 - Rakefile
 - lib/warning.rb
+- test/test_freeze_warning.rb
 - test/test_warning.rb
 homepage: https://github.com/jeremyevans/ruby-warning
 licenses:
@@ -56,7 +57,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: Add custom processing for warnings