adrift 0.0.1

data/spec/adrift/file_to_attach_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+module Adrift
+  describe FileToAttach do
+    context "when initialized with an unknown file representation" do
+      it "raises an error" do
+        expect {
+          FileToAttach.new(Object.new)
+        }.to raise_error(UnknownFileRepresentationError)
+      end
+    end
+
+    context "within a rails application" do
+      let(:file_to_attach) { FileToAttach.new(file_representation) }
+      let(:file_representation) do
+        double('uploaded file', {
+          :original_filename => 'me.png',
+          :tempfile => double('tempfile', :path => '/tmp/123')
+        })
+      end
+
+      describe "#original_filename" do
+        it "returns the original filename of the uploaded file" do
+          file_to_attach.original_filename.should ==
+            file_representation.original_filename
+        end
+      end
+
+      describe "#path" do
+        it "returns the uploaded tempfile's path" do
+          file_to_attach.path.should == file_representation.tempfile.path
+        end
+      end
+    end
+
+    context "within a rack (non rails) application" do
+      let(:file_to_attach) { FileToAttach.new(file_representation) }
+      let(:file_representation) do
+        {
+          :filename => 'me.png',
+          :tempfile => double('tempfile', :path => '/tmp/123')
+        }
+      end
+
+      describe "#original_filename" do
+        it "returns the original filename of the uploaded file" do
+          file_to_attach.original_filename.should ==
+            file_representation[:filename]
+        end
+      end
+
+      describe "#path" do
+        it "returns the uploaded tempfile's path" do
+          file_to_attach.path.should == file_representation[:tempfile].path
+        end
+      end
+    end
+
+    context "when initialized with a local file" do
+      let(:file_to_attach) { FileToAttach.new(file_representation) }
+      let(:file_representation) do
+        double('file', :to_path => '/avatars/me.png')
+      end
+
+      describe "#original_filename" do
+        it "returns the local file name" do
+          file_to_attach.original_filename.should == 'me.png'
+        end
+      end
+
+      describe "#path" do
+        it "returns the local file's path" do
+          file_to_attach.path.should == file_representation.to_path
+        end
+      end
+    end
+  end
+end

data/spec/adrift/integration/active_record_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+module Adrift::Integration
+  describe ActiveRecord do
+    it_behaves_like Adrift::Integration::Base do
+      before { klass.stub(after_save: nil, before_destroy: nil) }
+
+      describe ".attachment" do
+        it "registers an after save callback to save the attachments" do
+          klass.should_receive(:after_save).with(:save_attachments)
+          klass.attachment :avatar
+        end
+
+        it "registers a before destroy callback to destroy the attachments" do
+          klass.should_receive(:before_destroy).with(:destroy_attachments)
+          klass.attachment :avatar
+        end
+      end
+    end
+  end
+end

data/spec/adrift/integration/base_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+module Adrift::Integration
+  describe Base do
+    it_behaves_like Adrift::Integration::Base
+  end
+end

data/spec/adrift/integration/data_mapper_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+module Adrift::Integration
+  describe DataMapper do
+    it_behaves_like Adrift::Integration::Base do
+      before { klass.stub(after: nil, before:  nil) }
+
+      describe ".attachment" do
+        it "registers an after save callback to save the attachments" do
+          klass.should_receive(:after).with(:save, :save_attachments)
+          klass.attachment :avatar
+        end
+
+        it "registers a before destroy callback to destroy the attachments" do
+          klass.should_receive(:before).with(:destroy, :destroy_attachments)
+          klass.attachment :avatar
+        end
+      end
+    end
+  end
+end

data/spec/adrift/pattern_spec.rb

