slack_log_device 2.0.0 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3567b60b8adf372370ebeff154732ea9c0c85a6b
-  data.tar.gz: 1eaa0fea7a5ce04b969ef5ee2c5fd806e558d822
+  metadata.gz: 32885688b1cd9ac6a722a1a6f33555c509eb165f
+  data.tar.gz: 74ccc8aaa57329d2290dee6601fd6d82f9e31663
 SHA512:
-  metadata.gz: 721424872c5fbb4f9a997ef2c10b2c35f917255062fe469db09d7cf417295396ff955c23bfa1b7bcd827c5881a060c6e0c8bc157a633a8f8f9a66df9db88d08f
-  data.tar.gz: bd94f833858783c0ea6c4b649ea1a9cb7610302cf36e49cf9e42f2ebf4d0783d1c567061b70c0b282cc7ea8a6f5e51d8041d6ac0bc241a0406c3af3fc600bb9b
+  metadata.gz: 5fbf9de73db847487a88e5dd455865c4d892ad47fa2bd95c687655165c7da7701970d4a785931c1eb7c32ce1733c10963e3654940aefc3ff1590ee266a08c882
+  data.tar.gz: 2345e2526588ca41890abbd5e9864311b2ac19fd58c108608766eda3f2547629cba76846ef1d20f2d0a06f55db689d4c1afcc00b62fe51aea96ce1405742c6d0

data/VERSION

@@ -1 +1 @@
-2.0.0
+2.0.1

data/lib/slack_log_device.rb

@@ -88,14 +88,10 @@ class SlackLogDevice
     @mutex.synchronize do
       @buffer << message
     end
-    if flush?
+    @flush_thread.kill if @flush_thread
+    @flush_thread = Thread.new do
+      sleep(flush_delay) unless auto_flush?
       flush
-    else
-      @flush_thread.kill if @flush_thread
-      @flush_thread = Thread.new do
-        sleep(flush_delay)
-        flush
-      end
     end
     nil
   end

data/spec/slack_log_device_spec.rb

@@ -356,6 +356,7 @@ describe SlackLogDevice do
     it 'does nothing if log level is lower than specified one' do
       expect(HTTParty).not_to receive(:post)
       logger.debug('BIM!')
+      device.flush
     end
 
     it 'strips message' do
@@ -363,9 +364,16 @@ describe SlackLogDevice do
       expect(device.instance_variable_get(:@buffer)).to eq(['BAM  !'])
     end
 
+    it 'converts message to string' do
+      device.write(42)
+      expect(device.instance_variable_get(:@buffer)).to eq(['42'])
+    end
+
     it 'does nothing if message is blank' do
       expect(HTTParty).not_to receive(:post)
-      expect(device.write(" \n")).to be_nil
+      expect {
+        expect(device.write(" \n")).to be_nil
+      }.not_to change { device.instance_variable_get(:@buffer) }
     end
 
     it 'does nothing if message is nil' do
@@ -386,12 +394,6 @@ describe SlackLogDevice do
       device.flush
     end
 
-    it 'flush if auto flush is true' do
-      options[:auto_flush] = true
-      expect(HTTParty).to receive(:post)
-      device.write('BAM!')
-    end
-
     it 'does not post HTTP message if auto flush is false' do
       expect(HTTParty).not_to receive(:post)
       device.write('BAM!')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: slack_log_device
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.0.1
 platform: ruby
 authors:
 - Alexis Toulotte