durran-carrierwave 0.3.2.3

data/spec/spec_helper.rb

@@ -0,0 +1,99 @@
+# encoding: utf-8
+
+$TESTING=true
+$:.unshift File.join(File.dirname(__FILE__), '..', 'lib')
+
+require 'rubygems'
+
+if ENV["AS"]
+  puts "--> using ActiveSupport"
+  require 'activesupport'
+elsif ENV["EXTLIB"]
+  puts "--> using Extlib"
+  require 'extlib'
+end
+
+require 'tempfile'
+#require 'ruby-debug'
+require 'spec'
+require 'spec/autorun'
+
+require 'carrierwave'
+
+require 'logger'
+CarrierWave.logger = Logger.new(nil)
+
+alias :running :lambda
+
+def file_path( *paths )
+  File.expand_path(File.join(File.dirname(__FILE__), 'fixtures', *paths))
+end
+
+def public_path( *paths )
+  File.expand_path(File.join(File.dirname(__FILE__), 'public', *paths))
+end
+
+CarrierWave.config[:public] = public_path
+CarrierWave.config[:root] = File.expand_path(File.dirname(__FILE__))
+
+module CarrierWave
+  module Test
+    module MockStorage
+      def mock_storage(kind)
+        storage = mock("storage for #{kind} uploader")
+        storage.stub!(:setup!)
+        storage
+      end
+    end
+    
+    module MockFiles
+      def stub_merb_tempfile(filename)
+        raise "#{path} file does not exist" unless File.exist?(file_path(filename))
+
+        t = Tempfile.new(filename)
+        FileUtils.copy_file(file_path(filename), t.path)
+    
+        return t
+      end
+  
+      def stub_tempfile(filename, mime_type=nil, fake_name=nil)
+        raise "#{path} file does not exist" unless File.exist?(file_path(filename))
+
+        t = Tempfile.new(filename)
+        FileUtils.copy_file(file_path(filename), t.path)
+    
+        # This is stupid, but for some reason rspec won't play nice...
+        eval <<-EOF
+        def t.original_filename; '#{fake_name || filename}'; end
+        def t.content_type; '#{mime_type}'; end
+        def t.local_path; path; end
+        EOF
+
+        return t
+      end
+
+      def stub_stringio(filename, mime_type=nil, fake_name=nil)
+        if filename
+          t = StringIO.new( IO.read( file_path( filename ) ) )
+        else
+          t = StringIO.new
+        end
+        t.stub!(:local_path).and_return("")
+        t.stub!(:original_filename).and_return(filename || fake_name)
+        t.stub!(:content_type).and_return(mime_type)
+        return t
+      end
+
+      def stub_file(filename, mime_type=nil, fake_name=nil)
+        f = File.open(file_path(filename))
+        return f
+      end
+    end
+  end
+end
+
+Spec::Runner.configure do |config|
+  config.include CarrierWave::Test::Matchers
+  config.include CarrierWave::Test::MockFiles
+  config.include CarrierWave::Test::MockStorage
+end

data/spec/uploader/cache_spec.rb

@@ -0,0 +1,196 @@
+# 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_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 not move it if cache_to_cache_dir is false" do
+      CarrierWave.config[:cache_to_cache_dir] = false
+      path = file_path('test.jpg')
+      @uploader.cache!(File.open(path))
+      @uploader.current_path.should == path
+      @uploader.file.exists?.should be_true
+      CarrierWave.config[:cache_to_cache_dir] = 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
+      old_permissions = CarrierWave.config[:permissions]
+      CarrierWave.config[:permissions] = 0777
+
+      @uploader.cache!(File.open(file_path('test.jpg')))
+      @uploader.should have_permissions(0777)
+
+      CarrierWave.config[:permissions] = old_permissions
+    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/default_path_spec.rb

@@ -0,0 +1,68 @@
+# 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 path' do
+    before do
+      @uploader_class.class_eval do
+        def default_path
+          file_path('test.jpg')
+        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 the default path" do
+        @uploader.current_path.should == file_path('test.jpg')
+      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
+
+    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