carrierwave_direct 0.0.13 → 0.0.14

data/lib/carrierwave_direct/uploader/configuration.rb

@@ -1,7 +1,6 @@
 # encoding: utf-8
 
 module CarrierWaveDirect
-
   module Uploader
     module Configuration
       extend ActiveSupport::Concern
@@ -16,7 +15,10 @@ module CarrierWaveDirect
         add_config :min_file_size
         add_config :max_file_size
         add_config :upload_expiration
-        add_config :will_include_content_type
+
+        add_config :will_include_content_type #alias for default_content_type
+        add_config :default_content_type
+        add_config :allowed_content_types
 
         add_config :use_action_status
 

data/lib/carrierwave_direct/uploader/content_type.rb

@@ -0,0 +1,28 @@
+module CarrierWaveDirect
+  module Uploader
+    module ContentType
+
+      def content_type
+        default_content_type ? default_content_type : 'binary/octet-stream'
+      end
+
+      def content_types
+        types = allowed_content_types
+
+        return types if types.is_a? Array
+
+        %w(application/atom+xml application/ecmascript application/json
+          application/javascript application/octet-stream application/ogg
+          application/pdf application/postscript application/rss+xml
+          application/font-woff application/xhtml+xml application/xml
+          application/xml-dtd application/zip application/gzip audio/basic
+          audio/mp4 audio/mpeg audio/ogg audio/vorbis audio/vnd.rn-realaudio
+          audio/vnd.wave audio/webm image/gif image/jpeg image/pjpeg
+          image/png image/svg+xml image/tiff text/cmd text/css text/csv
+          text/html text/javascript text/plain text/vcard text/xml video/mpeg
+          video/mp4 video/ogg video/quicktime video/webm video/x-matroska
+          video/x-ms-wmv video/x-flv)
+      end
+    end
+  end
+end

data/lib/carrierwave_direct/uploader/direct_url.rb

@@ -0,0 +1,18 @@
+module CarrierWaveDirect
+  module Uploader
+    module DirectUrl
+
+      def direct_fog_url(options = {})
+        fog_uri = CarrierWave::Storage::Fog::File.new(self, CarrierWave::Storage::Fog.new(self), nil).public_url
+        if options[:with_path]
+          uri = URI.parse(fog_uri.chomp('/'))
+          path = "/#{URI.decode(key)}"
+          uri.path += URI.escape(path)
+          fog_uri = uri.to_s
+        end
+        fog_uri
+      end
+
+    end
+  end
+end

data/lib/carrierwave_direct/validations/active_model.rb

@@ -12,7 +12,8 @@ module CarrierWaveDirect
       class UniqueFilenameValidator < ::ActiveModel::EachValidator
         def validate_each(record, attribute, value)
           if record.new_record? && record.errors[attribute].empty? && (record.send("has_#{attribute}_upload?") || record.send("has_remote_#{attribute}_net_url?"))
-            if record.class.where(attribute => record.send(attribute).filename).exists?
+            column = record.class.uploader_options[attribute].fetch(:mount_on, attribute)
+            if record.class.where(column => record.send(attribute).filename).exists?
               record.errors.add(attribute, :carrierwave_direct_filename_taken)
             end
           end

data/lib/carrierwave_direct/version.rb

@@ -1,6 +1,6 @@
 # encoding: utf-8
 
 module CarrierwaveDirect
-  VERSION = "0.0.13"
+  VERSION = "0.0.14"
 end
 

data/spec/action_view_extensions/form_helper_spec.rb

@@ -8,28 +8,27 @@ describe CarrierWaveDirect::ActionViewExtensions::FormHelper do
   describe "#direct_upload_form_for" do
     it "should yield an instance of CarrierWaveDirect::FormBuilder" do
       direct_upload_form_for(direct_uploader) do |f|
-        f.should be_instance_of(CarrierWaveDirect::FormBuilder)
+        expect(f).to be_instance_of(CarrierWaveDirect::FormBuilder)
       end
     end
 
     context "the form" do
       before do
-        direct_uploader.stub(:direct_fog_url).and_return("http://example.com")
+        allow(direct_uploader).to receive(:direct_fog_url).and_return("http://example.com")
       end
 
       it "should post to the uploader's #direct_fog_url as a multipart form" do
-        form.should submit_to(
+        expect(form).to submit_to(
           :action => "http://example.com", :method => "post", :enctype => "multipart/form-data"
         )
       end
 
       it "should include any html options passed as through :html" do
-        form(:html => { :target => "_blank_iframe" }).should submit_to(
+        expect(form(:html => { :target => "_blank_iframe" })).to submit_to(
           :action => "http://example.com", :method => "post", :enctype => "multipart/form-data", :target => "_blank_iframe"
         )
       end
     end
   end
-
 end
 

data/spec/data/sample_data.rb

@@ -0,0 +1,40 @@
+SAMPLE_DATA = {
+  :path => "upload_dir/bliind.exe",
+  :path_with_special_chars => "upload_dir/some file & blah.exe",
+  :path_with_escaped_chars => "upload_dir/some%20file%20&%20blah.exe",
+  :key => "some key",
+  :guid => "guid",
+  :store_dir => "store_dir",
+  :extension_regexp => "(avi)",
+  :url => "http://example.com/some_url",
+  :expiration => 60,
+  :min_file_size => 1024,
+  :max_file_size => 10485760,
+  :file_url => "http://anyurl.com/any_path/video_dir/filename.avi",
+  :mounted_model_name => "Porno",
+  :mounted_as => :video,
+  :filename => "filename",
+  :extension => ".avi",
+  :version => :thumb,
+  :s3_bucket_url => "https://s3-bucket.s3.amazonaws.com"
+}
+
+SAMPLE_DATA.merge!(
+  :stored_filename_base => "#{sample(:guid)}/#{sample(:filename)}"
+)
+
+SAMPLE_DATA.merge!(
+  :stored_filename => "#{sample(:stored_filename_base)}#{sample(:extension)}",
+  :stored_version_filename => "#{sample(:stored_filename_base)}_#{sample(:version)}#{sample(:extension)}"
+)
+
+SAMPLE_DATA.merge!(
+  :s3_key => "#{sample(:store_dir)}/#{sample(:stored_filename)}"
+)
+
+SAMPLE_DATA.merge!(
+  :s3_file_url => "#{sample(:s3_bucket_url)}/#{sample(:s3_key)}"
+)
+
+SAMPLE_DATA.freeze
+

data/spec/form_builder_spec.rb

@@ -1,6 +1,45 @@
 # encoding: utf-8
 
 require 'spec_helper'
+require 'erb'
+
+class CarrierWaveDirect::FormBuilder
+  attr_accessor :template, :object
+
+  public :content_choices_options
+end
+
+shared_examples_for 'hidden values form' do
+  hidden_fields = [
+                    :key,
+                    {:aws_access_key_id => "AWSAccessKeyId"},
+                    :acl,
+                    :success_action_redirect,
+                    :policy,
+                    :signature
+                  ]
+
+  hidden_fields.each do |input|
+    if input.is_a?(Hash)
+      key = input.keys.first
+      name = input[key]
+    else
+      key = name = input
+    end
+
+    it "should have a hidden field for '#{name}'" do
+      allow(direct_uploader).to receive(key).and_return(key.to_s)
+      expect(subject).to have_input(
+        :direct_uploader,
+        key,
+        :type => :hidden,
+        :name => name,
+        :value => key,
+        :required => false
+      )
+    end
+  end
+end
 
 describe CarrierWaveDirect::FormBuilder do
   include FormBuilderHelpers
@@ -12,7 +51,11 @@ describe CarrierWaveDirect::FormBuilder do
     end
 
     def form_with_file_field_and_no_redirect
-      form {|f| f.file_field :video, use_action_status: true }
+      allow(@direct_uploader.class).to receive(:use_action_status).and_return(true)
+
+      form do |f|
+        f.file_field :video
+      end
     end
 
     default_hidden_fields = [
@@ -34,6 +77,8 @@ describe CarrierWaveDirect::FormBuilder do
 
     # http://aws.amazon.com/articles/1434?_encoding=UTF8
     context "form" do
+      let(:subject) {form_with_default_file_field}
+      it_should_behave_like 'hidden values form'
 
       default_hidden_fields.each do |input|
         if input.is_a?(Hash)
@@ -44,8 +89,8 @@ describe CarrierWaveDirect::FormBuilder do
         end
 
         it "should have a hidden field for '#{name}'" do
-          direct_uploader.stub(key).and_return(key.to_s)
-          form_with_default_file_field.should have_input(
+          allow(direct_uploader).to receive(key).and_return(key.to_s)
+          expect(form_with_default_file_field).to have_input(
             :direct_uploader,
             key,
             :type => :hidden,
@@ -65,8 +110,8 @@ describe CarrierWaveDirect::FormBuilder do
         end
 
         it "should have a hidden field for '#{name}'" do
-          direct_uploader.stub(key).and_return(key.to_s)
-          form_with_file_field_and_no_redirect.should have_input(
+          allow(direct_uploader).to receive(key).and_return(key.to_s)
+          expect(form_with_file_field_and_no_redirect).to have_input(
             :direct_uploader,
             key,
             :type => :hidden,
@@ -78,7 +123,7 @@ describe CarrierWaveDirect::FormBuilder do
       end
 
       it "should have an input for a file to upload" do
-        form_with_default_file_field.should have_input(
+        expect(form_with_default_file_field).to have_input(
           :direct_uploader,
           :video,
           :type => :file,
@@ -88,4 +133,92 @@ describe CarrierWaveDirect::FormBuilder do
       end
     end
   end
+
+  describe "#content_type_select" do
+    context "form" do
+      let(:subject) do
+        form do |f|
+          f.content_type_select
+        end
+      end
+
+      before do
+        allow(direct_uploader.class).to receive(:will_include_content_type).and_return(true)
+      end
+
+      it 'should select the default content type' do
+        allow(direct_uploader).to receive(:content_type).and_return('video/mp4')
+        expect(subject).to have_content_type 'video/mp4', true
+      end
+
+      it 'should include the default content types' do
+        allow(direct_uploader).to receive(:content_types).and_return(['text/foo','text/bar'])
+        expect(subject).to have_content_type 'text/foo', false
+        expect(subject).to have_content_type 'text/bar', false
+      end
+
+      it 'should select the passed in content type' do
+        dom = form {|f| f.content_type_select nil, 'video/mp4'}
+        expect(dom).to have_content_type 'video/mp4', true
+      end
+
+      it 'should include most content types' do
+        %w(application/atom+xml application/ecmascript application/json application/javascript application/octet-stream application/ogg application/pdf application/postscript application/rss+xml application/font-woff application/xhtml+xml application/xml application/xml-dtd application/zip application/gzip audio/basic audio/mp4 audio/mpeg audio/ogg audio/vorbis audio/vnd.rn-realaudio audio/vnd.wave audio/webm image/gif image/jpeg image/pjpeg image/png image/svg+xml image/tiff text/cmd text/css text/csv text/html text/javascript text/plain text/vcard text/xml video/mpeg video/mp4 video/ogg video/quicktime video/webm video/x-matroska video/x-ms-wmv video/x-flv).each do |type|
+          expect(subject).to have_content_type type
+        end
+      end
+    end
+  end
+
+  describe "#content_type_label" do
+    context "form" do
+      let(:subject) do
+        form do |f|
+          f.content_type_label
+        end
+      end
+
+    end
+  end
+
+  describe 'full form' do
+    let(:dom) do
+      form do |f|
+        f.content_type_label <<
+        f.content_type_select <<
+        f.file_field(:video)
+      end
+    end
+
+    before do
+      allow(direct_uploader).to receive('key').and_return('foo')
+      allow(direct_uploader.class).to receive(:will_include_content_type).and_return(true)
+    end
+
+    it 'should only include the hidden values once' do
+      expect(dom).to have_input(
+                   :direct_uploader,
+                   'key',
+                   :type => :hidden,
+                   :name => 'key',
+                   :value => 'foo',
+                   :required => false,
+                   :count => 1
+                 )
+    end
+
+    it 'should include Content-Type twice' do
+      expect(dom).to have_input(
+                   :direct_uploader,
+                   :content_type,
+                   :type => :hidden,
+                   :name => 'Content-Type',
+                   :value => 'binary/octet-stream',
+                   :required => false,
+                   :count => 1
+                 )
+
+      expect(dom).to have_selector :xpath, './/select[@name="Content-Type"]', :count => 1
+    end
+  end
 end

data/spec/mount_spec.rb

@@ -17,43 +17,30 @@ describe CarrierWaveDirect::Mount do
     it_should_have_accessor(:remote_video_net_url)
 
     describe "#has_video_upload?" do
-      context "video does not have a key" do
-        before { subject.video.stub(:has_key?).and_return(false) }
-
-        it "should return false" do
-          subject.should_not have_video_upload
-        end
+      it "returns false when video does not have a key" do
+        allow(subject.video).to receive(:has_key?).and_return(false)
+        expect(subject).to_not have_video_upload
       end
 
-      context "video has a key" do
-        before { subject.video.stub(:has_key?).and_return(true) }
-
-        it "should return true" do
-          subject.should have_video_upload
-        end
+      it "returns true when video has a key" do
+        allow(subject.video).to receive(:has_key?).and_return(true)
+        expect(subject).to have_video_upload
       end
     end
 
     describe "#has_remote_video_net_url?" do
-      context "remote_video_net_url is nil" do
-        before { subject.remote_video_net_url = nil }
-
-        it "should return false" do
-          subject.should_not have_remote_video_net_url
-        end
+      it "returns false when remote_video_net_url is nil" do
+        subject.remote_video_net_url = nil
+        expect(subject).to_not have_remote_video_net_url
       end
 
-      context "remote_video_net_url is not nil" do
-        before { subject.remote_video_net_url = "something" }
-
-        it "should return true" do
-          subject.should have_remote_video_net_url
-        end
+      it "returns true when remote_video_net_url is not nil" do
+        subject.remote_video_net_url = :not_nil
+        expect(subject).to have_remote_video_net_url
       end
     end
 
-    it_should_delegate(:key, :to => "video#key", :accessible => {"has_video_upload?" => false})
-
+    it_should_delegate(:key, :to => "video#key", :accessible => { "has_video_upload?" => false })
   end
 end
 

data/spec/orm/activerecord_spec.rb

@@ -15,10 +15,15 @@ describe CarrierWaveDirect::ActiveRecord do
       create_table :parties, :force => true do |t|
         t.column :video, :string
       end
+
+      create_table :dances, :force => true do |t|
+        t.column :location, :string
+      end
     end
 
     def self.down
       drop_table :parties
+      drop_table :dances
     end
   end
 
@@ -26,6 +31,9 @@ describe CarrierWaveDirect::ActiveRecord do
     mount_uploader :video, DirectUploader
   end
 
+  class Dance < ActiveRecord::Base
+  end
+
   ActiveRecord::Base.establish_connection(dbconfig)
 
   # turn off migration output
@@ -61,17 +69,17 @@ describe CarrierWaveDirect::ActiveRecord do
 
     shared_examples_for "an invalid filename" do
       it "should not be valid on create" do
-        subject.should_not be_valid
+        expect(subject).to_not be_valid
       end
 
       it "should be valid on update" do
         subject.save(:validate => false)
-        subject.should be_valid
+        expect(subject).to be_valid
       end
 
       it "should use i18n for the error messages" do
         subject.valid?
-        subject.errors[:video].should == [
+        expect(subject.errors[:video]).to eq [
           I18n.t("errors.messages.carrierwave_direct_filename_invalid") +
           I18n.t("errors.messages.carrierwave_direct_allowed_extensions", :extensions => %w{avi mp4}.to_sentence)
         ]
@@ -96,7 +104,7 @@ describe CarrierWaveDirect::ActiveRecord do
           end
         end
 
-        subject.errors[:remote_video_net_url].should == [ messages ]
+        expect(subject.errors[:remote_video_net_url]).to eq [ messages ]
       end
     end
 
@@ -150,6 +158,17 @@ describe CarrierWaveDirect::ActiveRecord do
         mount_uploader
       end
 
+      context "mount_on: option is used" do
+        let(:dance) { Dance.new }
+
+        before { Dance.mount_uploader(:non_existing_column, DirectUploader, mount_on: :location)    }
+        before { dance.non_existing_column.key = sample_key}
+
+        it "uses the column it's mounted on for checking uniqueness" do
+          expect { dance.valid? }.to_not raise_error
+        end
+      end
+
       context "another Party with a duplicate video filename" do
         before do
           subject.video.key = sample_key