carrierwave-rails3 0.4.5

data/spec/uploader/callback_spec.rb

@@ -0,0 +1,24 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CarrierWave::Uploader do
+  
+  it "should keep callbacks on different classes isolated" do
+    @uploader_class_1 = Class.new(CarrierWave::Uploader::Base)
+    
+    # First Uploader only has default before-callback
+    @uploader_class_1._before_callbacks[:cache].should == [:check_whitelist!]
+    
+    @uploader_class_2 = Class.new(CarrierWave::Uploader::Base)
+    @uploader_class_2.before :cache, :before_cache_callback
+
+    # Second Uploader defined with another callback
+    @uploader_class_2._before_callbacks[:cache].should == [:check_whitelist!, :before_cache_callback]
+    
+    # Make sure the first Uploader doesn't inherit the same callback
+    @uploader_class_1._before_callbacks[:cache].should == [:check_whitelist!]
+  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

data/spec/uploader/download_spec.rb

@@ -0,0 +1,75 @@
+# encoding: utf-8
+
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe CarrierWave::Uploader::Download do
+
+  before do
+    @uploader_class = Class.new(CarrierWave::Uploader::Base)
+    @uploader = @uploader_class.new
+  end
+  
+  after do
+    FileUtils.rm_rf(public_path)
+  end
+
+  describe '#download!' do
+
+    before do
+      CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+      response = mock('HTTP Response')
+      response.stub!(:body).and_return('Response Body')
+      Net::HTTP.stub!(:get_response).and_return(response)
+    end
+
+    it "should cache a file" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.file.should be_an_instance_of(CarrierWave::SanitizedFile)
+    end
+
+    it "should be cached" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.should be_cached
+    end
+
+    it "should store the cache name" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.cache_name.should == '20071201-1234-345-2255/file.png'
+    end
+
+    it "should set the filename to the file's sanitized filename" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.filename.should == 'file.png'
+    end
+
+    it "should move it to the tmp dir" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.file.path.should == public_path('uploads/tmp/20071201-1234-345-2255/file.png')
+      @uploader.file.exists?.should be_true
+    end
+
+    it "should set the url" do
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.url.should == '/uploads/tmp/20071201-1234-345-2255/file.png'
+    end
+
+    it "should do nothing when trying to download an empty file" do
+      @uploader.download!(nil)
+    end
+
+    it "should set permissions if options are given" do
+      @uploader_class.permissions = 0777
+
+      @uploader.download!('http://www.example.com/test/file.png')
+      @uploader.should have_permissions(0777)
+    end
+
+    it "should raise an error when trying to download a local file" do
+      running {
+        @uploader.download!('/etc/passwd')
+      }.should raise_error(CarrierWave::DownloadError)
+    end
+  end
+
+end
+

data/spec/uploader/extension_whitelist_spec.rb

@@ -0,0 +1,44 @@
+# 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 '#cache!' do
+
+    before do
+      CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+    end
+
+    it "should not raise an integiry error if there is no white list" do
+      @uploader.stub!(:extension_white_list).and_return(nil)
+      running {
+        @uploader.cache!(File.open(file_path('test.jpg')))
+      }.should_not raise_error(CarrierWave::IntegrityError)
+    end
+
+    it "should not raise an integiry error if there is a white list and the file is on it" do
+      @uploader.stub!(:extension_white_list).and_return(%w(jpg gif png))
+      running {
+        @uploader.cache!(File.open(file_path('test.jpg')))
+      }.should_not raise_error(CarrierWave::IntegrityError)
+    end
+
+    it "should raise an integiry error if there is a white list and the file is not on it" do
+      @uploader.stub!(:extension_white_list).and_return(%w(txt doc xls))
+      running {
+        @uploader.cache!(File.open(file_path('test.jpg')))
+      }.should raise_error(CarrierWave::IntegrityError)
+    end
+  end
+
+end

data/spec/uploader/mountable_spec.rb

@@ -0,0 +1,33 @@
+# 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 '#model' do
+    it "should be remembered from initialization" do
+      model = mock('a model object')
+      @uploader = @uploader_class.new(model)
+      @uploader.model.should == model
+    end
+  end
+
+  describe '#mounted_as' do
+    it "should be remembered from initialization" do
+      model = mock('a model object')
+      @uploader = @uploader_class.new(model, :llama)
+      @uploader.model.should == model
+      @uploader.mounted_as.should == :llama
+    end
+  end
+
+end

data/spec/uploader/paths_spec.rb

@@ -0,0 +1,22 @@
+# 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 '#root' do
+    it "should default to the config option" do
+      @uploader.root.should == public_path
+    end
+  end
+
+end

data/spec/uploader/processing_spec.rb

@@ -0,0 +1,73 @@
+# 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 '.process' do
+    it "should add a single processor when a symbol is given" do
+      @uploader_class.process :sepiatone
+      @uploader.should_receive(:sepiatone)
+      @uploader.process!
+    end
+
+    it "should add multiple processors when an array of symbols is given" do
+      @uploader_class.process :sepiatone, :desaturate, :invert
+      @uploader.should_receive(:sepiatone)
+      @uploader.should_receive(:desaturate)
+      @uploader.should_receive(:invert)
+      @uploader.process!
+    end
+
+    it "should add a single processor with an argument when a hash is given" do
+      @uploader_class.process :format => 'png'
+      @uploader.should_receive(:format).with('png')
+      @uploader.process!
+    end
+
+    it "should add a single processor with several argument when a hash is given" do
+      @uploader_class.process :resize => [200, 300]
+      @uploader.should_receive(:resize).with(200, 300)
+      @uploader.process!
+    end
+
+    it "should add multiple processors when an hash with multiple keys is given" do
+      @uploader_class.process :resize => [200, 300], :format => 'png'
+      @uploader.should_receive(:resize).with(200, 300)
+      @uploader.should_receive(:format).with('png')
+      @uploader.process!
+    end
+    
+    context "with 'enable_processing' set to false" do
+      it "should not do any processing" do
+        @uploader_class.enable_processing = false
+        @uploader_class.process :sepiatone, :desaturate, :invert
+        @uploader.should_not_receive(:sepiatone)
+        @uploader.should_not_receive(:desaturate)
+        @uploader.should_not_receive(:invert)
+        @uploader.process!
+      end
+    end
+  end
+
+  describe '#cache!' do
+    before do
+      CarrierWave.stub!(:generate_cache_id).and_return('20071201-1234-345-2255')
+    end
+
+    it "should trigger a process!" do
+      @uploader.should_receive(:process!)
+      @uploader.cache!(File.open(file_path('test.jpg')))
+    end
+  end
+
+end