Rename FormsController to DraftController

This controller is now only responsible for rendering the draft form
page. Rename it to reflect this, and move it into the Forms namespace
for consistency.

Author: stephencdaly

app/controllers/forms_controller.rb app/controllers/forms/draft_controller.rbapp/controllers/forms_controller.rb renamed to app/controllers/forms/draft_controller.rb

@@ -1,4 +1,4 @@
-class FormsController < WebController
+class Forms::DraftController < WebController
   after_action :verify_authorized
 
   def show

app/views/forms/show.html.erb app/views/forms/draft/show.html.erbapp/views/forms/show.html.erb renamed to app/views/forms/draft/show.html.erb

@@ -19,7 +19,7 @@
   end
 
   scope "forms/:form_id" do
-    get "/" => "forms#show", as: :form
+    get "/" => "forms/draft#show", as: :form
     get "/change-name" => "forms/change_name#edit", as: :change_form_name
     post "/change-name" => "forms/change_name#update"
 

config/routes.rb

@@ -1,6 +1,6 @@
 require "rails_helper"
 
-RSpec.describe FormsController, type: :request do
+RSpec.describe Forms::DraftController, type: :request do
   let(:form) { create(:form) }
   let(:group) { create(:group, organisation: standard_user.organisation) }
   let(:user) { standard_user }
@@ -22,7 +22,7 @@
       end
 
       it "renders the show template" do
-        expect(response).to render_template("forms/show")
+        expect(response).to render_template("show")
       end
 
       it "includes a task list" do
@@ -36,7 +36,7 @@
       end
 
       it "renders the show template" do
-        expect(response).to render_template("forms/show")
+        expect(response).to render_template("show")
       end
     end
 

spec/requests/forms_controller_spec.rb …/requests/forms/draft_controller_spec.rbspec/requests/forms_controller_spec.rb renamed to spec/requests/forms/draft_controller_spec.rb spec/requests/forms_controller_spec.rb renamed to spec/requests/forms/draft_controller_spec.rb

@@ -1,6 +1,6 @@
 require "rails_helper"
 
-describe "forms/show.html.erb" do
+describe "forms/draft/show.html.erb" do
   let(:user) { build :user }
   let(:form) { create :form, :with_pages, name: "Form 1", form_slug: "form-1" }
   let(:group) { create :group, name: "Group 1" }
@@ -16,7 +16,7 @@
       GroupForm.create!(form_id: form.id, group_id: group.id)
     end
 
-    render template: "forms/show"
+    render
   end
 
   it "contains page heading and sub-heading" do