@@ -0,0 +1,98 @@
+require 'spec_helper'
+
+module Adrift
+  describe Pattern do
+    describe "#specialize" do
+      let(:attachment) { double('attachment') }
+
+      it "replaces :attachment with the pluralized attachment's name" do
+        attachment.stub(name: 'avatar')
+        Pattern.new(':attachment').specialize(attachment: attachment).should == 'avatars'
+      end
+
+      it "replaces :style with the given style" do
+        Pattern.new(':style').specialize(style: :normal).should == 'normal'
+      end
+
+      it "replaces :url with the attachment's url for the given style" do
+        attachment.stub(:url) do |style|
+          case style
+          when :original then '/attachment/url'
+          when :small    then '/attachment/small/url'
+          end
+        end
+        pattern = Pattern.new(':url')
+        pattern.specialize(attachment: attachment, style: :original).should == '/attachment/url'
+        pattern.specialize(attachment: attachment, style: :small).should == '/attachment/small/url'
+      end
+
+      context "when the model's class is a top-level one" do
+        it "replaces :class with  the pluralized model's class name" do
+          attachment.stub_chain(:model, :class, :name).and_return('User')
+          Pattern.new(':class').specialize(attachment: attachment).should == 'users'
+        end
+
+        it "replaces :class_name with the pluralized model's class name" do
+          attachment.stub_chain(:model, :class, :name).and_return('User')
+          Pattern.new(':class_name').specialize(attachment: attachment).should == 'users'
+        end
+      end
+
+      context "when the model's class is not a top-level one" do
+        it "replaces :class with the pluralized model's class name namespaced" do
+          attachment.stub_chain(:model, :class, :name).and_return('App::Models::User')
+          Pattern.new(':class').specialize(attachment: attachment).should == 'app/models/users'
+        end
+
+        it "replaces :class_name with the pluralized model's class name" do
+          attachment.stub_chain(:model, :class, :name).and_return('App::Models::User')
+          Pattern.new(':class_name').specialize(attachment: attachment).should == 'users'
+        end
+      end
+
+      it "replaces :id with the model's id" do
+        attachment.stub_chain(:model, :id).and_return(1)
+        Pattern.new(':id').specialize(attachment: attachment).should == '1'
+      end
+
+      context "when Pattern::Tags::Root.path has been assigned" do
+        after { Pattern::Tags::Root.path = nil }
+
+        it "returns Root.path" do
+          Pattern::Tags::Root.path = '/root/path'
+          Pattern.new(':root').specialize.should == '/root/path'
+        end
+      end
+
+      context "when Root.path hasn't been assigned" do
+        it "returns '.' by default" do
+          Pattern.new(':root').specialize.should == '.'
+        end
+      end
+
+      it "replaces :filename with the attachment's file name" do
+        attachment.stub(filename: 'me.png')
+        Pattern.new(':filename').specialize(attachment: attachment).should == 'me.png'
+      end
+
+      it "replaces :basename with the attachment's file basename" do
+        attachment.stub(filename: 'me.png')
+        Pattern.new(':basename').specialize(attachment: attachment).should == 'me'
+      end
+
+      it "replaces :extension with the attachment's file extension" do
+        attachment.stub(filename: 'me.png')
+        Pattern.new(':extension').specialize(attachment: attachment).should == 'png'
+      end
+
+      it "replaces a tag every time it appears" do
+        attachment.stub(name: 'avatar')
+        Pattern.new(':attachment/:attachment/:attachment').specialize(attachment: attachment).should == 'avatars/avatars/avatars'
+      end
+
+      it "doesn't replace unknown tags" do
+        Pattern.new(':unknown_tag').specialize.should == ':unknown_tag'
+      end
+    end
+  end
+end

data/spec/adrift/processor_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+module Adrift
+  module Processor
+    describe Thumbnail::Cli do
+      describe "#run" do
+        it "calls convert with the proper syntax" do
+          cli = Thumbnail::Cli.new
+          cli.should_receive('`').with('convert /tmp/123 -resize "50x50^" -gravity "center" -extent "50x50" /tmp/small-123')
+          cli.run('/tmp/123', '/tmp/small-123', :resize => '50x50^', :gravity => 'center', :extent => '50x50')
+        end
+      end
+    end
+
+    describe Thumbnail do
+      let(:cli) { double('cli').as_null_object }
+      let(:processor) { Thumbnail.new(cli) }
+
+      describe "#processed_files" do
+        context "for a newly instantiated processor" do
+          it "returns an empty hash" do
+            processor.processed_files.should == {}
+          end
+        end
+
+        context "immediately after a proccess" do
+          it "returns the generated files in the last process" do
+            processor.process('/tmp/123', :small => '50x50', :normal => '100x100')
+            processor.processed_files.should == { :small => '/tmp/small-123', :normal => '/tmp/normal-123' }
+          end
+        end
+
+        context "after two proccess" do
+          it "returns the generated files in the last process" do
+            processor.process('/tmp/123', :small => '50x50')
+            processor.process('/tmp/456', :normal => '100x100')
+            processor.processed_files.should == {  :normal => '/tmp/normal-456' }
+          end
+        end
+      end
+
+      describe "#process" do
+        it "doesn't do anything if no styles are given" do
+          processor.process('/tmp/123', {})
+          processor.processed_files.should be_empty
+        end
+
+        it "creates the thumbnails for the given styles" do
+          cli.should_receive(:run).with('/tmp/123', '/tmp/small-123', :resize => '50x50!')
+          cli.should_receive(:run).with('/tmp/123', '/tmp/normal-123', :resize => '100x100')
+          processor.process('/tmp/123', :small => '50x50!', :normal => '100x100')
+        end
+
+        it "crops the image when the style definition ends with '#'" do
+          cli.should_receive(:run).with('/tmp/123', '/tmp/small-123', :resize => '50x50^', :gravity => 'center', :background => 'None', :extent => '50x50')
+          processor.process('/tmp/123', :small => '50x50#')
+        end
+      end
+    end
+  end
+end

data/spec/adrift/storage_spec.rb

@@ -0,0 +1,181 @@
+require 'spec_helper'
+
+module Adrift::Storage
+  describe Filesystem do
+    let(:storage) { Filesystem.new }
+
+    describe "#dirty?" do
+      context "for a newly instantiated storage" do
+        it "returns false" do
+          storage.should_not be_dirty
+        end
+      end
+
+      context "when it hasn't been flushed" do
+        context "and there's a file that needs to be stored" do
+          it "reutrns false" do
+            storage.store('/tmp/1', '/path/to/storage/1/file')
+            storage.should be_dirty
+          end
+        end
+
+        context "and there's a file that needs to be removed" do
+          it "returns false" do
+            storage.remove('/path/to/storage/1/file')
+            storage.should be_dirty
+          end
+        end
+      end
+
+      context "immediately after a flush" do
+        it "returns true" do
+          storage.store('/tmp/1', '/path/to/storage/1/file')
+          storage.remove('/path/to/storage/2/file')
+          storage.flush
+          storage.should_not be_dirty
+        end
+      end
+    end
+
+    describe "#removed" do
+      context "for a newly instantiated storage" do
+        it "returns an empty array" do
+          storage.removed.should == []
+        end
+      end
+
+      context "immediately after a flush" do
+        before do
+          storage.remove('/path/to/storage/1/file')
+          storage.remove('/path/to/storage/2/file')
+          storage.flush
+        end
+
+        it "returns the files removed in the last flush" do
+          storage.removed.should include('/path/to/storage/1/file')
+          storage.removed.should include('/path/to/storage/2/file')
+          storage.removed.size.should == 2
+        end
+
+        context "when a file has been enqueued for removal" do
+          before { storage.remove('/path/to/storage/3/file') }
+
+          it "doesn't include it before a flush" do
+            storage.removed.should_not include('/path/to/storage/3/file')
+          end
+
+          it "only contains it after a flush" do
+            storage.flush
+            storage.removed.should == ['/path/to/storage/3/file']
+          end
+        end
+      end
+    end
+
+    describe "#stored" do
+      context "for a newly instantiated storage" do
+        it "returns an empty array" do
+          storage.stored.should == []
+        end
+      end
+
+      context "immediately after a flush" do
+        before do
+          storage.store('/tmp/1', '/path/to/storage/1/file')
+          storage.store('/tmp/2', '/path/to/storage/2/file')
+          storage.flush
+        end
+
+        it "returns the files stored in the last flush" do
+          storage.stored.should include(['/tmp/1', '/path/to/storage/1/file'])
+          storage.stored.should include(['/tmp/2', '/path/to/storage/2/file'])
+          storage.stored.size.should == 2
+        end
+
+        context "when a file has been enqueued for storage" do
+          before { storage.store('/tmp/3', '/path/to/storage/3/file') }
+
+          it "doesn't include it before a flush" do
+            storage.stored.should_not include(['/tmp/3', '/path/to/storage/3/file'])
+          end
+
+          it "only contains it after a flush" do
+            storage.flush
+            storage.stored.should == [['/tmp/3', '/path/to/storage/3/file']]
+          end
+        end
+      end
+    end
+
+    describe "#remove!" do
+      before { File.stub(:exist? => true) }
+
+      context "when there're files that need to be removed" do
+        before do
+          storage.remove('/path/to/storage/1/file')
+          storage.remove('/path/to/storage/2/file')
+        end
+
+        it "removes the specified files" do
+          FileUtils.should_receive(:rm).with('/path/to/storage/1/file')
+          FileUtils.should_receive(:rm).with('/path/to/storage/2/file')
+          storage.remove!
+        end
+
+        it "doesn't try to remove inexistent files" do
+          File.stub(:exist?) { |path| path != '/path/to/storage/1/file' }
+          FileUtils.should_not_receive(:rm).with('/path/to/storage/1/file')
+          FileUtils.should_receive(:rm).with('/path/to/storage/2/file')
+          storage.remove!
+        end
+      end
+    end
+
+    describe "#store!" do
+      before { File.stub(:exist? => true) }
+
+      context "when there're files that need to be stored" do
+        before do
+          storage.store('/tmp/1', '/path/to/storage/1/file')
+          storage.store('/tmp/2', '/path/to/storage/2/file')
+        end
+
+        it "tries to create the directories for the files to store" do
+          FileUtils.should_receive(:mkdir_p).with('/path/to/storage/1')
+          FileUtils.should_receive(:mkdir_p).with('/path/to/storage/2')
+          storage.store!
+        end
+
+        it "moves the specified files to their destination path" do
+          FileUtils.should_receive(:cp).with('/tmp/1', '/path/to/storage/1/file')
+          FileUtils.should_receive(:cp).with('/tmp/2', '/path/to/storage/2/file')
+          storage.store!
+        end
+
+        it "changes the permissions of the stored files" do
+          FileUtils.should_receive(:chmod).with(0644, '/path/to/storage/1/file')
+          FileUtils.should_receive(:chmod).with(0644, '/path/to/storage/2/file')
+          storage.store!
+        end
+      end
+
+      describe "for each file" do
+        it "creates its directory, then moves it there and then sets its permissions" do
+          storage.store('/tmp/1', '/path/to/storage/1/file')
+          FileUtils.should_receive(:mkdir_p).ordered
+          FileUtils.should_receive(:cp).ordered
+          FileUtils.should_receive(:chmod).ordered
+          storage.store!
+        end
+      end
+    end
+
+    describe "#flush" do
+      it "removes the enqueued files for removal and then stores the enqueued for storage" do
+        storage.should_receive(:remove!).ordered
+        storage.should_receive(:store!).ordered
+        storage.flush
+      end
+    end
+  end
+end