carrierwave_backgrounder_revived 1.0.0

data/spec/backgrounder/workers/process_asset_spec.rb

@@ -0,0 +1,70 @@
+# encoding: utf-8
+require 'spec_helper'
+require 'backgrounder/workers/process_asset'
+
+RSpec.describe CarrierWave::Workers::ProcessAsset do
+  let(:worker_class) { CarrierWave::Workers::ProcessAsset }
+  let(:user)   { double('User') }
+  let!(:worker) { worker_class.new(user, '22', :image) }
+
+  describe ".perform" do
+    it 'creates a new instance and calls perform' do
+      args = [user, '22', :image]
+      expect(worker_class).to receive(:new).with(*args).and_return(worker)
+      expect_any_instance_of(worker_class).to receive(:perform)
+
+      worker_class.perform(*args)
+    end
+  end
+
+  describe "#perform" do
+    let(:image)  { double('UserAsset') }
+
+    before do
+      allow(user).to receive(:find).with('22').and_return(user).once
+      allow(user).to receive(:image).twice.and_return(image)
+      allow(user).to receive(:process_image_upload=).with(true).once
+      allow(image).to receive(:recreate_versions!).once.and_return(true)
+    end
+
+    it 'processes versions with image_processing column' do
+      expect(user).to receive(:respond_to?).with(:image_processing).once.and_return(true)
+      expect(user).to receive(:update_attribute).with(:image_processing, false).once
+      worker.perform
+    end
+
+    it 'processes versions without image_processing column' do
+      expect(user).to receive(:respond_to?).with(:image_processing).once.and_return(false)
+      expect(user).to receive(:update_attribute).never
+      worker.perform
+    end
+  end
+
+  describe '#perform with args' do
+    let(:admin) { double('Admin') }
+    let(:avatar)  { double('AdminAsset') }
+    let(:worker) { worker_class.new }
+
+    before do
+      allow(admin).to receive(:find).with('23').and_return(admin).once
+      allow(admin).to receive(:avatar).twice.and_return(avatar)
+      allow(admin).to receive(:process_avatar_upload=).with(true).once
+      allow(admin).to receive(:respond_to?).with(:avatar_processing).once.and_return(false)
+      allow(avatar).to receive(:recreate_versions!).once.and_return(true)
+
+      worker.perform admin, '23', :avatar
+    end
+
+    it 'sets klass' do
+      expect(worker.klass).to eql(admin)
+    end
+
+    it 'sets column' do
+      expect(worker.id).to eql('23')
+    end
+
+    it 'sets id' do
+      expect(worker.column).to eql(:avatar)
+    end
+  end
+end

data/spec/backgrounder/workers/store_asset_spec.rb

@@ -0,0 +1,137 @@
+# encoding: utf-8
+require 'spec_helper'
+require 'backgrounder/workers/store_asset'
+
+RSpec.describe CarrierWave::Workers::StoreAsset do
+  let(:fixtures_path) { File.expand_path('../fixtures/images', __FILE__) }
+  let(:worker_class) { CarrierWave::Workers::StoreAsset }
+  let(:user) { double('User') }
+  let!(:worker) { worker_class.new(user, '22', :image) }
+
+  describe ".perform" do
+    it 'creates a new instance and calls perform' do
+      args = [user, '22', :image]
+      expect(worker_class).to receive(:new).with(*args).and_return(worker)
+      expect_any_instance_of(worker_class).to receive(:perform)
+      worker_class.perform(*args)
+    end
+  end
+
+  describe "#perform" do
+    let(:image)  { double('UserAsset') }
+
+    before do
+      allow(image).to receive(:root).once.and_return(File.expand_path('..', __FILE__))
+      allow(image).to receive(:cache_dir).once.and_return('fixtures')
+      allow(user).to receive(:image_tmp).twice.and_return('images/test.jpg')
+      allow(user).to receive(:find).with('22').once.and_return(user)
+      allow(user).to receive(:image).once.and_return(image)
+      allow(user).to receive(:process_image_upload=).with(true).once
+      allow(user).to receive(:image=).once
+      allow(user).to receive(:image_tmp=).with(nil).once
+    end
+
+    it 'removes tmp directory on success' do
+      expect(FileUtils).to receive(:rm_r).with(fixtures_path, :force => true).once
+      expect(user).to receive(:save!).once.and_return(true)
+      worker.perform
+    end
+
+    it 'does not remove the tmp directory if save! fails' do
+      expect(FileUtils).to receive(:rm_r).never
+      expect(user).to receive(:save!).once.and_return(false)
+      worker.perform
+    end
+
+    it 'sets the cache_path' do
+      expect(user).to receive(:save!).once.and_return(false)
+      worker.perform
+      expect(worker.cache_path).to eql(fixtures_path + '/test.jpg')
+    end
+
+    it 'sets the tmp_directory' do
+      expect(user).to receive(:save!).once.and_return(false)
+      worker.perform
+      expect(worker.tmp_directory).to eql(fixtures_path)
+    end
+  end
+
+  describe '#perform with args' do
+    let(:admin) { double('Admin') }
+    let(:image)  { double('AdminAsset') }
+    let(:worker) { worker_class.new }
+
+    before do
+      allow(image).to receive(:root).once.and_return(File.expand_path('..', __FILE__))
+      allow(image).to receive(:cache_dir).once.and_return('fixtures')
+      allow(admin).to receive(:avatar_tmp).twice.and_return('images/test.jpg')
+      allow(admin).to receive(:find).with('23').once.and_return(admin)
+      allow(admin).to receive(:avatar).once.and_return(image)
+      allow(admin).to receive(:process_avatar_upload=).with(true).once
+      allow(admin).to receive(:avatar=).once
+      allow(admin).to receive(:avatar_tmp=).with(nil).once
+      allow(admin).to receive(:save!).once.and_return(false)
+      worker.perform admin, '23', :avatar
+    end
+
+    it 'sets klass' do
+      expect(worker.klass).to eql(admin)
+    end
+
+    it 'sets column' do
+      expect(worker.id).to eql('23')
+    end
+
+    it 'sets id' do
+      expect(worker.column).to eql(:avatar)
+    end
+  end
+
+  describe '#store_directories' do
+    let(:record) { double('Record') }
+
+    context 'cache_path' do
+      it 'sets the cache_path correctly if a full path is set for the cache_dir' do
+        root = '/Users/lar/Sites/bunker/public'
+        cache_dir = '/Users/lar/Sites/bunker/tmp/uploads'
+        asset = double(:cache_dir => cache_dir, :root => root)
+        expect(record).to receive(:image).and_return(asset)
+        expect(record).to receive(:image_tmp).and_return('images/test.jpg')
+        worker.send :store_directories, record
+        expect(worker.cache_path).to eql('/Users/lar/Sites/bunker/tmp/uploads/images/test.jpg')
+      end
+
+      it 'sets the cache_path correctly if a partial path is set for cache_dir' do
+        root = '/Users/lar/Sites/bunker/public'
+        cache_dir = 'uploads/tmp'
+        asset = double(:cache_dir => cache_dir, :root => root)
+        expect(record).to receive(:image).and_return(asset)
+        expect(record).to receive(:image_tmp).and_return('images/test.jpg')
+        worker.send :store_directories, record
+        expect(worker.cache_path).to eql('/Users/lar/Sites/bunker/public/uploads/tmp/images/test.jpg')
+      end
+    end
+
+    context 'tmp_directory' do
+      it 'sets the tmp_directory correctly if a full path is set for the cache_dir' do
+        root = '/Users/lar/Sites/bunker/public'
+        cache_dir = '/Users/lar/Sites/bunker/tmp/uploads'
+        asset = double(:cache_dir => cache_dir, :root => root)
+        expect(record).to receive(:image).and_return(asset)
+        expect(record).to receive(:image_tmp).and_return('images/test.jpg')
+        worker.send :store_directories, record
+        expect(worker.tmp_directory).to eql('/Users/lar/Sites/bunker/tmp/uploads/images')
+      end
+
+      it 'sets the tmp_directory correctly if a partial path is set for cache_dir' do
+        root = '/Users/lar/Sites/bunker/public'
+        cache_dir = 'uploads/tmp'
+        asset = double(:cache_dir => cache_dir, :root => root)
+        expect(record).to receive(:image).and_return(asset)
+        expect(record).to receive(:image_tmp).and_return('images/test.jpg')
+        worker.send :store_directories, record
+        expect(worker.tmp_directory).to eql('/Users/lar/Sites/bunker/public/uploads/tmp/images')
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,20 @@
+# encoding: utf-8
+require 'rubygems'
+require 'bundler/setup'
+require 'logger'
+require 'carrierwave_backgrounder'
+
+module WarningSuppression
+  def suppress_warnings
+    original_verbosity = $VERBOSE
+    $VERBOSE = nil
+    result = yield
+    $VERBOSE = original_verbosity
+    return result
+  end
+end
+
+RSpec.configure do |c|
+  c.include WarningSuppression
+end
+

data/spec/support/backend_constants.rb

@@ -0,0 +1,58 @@
+# Fixture module declarations for backend detection testing
+
+module GirlFriday
+  class WorkQueue
+  end
+end
+
+module Delayed
+  class Job
+    def self.column_names
+      %w(id priority attempts handler queue last_error run_at locked_at failed_at locked_by created_at updated_at)
+    end
+
+    column_names.each do |column_name|
+      define_method(column_name) { nil }
+    end
+  end
+end
+
+module Resque
+end
+
+module Qu
+end
+
+module Sidekiq
+  module Client
+  end
+
+  module Worker
+    def self.included(base)
+      base.extend(ClassMethods)
+    end
+
+    module ClassMethods
+      def sidekiq_options(opts = {})
+        opts
+      end
+
+      def client_push(item)
+      end
+    end
+  end
+end
+
+module QC
+end
+
+module SuckerPunch
+  class Queue
+  end
+end
+
+module Rails
+  def self.logger
+    @logger ||= Logger.new(STDOUT)
+  end
+end

data/spec/support/mock_worker.rb

@@ -0,0 +1,22 @@
+class MockWorker < Struct.new(:klass, :id, :column)
+  def self.perform(*args)
+    new(*args).perform
+  end
+
+  def perform(*args)
+    set_args(*args) unless args.empty?
+  end
+
+  def set_args(klass, id, column)
+    self.klass, self.id, self.column = klass, id, column
+  end
+end
+
+class MockSidekiqWorker < MockWorker
+  include Sidekiq::Worker
+end
+
+class MockNamedSidekiqWorker < MockWorker
+  include Sidekiq::Worker
+  sidekiq_options queue: :even_better_name
+end

metadata

@@ -0,0 +1,151 @@
+--- !ruby/object:Gem::Specification
+name: carrierwave_backgrounder_revived
+version: !ruby/object:Gem::Version
+  version: 1.0.0
+platform: ruby
+authors:
+- Larry Sprock
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2018-09-28 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: carrierwave
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0.5'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+- !ruby/object:Gem::Dependency
+  name: mime-types
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.99'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.99'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.5.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 3.5.0
+- !ruby/object:Gem::Dependency
+  name: rake
+  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: 
+email:
+- larry@lucidbleu.com
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- ".gitignore"
+- ".rspec"
+- ".travis.yml"
+- CHANGELOG.md
+- Gemfile
+- README.md
+- Rakefile
+- carrierwave_backgrounder.gemspec
+- lib/backgrounder/delay.rb
+- lib/backgrounder/orm/activemodel.rb
+- lib/backgrounder/orm/base.rb
+- lib/backgrounder/orm/data_mapper.rb
+- lib/backgrounder/railtie.rb
+- lib/backgrounder/support/backends.rb
+- lib/backgrounder/version.rb
+- lib/backgrounder/workers.rb
+- lib/backgrounder/workers/base.rb
+- lib/backgrounder/workers/class_methods.rb
+- lib/backgrounder/workers/process_asset.rb
+- lib/backgrounder/workers/process_asset_mixin.rb
+- lib/backgrounder/workers/store_asset.rb
+- lib/backgrounder/workers/store_asset_mixin.rb
+- lib/carrierwave_backgrounder.rb
+- lib/generators/carrierwave_backgrounder/USAGE
+- lib/generators/carrierwave_backgrounder/install_generator.rb
+- lib/generators/carrierwave_backgrounder/templates/config/initializers/carrierwave_backgrounder.rb
+- spec/backgrounder/orm/activemodel_spec.rb
+- spec/backgrounder/orm/base_spec.rb
+- spec/backgrounder/support/backends_spec.rb
+- spec/backgrounder/workers/fixtures/images/test.jpg
+- spec/backgrounder/workers/process_asset_spec.rb
+- spec/backgrounder/workers/store_asset_spec.rb
+- spec/spec_helper.rb
+- spec/support/backend_constants.rb
+- spec/support/mock_worker.rb
+homepage: https://github.com/lardawge/carrierwave_backgrounder
+licenses:
+- MIT
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - ">="
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.6.11
+signing_key: 
+specification_version: 4
+summary: Offload CarrierWave's image processing and storage to a background process
+  using Delayed Job, Resque, Sidekiq, Qu, Queue Classic or Girl Friday
+test_files:
+- spec/backgrounder/orm/activemodel_spec.rb
+- spec/backgrounder/orm/base_spec.rb
+- spec/backgrounder/support/backends_spec.rb
+- spec/backgrounder/workers/fixtures/images/test.jpg
+- spec/backgrounder/workers/process_asset_spec.rb
+- spec/backgrounder/workers/store_asset_spec.rb
+- spec/spec_helper.rb
+- spec/support/backend_constants.rb
+- spec/support/mock_worker.rb