refactoring tests to validate match_patient_event_procedure_user

Author: lucasleandro1

app/models/event_procedure.rb

@@ -32,13 +32,14 @@ class EventProcedure < ApplicationRecord
   validates :urgency, inclusion: [true, false], if: -> { health_insurance? }
   validates :payment, presence: true
 
-  validate :unique_user
+  validate :match_patient_event_procedure_user
 
   private
 
-  def unique_user
-    return unless patient.present? && user.present?
-    return if user == patient.user
+  def match_patient_event_procedure_user
+    return if patient.nil? || user.nil?
+
+    return unless patient.user_id != user_id
 
     errors.add(:base, "The patient must be associated with the same procedure user")
   end

spec/factories/event_procedures.rb

@@ -5,9 +5,9 @@
     cbhpm
     health_insurance
     hospital
-    patient
-    procedure
     user
+    patient { build(:patient, user: user) }
+    procedure
 
     patient_service_number { "202312150001" }
     date { "2023-12-15 16:02:00" }
@@ -22,24 +22,19 @@
 
     transient do
       health_insurance_attributes { nil }
-    end
-
-    transient do
       patient_attributes { nil }
-    end
-
-    transient do
       procedure_attributes { nil }
     end
 
     after(:build) do |event_procedure, evaluator|
-      event_procedure.patient = build(:patient, evaluator.patient_attributes) if evaluator.patient_attributes
+      event_procedure.user ||= build(:user)
+      unless event_procedure.patient
+        event_procedure.patient = build(:patient, evaluator.patient_attributes || { user: event_procedure.user })
+      end
       event_procedure.procedure = build(:procedure, evaluator.procedure_attributes) if evaluator.procedure_attributes
+
       if evaluator.health_insurance_attributes
-        event_procedure.health_insurance = build(
-          :health_insurance,
-          evaluator.health_insurance_attributes
-        )
+        event_procedure.health_insurance = build(:health_insurance, evaluator.health_insurance_attributes)
       end
     end
   end

spec/models/event_procedure_spec.rb

@@ -97,7 +97,9 @@
     context "when patient_attributes are provided" do
       it "creates patient" do
         event_procedure = build(
-          :event_procedure, user: user,
+          :event_procedure,
+          user: user,
+          patient: nil,
           patient_attributes: { name: "John Doe", user_id: user.id }
         )
 
@@ -109,7 +111,11 @@
 
     context "when patient_attributes are not provided" do
       it "does not create patient" do
-        event_procedure = build(:event_procedure, patient_attributes: { id: nil, name: nil })
+        event_procedure = build(
+          :event_procedure,
+          patient: nil,
+          patient_attributes: { name: nil, user_id: nil }
+        )
 
         expect(event_procedure.save).to be_falsey
         expect(event_procedure.errors[:"patient.name"]).to include("can't be blank")

spec/models/patient_spec.rb

@@ -15,16 +15,17 @@
   describe "deletable?" do
     subject { patient.deletable? }
 
-    let(:patient) { create(:patient, event_procedures:) }
+    let(:user) { create(:user) }
+    let(:patient) { create(:patient, user: user) }
 
     context "when patient has no event_procedures" do
-      let(:event_procedures) { [] }
-
       it { is_expected.to be_truthy }
     end
 
     context "when patient has event_procedures" do
-      let(:event_procedures) { create_list(:event_procedure, 1) }
+      before do
+        create(:event_procedure, patient: patient, user: user)
+      end
 
       it { is_expected.to be_falsy }
     end

spec/operations/event_procedures/build_total_amount_cents_spec.rb

@@ -5,12 +5,13 @@
 RSpec.describe EventProcedures::BuildTotalAmountCents, type: :operation do
   describe ".result" do
     it "returns a success" do
+      user = create(:user)
       cbhpm = create(:cbhpm)
       procedure = create(:procedure)
       create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
       create(:port_value, cbhpm: cbhpm, anesthetic_port: "1A", amount_cents: 1000)
-      event_procedure = create(:event_procedure, procedure: procedure, cbhpm: cbhpm)
-
+      patient = create(:patient, user: user)
+      event_procedure = create(:event_procedure, user: user, procedure: procedure, cbhpm: cbhpm, patient: patient)
       expect(described_class.result(event_procedure: event_procedure)).to be_success
     end
 

spec/operations/event_procedures/create_spec.rb

@@ -3,10 +3,12 @@
 require "rails_helper"
 
 RSpec.describe EventProcedures::Create, type: :operation do
+  let(:user) { create(:user) }
+  let(:patient) { create(:patient, user: user) }
+
   describe ".result" do
     context "when params are valid" do
       it "is successful" do
-        user = create(:user)
         cbhpm = create(:cbhpm)
         procedure = create(:procedure)
         create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
@@ -19,7 +21,7 @@
           urgency: false,
           room_type: EventProcedures::RoomTypes::WARD,
           payment: EventProcedures::Payments::HEALTH_INSURANCE,
-          patient_attributes: { id: create(:patient).id },
+          patient_attributes: { id: patient.id, user_id: user.id },
           procedure_attributes: { id: procedure.id },
           health_insurance_attributes: { id: create(:health_insurance).id }
         }
@@ -30,7 +32,6 @@
       end
 
       it "creates a new event_procedure" do
-        user = create(:user)
         cbhpm = create(:cbhpm)
         procedure = create(:procedure)
         create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
@@ -43,7 +44,7 @@
           urgency: true,
           room_type: EventProcedures::RoomTypes::WARD,
           payment: EventProcedures::Payments::HEALTH_INSURANCE,
-          patient_attributes: { id: create(:patient).id },
+          patient_attributes: { id: patient.id, user_id: user.id },
           procedure_attributes: { id: procedure.id },
           health_insurance_attributes: { id: create(:health_insurance).id }
         }
@@ -68,7 +69,6 @@
 
       context "when create a new patient" do
         it "creates and does not duplicate the creation" do
-          user = create(:user)
           cbhpm = create(:cbhpm)
           procedure = create(:procedure)
           create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
@@ -94,7 +94,6 @@
       context "when create a new procedure" do
         context "when procedure attributes are valid" do
           it "does not duplicate the creation" do
-            user = create(:user)
             cbhpm = create(:cbhpm)
             procedure = create(:procedure)
             create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
@@ -117,7 +116,7 @@
               urgency: true,
               room_type: EventProcedures::RoomTypes::WARD,
               payment: EventProcedures::Payments::HEALTH_INSURANCE,
-              patient_attributes: { id: create(:patient).id },
+              patient_attributes: { id: patient.id, user_id: user.id },
               procedure_attributes: procedure_attributes,
               health_insurance_attributes: { id: create(:health_insurance).id }
             }
@@ -163,7 +162,6 @@
       context "when create a new health_insurance" do
         context "when health_insurance attributes are valid" do
           it "does not duplicate the creation" do
-            user = create(:user)
             cbhpm = create(:cbhpm)
             procedure = create(:procedure)
             create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
@@ -182,7 +180,7 @@
               urgency: nil,
               room_type: nil,
               payment: EventProcedures::Payments::OTHERS,
-              patient_attributes: { id: create(:patient).id },
+              patient_attributes: { id: patient.id, user_id: user.id },
               procedure_attributes: { id: procedure.id },
               health_insurance_attributes: health_insurance_attributes
             }
@@ -249,9 +247,7 @@
       end
 
       it "returns errors" do
-        user = create(:user)
         procedure = create(:procedure)
-        patient = create(:patient)
         health_insurance = create(:health_insurance)
         attributes = {
           patient_attributes: { id: patient.id },

spec/operations/event_procedures/list_spec.rb

@@ -24,7 +24,7 @@
     it "includes the associations" do # rubocop:disable RSpec/MultipleExpectations
       user = create(:user)
       procedure = create(:procedure, name: "Tireoidectomia")
-      patient = create(:patient, name: "John Doe")
+      patient = create(:patient, name: "John Doe", user: user)
       hospital = create(:hospital, name: "General Hospital")
       health_insurance = create(:health_insurance, name: "Insurance Corp")
       create_list(

spec/operations/event_procedures/total_amount_cents_spec.rb

@@ -6,7 +6,8 @@
   describe ".result" do
     it "is successful" do
       user = create(:user)
-      event_procedure = create(:event_procedure, user_id: user.id)
+      patient = create(:patient, user: user)
+      event_procedure = create(:event_procedure, user_id: user.id, patient: patient)
       expect(described_class.result(event_procedures: [event_procedure])).to be_success
     end
 

spec/operations/event_procedures/update_spec.rb

@@ -22,17 +22,21 @@
 
       context "when patient_attributes are provided" do
         it "update event_procedure association patient" do
-          old_patient = create(:patient, name: "Old Patient")
-          new_patient = create(:patient, name: "New Patient name")
+          user = create(:user)
+          old_patient = create(:patient, name: "Old Patient", user: user)
+          new_patient = create(:patient, name: "New Patient name", user: user)
           cbhpm = create(:cbhpm)
           procedure = create(:procedure)
           create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
           create(:port_value, cbhpm: cbhpm, anesthetic_port: "1A", amount_cents: 1000)
-          event_procedure = create(:event_procedure, procedure: procedure, cbhpm: cbhpm, patient: old_patient)
+          event_procedure = create(
+            :event_procedure, procedure: procedure, cbhpm: cbhpm, patient: old_patient,
+            user: user
+          )
           attributes = {
             urgency: true,
             room_type: EventProcedures::RoomTypes::WARD,
-            patient_attributes: { id: new_patient.id, name: nil }
+            patient_attributes: { id: new_patient.id, name: nil, user: user }
           }
           result = described_class.result(id: event_procedure.id.to_s, attributes: attributes)
 
@@ -41,13 +45,13 @@
         end
 
         it "creates a new patient and does not duplicate the creation" do
-          patient = create(:patient, name: "Old Patient")
           user = create(:user)
+          patient = create(:patient, name: "Old Patient", user: user)
           cbhpm = create(:cbhpm)
           procedure = create(:procedure)
           create(:cbhpm_procedure, procedure: procedure, cbhpm: cbhpm, anesthetic_port: "1A")
           create(:port_value, cbhpm: cbhpm, anesthetic_port: "1A", amount_cents: 1000)
-          event_procedure = create(:event_procedure, procedure: procedure, cbhpm: cbhpm, patient: patient)
+          event_procedure = create(:event_procedure, procedure: procedure, cbhpm: cbhpm, patient: patient, user: user)
           attributes = {
             urgency: true,
             room_type: EventProcedures::RoomTypes::WARD,

spec/pdfs/event_procedures_report_pdf_spec.rb

@@ -5,8 +5,9 @@
 RSpec.describe EventProceduresReportPdf, type: :pdf do
   it "generates a report with the correct content" do
     user = create(:user)
+    patient = create(:patient, user: user)
     pdf = Prawn::Document.new
-    event_procedures = create_list(:event_procedure, 3, user_id: user.id)
+    event_procedures = create_list(:event_procedure, 3, user_id: user.id, patient: patient)
     amount = EventProcedures::TotalAmountCents.call(event_procedures: event_procedures)
 
     described_class.new(