Merge pull request #7 from ketan/add-does-not-pass-args-correctly

Ensure that arguments to Mixlib::Log#add are passed as is to all loggers

2 files changed, 10 insertions, 1 deletions

diff --git a/lib/mixlib/log.rb b/lib/mixlib/log.rb
index aa477c8..5ff243d 100644
--- a/lib/mixlib/log.rb
+++ b/lib/mixlib/log.rb
@@ -133,7 +133,7 @@ module Mixlib
     end
 
     def add(severity, message = nil, progname = nil, &block)
-      loggers.each {|l| l.add(severity, message = nil, progname = nil, &block) }
+      loggers.each {|l| l.add(severity, message, progname, &block) }
     end
 
     alias :log :add
diff --git a/spec/mixlib/log_spec.rb b/spec/mixlib/log_spec.rb
index 1a4a33e..640ae81 100644
--- a/spec/mixlib/log_spec.rb
+++ b/spec/mixlib/log_spec.rb
@@ -130,6 +130,15 @@ describe Mixlib::Log do
     lambda { Logit.debug("Gimme some sugar!") }.should_not raise_error
   end
 
+  it "should pass add method calls directly to logger" do
+    logdev = StringIO.new
+    Logit.init(logdev)
+    Logit.level = :debug
+    Logit.should be_debug
+    lambda { Logit.add(Logger::DEBUG, "Gimme some sugar!") }.should_not raise_error
+    logdev.string.should match(/Gimme some sugar/)
+  end
+
   it "should default to STDOUT if init is called with no arguments" do
     logger_mock = Struct.new(:formatter, :level).new
     Logger.stub!(:new).and_return(logger_mock)