mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
Fix Enumerable#tally with some arguments pattern [Feature #17744]
* Add test cases for Enumerable#tally with hash argument * Add ruby/spec for Enumerable#tally with hash argument * Fix Enumerable#tally does not update given frozen hash * Add test cases for Enumerable#tally with hash convertible arguments * Fix SEGV when Enumerable#tally takes non Hash convertible * FIx cosmetic damage enum.c
This commit is contained in:
parent
785c77d782
commit
aceb8c0b4b
Notes:
git
2021-03-27 12:56:06 +09:00
Merged: https://github.com/ruby/ruby/pull/4327 Merged-By: nobu <nobu@ruby-lang.org>
3 changed files with 54 additions and 3 deletions
|
@ -45,6 +45,25 @@ ruby_version_is "3.1" do
|
|||
enum.tally({ 'foo' => 1 }).should == { 'foo' => 3, 'bar' => 1, 'baz' => 1}
|
||||
end
|
||||
|
||||
it "returns the given hash" do
|
||||
enum = EnumerableSpecs::Numerous.new('foo', 'bar', 'foo', 'baz')
|
||||
hash = { 'foo' => 1 }
|
||||
enum.tally(hash).should equal(hash)
|
||||
end
|
||||
|
||||
it "raises a FrozenError and does not udpate the given hash when the hash is frozen" do
|
||||
enum = EnumerableSpecs::Numerous.new('foo', 'bar', 'foo', 'baz')
|
||||
hash = { 'foo' => 1 }.freeze
|
||||
-> { enum.tally(hash) }.should raise_error(FrozenError)
|
||||
hash.should == { 'foo' => 1 }
|
||||
end
|
||||
|
||||
it "does not call given block" do
|
||||
enum = EnumerableSpecs::Numerous.new('foo', 'bar', 'foo', 'baz')
|
||||
enum.tally({ 'foo' => 1 }) { |v| ScratchPad << v }
|
||||
ScratchPad.recorded.should == []
|
||||
end
|
||||
|
||||
it "ignores the default value" do
|
||||
enum = EnumerableSpecs::Numerous.new('foo', 'bar', 'foo', 'baz')
|
||||
enum.tally(Hash.new(100)).should == { 'foo' => 2, 'bar' => 1, 'baz' => 1}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue