carrierwave_direct 0.0.16 → 3.0.0

data/spec/spec_helper.rb

@@ -3,6 +3,11 @@
 require 'carrierwave_direct'
 require 'json'
 require 'timecop'
+begin
+  require 'byebug'
+rescue LoadError
+  puts "Byebug not installed"
+end
 
 require File.dirname(__FILE__) << '/support/view_helpers' # Catch dependency order
 

data/spec/support/carrier_wave_config.rb

@@ -1,6 +1,7 @@
 # encoding: utf-8
 
 CarrierWave.configure do |config|
+  config.fog_provider = 'fog/aws'
   config.fog_credentials = {
     :provider               => 'AWS',
     :aws_access_key_id      => 'AWS_ACCESS_KEY_ID',

data/spec/test/capybara_helpers_spec.rb

@@ -15,8 +15,8 @@ describe CarrierWaveDirect::Test::CapybaraHelpers do
     allow(subject).to receive(:page).and_return(page)
   end
 
-  def find_element_value(css, value, options = nil)
-    if options
+  def find_element_value(css, value, **options)
+    if options.keys.any?
       allow(page).to receive(:find).with(css, options).and_return(selector)
     else
       allow(page).to receive(:find).with(css).and_return(selector)
@@ -36,10 +36,10 @@ describe CarrierWaveDirect::Test::CapybaraHelpers do
   describe "#upload_directly" do
     let(:uploader) { DirectUploader.new }
 
-    def upload_directly(options = {})
+    def upload_directly(**options)
       options[:button_locator] ||= ""
       button_locator = options.delete(:button_locator)
-      subject.upload_directly(uploader, button_locator, options)
+      subject.upload_directly(uploader, button_locator, **options)
     end
 
     def stub_common

data/spec/test/helpers_spec.rb

@@ -19,7 +19,7 @@ describe CarrierWaveDirect::Test::Helpers do
 
         context "['exe', 'bmp']" do
           before do
-            allow(direct_uploader).to receive(:extension_white_list).and_return(%w{exe bmp})
+            allow(direct_uploader).to receive(:extension_allowlist).and_return(%w{exe bmp})
           end
 
           it "should return '*/guid/filename.exe'" do
@@ -29,7 +29,7 @@ describe CarrierWaveDirect::Test::Helpers do
 
         context "[]" do
           before do
-            allow(direct_uploader).to receive(:extension_white_list).and_return([])
+            allow(direct_uploader).to receive(:extension_allowlist).and_return([])
           end
 
           it_should_behave_like "returning the default extension"
@@ -37,7 +37,7 @@ describe CarrierWaveDirect::Test::Helpers do
 
         context "nil" do
           before do
-            allow(direct_uploader).to receive(:extension_white_list).and_return(nil)
+            allow(direct_uploader).to receive(:extension_allowlist).and_return(nil)
           end
 
           it_should_behave_like "returning the default extension"

data/spec/uploader_spec.rb

@@ -7,10 +7,8 @@ describe CarrierWaveDirect::Uploader do
   include ModelHelpers
 
   let(:subject) { DirectUploader.new }
-  let(:mounted_model) { double(sample(:mounted_model_name), video_identifier: sample(:stored_filename)) }
+  let(:mounted_model) { MountedClass.new }
   let(:mounted_subject) { DirectUploader.new(mounted_model, sample(:mounted_as)) }
-  let(:direct_subject) { DirectUploader.new }
-
 
   DirectUploader.fog_credentials.keys.each do |key|
     describe "##{key}" do
@@ -74,8 +72,9 @@ describe CarrierWaveDirect::Uploader do
       context "but the uploaders url is '#{sample(:s3_file_url)}'" do
         before do
           allow(mounted_subject).to receive(:store_dir).and_return(sample(:store_dir))
-          allow(mounted_subject).to receive(:url).and_return(sample(:s3_file_url))
           allow(mounted_subject).to receive(:present?).and_return(true)
+          allow(mounted_model).to   receive(:video_identifier).and_return(sample(:stored_filename))
+          mounted_model.remote_video_url = sample(:s3_file_url)
         end
 
         it "should return '#{sample(:s3_key)}'" do
@@ -104,6 +103,14 @@ describe CarrierWaveDirect::Uploader do
     end
   end
 
+  describe "#direct_fog_url" do
+    it "should return the result from CarrierWave::Storage::Fog::File#public_url" do
+      expect(subject.direct_fog_url).to eq CarrierWave::Storage::Fog::File.new(
+        subject, nil, nil
+      ).public_url
+    end
+  end
+
   describe "#key_regexp" do
     it "should return a regexp" do
       expect(subject.key_regexp).to be_a(Regexp)
@@ -138,26 +145,26 @@ describe CarrierWaveDirect::Uploader do
       expect(subject.extension_regexp).to be_a(String)
     end
 
-    context "where #extension_white_list returns nil" do
+    context "where #extension_allowlist returns nil" do
       before do
-        allow(subject).to receive(:extension_white_list).and_return(nil)
+        allow(subject).to receive(:extension_allowlist).and_return(nil)
       end
 
       it_should_behave_like "a globally allowed file extension"
     end
 
-    context "where #extension_white_list returns []" do
+    context "where #extension_allowlist returns []" do
       before do
-        allow(subject).to receive(:extension_white_list).and_return([])
+        allow(subject).to receive(:extension_allowlist).and_return([])
       end
 
       it_should_behave_like "a globally allowed file extension"
     end
 
-    context "where #extension_white_list returns ['exe', 'bmp']" do
+    context "where #extension_allowlist returns ['exe', 'bmp']" do
 
       before do
-        allow(subject).to receive(:extension_white_list).and_return(%w{exe bmp})
+        allow(subject).to receive(:extension_allowlist).and_return(%w{exe bmp})
       end
 
       it "should return '(exe|bmp)'" do
@@ -275,6 +282,7 @@ describe CarrierWaveDirect::Uploader do
   end
 
   # http://aws.amazon.com/articles/1434?_encoding=UTF8
+  #http://docs.aws.amazon.com/AmazonS3/latest/API/sigv4-UsingHTTPPOST.html
   describe "#policy" do
 
 
@@ -473,31 +481,17 @@ describe CarrierWaveDirect::Uploader do
     end
   end
 
-  describe "#signature" do
-    it "should not contain any new lines" do
-      expect(subject.signature).to_not include("\n")
-    end
-
-    it "should return a base64 encoded 'sha1' hash of the secret key and policy document" do
-      expect(Base64.decode64(subject.signature)).to eq OpenSSL::HMAC.digest(
-        OpenSSL::Digest.new('sha1'),
-        subject.aws_secret_access_key, subject.policy
-      )
-    end
-  end
-
-
   # note that 'video' is hardcoded into the MountedClass support file
   # so changing the sample will cause the tests to fail
   context "a class has a '#{sample(:mounted_as)}' mounted" do
     describe "#{sample(:mounted_as).to_s.capitalize}Uploader" do
       describe "##{sample(:mounted_as)}" do
         it "should be defined" do
-          expect(direct_subject).to be_respond_to(sample(:mounted_as))
+          expect(subject).to be_respond_to(sample(:mounted_as))
         end
 
         it "should return itself" do
-          expect(direct_subject.send(sample(:mounted_as))).to eq direct_subject
+          expect(subject.send(sample(:mounted_as))).to eq subject
         end
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: carrierwave_direct
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 3.0.0
 platform: ruby
 authors:
 - David Wilkie
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-02 00:00:00.000000000 Z
+date: 2021-06-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: carrierwave
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 2.2.0
 - !ruby/object:Gem::Dependency
   name: fog-aws
   requirement: !ruby/object:Gem::Requirement
@@ -42,16 +42,16 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '3.0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -72,14 +72,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.12
+        version: 5.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 3.2.12
+        version: 5.1.0
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -108,6 +108,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Process your uploads in the background by uploading directly to S3
 email:
 - dwilkie@gmail.com
@@ -123,9 +137,10 @@ files:
 - README.md
 - Rakefile
 - carrierwave_direct.gemspec
-- gemfiles/3.2.gemfile
-- gemfiles/4.0.gemfile
-- gemfiles/4.1.gemfile
+- gemfiles/5.1.gemfile
+- gemfiles/5.2.gemfile
+- gemfiles/6.0.gemfile
+- gemfiles/6.1.gemfile
 - lib/carrierwave_direct.rb
 - lib/carrierwave_direct/action_view_extensions/form_helper.rb
 - lib/carrierwave_direct/form_builder.rb
@@ -133,12 +148,14 @@ files:
 - lib/carrierwave_direct/locale/nl.yml
 - lib/carrierwave_direct/mount.rb
 - lib/carrierwave_direct/orm/activerecord.rb
+- lib/carrierwave_direct/policies/aws4_hmac_sha256.rb
+- lib/carrierwave_direct/policies/aws_base64_sha1.rb
+- lib/carrierwave_direct/policies/base.rb
 - lib/carrierwave_direct/test/capybara_helpers.rb
 - lib/carrierwave_direct/test/helpers.rb
 - lib/carrierwave_direct/uploader.rb
 - lib/carrierwave_direct/uploader/configuration.rb
 - lib/carrierwave_direct/uploader/content_type.rb
-- lib/carrierwave_direct/uploader/direct_url.rb
 - lib/carrierwave_direct/validations/active_model.rb
 - lib/carrierwave_direct/version.rb
 - spec/action_view_extensions/form_helper_spec.rb
@@ -147,6 +164,8 @@ files:
 - spec/mount_spec.rb
 - spec/orm/activerecord_spec.rb
 - spec/orm/indirect_activerecord_spec.rb
+- spec/policies/aws4_hmac_sha256_spec.rb
+- spec/policies/aws_base64_sha1_spec.rb
 - spec/spec_helper.rb
 - spec/support/carrier_wave_config.rb
 - spec/support/direct_uploader.rb
@@ -160,7 +179,6 @@ files:
 - spec/test/helpers_spec.rb
 - spec/uploader/configuration_spec.rb
 - spec/uploader/content_type_spec.rb
-- spec/uploader/direct_url_spec.rb
 - spec/uploader_spec.rb
 homepage: https://github.com/dwilkie/carrierwave_direct
 licenses: []
@@ -180,8 +198,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: carrierwave_direct
-rubygems_version: 2.5.2.3
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Upload direct to S3 using CarrierWave
@@ -192,6 +209,8 @@ test_files:
 - spec/mount_spec.rb
 - spec/orm/activerecord_spec.rb
 - spec/orm/indirect_activerecord_spec.rb
+- spec/policies/aws4_hmac_sha256_spec.rb
+- spec/policies/aws_base64_sha1_spec.rb
 - spec/spec_helper.rb
 - spec/support/carrier_wave_config.rb
 - spec/support/direct_uploader.rb
@@ -205,5 +224,4 @@ test_files:
 - spec/test/helpers_spec.rb
 - spec/uploader/configuration_spec.rb
 - spec/uploader/content_type_spec.rb
-- spec/uploader/direct_url_spec.rb
 - spec/uploader_spec.rb

data/lib/carrierwave_direct/uploader/direct_url.rb

@@ -1,15 +0,0 @@
-module CarrierWaveDirect
-  module Uploader
-    module DirectUrl
-
-      def direct_fog_url(options = {})
-        if options[:with_path]
-          url
-        else
-          CarrierWave::Storage::Fog::File.new(self, CarrierWave::Storage::Fog.new(self), nil).public_url
-        end
-      end
-
-    end
-  end
-end

data/spec/uploader/direct_url_spec.rb

@@ -1,26 +0,0 @@
-# encoding: utf-8
-#
-require 'spec_helper'
-require 'data/sample_data'
-
-describe CarrierWaveDirect::Uploader::DirectUrl do
-
-  let(:subject) { DirectUploader.new }
-
-  let(:mounted_subject) { DirectUploader.new(mounted_model, sample(:mounted_as)) }
-
-  describe "#direct_fog_url" do
-    it "should return the result from CarrierWave::Storage::Fog::File#public_url" do
-      expect(subject.direct_fog_url).to eq CarrierWave::Storage::Fog::File.new(
-        subject, nil, nil
-      ).public_url
-    end
-
-    context ":with_path => true" do
-      it "should return the full url set by carrierwave" do
-        allow(subject).to receive(:url).and_return("url")
-        expect(subject.direct_fog_url(:with_path => true)).to eq "url"
-      end
-    end
-  end
-end