diff options
author | Yusuke Endoh <mame@ruby-lang.org> | 2020-01-06 18:22:43 +0900 |
---|---|---|
committer | Yusuke Endoh <mame@ruby-lang.org> | 2020-01-17 17:20:38 +0900 |
commit | 7cfe93c028fbf7aa0022ca8a4ac6a66d0103337a (patch) | |
tree | 505d214877b1c84986ab221a701938a5bd1a6c0b /test | |
parent | b23fd59cbb3f097bcd559d0c85a86ff7a1eeeb7e (diff) |
hash.c: Add a feature to manipulate ruby2_keywords flag
It was found that a feature to check and add ruby2_keywords flag to an
existing Hash is needed when arguments are serialized and deserialized.
It is possible to do the same without explicit APIs, but it would be
good to provide them as a core feature.
https://github.com/rails/rails/pull/38105#discussion_r361863767
Hash.ruby2_keywords_hash?(hash) checks if hash is flagged or not.
Hash.ruby2_keywords_hash(hash) returns a duplicated hash that has a
ruby2_keywords flag,
[Bug #16486]
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/2818
Diffstat (limited to 'test')
-rw-r--r-- | test/ruby/test_hash.rb | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/test/ruby/test_hash.rb b/test/ruby/test_hash.rb index ef32cff868..2c449739f1 100644 --- a/test/ruby/test_hash.rb +++ b/test/ruby/test_hash.rb @@ -1756,4 +1756,28 @@ class TestHash < Test::Unit::TestCase super end end + + ruby2_keywords def get_flagged_hash(*args) + args.last + end + + def check_flagged_hash(k: :NG) + k + end + + def test_ruby2_keywords_hash? + flagged_hash = get_flagged_hash(k: 1) + assert_equal(true, Hash.ruby2_keywords_hash?(flagged_hash)) + assert_equal(false, Hash.ruby2_keywords_hash?({})) + assert_raise(TypeError) { Hash.ruby2_keywords_hash?(1) } + end + + def test_ruby2_keywords_hash! + hash = {k: 1} + assert_equal(false, Hash.ruby2_keywords_hash?(hash)) + hash = Hash.ruby2_keywords_hash(hash) + assert_equal(true, Hash.ruby2_keywords_hash?(hash)) + assert_equal(1, check_flagged_hash(*[hash])) + assert_raise(TypeError) { Hash.ruby2_keywords_hash(1) } + end end |