stackone_migration 1.2.2 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ed058e52bc6531d4776c4309fa4a63c1f17c050cf7a3a79cbdd5cdb126314a2
-  data.tar.gz: dc2432ab2547c52815d94de47f2e837937e47035b0ec582f723a547c00050596
+  metadata.gz: 67a03369917c2e312bf393a5f96d3493d5b6430647fc8f080d3b2a37be59c29a
+  data.tar.gz: ac3404c9425e9a1bddbf0613025bfaa84e1091e6afc5b137dc12f4eaa1cdfd18
 SHA512:
-  metadata.gz: bd971a17ee5e0ae0788e75ed6203ec7cf0907988c3e8c6ffab34bd29c1dde4487184a7161a68b3a1c7e1bbea3cf1a8f55a62a9b80a1b0d7816897752881bbce6
-  data.tar.gz: 799d35be2943ac5678d2bdf1c4e2ce09269c09fe736bc89480fa62d981318eec8a2ac7bf02f464145961188b868dda0a969ddf3b3aa8cc8634ecbdce2c297aa6
+  metadata.gz: 603295af5e304db4088764640a04a6c5d61a5cb0f569d66aec876732b49c659e6054eaa7b870c511d244a244dc7f47b194e8c20440c22cc1bd9e18792c0f59ed
+  data.tar.gz: 1e766c8704c8ab85cdb04607819dced4e65b6516c550c322975bb95de60cd6bc20e7419d013389158466b8353ed1f87f391f180567a87fdfe70839975196dfec

data/Gemfile

@@ -11,4 +11,4 @@ group :development, :test do
 end
 
 gem 'merge_hris_client', '~> 3.0'
-gem 'stackone_hris_client', '~> 1.3.2'
+gem 'stackone_hris_client', '~> 1.5.1'

data/Gemfile.lock

@@ -1,9 +1,9 @@
 PATH
   remote: .
   specs:
-    stackone_migration (1.2.2)
+    stackone_migration (1.4.0)
       merge_hris_client (~> 3.0)
-      stackone_hris_client (~> 1.3.2)
+      stackone_hris_client (~> 1.5.1)
 
 GEM
   remote: https://rubygems.org/
@@ -58,7 +58,7 @@ GEM
     rubocop-ast (1.27.0)
       parser (>= 3.2.1.0)
     ruby-progressbar (1.13.0)
-    stackone_hris_client (1.3.2)
+    stackone_hris_client (1.5.1)
       typhoeus (~> 1.0, >= 1.0.1)
     typhoeus (1.4.0)
       ethon (>= 0.9.0)
@@ -73,7 +73,7 @@ DEPENDENCIES
   rake (~> 13.0.1)
   rspec (~> 3.6, >= 3.6.0)
   rubocop (~> 1.48.0)
-  stackone_hris_client (~> 1.3.2)
+  stackone_hris_client (~> 1.5.1)
   stackone_migration!
 
 BUNDLED WITH

data/lib/stackone_migration/mappers/employees_mapper.rb

@@ -101,14 +101,14 @@ module StackOneMigration
         :personal_email => employee.personal_email,
         :work_email => employee.work_email,
         :mobile_phone_number => employee.personal_phone_number,
-        :gender => MergeHRISClient::GenderEnum.build_from_hash(gender_enum_hash[employee.gender&.to_sym]),
+        :gender => MergeHRISClient::GenderEnum.build_from_hash(map_enum(gender_enum_hash,employee.gender)),
         :date_of_birth => employee.date_of_birth,
         :start_date => employee.start_date,
         :manager => employee.manager_id,
-        :ethnicity => MergeHRISClient::EthnicityEnum.build_from_hash(ethnicity_enum_hash[employee.ethnicity&.to_sym]),
-        :marital_status => MergeHRISClient::MaritalStatusEnum.build_from_hash(marital_status_enum_hash[employee.marital_status&.to_sym]),
+        :ethnicity => MergeHRISClient::EthnicityEnum.build_from_hash(map_enum(ethnicity_enum_hash, employee.ethnicity)),
+        :marital_status => MergeHRISClient::MaritalStatusEnum.build_from_hash(map_enum(marital_status_enum_hash, employee.marital_status)),
         :hire_date => employee.hire_date,
-        :employment_status => MergeHRISClient::EmploymentStatusEnum.build_from_hash(employment_status_enum_hash[employee.employment_status&.to_sym]),
+        :employment_status => MergeHRISClient::EmploymentStatusEnum.build_from_hash(map_enum(employment_status_enum_hash, employee.employment_status)),
         :termination_date => employee.termination_date,
         :home_location => map_to_location_model(employee.home_location),
         :work_location => map_to_location_model(employee.work_location),
@@ -129,8 +129,8 @@ module StackOneMigration
         :city => location.city,
         :state => location.state,
         :zip_code => location.zip_code,
-        :country => MergeHRISClient::CountryEnum.build_from_hash(location.country),
-        :location_type => MergeHRISClient::LocationTypeEnum.build_from_hash(location_type_enum_hash[location.location_type&.to_sym]),
+        :country => map_country_enum(location.country),
+        :location_type => MergeHRISClient::LocationTypeEnum.build_from_hash(map_enum(location_type_enum_hash, location.location_type)),
       )
     end
 
@@ -140,11 +140,11 @@ module StackOneMigration
       MergeHRISClient::Employment.new(
         :job_title => employment.job_title,
         :pay_rate => employment.pay_rate,
-        :pay_period => MergeHRISClient::PayPeriodEnum.build_from_hash(pay_period_enum_hash[employment.pay_period&.to_sym]),
-        :pay_frequency => MergeHRISClient::PayFrequencyEnum.build_from_hash(pay_frequency_enum_hash[employment.pay_frequency&.to_sym]),
+        :pay_period => MergeHRISClient::PayPeriodEnum.build_from_hash(map_enum(pay_period_enum_hash, employment.pay_period)),
+        :pay_frequency => MergeHRISClient::PayFrequencyEnum.build_from_hash(map_enum(pay_frequency_enum_hash, employment.pay_frequency)),
         :pay_currency => MergeHRISClient::PayCurrencyEnum.build_from_hash(employment.pay_currency&.to_s&.upcase),
         :effective_date => employment.effective_date,
-        :employment_type => MergeHRISClient::EmploymentTypeEnum.build_from_hash(employment_type_enum_hash[employment.employment_type&.to_sym]),
+        :employment_type => MergeHRISClient::EmploymentTypeEnum.build_from_hash(map_enum(employment_type_enum_hash, employment.employment_type)),
       )
     end
 
@@ -187,5 +187,22 @@ module StackOneMigration
         :previous => nil
       )
     end
+
+    def self.map_enum(enum_hash, enum)
+      return nil if enum.nil? || enum.value.nil?
+
+      mapped_enum = enum_hash[enum&.value&.to_sym]
+      return mapped_enum unless mapped_enum.nil? || mapped_enum == 'MERGE_NONSTANDARD_VALUE'
+
+      enum.source_value
+    end
+
+    def self.map_country_enum(enum)
+      return nil if enum.nil? || enum.value.nil?
+
+      enum_value = enum.value != 'unmapped_value' ? enum.value : enum.source_value
+
+      MergeHRISClient::CountryEnum.build_from_hash(enum_value)
+    end
   end
 end

data/lib/stackone_migration/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module StackOneMigration
-  VERSION = '1.2.2'
+  VERSION = '1.4.0'
 end

data/spec/mappers/employees_mapper_spec.rb

@@ -14,13 +14,13 @@ describe StackOneMigration::EmployeesMapper do
       :personal_email => 'ija@egje.wf',
       :work_email => 'kota@efpiphi.kw',
       :personal_phone_number => '(715) 658-9486',
-      :gender => 'female',
+      :gender => StackOneHRIS::GenderEnum.new(value: 'female', source_value: 'female'),
       :date_of_birth => '1990-01-01',
       :start_date => '2018-01-01',
       :manager_id => '417b5e62-e011-5e69-a906-0aefade9ded1',
-      :ethnicity => 'white',
-      :marital_status => 'married',
-      :employment_status => 'terminated',
+      :ethnicity => StackOneHRIS::EthnicityEnum.new(value: 'white', source_value: 'white'),
+      :marital_status => StackOneHRIS::MaritalStatusEnum.new(value: 'married', source_value: 'married'),
+      :employment_status => StackOneHRIS::EmploymentStatusEnum.new(value: 'terminated', source_value: 'terminated'),
       :work_phone_number => '(456) 787-3508',
       :job_title => 'Manager',
       :department => 'Human Resources',
