steno 1.0.4 → 1.1.0

data/lib/steno/logger.rb

@@ -133,7 +133,7 @@ class Steno::Logger
 
     data = @context.data.merge(user_data || {})
 
-    record = Steno::Record.new(@name, level, message, loc, data)
+    record = Steno::Record.new(@name, level_name, message, loc, data)
 
     @sinks.each { |sink| sink.add_record(record) }
 

data/lib/steno/record.rb

@@ -25,6 +25,8 @@ class Steno::Record
   #        Format is [<filename>, <lineno>, <method>].
   # @param [Hash]   data       User-supplied data
   def initialize(source, log_level, message, loc = [], data = {})
+    raise "Log level must be a Symbol" unless log_level.is_a? Symbol
+
     @timestamp  = Time.now
     @source     = source
     @log_level  = log_level

data/lib/steno/sink/counter.rb

@@ -14,7 +14,7 @@ class Steno::Sink::Counter < Steno::Sink::Base
   end
 
   def add_record(record)
-    level = record.log_level.name.to_s
+    level = record.log_level.to_s
 
     @mutex.synchronize do
       unless @counts[level]

data/lib/steno/sink/syslog.rb

@@ -34,7 +34,7 @@ class Steno::Sink::Syslog < Steno::Sink::Base
   def add_record(record)
     record = truncate_record(record)
     msg = @codec.encode_record(record)
-    pri = LOG_LEVEL_MAP[record.log_level.name]
+    pri = LOG_LEVEL_MAP[record.log_level]
     @syslog_lock.synchronize { @syslog.log(pri, "%s", msg) }
   end
 

data/lib/steno/version.rb

@@ -1,3 +1,3 @@
 module Steno
-  VERSION = "1.0.4"
+  VERSION = "1.1.0"
 end

data/spec/unit/json_codec_spec.rb

@@ -40,7 +40,7 @@ describe Steno::Codec::Json do
 
   def make_record(opts = {})
     Steno::Record.new(opts[:source]  || "my_source",
-                      opts[:level]   || Steno::LogLevel.new("debug", 0),
+                      opts[:level]   || :debug,
                       opts[:message] || "test message",
                       nil,
                       opts[:data]    || {})

data/spec/unit/logger_spec.rb

@@ -72,6 +72,16 @@ describe Steno::Logger do
       logger.warn { invoked = true }
       invoked.should be_true
     end
+
+    it "creates a record with the proper level" do
+      sink = mock("sink")
+      Steno::Record.should_receive(:new).with("test", :warn, "message", anything, anything).and_call_original
+      sink.stub(:add_record)
+
+      my_logger = Steno::Logger.new("test", [sink])
+
+      my_logger.warn("message")
+    end
   end
 
   describe "#logf" do

data/spec/unit/sink/counter_spec.rb

@@ -6,7 +6,7 @@ describe Steno::Sink::Counter do
   end
 
   let(:record) do
-    Steno::Record.new("source", level, "message")
+    Steno::Record.new("source", level.name, "message")
   end
 
   describe "add_record" do

data/spec/unit/sink/fluentd_spec.rb

@@ -6,7 +6,7 @@ describe Steno::Sink::IO do
   end
 
   let(:record) do
-    Steno::Record.new("source", level, "message")
+    Steno::Record.new("source", level.name, "message")
   end
 
   describe "#initialize" do

data/spec/unit/sink/io_spec.rb

@@ -6,7 +6,7 @@ describe Steno::Sink::IO do
   end
 
   let(:record) do
-    Steno::Record.new("source", level, "message")
+    Steno::Record.new("source", level.name, "message")
   end
 
   describe ".for_file" do

data/spec/unit/sink/syslog_spec.rb

@@ -6,11 +6,11 @@ describe Steno::Sink::Syslog do
   end
 
   let(:record) do
-    Steno::Record.new("source", level, "message")
+    Steno::Record.new("source", level.name, "message")
   end
 
   let(:record_with_big_message) do
-    Steno::Record.new("source", level,
+    Steno::Record.new("source", level.name,
                       "a" * (Steno::Sink::Syslog::MAX_MESSAGE_SIZE + 1))
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: steno
 version: !ruby/object:Gem::Version
-  version: 1.0.4
+  version: 1.1.0
   prerelease: 
 platform: ruby
 authors: