feat(customer-type): Update anrok and xero payloads with new fields

app/models/customer.rb

@@ -82,8 +82,9 @@ def self.ransackable_attributes(_auth_object = nil)
     %w[id name firstname lastname external_id email]
   end
 
-  def display_name
-    names = [legal_name.presence || name.presence]
+  def display_name(prefer_legal_name: true)
+    names = prefer_legal_name ? [legal_name.presence || name.presence] : [name.presence]
+
     if firstname.present? || lastname.present?
       names << '-' if names.compact.present?
       names << firstname

app/services/integrations/aggregator/contacts/payloads/anrok.rb

@@ -5,6 +5,34 @@ module Aggregator
     module Contacts
       module Payloads
         class Anrok < BasePayload
+          def create_body
+            [
+              {
+                'name' => customer.display_name(prefer_legal_name: false),
+                'city' => customer.city,
+                'zip' => customer.zipcode,
+                'country' => customer.country,
+                'state' => customer.state,
+                'email' => email,
+                'phone' => phone
+              }
+            ]
+          end
+
+          def update_body
+            [
+              {
+                'id' => integration_customer.external_customer_id,
+                'name' => customer.display_name(prefer_legal_name: false),
+                'city' => customer.city,
+                'zip' => customer.zipcode,
+                'country' => customer.country,
+                'state' => customer.state,
+                'email' => email,
+                'phone' => phone
+              }
+            ]
+          end
         end
       end
     end

app/services/integrations/aggregator/contacts/payloads/base_payload.rb

@@ -17,6 +17,8 @@ def create_body
             [
               {
                 'name' => customer.name,
+                'firstname' => customer.firstname,
+                'lastname' => customer.lastname,
                 'city' => customer.city,
                 'zip' => customer.zipcode,
                 'country' => customer.country,
@@ -32,6 +34,8 @@ def update_body
               {
                 'id' => integration_customer.external_customer_id,
                 'name' => customer.name,
+                'firstname' => customer.firstname,
+                'lastname' => customer.lastname,
                 'city' => customer.city,
                 'zip' => customer.zipcode,
                 'country' => customer.country,

spec/models/customer_spec.rb

@@ -117,21 +117,30 @@
       let(:legal_name) { nil }
 
       it 'returns name with firstname and lastname' do
-        expect(customer.display_name).to eq('ACME Inc - Thomas Anderson')
+        aggregate_failures do
+          expect(customer.display_name).to eq('ACME Inc - Thomas Anderson')
+          expect(customer.display_name(prefer_legal_name: false)).to eq('ACME Inc - Thomas Anderson')
+        end
       end
     end
 
     context 'when legal_name is present and both firstname and lastname are present' do
       let(:name) { nil }
 
       it 'returns legal_name with firstname and lastname' do
-        expect(customer.display_name).to eq('ACME International Corporation - Thomas Anderson')
+        aggregate_failures do
+          expect(customer.display_name).to eq('ACME International Corporation - Thomas Anderson')
+          expect(customer.display_name(prefer_legal_name: false)).to eq('Thomas Anderson')
+        end
       end
     end
 
     context 'when all fields are present' do
-      it 'returns legal_name with firstname and lastname' do
-        expect(customer.display_name).to eq('ACME International Corporation - Thomas Anderson')
+      it 'returns display name' do
+        aggregate_failures do
+          expect(customer.display_name).to eq('ACME International Corporation - Thomas Anderson')
+          expect(customer.display_name(prefer_legal_name: false)).to eq('ACME Inc - Thomas Anderson')
+        end
       end
     end
   end

spec/services/integrations/aggregator/contacts/create_service_spec.rb

@@ -115,6 +115,8 @@
           [
             {
               'name' => customer.name,
+              'firstname' => customer.firstname,
+              'lastname' => customer.lastname,
               'email' => customer.email,
               'city' => customer.city,
               'zip' => customer.zipcode,

spec/services/integrations/aggregator/contacts/payloads/anrok_spec.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Integrations::Aggregator::Contacts::Payloads::Anrok do
+  let(:integration) { integration_customer.integration }
+  let(:integration_customer) { FactoryBot.create(:anrok_customer, customer:) }
+  let(:customer) { create(:customer) }
+  let(:payload) { described_class.new(integration:, customer:, integration_customer:) }
+  let(:customer_link) { payload.__send__(:customer_url) }
+
+  describe "#create_body" do
+    subject(:create_body_call) { payload.create_body }
+
+    let(:payload_body) do
+      [
+        {
+          'name' => customer.display_name(prefer_legal_name: false),
+          'city' => customer.city,
+          'zip' => customer.zipcode,
+          'country' => customer.country,
+          'state' => customer.state,
+          'email' => customer.email,
+          'phone' => customer.phone
+        }
+      ]
+    end
+
+    it 'returns the payload body' do
+      expect(subject).to eq payload_body
+    end
+  end
+
+  describe "#update_body" do
+    subject(:update_body_call) { payload.update_body }
+
+    let(:payload_body) do
+      [
+        {
+          'id' => integration_customer.external_customer_id,
+          'name' => customer.display_name(prefer_legal_name: false),
+          'city' => customer.city,
+          'zip' => customer.zipcode,
+          'country' => customer.country,
+          'state' => customer.state,
+          'email' => customer.email,
+          'phone' => customer.phone
+        }
+      ]
+    end
+
+    it "returns the payload body" do
+      expect(subject).to eq payload_body
+    end
+  end
+end

spec/services/integrations/aggregator/contacts/payloads/factory_spec.rb

@@ -62,7 +62,7 @@
       let(:integration_customer) { FactoryBot.create(:anrok_customer) }
 
       it 'returns payload body' do
-        expect(subject.first['name']).to eq(customer.name)
+        expect(subject.first['name']).to eq(customer.display_name(prefer_legal_name: false))
       end
     end
   end
@@ -100,7 +100,7 @@
       it 'returns payload body' do
         aggregate_failures do
           expect(subject.first['id']).to eq(integration_customer.external_customer_id)
-          expect(subject.first['name']).to eq(customer.name)
+          expect(subject.first['name']).to eq(customer.display_name(prefer_legal_name: false))
         end
       end
     end

spec/services/integrations/aggregator/contacts/payloads/xero_spec.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe Integrations::Aggregator::Contacts::Payloads::Xero do
+  let(:integration) { integration_customer.integration }
+  let(:integration_customer) { FactoryBot.create(:xero_customer, customer:) }
+  let(:customer) { create(:customer) }
+  let(:payload) { described_class.new(integration:, customer:, integration_customer:) }
+  let(:customer_link) { payload.__send__(:customer_url) }
+
+  describe "#create_body" do
+    subject(:create_body_call) { payload.create_body }
+
+    let(:payload_body) do
+      [
+        {
+          'name' => customer.name,
+          'firstname' => customer.firstname,
+          'lastname' => customer.lastname,
+          'city' => customer.city,
+          'zip' => customer.zipcode,
+          'country' => customer.country,
+          'state' => customer.state,
+          'email' => customer.email,
+          'phone' => customer.phone
+        }
+      ]
+    end
+
+    it 'returns the payload body' do
+      expect(subject).to eq payload_body
+    end
+  end
+
+  describe "#update_body" do
+    subject(:update_body_call) { payload.update_body }
+
+    let(:payload_body) do
+      [
+        {
+          'id' => integration_customer.external_customer_id,
+          'name' => customer.name,
+          'firstname' => customer.firstname,
+          'lastname' => customer.lastname,
+          'city' => customer.city,
+          'zip' => customer.zipcode,
+          'country' => customer.country,
+          'state' => customer.state,
+          'email' => customer.email,
+          'phone' => customer.phone
+        }
+      ]
+    end
+
+    it "returns the payload body" do
+      expect(subject).to eq payload_body
+    end
+  end
+end

spec/services/integrations/aggregator/contacts/update_service_spec.rb

@@ -89,6 +89,8 @@
             {
               'id' => integration_customer.external_customer_id,
               'name' => customer.name,
+              'firstname' => customer.firstname,
+              'lastname' => customer.lastname,
               'email' => customer.email,
               'city' => customer.city,
               'zip' => customer.zipcode,