@@ -32,18 +32,19 @@ describe StackOneMigration::EmployeesMapper do
            :city => "San Francisco",
            :state => "CA",
            :zip_code => "94105",
-           :country => "US"
+           :country => StackOneHRIS::CountryCodeEnum.new(value: 'US', source_value: 'United States'),
+           :location_type => StackOneHRIS::LocationTypeEnum.new(value: 'home', source_value: 'home'),
         ),
       :work_location => nil,
       :employments => [
         StackOneHRIS::Employment.new( 
           :job_title => 'Manager',
           :pay_rate => 100000,
-          :pay_period => 'year',
-          :pay_frequency => 'weekly',
+          :pay_period => StackOneHRIS::PayPeriodEnum.new(value: 'year', source_value: 'year'),
+          :pay_frequency => StackOneHRIS::PayFrequencyEnum.new(value: 'weekly', source_value: 'weekly'),
           :pay_currency => 'USD',
           :effective_date => '2018-01-01',
-          :employment_type => 'full_time',
+          :employment_type => StackOneHRIS::EmploymentTypeEnum.new(value: 'full_time', source_value: 'full_time'),
         )
       ]
     )
@@ -59,7 +60,7 @@ describe StackOneMigration::EmployeesMapper do
       :personal_email => 'ija@egje.wf',
       :work_email => 'kota@efpiphi.kw',
       :personal_phone_number => '(715) 658-9486',
-      :gender => 'female',
+      :gender => StackOneHRIS::GenderEnum.new(value: 'female', source_value: 'female'),
       :date_of_birth => '1990-01-01',
       :start_date => '2018-01-01',
       :manager_id => '417b5e62-e011-5e69-a906-0aefade9ded1',
@@ -119,7 +120,7 @@ describe StackOneMigration::EmployeesMapper do
         :city => "San Francisco",
         :state => "CA",
         :zip_code => "94105",
-        :country => "US"
+        :country => StackOneHRIS::CountryCodeEnum.new(value: 'US', source_value: 'United States'),
       )
 
       target_model = described_class.map_to_location_model(sample_location)
@@ -137,8 +138,8 @@ describe StackOneMigration::EmployeesMapper do
         :city => "San Francisco",
         :state => "CA",
         :zip_code => "94105",
-        :country => "US",
-        :location_type => 'home'
+        :country => StackOneHRIS::CountryCodeEnum.new(value: 'US', source_value: 'United States'),
+        :location_type => StackOneHRIS::LocationTypeEnum.new(value: 'home', source_value: 'home'),
       )
 
       target_model = described_class.map_to_location_model(sample_location)
@@ -152,21 +153,49 @@ describe StackOneMigration::EmployeesMapper do
       expect(target_model.city).to eq("San Francisco")
       expect(target_model.state).to eq("CA")
       expect(target_model.zip_code).to eq("94105")
-      expect(target_model.country).to have_attributes(:raw_value => 'US')
+      expect(target_model.country).to have_attributes(:value => 'US')
       expect(target_model.location_type).to have_attributes(:value => 'HOME')
     end
   end
 
+  context '.map_country_enum' do
+    it 'maps a country to correct class' do
+      sample_country = StackOneHRIS::CountryCodeEnum.new(value: 'US', source_value: 'United States')
+
+      target_model = described_class.map_country_enum(sample_country)
+
+      expect(target_model).to be_a(MergeHRISClient::CountryEnum)
+    end
+
+    it 'maps to a country with the right attributes' do
+      sample_country = StackOneHRIS::CountryCodeEnum.new(value: 'US', source_value: 'United States')
+
+      target_model = described_class.map_country_enum(sample_country)
+
+      expect(target_model).to have_attributes(:value => 'US', :raw_value => 'US')
+    end
+
+    context 'when the original enum is an unmapped value' do
+      it 'maps to a country with non standard value' do
+        sample_country = StackOneHRIS::CountryCodeEnum.new(value: 'unmapped_value', source_value: 'United States')
+
+        target_model = described_class.map_country_enum(sample_country)
+
+        expect(target_model).to have_attributes(:value => 'MERGE_NONSTANDARD_VALUE', :raw_value => 'United States')
+      end
+    end
+  end
+
   context '.map_to_employment_model' do
     it 'maps a employeement to correct class' do
       sample_employment = StackOneHRIS::Employment.new( 
         :job_title => 'Manager',
         :pay_rate => 100000,
-        :pay_period => 'year',
-        :pay_frequency => 'weekly',
+        :pay_period => StackOneHRIS::PayPeriodEnum.new(value: 'year', source_value: 'year'),
+        :pay_frequency => StackOneHRIS::PayFrequencyEnum.new(value: 'weekly', source_value: 'weekly'),
         :pay_currency => 'USD',
         :effective_date => '2018-01-01',
-        :employment_type => 'full_time',
+        :employment_type => StackOneHRIS::EmploymentTypeEnum.new(value: 'full_time', source_value: 'full_time'),
       )
 
       target_model = described_class.map_to_employment_model(sample_employment)
