rivendell-import 0.0.1

data/spec/fixtures/mail-body.erb

@@ -0,0 +1,5 @@
+<% if tasks.count > 1 %>
+  <%= tasks.count %> tasks have been processed
+<% else %>
+A task is processed
+<% end %>

data/spec/rivendell/import/base_spec.rb

@@ -0,0 +1,125 @@
+require 'spec_helper'
+
+describe Rivendell::Import::Base do
+
+  describe "#prepare_task" do
+
+    let(:task) { mock }
+    
+    it "should prepare task with to_prepare block" do
+      subject.to_prepare = Proc.new {}
+      task.should_receive :prepare
+      subject.prepare_task task
+    end
+
+  end
+
+  describe "#create_task" do
+
+    let(:file) { Rivendell::Import::File.new "dummy.wav" }
+    
+    it "should create a task with given file" do
+      Rivendell::Import::Task.should_receive(:create).with({:file => file}, {})
+      subject.create_task file
+    end
+
+    it "should prepare task" do
+      subject.should_receive(:prepare_task)
+      subject.create_task file
+    end
+
+  end
+
+  describe "#file" do
+
+    let(:file) { Rivendell::Import::File.new("dummy.wav") }
+    
+    it "should create a File with given path and base_directory" do
+      Rivendell::Import::File.should_receive(:new).with("path", :base_directory => "base_directory")
+      subject.file "path", "base_directory"
+    end
+
+    it "should create a File with given path and base_directory" do
+      Rivendell::Import::File.stub :new=> file
+      subject.should_receive(:create_task).with(file)
+      subject.file "path", "base_directory"
+    end
+
+  end
+
+  describe "#directory" do
+    
+    it "should look for files in given directory" do
+      Dir.mktmpdir do |directory| 
+        FileUtils.mkdir "#{directory}/subdirectory"
+        
+        file = "#{directory}/subdirectory/dummy.wav"
+        FileUtils.touch file
+
+        subject.should_receive(:file).with(file, directory)
+        subject.directory directory
+      end
+    end
+
+  end
+  
+  describe "#process" do
+    
+    it "should use file method when path isn't a directory" do
+      File.stub :directory? => false
+      subject.should_receive(:file).with("dummy")
+      subject.process("dummy")
+    end
+
+    it "should use directory method when path is a directory" do
+      File.stub :directory? => true
+      subject.should_receive(:directory).with("dummy")
+      subject.process("dummy")
+    end
+
+  end
+
+  describe "#listen" do
+
+    before(:each) do
+      Listen.stub :to => true
+    end
+
+    let(:directory) { "directory" }
+    let(:worker) { mock }
+
+    before(:each) do
+      worker.stub :start => worker
+      Rivendell::Import::Worker.stub :new => worker
+    end
+    
+    it "should create a Worker" do
+      Rivendell::Import::Worker.should_receive(:new).with(subject).and_return(worker)
+      subject.listen directory
+      subject.workers.should == [ worker ]
+    end
+
+    it "should start Worker" do
+      worker.should_receive(:start).and_return(worker)
+      subject.listen directory
+    end
+
+    it "should not create Worker with dry_run option" do
+      subject.listen directory, :dry_run => true
+      subject.workers.should be_empty
+    end
+
+    it "should invoke Listen.to with given directory" do
+      Listen.should_receive(:to).with(directory)
+      subject.listen directory
+    end
+
+    it "should invoke file with added files" do
+      Listen.stub(:to).and_yield(nil,%w{file},nil)
+      subject.should_receive(:file).with("file", directory)
+      subject.listen directory
+    end
+
+  end
+
+end

data/spec/rivendell/import/cart_spec.rb

@@ -0,0 +1,132 @@
+require 'spec_helper'
+
+describe Rivendell::Import::Cart do
+
+  let(:task) { mock }
+  subject { Rivendell::Import::Cart.new task }
+
+  describe "initialization" do
+
+    it "should use the given task" do
+      Rivendell::Import::Cart.new(task).task.should == task
+    end
+    
+  end
+
+  describe "#xport" do
+
+    before(:each) do
+      task.stub :xport => mock
+    end
+    
+    it "should be task xport" do
+      subject.xport.should == subject.task.xport
+    end
+
+  end
+
+  describe "#create" do
+
+    before(:each) do
+      subject.stub :xport => mock
+    end
+
+    it "should use Xport#add_cart with Cart group" do
+      subject.group = "dummy"
+      subject.xport.should_receive(:add_cart).with(:group => subject.group).and_return(mock(:number => 123))
+      subject.create
+    end
+
+    it "should use the number returned by Xport#add_cart" do
+      subject.xport.stub(:add_cart).and_return(mock(:number => 123))
+      subject.group = "dummy"
+      subject.create
+      subject.number.should == 123
+    end
+
+    context "when number is already defined" do
+
+      before(:each) do
+        subject.number = 666
+      end
+
+      it "should not invoke Xport#add_cart" do
+        subject.xport.stub(:add_cart).and_return(mock(:number => 123))
+        subject.create
+        subject.number.should == 666
+      end
+
+    end
+
+    context "when group isn't defined" do
+
+      it "should raise an error" do
+        subject.group = nil
+        lambda { subject.create }.should raise_error
+      end
+                                         
+    end
+
+  end
+
+  describe "#cut" do
+
+    it "should return a Cut associated to this Cart" do
+      subject.cut.cart.should == subject
+    end
+
+  end
+
+  describe "#import" do
+
+    let(:file) { mock :path => "dummy", :exists? => true }
+
+    before(:each) do
+      subject.number = 123
+      subject.stub :xport => mock(:import => true)
+      subject.cut.stub :create => true, :number => 1, :update => true
+    end
+    
+    it "should create Cut" do
+      subject.cut.should_receive :create
+      subject.import file
+    end
+
+    it "should import file via xport with Cart and Cut numbers" do
+      subject.xport.should_receive(:import).with(subject.number, subject.cut.number, file.path)
+      subject.import file
+    end
+
+    it "should update Cut" do
+      subject.cut.should_receive :update
+      subject.import file
+    end
+
+  end
+
+  describe "#find_by_title" do
+
+    let(:cart) { mock :title => "The Title of the Cart", :number => 123 }
+
+    before(:each) do
+      subject.stub_chain("xport.list_carts").and_return([cart])
+    end
+
+    it "should find an exact title" do
+      subject.find_by_title(cart.title)
+      subject.number.should == cart.number
+    end
+
+    it "should find with a 'matching' filename ('the-title_of_the Cart' for 'The Title of the Cart')" do
+      subject.find_by_title("the-title_of_the Cart")
+      subject.number.should == cart.number
+    end
+
+    it "should use specified options to find carts" do
+      subject.xport.should_receive(:list_carts).with(:group => "TEST").and_return([cart])
+      subject.find_by_title("dummy", :group => "TEST")
+    end
+
+  end
+
+end

data/spec/rivendell/import/carts_cache_spec.rb

@@ -0,0 +1,110 @@
+require 'spec_helper'
+
+describe Rivendell::Import::CartsCache do
+
+  let(:carts_cache) { Rivendell::Import::CartsCache.new(xport) }
+  subject { carts_cache }
+
+  let(:cart) { mock :title => "dummy" }
+  let(:carts) { [ cart ] }
+
+  let(:xport) { mock }
+
+  before do
+    subject.stub :xport => xport
+  end
+
+  describe "#find_all_by_title" do
+
+    let(:options) { Hash.new }
+
+    before do
+      subject.stub(:carts).and_return(carts)
+    end
+    
+    it "should use carts with given options" do
+      subject.should_receive(:carts).with(options).and_return(carts)
+      subject.find_all_by_title("dummy", options)
+    end
+
+    it "should select with given title" do
+      subject.find_all_by_title(cart.title).should == [ cart ]
+    end
+
+    it "should use the given normalizer" do
+      normalizer = Proc.new { |s| s.downcase }
+      cart.stub :title => "the Title"
+      subject.find_all_by_title("The TITLE", :normalizer => normalizer).should == [ cart ]
+    end
+
+  end
+
+  describe "#default_normalizer" do
+
+    subject { carts_cache.default_normalizer }
+
+    it "should downcase string" do
+      subject.call("ABC").should == "abc"
+    end
+
+    it "should replace no alphanumeric characters by space" do
+      subject.call("a'b-c").should == "a b c"
+    end
+
+    it "should remove double spaces" do
+      subject.call("a  b   c").should == "a b c"
+    end
+    
+  end
+
+  describe "#carts" do
+
+    before do
+      subject.xport.stub :list_carts => carts
+    end
+    
+    it "should return Xport#list_carts result" do
+      subject.carts.should == carts
+    end
+
+    it "should cache the result" do
+      subject.xport.should_receive(:list_carts).once.and_return(carts)
+      2.times { subject.carts }
+    end
+
+    it "should reset cache after delay defined by cache_time_to_live" do
+      subject.carts # fill cache
+      subject.purged_at = Time.now - subject.time_to_live - 1
+
+      subject.xport.should_receive(:list_carts).and_return(carts)
+      subject.carts
+    end
+
+  end
+
+  describe "#find_by_title" do
+
+    before do
+      subject.stub(:carts).and_return(carts)
+    end
+    
+    it "should try an exact match" do
+      subject.find_by_title(cart.title).should == cart
+    end
+
+    it "should try a match with default normalizer" do
+      subject.find_by_title(cart.title.upcase).should == cart
+    end
+
+    it "should return nil when no cart matchs" do
+      subject.find_by_title("nothing").should be_nil
+    end
+
+    it "should return nil when several carts match" do
+      carts << cart
+      subject.find_by_title(cart.title).should be_nil
+    end
+
+  end
+
+end

