Add role to invite (#140)

* Add role to invite

* Make role field a radio button

* Fix broken specs on main

app/controllers/users_controller.rb

@@ -46,6 +46,6 @@ private
   end
 
   def user_params
-    params.require(:user).permit(:email, :name, :password)
+    params.require(:user).permit(:email, :name, :password, :role)
   end
 end

app/views/users/new.html.erb

@@ -26,6 +26,10 @@
         value: @resource.email
       %>
 
+      <%= roles = User::ROLES.map { |key, value| OpenStruct.new(id:key, name: key.humanize) }
+        f.govuk_collection_radio_buttons :role, roles, :id, :name, legend: { text: "Role", size: "m" }
+      %>
+
       <%= f.govuk_submit "Continue" %>
     </div>
   </div>

spec/features/form/form_navigation_spec.rb

@@ -31,7 +31,7 @@ RSpec.describe "Form Navigation" do
       visit("/case-logs")
       click_link("Create new log")
       id = CaseLog.order(created_at: :desc).first.id
-      expect(page).to have_content("Tasklist for log #{id}")
+      expect(page).to have_content("Case log #{id}")
     end
   end
 
@@ -55,7 +55,7 @@ RSpec.describe "Form Navigation" do
         visit("/case-logs/#{id}")
         visit("/case-logs/#{id}/tenant-code")
         click_link(text: "Back")
-        expect(page).to have_content("Tasklist for log #{id}")
+        expect(page).to have_content("Case log #{id}")
       end
 
       it "go back to tenant code page from tenant age page", js: true do

spec/features/organisation_spec.rb

@@ -38,8 +38,10 @@ RSpec.describe "User Features" do
         expect(page).to have_content("Invite user to submit CORE data")
         fill_in("user[name]", with: "New User")
         fill_in("user[email]", with: "new_user@example.com")
+        choose("user-role-data-provider-field")
         expect { click_button("Continue") }.to change { ActionMailer::Base.deliveries.count }.by(1)
         expect(page).to have_current_path("/organisations/#{org_id}/users")
+        expect(User.last.role).to eq("data_provider")
       end
     end
   end

spec/requests/case_log_controller_spec.rb

@@ -187,7 +187,7 @@ RSpec.describe CaseLogsController, type: :request do
           end
 
           it "shows the tasklist for case logs you have access to" do
-            expect(response.body).to match("Tasklist for log")
+            expect(response.body).to match("Case log")
             expect(response.body).to match(case_log.id.to_s)
           end