@@ -178,11 +207,11 @@ describe StackOneMigration::EmployeesMapper do
       sample_employment = StackOneHRIS::Employment.new( 
         :job_title => 'Manager',
         :pay_rate => 100000,
-        :pay_period => 'year',
-        :pay_frequency => 'weekly',
+        :pay_period => StackOneHRIS::PayPeriodEnum.new(value: 'year', source_value: 'year'),
+        :pay_frequency => StackOneHRIS::PayFrequencyEnum.new(value: 'weekly', source_value: 'weekly'),
         :pay_currency => 'USD',
         :effective_date => '2018-01-01',
-        :employment_type => 'full_time',
+        :employment_type => StackOneHRIS::EmploymentTypeEnum.new(value: 'full_time', source_value: 'full_time'),
       )
 
       target_model = described_class.map_to_employment_model(sample_employment)
@@ -266,4 +295,59 @@ describe StackOneMigration::EmployeesMapper do
        end
      end
   end
+
+  context '.map_enum' do
+    it 'returns the mapped enum' do
+      target_value = described_class.map_enum(
+        described_class.pay_period_enum_hash,
+        StackOneHRIS::PayPeriodEnum.new(value: 'year', source_value: 'year')
+      )
+
+      expect(target_value).to eq('YEAR')
+    end
+
+    context 'when the enum is nil' do
+      it 'returns nil' do
+        target_value = described_class.map_enum(
+          described_class.pay_period_enum_hash,
+          nil
+        )
+
+        expect(target_value).to be_nil
+      end
+    end
+
+    context 'when the enum value is nil' do
+      it 'returns nil' do
+        target_value = described_class.map_enum(
+          described_class.pay_period_enum_hash,
+          StackOneHRIS::PayPeriodEnum.new(value: nil, source_value: 'year')
+        )
+
+        expect(target_value).to be_nil
+      end
+    end
+
+    context 'when the enum value is an unmapped value' do
+      it 'returns the source value' do
+        target_value = described_class.map_enum(
+          described_class.pay_period_enum_hash,
+          StackOneHRIS::PayPeriodEnum.new(value: 'unmapped_value', source_value: 'year_custom_value')
+        )
+
+        expect(target_value).to eq('year_custom_value')
+      end
+    end
+
+    context 'when the enum value maps to a MERGE_NONSTANDARD_VALUE' do
+      it 'returns the mapped enum' do
+        target_value = described_class.map_enum(
+          described_class.marital_status_enum_hash,
+          StackOneHRIS::MaritalStatusEnum.new(value: 'other', source_value: 'other_status')
+        )
+
+        expect(target_value).to eq('other_status')
+      end
+    end
+  end
 end

data/stackone_migration.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |s|
   s.required_ruby_version = '>= 2.7'
 
   s.add_runtime_dependency 'merge_hris_client', '~> 3.0'
-  s.add_runtime_dependency 'stackone_hris_client', '~> 1.3.2'
+  s.add_runtime_dependency 'stackone_hris_client', '~> 1.5.1'
 
   s.add_development_dependency 'rspec', '~> 3.6', '>= 3.6.0'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stackone_migration
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.4.0
 platform: ruby
 authors:
 - StackOne
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-03 00:00:00.000000000 Z
+date: 2023-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: merge_hris_client
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.2
+        version: 1.5.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.2
+        version: 1.5.1
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement