test/devise/async/model_test.rb in devise-async-0.3.0 vs test/devise/async/model_test.rb in devise-async-0.3.1

@@ -1,21 +1,41 @@
 require "test_helper"
 
 module Devise
   module Async
     describe "Model" do
-      it "accumulates notifications to be send after commit" do
+      it "accumulates notifications to be sent after commit on Model creation" do
+        Admin.transaction do
+          admin = create_admin
+          admin.send(:devise_pending_notifications).must_equal [:confirmation_instructions]
+        end
+      end
+      
+      it "immediately sends notifications when the model has not changed" do
         admin = create_admin
+        Worker.expects(:enqueue).with(:confirmation_instructions, "Admin", admin.id.to_s)
         admin.send_confirmation_instructions
-        admin.send(:devise_pending_notifications).must_equal [:confirmation_instructions]
-        Worker.expects(:enqueue).never
       end
+      
+      it "accumulates notifications to be sent after commit when Model has been changed" do
+        admin = create_admin
+        Admin.transaction do
+          admin[:username] = "changed_username"
+          admin.send_confirmation_instructions
+          admin.send(:devise_pending_notifications).must_equal [:confirmation_instructions]
+          Worker.expects(:enqueue).never # after_commit will not fire without save
+        end
+      end
 
-      it "triggers enqueues the notifications on save" do
+      it "triggers the enqueued notifications on save" do
         admin = create_admin
-        admin.send_confirmation_instructions
-        Worker.expects(:enqueue).with(:confirmation_instructions, "Admin", admin.id.to_s)
-        admin.update_attribute(:username, "newusername")
+        Admin.transaction do
+          admin[:username] = "changed_username"
+          admin.send_confirmation_instructions
+          admin.send(:devise_pending_notifications).must_equal [:confirmation_instructions]
+          admin.save
+          Worker.expects(:enqueue).with(:confirmation_instructions, "Admin", admin.id.to_s)
+        end
       end
     end
   end
 end