logging_worker 0.3.0 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: db4ba954cf6bbd1641c36fb30a943145728ffe4c
-  data.tar.gz: 53c8b58c918317b7414927b05656c47d17f47cc3
+  metadata.gz: cee20c47d414820e4304760caf55d2cf166844cd
+  data.tar.gz: 7cc1fc4d0e0dcae4adc8e714d84a92905727180f
 SHA512:
-  metadata.gz: 9130117d5dfe30aaecd34c60c4e377e35fda12591356009a23d6d739cf1fa4b63474ee7dd730417bdc0e90181be312c564f3ddaed1f294e98a4441331c68980e
-  data.tar.gz: e903262551220f97af34e8a57f5c70c3313af5c18119a0e0953fc0164a0d9d07f11bfa87646f082506b4f22102512086c664d0a776ba29d6189f7d95f41a0cb5
+  metadata.gz: 69f459050404932c790158e123c2e82b90b54cf6a573a2331d7f6c3cba3bfabdc4b247a87e8caad9bb17fe7711f40e82648d4ef1fb2ebe125708e8b97fd35ef8
+  data.tar.gz: f7d394a448b46448e1cd43b427b6ea14ce3b4fce7e58aac340a28d5b5765750ef6b49da033cee46362b0cf1d02977a2ccde9c42a571a3b4e29d589e53373c579

data/app/models/logging_worker/job_run.rb

@@ -5,12 +5,19 @@ class LoggingWorker::JobRun < ApplicationRecord
   end
 
   def logger
-    @logger ||= Logger.new(StringIO.new(log))
+    @io ||= StringIO.new
+    @logger ||= Logger.new(@io)
   end
 
-  def flush_log!
-    log_will_change!
-    save!
+  def flush_log!(save_to_db = true)
+    @io.rewind
+    content = @io.read
+    self.log += content
+    save! if save_to_db
+    @io.close
+    @logger.close
+    @io = nil
+    @logger = nil
   end
 
   def do_not_record!
@@ -27,6 +34,8 @@ class LoggingWorker::JobRun < ApplicationRecord
     else
       self.successful = false if successful.nil?
       self.completed_at = Time.now
+      logger # make sure we've got it
+      flush_log!(false)
       save!
     end
   end

data/lib/logging_worker/version.rb

@@ -1,3 +1,3 @@
 module LoggingWorker
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/lib/logging_worker/worker.rb

@@ -19,8 +19,6 @@ module LoggingWorker
       job_run.arguments = args
       job_run.save!
 
-      job_run.log_will_change!
-
       super
 
       job_run.successful!

data/spec/models/job_run_spec.rb

@@ -10,6 +10,7 @@ describe JobRun do
   specify "logging during the run" do
     job_run = JobRun.new
     job_run.logger.info("test")
+    job_run.flush_log!
     expect(job_run.log).to include("test")
   end
 end

data/spec/worker_spec.rb

@@ -79,9 +79,11 @@ describe LoggingWorker::Worker do
     run.flush_log!
     expect(LoggingWorker::JobRun.first.log).to include("Test this")
     expect(LoggingWorker::JobRun.first.log).to include("Testing more")
+    run.logger.info "Testing even more"
     run.completed!
     expect(LoggingWorker::JobRun.first.log).to include("Test this")
     expect(LoggingWorker::JobRun.first.log).to include("Testing more")
+    expect(LoggingWorker::JobRun.first.log).to include("Testing even more")
   end
 
   specify "self desctruct on complete" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: logging_worker
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Eric Oestrich
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-02-08 00:00:00.000000000 Z
+date: 2018-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails