spec/backgrounder/orm/activemodel_spec.rb in carrierwave_backgrounder-0.1.2 vs spec/backgrounder/orm/activemodel_spec.rb in carrierwave_backgrounder-0.1.3

@@ -11,16 +11,73 @@
       def remote_avatar_url; OpenStruct.new(:present? => true); end
       def previous_changes; {}; end
     end
 
     @mock_class.extend CarrierWave::Backgrounder::ORM::ActiveModel
-    @mock_class.process_in_background :avatar
   end
 
+  describe '.store_in_background' do
+    context 'setting up callbacks' do
+      it 'creates an after_commit hook' do
+        @mock_class.expects(:after_commit).with(:enqueue_avatar_background_job, :if => :enqueue_avatar_background_job?)
+        @mock_class.store_in_background :avatar
+      end
+    end
+  end
+
+  describe '.process_in_background' do
+    context 'setting up callbacks' do
+      it 'creates a before_save hook' do
+        @mock_class.expects(:before_save).with(:set_avatar_processing, :if => :enqueue_avatar_background_job?)
+        @mock_class.process_in_background :avatar
+      end
+
+      it 'creates an after_save hook' do
+        @mock_class.expects(:after_commit).with(:enqueue_avatar_background_job, :if => :enqueue_avatar_background_job?)
+        @mock_class.process_in_background :avatar
+      end
+    end
+  end
+
   describe '#trigger_column_background_processing?' do
-    it "calls up to processs column upload getter in the base class" do
-      instance = @mock_class.new
+    let(:instance) { @mock_class.new }
+
+    before do
+      @mock_class.process_in_background :avatar
+    end
+
+    it "returns true if process_avatar_upload is false" do
       instance.expects(:process_avatar_upload)
-      instance.trigger_avatar_background_processing?
+      expect(instance.enqueue_avatar_background_job?).to be_true
+    end
+
+    it "calls column_changed?" do
+      instance.expects(:process_avatar_upload).returns(false)
+      instance.expects(:avatar_changed?)
+      expect(instance.enqueue_avatar_background_job?).to be_true
+    end
+
+    it "calls previous_changes" do
+      instance.expects(:process_avatar_upload).returns(false)
+      instance.expects(:avatar_changed?).returns(false)
+      instance.expects(:previous_changes).returns({:avatar => true})
+      expect(instance.enqueue_avatar_background_job?).to be_true
+    end
+
+    it "calls avatar_remote_url" do
+      instance.expects(:process_avatar_upload).returns(false)
+      instance.expects(:avatar_changed?).returns(false)
+      instance.expects(:previous_changes).returns({})
+      instance.expects(:remote_avatar_url).returns('yup')
+      expect(instance.enqueue_avatar_background_job?).to be_true
+    end
+
+    it "calls avatar_cache" do
+      instance.expects(:process_avatar_upload).returns(false)
+      instance.expects(:avatar_changed?).returns(false)
+      instance.expects(:previous_changes).returns({})
+      instance.expects(:remote_avatar_url).returns(nil)
+      instance.expects(:avatar_cache).returns('yup')
+      expect(instance.enqueue_avatar_background_job?).to be_true
     end
   end
 end