wicoris-postman 0.11.1 → 0.11.2

data/lib/wicoris/postman/copier.rb

@@ -4,6 +4,8 @@ require 'digest/md5'
 module Wicoris
   module Postman
     class Copier
+      attr_reader :logger
+
       def initialize(job, opts = {})
         @job = job
         @opts = opts
@@ -13,6 +15,10 @@ module Wicoris
       # Copy letter to destination.
       def run
         FileUtils.cp(source, destination, :noop => (@opts[:noop] == true))
+        logger.debug :message => 'Letter copied',
+                     :destination => destination,
+                     :letter => @job.letter,
+                     :job => @job
       end
 
       private

data/lib/wicoris/postman/fax_machine.rb

@@ -8,6 +8,8 @@ module Wicoris
         \d{7,}$  # followed by at least 8 digits
       }x
 
+      attr_reader :logger
+
       def initialize(job, opts = {})
         @job = job
         @opts = opts
@@ -17,6 +19,10 @@ module Wicoris
       # Actually fax the letter.
       def run
         system(command) unless @opts[:noop]
+        logger.debug :message => 'Letter faxed',
+                     :phone => validated_phone,
+                     :letter => @job.letter,
+                     :job => @job
       end
 
       # @returns [String] Validated phone number.

data/lib/wicoris/postman/job.rb

@@ -50,6 +50,10 @@ module Wicoris
         'Unknown'
       end
 
+      def to_s
+        @json_file
+      end
+
       private
 
       def deliver

data/lib/wicoris/postman/version.rb

@@ -1,5 +1,5 @@
 module Wicoris
   module Postman
-    VERSION = '0.11.1'
+    VERSION = '0.11.2'
   end
 end

data/spec/wicoris/postman/copier_spec.rb

@@ -4,12 +4,15 @@ module Wicoris::Postman
   describe Copier do
     let(:job) { double('job') }
     let(:copier) { Copier.new(job, {}) }
+    let(:logger) { double('logger') }
 
     describe '#run' do
 
       before do
+        logger.stub(:debug)
         job.stub(:letter).and_return('/tmp/foo.pdf')
         copier.stub(:destination).and_return('/export')
+        copier.stub(:logger).and_return(logger)
       end
 
       context 'without noop' do

data/spec/wicoris/postman/fax_machine_spec.rb

@@ -12,6 +12,13 @@ module Wicoris::Postman
     describe '#run' do
       let(:logger) { double('logger') }
 
+      before do
+        logger.stub(:debug)
+        fax.stub(:logger).and_return(logger)
+        fax.stub(:validated_phone)
+        job.stub(:letter)
+      end
+
       it 'faxes the document' do
         fax.stub(:command).and_return('chunky bacon')
         fax.should_receive(:system).with('chunky bacon')

data/spec/wicoris/postman/job_spec.rb

@@ -154,6 +154,12 @@ module Wicoris::Postman
       end
     end
 
+    describe '#to_s' do
+      it 'returns the JSON file path' do
+        expect(job.to_s).to eq 'example.json'
+      end
+    end
+
     describe '#clear!' do
       before do
         job.stub(:json).and_return('')

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: wicoris-postman
 version: !ruby/object:Gem::Version 
-  hash: 49
+  hash: 55
   prerelease: 
   segments: 
   - 0
   - 11
-  - 1
-  version: 0.11.1
+  - 2
+  version: 0.11.2
 platform: ruby
 authors: 
 - "Bjo\xCC\x88rn Albers"
@@ -223,7 +223,7 @@ rubyforge_project:
 rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
-summary: wicoris-postman-0.11.1
+summary: wicoris-postman-0.11.2
 test_files: 
 - features/copy.feature
 - features/fax.feature