actually calls update method in mutate + coverage test

webbob · May 8, 2013 · 7a64369 · 7a64369
1 parent 3efcb9e
commit 7a64369
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/logstash/filters/mutate.rb b/lib/logstash/filters/mutate.rb
@@ -201,6 +201,7 @@ def filter(event)
     return unless filter?(event)
 
     rename(event) if @rename
+    update(event) if @update
     replace(event) if @replace
     convert(event) if @convert
     gsub(event) if @gsub

diff --git a/spec/filters/mutate.rb b/spec/filters/mutate.rb
@@ -15,6 +15,7 @@
           replace => [ "replaceme", "hello world" ]
           replace => [ "newfield", "newnew" ]
           update => [ "nosuchfield", "weee" ]
+          update => [ "updateme", "updated" ]
           remove => [ "removeme" ]
         }
       }
@@ -26,6 +27,7 @@
       "intme" => [ "1234", "7890.4", "7.9" ],
       "floatme" => [ "1234.455" ],
       "rename1" => [ "hello world" ],
+      "updateme" => [ "who cares" ],
       "replaceme" => [ "who cares" ],
       "removeme" => [ "something" ]
       }
@@ -43,6 +45,7 @@
       insist { subject }.include?("newfield")
       insist { subject["newfield"] } == "newnew"
       reject { subject }.include?("nosuchfield")
+      insist { subject["updateme"] } == "updated"
     end
   end