slogger 0.0.8 → 0.0.9

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    slogger (0.0.8)
+    slogger (0.0.9)
 
 GEM
   remote: http://rubygems.org/

data/lib/slogger/base.rb

@@ -1,3 +1,5 @@
+require 'thread'
+
 module Slogger
   class Base
     
@@ -102,6 +104,7 @@ module Slogger
       @facility = facility
       @facility_as_int = SYSLOG_FACILITIES[facility]
       @custom_severity_levels = custom_severity_levels
+      @mutex = Mutex.new
     end
 
     def severity=(value)
@@ -119,7 +122,9 @@ module Slogger
         message = "[time: #{benchmark.real}] #{message}"
       end
       
-      Syslog.open(@app_name, Syslog::LOG_PID, @facility_as_int) { |s| s.send severity, '%s', message }
+      @mutex.synchronize do
+        Syslog.open(@app_name, Syslog::LOG_PID, @facility_as_int) { |s| s.send severity, '%s', message }
+      end
     end
 
     def raise_argument_error_to_required_parameter(param)

data/lib/slogger/common_logger.rb

@@ -21,7 +21,7 @@ module Slogger
       :err     => Syslog::LOG_ERR,
       :info    => Syslog::LOG_INFO,
       :debug   => Syslog::LOG_DEBUG,
-      :warn    => Syslog::LOG_WARNING,
+      :warning => Syslog::LOG_WARNING,
     }
 
     #
@@ -55,7 +55,7 @@ module Slogger
     # Raises an ArgumentError if app_name, severity, or facility is nil.
     #
     def initialize(app_name, severity, facility)
-      super app_name, severity, facility, SEVERITIES
+      super app_name, BRIDGE_SEVERITIES[severity], facility, SEVERITIES
     end
 
     def log(severity, message = nil, &block)
@@ -66,6 +66,18 @@ module Slogger
       end
     end
 
+    def level
+      ::Logger.const_get(BRIDGE_SEVERITIES.to_a.rassoc(@severity)[0].to_s.upcase)
+    end
+
+    def severity
+      BRIDGE_SEVERITIES.to_a.rassoc(@severity)[0]
+    end
+
+    def severity=(value)
+      super(BRIDGE_SEVERITIES[value])
+    end
+
     def add(severity, message = nil, progname = nil, &block)
       (BRIDGE_SEVERITIES.keys - [:unknow]).each do |key|
         if ::Logger.const_get(key.to_s.upcase) == severity
@@ -77,12 +89,13 @@ module Slogger
     end
 
     BRIDGE_SEVERITIES.each_key do |severity|
-      define_method severity do |message = nil, &block|
+      define_method severity do |*args, &block|
+        message = args.at(0)
         log BRIDGE_SEVERITIES[severity], message, &block
       end
 
       define_method "#{severity}?" do
-        SEVERITIES[BRIDGE_SEVERITIES[severity]] <= SEVERITIES[BRIDGE_SEVERITIES[@severity]]
+        SEVERITIES[BRIDGE_SEVERITIES[severity]] <= SEVERITIES[@severity]
       end
     end
   end

data/lib/slogger/version.rb

@@ -2,7 +2,7 @@ module Slogger
   module Version
     MAJOR = 0
     MINOR = 0
-    PATCH = 8
+    PATCH = 9
     STRING = "#{MAJOR}.#{MINOR}.#{PATCH}"
   end
 end

data/spec/slogger/common_logger_spec.rb

@@ -17,6 +17,23 @@ describe Slogger::CommonLogger do
     end
   end
 
+  describe "#level" do
+    mappings = {
+      :debug => Logger::DEBUG,
+      :info => Logger::INFO,
+      :warn => Logger::WARN,
+      :error => Logger::ERROR,
+      :fatal => Logger::FATAL
+    }
+
+    mappings.each_pair do |key, value|
+      it "should map #{key.inspect} to #{value}" do
+        logger = Slogger::CommonLogger.new "test_app", key, :local0
+        logger.level.should == value
+      end
+    end
+  end
+
   describe "invalid state" do
     it "should raise ArgumentError if doesn't have app_name" do
       lambda { Slogger::CommonLogger.new nil, :debug, :local0 }.should raise_error
@@ -114,9 +131,9 @@ describe Slogger::CommonLogger do
         subject.severity = :info
 
         messenger = mock('messenger')
-        messenger.should_receive(:message).and_return('this is a message %{name}')
+        messenger.should_receive(:message).and_return('this is a message %s')
         Syslog.should_receive(:info).with('%s','this is a message logger').and_return(Syslog)
-        subject.info { messenger.message % {:name => 'logger'} }
+        subject.info { messenger.message % ('logger') }
       end
     end
 

metadata

@@ -2,7 +2,7 @@
 name: slogger
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.0.8
+  version: 0.0.9
 platform: ruby
 authors: 
 - Leandro Silva
@@ -10,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-04-23 00:00:00 -03:00
+date: 2012-06-23 00:00:00 -03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency