More testing for the creation/update process

spec/controllers/admin/log_entries_controller_spec.rb

@@ -2,27 +2,94 @@
 
 describe Admin::LogEntriesController, type: :controller do
   describe "POST create" do
-    def post_create(logeable: create(:customer))
+    def post_create(action: "create", logeable: create(:customer))
       resource_params = attributes_for(:log_entry).merge(
+        action: action,
         logeable: {
           type: "Administrate::Field::Polymorphic",
           value: logeable.to_global_id.to_s,
-        }
+        },
       )
       post :create, log_entry: resource_params
     end
 
-    it "creates a new LogEntry" do
-      customer = create(:customer)
-      expect {
-        post_create(logeable: customer)
-      }.to change(LogEntry, :count).by(1)
-      expect(LogEntry.last.logeable).to eq(customer)
+    describe "with valid params" do
+      it "creates a new LogEntry" do
+        customer = create(:customer)
+        expect do
+          post_create(logeable: customer)
+        end.to change(LogEntry, :count).by(1)
+        expect(LogEntry.last.logeable).to eq(customer)
+      end
+
+      it "redirects to the created customer" do
+        post_create
+        expect(response).to redirect_to([:admin, LogEntry.last])
+      end
+    end
+
+    describe "with invalid params" do
+      it "passes a form page object to the view" do
+        customer = create(:customer)
+
+        locals = capture_view_locals do
+          post_create(action: "cancel", logeable: customer)
+        end
+
+        page = locals[:page]
+        expect(page).to be_instance_of(Administrate::Page::Form)
+        expect(page.resource).to be_a_new(LogEntry)
+      end
+    end
+  end
+
+  describe "PUT update" do
+    def put_update(original, action: "create", logeable: create(:customer))
+      resource_params = attributes_for(:log_entry).merge(
+        action: action,
+        logeable: {
+          type: "Administrate::Field::Polymorphic",
+          value: logeable.to_global_id.to_s,
+        },
+      )
+      put :update, id: original.to_param, log_entry: resource_params
     end
 
-    it "redirects to the created customer" do
-      post_create
-      expect(response).to redirect_to([:admin, LogEntry.last])
+    describe "with valid params" do
+      it "updates the requested log entry" do
+        log_entry = create(:log_entry)
+        new_action = "test-action"
+
+        put_update(log_entry, action: new_action)
+
+        log_entry.reload
+        expect(log_entry.action).to eq new_action
+      end
+
+      it "redirects to the customer" do
+        log_entry = create(:log_entry)
+
+        put_update(log_entry)
+
+        expect(response).to redirect_to([:admin, log_entry])
+      end
+    end
+
+    describe "with invalid params" do
+      it "passes a form page object to the view" do
+        customer = create(:customer)
+        log_entry = create(:log_entry, logeable: customer)
+        new_action = "cancel"
+
+        locals = capture_view_locals do
+          put_update(log_entry, action: new_action, logeable: customer)
+        end
+
+        page = locals[:page]
+        expect(page).to be_instance_of(Administrate::Page::Form)
+        expect(page.resource).to eq(log_entry)
+        expect(page.resource.action).to eq(new_action)
+      end
     end
   end
 end

spec/example_app/app/models/log_entry.rb

@@ -1,3 +1,9 @@
 class LogEntry < ActiveRecord::Base
   belongs_to :logeable, polymorphic: true
+
+  validate do |log_entry|
+    if log_entry.action == "cancel" && log_entry.logeable.is_a?(Customer)
+      log_entry.errors.add(:logeable, "A customer cannot be cancelled")
+    end
+  end
 end