tobacco 0.0.1 → 0.0.2

data/lib/tobacco/exhaler.rb

@@ -17,7 +17,7 @@ module Tobacco
     end
 
     def write_content_to_file
-      File.open(filepath, 'w') do |f|
+      File.open(filepath, 'wb') do |f|
         f.write content
       end
     end

data/lib/tobacco/smoker.rb

@@ -41,6 +41,10 @@ module Tobacco
         callback(:on_success, modified_content)
 
       rescue => e
+        Tobacco.log("ErrorWriting: #{filepath}")
+
+        # Remove the empty file
+        File.delete(filepath) if File.exists?(filepath)
 
         error = error_object('Error Writing', modified_content, e)
         callback(:on_write_error, error)
@@ -126,7 +130,7 @@ module Tobacco
     def error_object(msg, modified_content, e)
       Tobacco::Error.new(
         msg: msg,
-        filepath: file_path_generator.filepath,
+        filepath: file_path_generator.output_filepath,
         content: modified_content,
         object: e
       )

data/lib/tobacco/version.rb

@@ -1,3 +1,3 @@
 module Tobacco
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/lib/tobacco.rb

@@ -27,12 +27,14 @@ module Tobacco
   end
 
   def self.log(msg)
-    msg = "LOGGING: #{msg}"
+    log_msg =  "\n*******************************************\n"
+    log_msg += "Tobacco::Log: #{msg}\n"
+    log_msg += "*******************************************\n"
 
     if defined? Rails
-      Rails.logger.info(msg)
+      Rails.logger.info(log_msg)
     else
-      puts msg
+      puts log_msg
     end
   end
 end

data/spec/tobacco/smoker_spec.rb

@@ -21,10 +21,10 @@ describe Tobacco::Smoker do
 
     context 'when content is empty' do
       let(:inhaler)  { mock('inhaler', read: '') }
-      let(:filepath) { mock('roller', content_url: '/video', filepath: '/user') }
+      let(:filepath) { mock('roller', content_url: '/video', output_filepath: '/user') }
 
       before do
-        Tobacco::Inhaler.should_receive(:new).and_return(inhaler)
+        Tobacco::Inhaler.stub(:new).and_return(inhaler)
         subject.file_path_generator = filepath
       end
 
@@ -39,10 +39,10 @@ describe Tobacco::Smoker do
     context 'content' do
       let(:content) { 'Directly set content' }
       let(:exhaler) { mock('exhaler', write!: true) }
-      let(:filepath) { mock('roller', content_url: '/video', filepath: '/user', output_filepath: '/desktop') }
+      let(:filepath) { mock('roller', content_url: '/video', output_filepath: '/desktop') }
 
       before do
-        Tobacco::Exhaler.should_receive(:new).and_return(exhaler)
+        Tobacco::Exhaler.stub(:new).and_return(exhaler)
         subject.file_path_generator = filepath
       end
 
@@ -121,7 +121,7 @@ describe Tobacco::Smoker do
       let(:filepath) { mock('roller', content_url: '/video') }
 
       before do
-        Tobacco::Inhaler.should_receive(:new).and_return(inhaler)
+        Tobacco::Inhaler.stub(:new).and_return(inhaler)
         subject.file_path_generator = filepath
       end
 
@@ -152,7 +152,7 @@ describe Tobacco::Smoker do
       let(:filepath) { mock('roller', content_url: '/video') }
 
       before do
-        Tobacco::Inhaler.should_receive(:new).and_return(inhaler)
+        Tobacco::Inhaler.stub(:new).and_return(inhaler)
         subject.file_path_generator = filepath
       end
 
@@ -176,11 +176,11 @@ describe Tobacco::Smoker do
 
       describe '#on_read_error' do
         let(:inhaler)  { mock('inhaler', read: '') }
-        let(:filepath) { mock('roller', content_url: '/video', filepath: '/file/path') }
+        let(:filepath) { mock('roller', content_url: '/video', output_filepath: '/file/path') }
         let(:smoker)   { Writer.new }
 
         before do
-          Tobacco::Inhaler.should_receive(:new).and_return(inhaler)
+          Tobacco::Inhaler.stub(:new).and_return(inhaler)
           subject.file_path_generator = filepath
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tobacco
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors: