dsturnbull-carrierwave 0.4.5

data/spec/storage/grid_fs_spec.rb

@@ -0,0 +1,82 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CarrierWave::Storage::GridFS do
+
+  before do
+    @database = Mongo::Connection.new('localhost').db('carrierwave_test')
+    @uploader = mock('an uploader')
+    @uploader.stub!(:grid_fs_database).and_return("carrierwave_test")
+    @uploader.stub!(:grid_fs_host).and_return("localhost")
+    @uploader.stub!(:grid_fs_access_url).and_return(nil)
+    @uploader.stub!(:grid_fs_username).and_return(nil)
+    @uploader.stub!(:grid_fs_password).and_return(nil)
+
+    @storage = CarrierWave::Storage::GridFS.new(@uploader)
+    @file = stub_tempfile('test.jpg', 'application/xml')
+  end
+  
+  after do
+    GridFS::GridStore.unlink(@database, 'uploads/bar.txt')
+  end
+
+  describe '#store!' do
+    before do
+      @uploader.stub!(:store_path).and_return('uploads/bar.txt')
+      @grid_fs_file = @storage.store!(@file)
+    end
+    
+    it "should upload the file to gridfs" do
+      GridFS::GridStore.read(@database, 'uploads/bar.txt').should == 'this is stuff'
+    end
+    
+    it "should not have a path" do
+      @grid_fs_file.path.should be_nil
+    end
+    
+    it "should not have a URL" do
+      @grid_fs_file.url.should be_nil
+    end
+    
+    it "should be deletable" do
+      @grid_fs_file.delete
+      GridFS::GridStore.read(@database, 'uploads/bar.txt').should == ''
+    end
+    
+    it "should store the content type on GridFS" do
+      @grid_fs_file.content_type.should == 'application/xml'
+    end
+  end
+  
+  describe '#retrieve!' do
+    before do
+      GridFS::GridStore.open(@database, 'uploads/bar.txt', 'w') { |f| f.puts "A test, 1234" }
+      @uploader.stub!(:store_path).with('bar.txt').and_return('uploads/bar.txt')
+      @grid_fs_file = @storage.retrieve!('bar.txt')
+    end
+
+    it "should retrieve the file contents from gridfs" do
+      @grid_fs_file.read.chomp.should == "A test, 1234"
+    end
+    
+    it "should not have a path" do
+      @grid_fs_file.path.should be_nil
+    end
+    
+    it "should not have a URL unless set" do
+      @grid_fs_file.url.should be_nil
+    end
+    
+    it "should return a URL if configured" do
+      @uploader.stub!(:grid_fs_access_url).and_return("/image/show")
+      @grid_fs_file.url.should == "/image/show/uploads/bar.txt"
+    end
+    
+    it "should be deletable" do
+      @grid_fs_file.delete
+      GridFS::GridStore.read(@database, 'uploads/bar.txt').should == ''
+    end
+  end
+
+end

data/spec/storage/right_s3_spec.rb

@@ -0,0 +1,75 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+require 'right_aws'
+
+if ENV['S3_SPEC']
+  describe CarrierWave::Storage::RightS3 do
+    before do
+      @bucket = ENV['CARRIERWAVE_TEST_BUCKET']
+      @uploader = mock('an uploader')
+      @uploader.stub!(:s3_access_key_id).and_return(ENV["S3_ACCESS_KEY_ID"])
+      @uploader.stub!(:s3_secret_access_key).and_return(ENV["S3_SECRET_ACCESS_KEY"])
+      @uploader.stub!(:s3_bucket).and_return(@bucket)
+      @uploader.stub!(:s3_access_policy).and_return('public-read')
+      @uploader.stub!(:s3_cnamed).and_return(false)
+
+      @storage = CarrierWave::Storage::RightS3.new(@uploader)
+      @file = CarrierWave::SanitizedFile.new(file_path('test.jpg'))
+    end
+  
+    after do
+      @storage.connection.delete(@bucket, 'uploads/bar.txt')
+    end
+
+    describe '#store!' do
+      before do
+        @uploader.stub!(:store_path).and_return('uploads/bar.txt')
+        @s3_file = @storage.store!(@file)
+      end
+    
+      it "should upload the file to s3" do
+        @storage.connection.get_object(@bucket, 'uploads/bar.txt').should == 'this is stuff'
+      end
+    
+      it "should have a path" do
+        @s3_file.path.should == 'uploads/bar.txt'
+      end
+    
+      it "should have an Amazon URL" do
+        @s3_file.url.should == "http://#{@bucket}.s3.amazonaws.com/uploads/bar.txt"
+      end
+    
+      it "should be deletable" do
+        @s3_file.delete
+        lambda {@storage.connection.head(@bucket, 'uploads/bar.txt')}.should raise_error(RightAws::AwsError)
+      end
+    end
+  
+    describe '#retrieve!' do
+      before do
+        @storage.connection.put(@bucket, "uploads/bar.txt", "A test, 1234", {'a-amz-acl' => 'public-read'})
+        @uploader.stub!(:store_path).with('bar.txt').and_return('uploads/bar.txt')
+        @s3_file = @storage.retrieve!('bar.txt')
+      end
+
+      it "should retrieve the file contents from s3" do
+        @s3_file.read.chomp.should == "A test, 1234"
+      end
+    
+      it "should have a path" do
+        @s3_file.path.should == 'uploads/bar.txt'
+      end
+    
+      it "should have an Amazon URL" do
+        @s3_file.url.should == "http://#{@bucket}.s3.amazonaws.com/uploads/bar.txt"
+      end
+    
+      it "should be deletable" do
+        @s3_file.delete
+        lambda {@storage.connection.head(@bucket, 'uploads/bar.txt')}.should raise_error(RightAws::AwsError)
+      end
+    end
+
+  end
+end

data/spec/storage/s3_spec.rb

@@ -0,0 +1,95 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+require 'aws/s3'
+require 'net/http'
+
+if ENV['S3_SPEC']
+  describe CarrierWave::Storage::S3 do
+    before do
+      @uploader = mock('an uploader')
+      @uploader.stub!(:s3_access_key_id).and_return(ENV["S3_ACCESS_KEY_ID"])
+      @uploader.stub!(:s3_secret_access_key).and_return(ENV["S3_SECRET_ACCESS_KEY"])
+      @uploader.stub!(:s3_bucket).and_return(ENV['CARRIERWAVE_TEST_BUCKET'])
+      @uploader.stub!(:s3_access).and_return(:public_read)
+      @uploader.stub!(:s3_cnamed).and_return(false)
+      @uploader.stub!(:s3_headers).and_return({'Expires' => 'Fri, 21 Jan 2021 16:51:06 GMT'})
+
+      @storage = CarrierWave::Storage::S3.new(@uploader)
+      @file = stub_tempfile('test.jpg', 'application/xml')
+    end
+  
+    after do
+      AWS::S3::S3Object.delete('uploads/bar.txt', @uploader.s3_bucket)
+    end
+
+    describe '#store!' do
+      before do
+        @uploader.stub!(:store_path).and_return('uploads/bar.txt')
+        @s3_file = @storage.store!(@file)
+      end
+    
+      it "should upload the file to s3" do
+        AWS::S3::S3Object.value('uploads/bar.txt', @uploader.s3_bucket).should == 'this is stuff'
+      end
+    
+      it "should have a path" do
+        @s3_file.path.should == 'uploads/bar.txt'
+      end
+    
+      it "should have an Amazon URL" do
+        @s3_file.url.should == "http://s3.amazonaws.com/#{@uploader.s3_bucket}/uploads/bar.txt"
+      end
+      
+      context "with cnamed bucket" do
+        it "should have a CNAMED URL" do
+          @uploader.stub!(:s3_cnamed).and_return(true)
+          @uploader.stub!(:s3_bucket).and_return('foo.bar')
+          @s3_file.url.should == 'http://foo.bar/uploads/bar.txt'
+        end
+      end
+    
+      it "should be deletable" do
+        @s3_file.delete
+        AWS::S3::S3Object.exists?('uploads/bar.txt', @uploader.s3_bucket).should be_false
+      end
+      
+      it "should store the content type on S3" do
+        @s3_file.content_type.should == 'application/xml'
+      end
+
+      it "should set headers" do
+        client = Net::HTTP.new("s3.amazonaws.com")
+        headers = client.request_head(URI.parse(@s3_file.url).path)
+        headers['Expires'].should == 'Fri, 21 Jan 2021 16:51:06 GMT'
+      end
+    end
+  
+    describe '#retrieve!' do
+      before do
+        AWS::S3::S3Object.store('uploads/bar.txt', "A test, 1234", @uploader.s3_bucket)
+  
+        @uploader.stub!(:store_path).with('bar.txt').and_return('uploads/bar.txt')
+        @s3_file = @storage.retrieve!('bar.txt')
+      end
+
+      it "should retrieve the file contents from s3" do
+        @s3_file.read.chomp.should == "A test, 1234"
+      end
+    
+      it "should have a path" do
+        @s3_file.path.should == 'uploads/bar.txt'
+      end
+    
+      it "should have an Amazon URL" do
+        @s3_file.url.should == "http://s3.amazonaws.com/#{@uploader.s3_bucket}/uploads/bar.txt"
+      end
+    
+      it "should be deletable" do
+        @s3_file.delete
+        AWS::S3::S3Object.exists?('uploads/bar.txt', @uploader.s3_bucket).should be_false
+      end
+    end
+
+  end
+end

data/spec/uploader/cache_spec.rb

@@ -0,0 +1,209 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CarrierWave::Uploader do
+
+  before do
+    @uploader_class = Class.new(CarrierWave::Uploader::Base)
+    @uploader = @uploader_class.new
+  end
+  
+  after do
+    FileUtils.rm_rf(public_path)
+  end
+
+  describe '.clean_cached_files!' do
+    before do
+      @cache_dir = File.expand_path(@uploader_class.cache_dir, CarrierWave.root)
+      FileUtils.mkdir_p File.expand_path('20071201-1234-234-2213', @cache_dir)
+      FileUtils.mkdir_p File.expand_path('20071203-1234-234-2213', @cache_dir)
+      FileUtils.mkdir_p File.expand_path('20071205-1234-234-2213', @cache_dir)
+    end
+
+    after { FileUtils.rm_rf(@cache_dir) }
+
+    it "should clear all files older than 24 hours in the default cache directory" do
+      Timecop.freeze(Time.parse('2007-12-06 10:12')) do
+        @uploader_class.clean_cached_files!
+      end
+      Dir.glob("#{@cache_dir}/*").should have(1).element
+    end
+
+    it "should be aliased on the CarrierWave module" do
+      Timecop.freeze(Time.parse('2007-12-06 10:12')) do
+        CarrierWave.clean_cached_files!
+      end
+      Dir.glob("#{@cache_dir}/*").should have(1).element
+    end
+  end
+
+  describe '#cache_dir' do
+    it "should default to the config option" do
+      @uploader.cache_dir.should == 'uploads/tmp'
+    end
+  end
+
+  describe '#cache!' do
+
+    before do
+      CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+    end
+
+    it "should cache a file" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.file.should be_an_instance_of(CarrierWave::SanitizedFile)
+    end
+
+    it "should be cached" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.should be_cached
+    end
+
+    it "should store the cache name" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.cache_name.should == '20071201-1234-345-2255/test.jpg'
+    end
+
+    it "should set the filename to the file's sanitized filename" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.filename.should == 'test.jpg'
+    end
+
+    it "should move it to the tmp dir" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.file.path.should == public_path('uploads/tmp/20071201-1234-345-2255/test.jpg')
+      @uploader.file.exists?.should be_true
+    end
+
+    it "should set the url" do
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.url.should == '/uploads/tmp/20071201-1234-345-2255/test.jpg'
+    end
+
+    it "should raise an error when trying to cache a string" do
+      running {
+        @uploader.cache!(file_path('test.jpg'))
+      }.should raise_error(CarrierWave::FormNotMultipart)
+    end
+
+    it "should raise an error when trying to cache a pathname" do
+      running {
+        @uploader.cache!(Pathname.new(file_path('test.jpg')))
+      }.should raise_error(CarrierWave::FormNotMultipart)
+    end
+
+    it "should do nothing when trying to cache an empty file" do
+      @uploader.cache!(nil)
+    end
+
+    it "should set permissions if options are given" do
+      @uploader_class.permissions = 0777
+
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.should have_permissions(0777)
+    end
+  end
+
+  describe '#retrieve_from_cache!' do
+    it "should cache a file" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.file.should be_an_instance_of(CarrierWave::SanitizedFile)
+    end
+
+    it "should be cached" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.should be_cached
+    end
+
+    it "should set the path to the tmp dir" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.current_path.should == public_path('uploads/tmp/20071201-1234-345-2255/test.jpeg')
+    end
+
+    it "should overwrite a file that has already been cached" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/bork.txt')
+      @uploader.current_path.should == public_path('uploads/tmp/20071201-1234-345-2255/bork.txt')
+    end
+
+    it "should store the cache_name" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.cache_name.should == '20071201-1234-345-2255/test.jpeg'
+    end
+
+    it "should store the filename" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.filename.should == 'test.jpeg'
+    end
+
+    it "should set the url" do
+      @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpeg')
+      @uploader.url.should == '/uploads/tmp/20071201-1234-345-2255/test.jpeg'
+    end
+
+    it "should raise an error when the cache_id has an invalid format" do
+      running {
+        @uploader.retrieve_from_cache!('12345/test.jpeg')
+      }.should raise_error(CarrierWave::InvalidParameter)
+
+      @uploader.file.should be_nil
+      @uploader.filename.should be_nil
+      @uploader.cache_name.should be_nil
+    end
+
+    it "should raise an error when the original_filename contains invalid characters" do
+      running {
+        @uploader.retrieve_from_cache!('20071201-1234-345-2255/te/st.jpeg')
+      }.should raise_error(CarrierWave::InvalidParameter)
+      running {
+        @uploader.retrieve_from_cache!('20071201-1234-345-2255/te??%st.jpeg')
+      }.should raise_error(CarrierWave::InvalidParameter)
+
+      @uploader.file.should be_nil
+      @uploader.filename.should be_nil
+      @uploader.cache_name.should be_nil
+    end
+  end
+
+  describe 'with an overridden, reversing, filename' do
+    before do
+      @uploader_class.class_eval do
+        def filename
+          super.reverse unless super.blank?
+        end
+      end
+    end
+
+    describe '#cache!' do
+
+      before do
+        CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+      end
+
+      it "should set the filename to the file's reversed filename" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.filename.should == "gpj.tset"
+      end
+
+      it "should move it to the tmp dir with the filename unreversed" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.current_path.should == public_path('uploads/tmp/20071201-1234-345-2255/test.jpg')
+        @uploader.file.exists?.should be_true
+      end
+    end
+
+    describe '#retrieve_from_cache!' do
+      it "should set the path to the tmp dir" do
+        @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpg')
+        @uploader.current_path.should == public_path('uploads/tmp/20071201-1234-345-2255/test.jpg')
+      end
+
+      it "should set the filename to the reversed name of the file" do
+        @uploader.retrieve_from_cache!('20071201-1234-345-2255/test.jpg')
+        @uploader.filename.should == "gpj.tset"
+      end
+    end
+  end
+
+end

data/spec/uploader/configuration_spec.rb

@@ -0,0 +1,105 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+
+describe CarrierWave do
+  before do
+    @uploader_class = Class.new(CarrierWave::Uploader::Base)
+  end
+
+  describe '.configure' do
+    it "should proxy to Uploader configuration" do
+      CarrierWave::Uploader::Base.add_config :test_config
+      CarrierWave.configure do |config|
+        config.test_config = "foo"
+      end
+      CarrierWave::Uploader::Base.test_config.should == 'foo'
+    end
+  end
+end
+
+describe CarrierWave::Uploader::Base do
+  before do
+    @uploader_class = Class.new(CarrierWave::Uploader::Base)
+  end
+
+  describe '.configure' do
+    it "should set a configuration parameter" do
+      @uploader_class.add_config :foo_bar
+      @uploader_class.configure do |config|
+        config.foo_bar = "monkey"
+      end
+      @uploader_class.foo_bar.should == 'monkey'
+    end
+  end
+  
+  describe ".storage" do
+    it "should set the storage if an argument is given" do
+      storage = mock('some kind of storage')
+      @uploader_class.storage storage
+      @uploader_class.storage.should == storage
+    end
+
+    it "should default to file" do
+      @uploader_class.storage.should == CarrierWave::Storage::File
+    end
+
+    it "should set the storage from the configured shortcuts if a symbol is given" do
+      @uploader_class.storage :file
+      @uploader_class.storage.should == CarrierWave::Storage::File
+    end
+
+    it "should remember the storage when inherited" do
+      @uploader_class.storage :s3
+      subclass = Class.new(@uploader_class)
+      subclass.storage.should == CarrierWave::Storage::S3
+    end
+
+    it "should be changeable when inherited" do
+      @uploader_class.storage :s3
+      subclass = Class.new(@uploader_class)
+      subclass.storage.should == CarrierWave::Storage::S3
+      subclass.storage :file
+      subclass.storage.should == CarrierWave::Storage::File
+    end
+  end
+  
+  
+  describe '.add_config' do
+    it "should add a class level accessor" do
+      @uploader_class.add_config :foo_bar
+      @uploader_class.foo_bar = 'foo'
+      @uploader_class.foo_bar.should == 'foo'
+    end
+    
+    ['foo', :foo, 45, ['foo', :bar]].each do |val|
+      it "should be inheritable for a #{val.class}" do
+        @uploader_class.add_config :foo_bar
+        @child_class = Class.new(@uploader_class)
+
+        @uploader_class.foo_bar = val
+        @uploader_class.foo_bar.should == val
+        @child_class.foo_bar.should == val
+
+        @child_class.foo_bar = "bar"
+        @child_class.foo_bar.should == "bar"
+
+        @uploader_class.foo_bar.should == val
+      end
+    end
+    
+    
+    it "should add an instance level accessor" do
+      @uploader_class.add_config :foo_bar
+      @uploader_class.foo_bar = 'foo'
+      @uploader_class.new.foo_bar.should == 'foo'
+    end
+    
+    it "should add a convenient in-class setter" do
+      @uploader_class.add_config :foo_bar
+      @uploader_class.foo_bar "monkey"
+      @uploader_class.foo_bar.should == "monkey"
+    end
+  end
+end

