Merge pull request #4455 from nhsuk/prescriber-record-psd

Allow prescribers to record PSDs

Author: thomasleese

app/policies/patient_specific_direction_policy.rb

@@ -2,6 +2,6 @@
 
 class PatientSpecificDirectionPolicy < ApplicationPolicy
   def create?
-    user.is_nurse?
+    user.is_nurse? || user.is_prescriber?
   end
 end

app/views/sessions/patient_specific_directions/show.html.erb

@@ -26,7 +26,8 @@
         There are <%= @eligible_for_bulk_psd_count %> children with consent for the nasal flu vaccine
         who do not require triage and do not yet have a PSD in place.
       </p>
-      <% if current_user.is_nurse? %>
+
+      <% if policy(PatientSpecificDirection).create? %>
         <%= link_to new_session_patient_specific_directions_path(@session), class: "nhsuk-action-link" do %>
           <svg class="nhsuk-icon nhsuk-icon--arrow-right-circle" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16" focusable="false" aria-hidden="true">
             <path d="M12 2a10 10 0 0 0-10 9h11.7l-4-4a1 1 0 0 1 1.5-1.4l5.6 5.7a1 1 0 0 1 0 1.4l-5.6 5.7a1 1 0 0 1-1.5 0 1 1 0 0 1 0-1.4l4-4H2A10 10 0 1 0 12 2z" />

spec/policies/patient_specific_direction_policy_spec.rb

@@ -1,25 +1,27 @@
 # frozen_string_literal: true
 
-RSpec.describe PatientSpecificDirectionPolicy do
+describe PatientSpecificDirectionPolicy do
   subject(:policy) { described_class.new(user, PatientSpecificDirection) }
 
-  context "cis2 is disabled", cis2: :disabled do
-    describe "#create?" do
-      context "when user is a nurse" do
-        let(:user) { build(:user, :nurse) }
+  describe "#create?" do
+    subject(:create?) { policy.create? }
 
-        it "permits creation" do
-          expect(policy.create?).to be(true)
-        end
-      end
+    context "when user is a nurse" do
+      let(:user) { build(:user, :nurse) }
 
-      context "when user is not a nurse" do
-        let(:user) { build(:user, :healthcare_assistant) }
+      it { should be(true) }
+    end
+
+    context "when user is a prescriber" do
+      let(:user) { build(:user, :prescriber) }
+
+      it { should be(true) }
+    end
+
+    context "when user is a healthcare assistant" do
+      let(:user) { build(:user, :healthcare_assistant) }
 
-        it "denies creation" do
-          expect(policy.create?).to be(false)
-        end
-      end
+      it { should be(false) }
     end
   end
 end