data/spec/rivendell/import/cli_spec.rb

@@ -0,0 +1,149 @@
+require 'spec_helper'
+
+require 'rivendell/import/cli'
+
+describe Rivendell::Import::CLI do
+
+  describe "#config_file" do
+    
+    it "should return file specified with --config" do
+      subject.arguments << "--config" << "dummy"
+      subject.config_file.should == "dummy"
+    end
+
+  end
+
+  describe "listen_mode?" do
+
+    it "should return true when --listen is specified" do
+      subject.arguments << "--listen"
+      subject.should be_listen_mode
+    end
+    
+  end
+
+  describe "dry_run?" do
+    
+    it "should return true when --dry-run is specified" do
+      subject.arguments << "--dry-run"
+      subject.should be_dry_run
+    end
+
+  end
+
+  describe "debug?" do
+    
+    it "should return true when --debug is specified" do
+      subject.arguments << "--debug"
+      subject.should be_debug
+    end
+
+  end
+
+  describe "#import" do
+
+    it "should return a Rivendell::Import::Base instance" do
+      subject.import.should be_instance_of(Rivendell::Import::Base)
+    end
+
+  end
+
+  describe "#paths" do
+    
+    it "should return arguments after options" do
+      subject.arguments << "--listen"
+      subject.arguments << "file1" << "file2"
+      subject.paths.should == %w{file1 file2}
+    end
+
+  end
+
+  describe "#database" do
+    
+    it "should return file specified with --dabase" do
+      subject.arguments << "--database" << "tasks.sqlite3"
+      subject.database.should == "tasks.sqlite3"
+    end
+
+  end
+
+  describe "#run" do
+
+    before(:each) do
+      subject.stub :paths => %w{file1 file2}
+      subject.import.tasks.stub :run => true
+    end
+
+    it "should load config_file" do
+      subject.stub :config_file => "dummy.rb"
+      subject.should_receive(:load).with(subject.config_file)
+
+      subject.run
+    end
+
+    it "should establish_connection with specified database" do
+      subject.stub :database => "tasks.sqlite3"
+      Rivendell::Import.should_receive(:establish_connection).with("tasks.sqlite3")
+
+      subject.run
+    end
+
+    context "in listen_mode" do
+
+      let(:directory) { "directory" }
+
+      before(:each) do
+        subject.stub :listen_mode? => true
+        subject.stub :paths => [directory]
+      end
+  
+      it "should use listen import" do
+        subject.import.should_receive(:listen).with(directory, {})
+        subject.run
+      end
+
+      context "when dry_run" do
+        before(:each) do
+          subject.stub :dry_run? => true
+        end
+
+        it "should use dry_run listen option" do
+          subject.import.should_receive(:listen).with(anything, hash_including(:dry_run => true))
+          subject.run
+        end
+      end
+
+    end
+
+    context "without listen_mode" do
+
+      before(:each) do
+        subject.stub :listen_mode? => false
+      end
+  
+      it "should use process import" do
+        subject.import.should_receive(:process).with(subject.paths)
+        subject.run
+      end
+
+      it "should run tasks" do
+        subject.import.tasks.should_receive(:run)
+        subject.run
+      end
+
+      context "when dry_run" do
+        before(:each) do
+          subject.stub :dry_run? => true
+        end
+
+        it "should not run tasks" do
+          subject.import.tasks.should_not_receive(:run)
+          subject.run
+        end
+      end
+
+    end
+    
+  end
+
+end