diff options
| author | jfrazx <[email protected]> | 2021-06-14 08:53:20 -0500 |
|---|---|---|
| committer | <[email protected]> | 2021-06-14 09:53:20 -0400 |
| commit | 931ea7cfbec6d863cd8b48308804323704a2696c () | |
| tree | a2585c739c839d5aa86a4488244087d93dd95bb4 | |
| parent | 90cad6e14745d812f042df61a6455db022be7389 (diff) | |
Add fallback block to `OpenStruct#delete_field` (#1409)
Notes: Merged-By: marcandre <@marc-andre.ca>
| -rw-r--r-- | lib/ostruct.rb | 11 | ||||
| -rw-r--r-- | test/ostruct/test_ostruct.rb | 12 |
2 files changed, 20 insertions, 3 deletions
@@ -326,8 +326,10 @@ class OpenStruct end # - # Removes the named field from the object. Returns the value that the field - # contained if it was defined. # # require "ostruct" # @@ -341,6 +343,10 @@ class OpenStruct # person.pension = nil # person # => #<OpenStruct name="John", pension=nil> # def delete_field(name) sym = name.to_sym begin @@ -348,6 +354,7 @@ class OpenStruct rescue NameError end @table.delete(sym) do raise! NameError.new("no field `#{sym}' in #{self}", sym) end end @@ -89,7 +89,7 @@ class TC_OpenStruct < Test::Unit::TestCase a = o.delete_field :a assert_not_respond_to(o, :a, bug) assert_not_respond_to(o, :a=, bug) - assert_equal(a, 'a') s = Object.new def s.to_sym :foo @@ -100,6 +100,16 @@ class TC_OpenStruct < Test::Unit::TestCase o.delete_field s assert_not_respond_to(o, :foo) assert_not_respond_to(o, :foo=) end def test_setter |