API-41897-remove-updatepoaaccess-for-dependent-claimant

modules/claims_api/app/sidekiq/claims_api/poa_updater.rb

@@ -30,7 +30,9 @@ def perform(power_of_attorney_id, rep = nil) # rubocop:disable Metrics/MethodLen
 
         ClaimsApi::VANotifyJob.perform_async(poa_form.id, rep) if vanotify?(poa_form.auth_headers, rep)
 
-        ClaimsApi::PoaVBMSUpdater.perform_async(poa_form.id) if enable_vbms_access?(poa_form:)
+        if enable_vbms_access?(poa_form:) && update_poa_access?(poa_form.auth_headers)
+          ClaimsApi::PoaVBMSUpdater.perform_async(poa_form.id)
+        end
       else
         poa_form.status = ClaimsApi::PowerOfAttorney::ERRORED
         poa_form.vbms_error_message = "BGS Error: update_birls_record failed with code #{response[:return_code]}"
@@ -42,12 +44,21 @@ def perform(power_of_attorney_id, rep = nil) # rubocop:disable Metrics/MethodLen
 
     private
 
+    # If we are in the dependent workflow we do not want this job to run
+    def update_poa_access?(auth_headers)
+      !depedent_auth_headers_present?(auth_headers)
+    end
+
     def vanotify?(auth_headers, rep)
       if Flipper.enabled?(:lighthouse_claims_api_v2_poa_va_notify)
         auth_headers.key?(ClaimsApi::V2::Veterans::PowerOfAttorney::BaseController::VA_NOTIFY_KEY) && rep.present?
       else
         false
       end
     end
+
+    def depedent_auth_headers_present?(auth_headers)
+      auth_headers.key?('dependent')
+    end
   end
 end

modules/claims_api/spec/sidekiq/poa_updater_spec.rb

@@ -111,6 +111,37 @@
     end
   end
 
+  context 'deciding to call PoaVBMSUpdater' do
+    before do
+      create_mock_lighthouse_service
+    end
+
+    let(:poa) { create_poa }
+
+    context 'when the dependent header key is present' do
+      it 'does not call PoaVBMSUpdater' do
+        expect(ClaimsApi::PoaVBMSUpdater).not_to receive(:perform_async)
+
+        poa.auth_headers.merge!({ dependent: { first_name: 'Test' } })
+        poa.form_data.merge!({ recordConsent: true, consentLimits: [] })
+        poa.save!
+
+        subject.new.perform(poa.id, 'Rep Data')
+      end
+    end
+
+    context 'when the dependent header key is not present' do
+      it 'does call PoaVBMSUpdater' do
+        expect(ClaimsApi::PoaVBMSUpdater).to receive(:perform_async)
+
+        poa.form_data.merge!({ recordConsent: true, consentLimits: [] })
+        poa.save!
+
+        subject.new.perform(poa.id, 'Rep Data')
+      end
+    end
+  end
+
   context 'deciding to send a VA Notify email' do
     before do
       create_mock_lighthouse_service