From 46e96fb0a2a16a44a7d5980f81d0b76c201c705a Mon Sep 17 00:00:00 2001 From: Suyog Rao Date: Wed, 21 Jan 2015 11:52:38 -0800 Subject: [PATCH] Added specs for accessors del method Fixes #2384 --- spec/util/accessors_spec.rb | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/spec/util/accessors_spec.rb b/spec/util/accessors_spec.rb index 2ae526d3d..864ae5b05 100644 --- a/spec/util/accessors_spec.rb +++ b/spec/util/accessors_spec.rb @@ -36,6 +36,17 @@ describe LogStash::Util::Accessors, :if => true do expect(data).to be_empty end + it "should handle delete on non-existent field" do + str = "[foo][bar]" + data = { "hello" => "world" } + accessors = LogStash::Util::Accessors.new(data) + expect(accessors.del(str)).to be_nil + expect(data).not_to be_empty + # assert no side effects + expect(accessors.get("foo")).to be_nil + expect(accessors.get("hello")).to eq("world") + end + it "should set string value" do str = "simple" data = {}