Redirect to #edit after updating

On a successful update AdminPublicBodyHeadingsController and
AdminPublicBodyCategoriesController #update should redirect to the #edit
action rather than simply rendering the ‘edit’ template.

Fixes https://github.com/mysociety/alaveteli/issues/1977

2 files changed, 16 insertions, 0 deletions

diff --git a/spec/controllers/admin_public_body_categories_controller_spec.rb b/spec/controllers/admin_public_body_categories_controller_spec.rb
index 1e2a8d116..58e87d441 100644
--- a/spec/controllers/admin_public_body_categories_controller_spec.rb
+++ b/spec/controllers/admin_public_body_categories_controller_spec.rb
@@ -182,6 +182,14 @@ describe AdminPublicBodyCategoriesController do
             pbc = PublicBodyCategory.find(category.id)
             pbc.category_tag.should == "renamed"
         end
+
+        it "redirects to the edit page after a successful update" do
+            post :update, { :id => @category.id,
+                            :public_body_category => { :title => "Renamed" } }
+
+            expect(response).to redirect_to(edit_admin_category_path(@category))
+        end
+
     end
 
     context 'when destroying a public body category' do
diff --git a/spec/controllers/admin_public_body_headings_controller_spec.rb b/spec/controllers/admin_public_body_headings_controller_spec.rb
index 39d524013..7058e1ab8 100644
--- a/spec/controllers/admin_public_body_headings_controller_spec.rb
+++ b/spec/controllers/admin_public_body_headings_controller_spec.rb
@@ -102,6 +102,14 @@ describe AdminPublicBodyHeadingsController do
                heading.name.should == @name
             end
         end
+
+        it "redirects to the edit page after a successful update" do
+            post :update, { :id => @heading.id,
+                            :public_body_heading => { :name => "Renamed" } }
+
+            expect(response).to redirect_to(edit_admin_heading_path(@heading))
+        end
+
     end
 
     context 'when destroying a public body heading' do