influxdb 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3cfda4b23fa6e69256f6309aa37b2b84b4b00fe9
-  data.tar.gz: d9ca155910d5323850f3dc11f76a20f6d4520747
+  metadata.gz: 89e34f73dc79fdd7911d7b95222437c9a36b3839
+  data.tar.gz: 2e62a50a70a4b7848a66a84d1033d49194d09445
 SHA512:
-  metadata.gz: 755051222fdd660fbc2b0b43bd6e3b07e153fe81193b9152f0b03e2eeedb6bccdcac0aeb67ec10b824efc0446597b9c2629fbb0117ca27739af12d35f1ff4466
-  data.tar.gz: 764110f71894216a9032505fc2bdd8f0ad5580fce387a8b420a3d0eff01b16ab86b8525543c5d42793ebc090e090ac1281d42d70b7d264693a8ac4ed64fd2560
+  metadata.gz: 04a951e2106d61547d4409ebac4dcb2055c5d38012fc9b40cb97d46af7dfa9426fc27f7a27b7657e9587faf7d15d322d2e80a4b7f8da6967e2fc06edcff1458b
+  data.tar.gz: 51ec22a345e521fc8adcfd7ec71a2bc488098d389b9b9463a51acd2ea262c86297d4d015fd5ceb8fb026eba9a9ecf6f82ed985f3dce62ee8641da7668f298e48

data/lib/influxdb/client.rb

@@ -15,7 +15,7 @@ module InfluxDB
 
     attr_accessor :queue, :worker
 
-    include InfluxDB::Logger
+    include InfluxDB::Logging
 
     # Initializes a new InfluxDB client
     #

data/lib/influxdb/logger.rb

@@ -1,10 +1,20 @@
+require 'logger'
+
 module InfluxDB
-  module Logger
+  module Logging
     PREFIX = "[InfluxDB] "
 
+    def self.logger=(new_logger)
+      @logger = new_logger
+    end
+
+    def self.logger
+      @logger ||= ::Logger.new(STDERR).tap {|logger| logger.level = Logger::INFO}
+    end
+
     private
     def log(level, message)
-      STDERR.puts(PREFIX + "(#{level}) #{message}") unless level == :debug
+      InfluxDB::Logging.logger.send(level.to_sym, PREFIX + message)
     end
   end
 end

data/lib/influxdb/version.rb

@@ -1,3 +1,3 @@
 module InfluxDB
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/lib/influxdb/worker.rb

@@ -7,7 +7,7 @@ module InfluxDB
     attr_reader :client
     attr_accessor :queue
 
-    include InfluxDB::Logger
+    include InfluxDB::Logging
 
     MAX_POST_POINTS = 1000
     NUM_WORKER_THREADS = 3

data/spec/influxdb/logger_spec.rb

@@ -1,29 +1,36 @@
 require 'spec_helper'
+require 'logger'
 
-describe InfluxDB::Logger do
+describe InfluxDB::Logging do
   class LoggerTest
-    include InfluxDB::Logger
+    include InfluxDB::Logging
 
     def write_to_log(level, message)
       log(level, message)
     end
+
   end
 
-  subject { LoggerTest.new }
+  before { @old_logger = InfluxDB::Logging.logger }
+  after { InfluxDB::Logging.logger = @old_logger }
 
-  context 'with DEBUG level' do
-    it 'should not write a log message to STDERR' do
-      expect(STDERR).to_not receive(:puts)
+  it "has a default logger" do
+    expect(InfluxDB::Logging.logger).to be_a(Logger)
+  end
 
-      subject.write_to_log(:debug, 'debug')
-    end
+  it "allows setting of a logger" do
+    new_logger = Logger.new(STDOUT)
+    InfluxDB::Logging.logger = new_logger
+    expect(InfluxDB::Logging.logger).to eq(new_logger)
   end
 
-  context 'with non-DEBUG level' do
-    it 'should write a log message to STDERR' do
-      expect(STDERR).to receive(:puts).with('[InfluxDB] (info) info')
+  context "when included in classes" do
+
+    subject { LoggerTest.new }
 
-      subject.write_to_log(:info, 'info')
+    it "logs" do
+      expect(InfluxDB::Logging.logger).to receive(:debug).with(an_instance_of(String)).once
+      subject.write_to_log(:debug, 'test')
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: influxdb
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Todd Persen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-08 00:00:00.000000000 Z
+date: 2014-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json