🥗🥔✨ Agreements: Tea up the #update action

- #1364

app/controllers/space/agreements_controller.rb

@@ -15,6 +15,14 @@ def create
       end
     end
 
+    def update
+      if agreement.update(agreement_params)
+        redirect_to space.location(:edit), notice: t(".success", name: agreement.name)
+      else
+        render :edit, status: :unprocessable_entity
+      end
+    end
+
     def destroy
       agreement.destroy
       redirect_to space.location(:edit), notice: t(".success", name: agreement.name)

app/lib/space_routes.rb

@@ -1,6 +1,6 @@
 module SpaceRoutes
   def self.append_routes(router)
-    router.resources :agreements, only: %i[show new create destroy], controller: "space/agreements"
+    router.resources :agreements, only: %i[show new create update destroy], controller: "space/agreements"
     router.resource :authenticated_session, only: %i[new create update destroy show]
     router.resources :invitations, only: %i[create destroy index] do
       router.resource :rsvp, only: %i[show update]

app/policies/space/agreement_policy.rb

@@ -11,6 +11,7 @@ def create?
     end
 
     alias_method :destroy?, :create?
+    alias_method :update?, :create?
 
     def permitted_attributes(_)
       %i[name body]

config/locales/agreement/en.yml

@@ -8,6 +8,8 @@ en:
         link_to: "Add Agreement"
       create:
         success: "Added Agreement '%{name}'"
+      update:
+        success: "Changed Agreement '%{name}'"
       destroy:
         link_to: "Remove Agreement '%{name}'"
         success: "Removed Agreement '%{name}'"

spec/requests/space/agreements_controller_request_spec.rb

@@ -69,6 +69,35 @@
     end
   end
 
+  describe "#update" do
+    subject(:perform_request) do
+      put polymorphic_path(agreement.location, params: {agreement: agreement_params})
+      agreement.reload
+      response
+    end
+
+    let(:agreement_params) { {name: "Ooh"} }
+    let(:agreement) { create(:space_agreement, space: space) }
+
+    it { is_expected.to be_not_found }
+
+    describe "when signed in as a member" do
+      before { sign_in(space, member) }
+
+      it { is_expected.to redirect_to(space.location(:edit)) }
+
+      specify do
+        expect { perform_request }.to change(agreement, :name).to("Ooh")
+      end
+
+      context "when the params are invalid" do
+        let(:agreement_params) { {name: ""} }
+
+        it { is_expected.to render_template(:edit).and(be_unprocessable) }
+      end
+    end
+  end
+
   describe "#destroy" do
     subject(:perform_request) do
       delete polymorphic_path(agreement.location)