data/spec/uploader/default_url_spec.rb

@@ -0,0 +1,85 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CarrierWave::Uploader do
+
+  before do
+    @uploader_class = Class.new(CarrierWave::Uploader::Base)
+    @uploader = @uploader_class.new
+  end
+
+  after do
+    FileUtils.rm_rf(public_path)
+  end
+
+  describe 'with a default url' do
+    before do
+      @uploader_class.class_eval do
+        version :thumb
+        def default_url
+          ["http://someurl.example.com", version_name].compact.join('/')
+        end
+      end
+      @uploader = @uploader_class.new
+    end
+
+    describe '#blank?' do
+      it "should be true by default" do
+        @uploader.should be_blank
+      end
+    end
+
+    describe '#current_path' do
+      it "should return nil" do
+        @uploader.current_path.should be_nil
+      end
+    end
+    
+    describe '#url' do
+      it "should return the default url" do
+        @uploader.url.should == 'http://someurl.example.com'
+      end
+      
+      it "should return the default url with version when given" do
+        @uploader.url(:thumb).should == 'http://someurl.example.com/thumb'
+      end
+    end
+
+    describe '#cache!' do
+
+      before do
+        CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+      end
+
+      it "should cache a file" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.file.should be_an_instance_of(CarrierWave::SanitizedFile)
+      end
+
+      it "should be cached" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.should be_cached
+      end
+
+      it "should no longer be blank" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.should_not be_blank
+      end
+
+      it "should set the current_path" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.current_path.should == public_path('uploads/tmp/20071201-1234-345-2255/test.jpg')
+      end
+      
+      it "should set the url" do
+        @uploader.cache!(File.open(file_path('test.jpg')))
+        @uploader.url.should_not == 'http://someurl.example.com'
+        @uploader.url.should == '/uploads/tmp/20071201-1234-345-2255/test.jpg'
+      end
+
+    end
+
+  end
+
+end