map 5.4.0 → 5.5.0

data/lib/map.rb

@@ -1,6 +1,6 @@
 # -*- encoding : utf-8 -*-
 class Map < Hash
-  Version = '5.4.0' unless defined?(Version)
+  Version = '5.5.0' unless defined?(Version)
   Load = Kernel.method(:load) unless defined?(Load)
 
   class << Map
@@ -420,6 +420,14 @@ class Map < Hash
     self
   end
 
+  # See: https://github.com/rubinius/rubinius/blob/98c516820d9f78bd63f29dab7d5ec9bc8692064d/kernel/common/hash19.rb#L476-L484
+  def keep_if( &block )
+    raise RuntimeError.new( "can't modify frozen #{ self.class.name }" ) if frozen?
+    return to_enum( :keep_if ) unless block_given?
+    each { | key , val | delete key unless yield( key , val ) }
+    self
+  end
+
   def replace(*args)
     clear
     update(*args)

data/map.gemspec

@@ -3,7 +3,7 @@
 
 Gem::Specification::new do |spec|
   spec.name = "map"
-  spec.version = "5.4.0"
+  spec.version = "5.5.0"
   spec.platform = Gem::Platform::RUBY
   spec.summary = "map"
   spec.description = "description: map kicks the ass"

data/test/map_test.rb

@@ -634,6 +634,41 @@ Testing Map do
     end
   end
 
+## Map#keep_if tests
+#
+# See: https://github.com/rubyspec/rubyspec/blob/ebd1ea400cb06807dbd6aa481c6c3d7a0b8fc7b4/core/hash/keep_if_spec.rb
+#
+  # original failing test
+  testing 'that Map#keep_if properly removes k/v pairs for which the passed block evaluates to false' do
+    m = Map.new( { 1 => "hi" , 2 => "there" } )
+    assert{ !( m.keep_if { |k,v| k == 2 }.keys.include? 1 ) }
+  end
+
+  testing 'yields two arguments: key and value' do
+    all_args = []
+    m = Map.new( { 1 => 2 , 3 => 4 } )
+    m.keep_if { |*args| all_args << args }
+    assert{ all_args == [[1, 2], [3, 4]] }
+  end
+
+  testing 'keeps every entry for which block is true and returns self' do
+    m = Map.new( { :a => 1 , :b => 2 , :c => 3 , :d => 4 } )
+    assert{ m.keep_if { |k,v| v % 2 == 0 }.object_id == m.object_id }
+    assert{ m == Map.new( { :b => 2 , :d => 4 } ) }
+  end
+
+  testing 'it raises a RuntimeError if called on a frozen instance' do
+    m = Map.new( { :a => 1 } ).freeze
+
+    assert do
+      begin
+        m.keep_if { |*_| false }
+      rescue Object => e
+        e.is_a?( RuntimeError )
+      end
+    end
+  end
+
 protected
   def new_int_map(n = 1024)
     map = assert{ Map.new }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: map
 version: !ruby/object:Gem::Version
-  version: 5.4.0
+  version: 5.5.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-18 00:00:00.000000000 Z
+date: 2012-03-29 00:00:00.000000000 Z
 dependencies: []
 description: ! 'description: map kicks the ass'
 email: ara.t.howard@gmail.com