hubrise_app 0.1.2 → 1.1.1

data/app/lib/hubrise_app/refresher/user.rb

@@ -0,0 +1,16 @@
+module HubriseApp::Refresher
+  class User < Base
+    class << self
+      def fetch_attributes(resource, api_client)
+        api_data = api_client.get_user.data
+        {
+          access_token: api_client.access_token,
+          email: api_data["email"],
+          first_name: api_data["first_name"],
+          last_name: api_data["last_name"],
+          locales: api_data["locales"]
+        }
+      end
+    end
+  end
+end

data/app/lib/hubrise_app/services/assign_app_instance.rb

@@ -0,0 +1,8 @@
+class HubriseApp::Services::AssignAppInstance
+  def self.run(hr_user, hr_app_instance, _ctx)
+    UserAppInstance.find_or_initialize_by(
+      hr_app_instance_id: hr_app_instance.hr_id,
+      hr_user_id: hr_user.hr_id
+    ).update!(refreshed_at: Time.now)
+  end
+end

data/app/lib/hubrise_app/services/connect_app_instance.rb

@@ -1,6 +1,5 @@
 class HubriseApp::Services::ConnectAppInstance
-  prepend HubriseApp::Services::Override::ConnectAppInstance
-
-  class << self; delegate :run, to: :new; end
-  def run(hr_app_instance, hubrise_callback_event_url:); end
+  def self.run(api_client, _ctx)
+    HubriseApp::Refresher::AppInstance.from_api_client(api_client)
+  end
 end

data/app/lib/hubrise_app/services/resolve_app_instance.rb

@@ -0,0 +1,5 @@
+class HubriseApp::Services::ResolveAppInstance
+  def self.run(scope, id, _ctx)
+    scope.where(hr_id: id).take
+  end
+end

data/app/models/account.rb

@@ -0,0 +1,2 @@
+class Account < HubriseApp::AccountBase
+end

data/app/models/app_instance.rb

@@ -0,0 +1,2 @@
+class AppInstance < HubriseApp::AppInstanceBase
+end

data/app/models/catalog.rb

@@ -0,0 +1,2 @@
+class Catalog < HubriseApp::CatalogBase
+end

data/app/models/customer_list.rb

@@ -0,0 +1,2 @@
+class CustomerList < HubriseApp::CustomerListBase
+end

data/app/models/hubrise_app/account_base.rb

@@ -0,0 +1,7 @@
+module HubriseApp
+  class AccountBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    store_accessor :api_data, :name, :currency
+  end
+end

data/app/models/hubrise_app/app_instance_base.rb

@@ -0,0 +1,10 @@
+module HubriseApp
+  class AppInstanceBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    belongs_to :account, optional: true, primary_key: :hr_id, foreign_key: :hr_account_id
+    belongs_to :location, optional: true, primary_key: :hr_id, foreign_key: :hr_location_id
+    belongs_to :catalog, optional: true, primary_key: :hr_id, foreign_key: :hr_catalog_id
+    belongs_to :customer_list, optional: true, primary_key: :hr_id, foreign_key: :hr_customer_list_id
+  end
+end

data/app/models/hubrise_app/catalog_base.rb

@@ -0,0 +1,7 @@
+module HubriseApp
+  class CatalogBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    store_accessor :api_data, :name
+  end
+end

data/app/models/hubrise_app/customer_list_base.rb

@@ -0,0 +1,7 @@
+module HubriseApp
+  class CustomerListBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    store_accessor :api_data, :name
+  end
+end

data/app/models/hubrise_app/location_base.rb

@@ -0,0 +1,11 @@
+module HubriseApp
+  class LocationBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    store_accessor :api_data, :name, :country
+
+    def timezone
+      api_data.dig("timezone", "name")
+    end
+  end
+end

data/app/models/hubrise_app/user_app_instance_base.rb

@@ -0,0 +1,12 @@
+module HubriseApp
+  class UserAppInstanceBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    belongs_to :app_instance, primary_key: :hr_id, foreign_key: :hr_app_instance_id
+
+    REFRESH_THRESHOLD = 1.day
+    def self.fresh(time: Time.now)
+      where("user_app_instances.refreshed_at > ?", time - REFRESH_THRESHOLD)
+    end
+  end
+end

data/app/models/hubrise_app/user_base.rb

@@ -0,0 +1,18 @@
+module HubriseApp
+  class UserBase < HubriseApp::ApplicationRecord
+    self.abstract_class = true
+
+    has_many :user_app_instances, -> { fresh }, primary_key: :hr_id, foreign_key: :hr_user_id
+    has_many :app_instances, through: :user_app_instances
+
+    serialize :locales
+
+    def primary_locale
+      locales.first
+    end
+
+    def full_name
+      [first_name, last_name].join(" ")
+    end
+  end
+end

data/app/models/location.rb

@@ -0,0 +1,2 @@
+class Location < HubriseApp::LocationBase
+end

data/app/models/user.rb

@@ -0,0 +1,2 @@
+class User < HubriseApp::UserBase
+end

data/app/models/user_app_instance.rb

@@ -0,0 +1,2 @@
+class UserAppInstance < HubriseApp::UserAppInstanceBase
+end

data/config/initializers/hubrise_app_config.rb

@@ -1 +1 @@
-HubriseApp::CONFIG = Rails.application.config_for("hubrise_app/config").deep_symbolize_keys
+HubriseApp::CONFIG = HubriseApp::ConfigLoader.load

data/config/routes.rb

@@ -9,4 +9,6 @@ HubriseApp::Engine.routes.draw do
     post :event
     get  :disconnect
   end
+
+  resource :sessions, only: :destroy
 end

data/db/migrate/20190116155419_create_base_tables.rb

@@ -1,44 +1,48 @@
 class CreateBaseTables < ActiveRecord::Migration[5.2]
   def change
-    create_table :hr_accounts do |t|
-      t.string   :hr_id,        null: false, index: { unique: true }
-      t.json     :hr_api_data,  null: false
+    create_table :accounts do |t|
+      t.string :hr_id, null: false, index: { unique: true }
+      t.string :name, null: false
+      t.string :currency, null: false
       t.datetime :refreshed_at, null: false
     end
 
-    create_table :hr_locations do |t|
-      t.string   :hr_id,        null: false, index: { unique: true }
-      t.json     :hr_api_data,  null: false
+    create_table :locations do |t|
+      t.string :hr_id, null: false, index: { unique: true }
+      t.string :name, null: false
       t.datetime :refreshed_at, null: false
     end
 
-    create_table :hr_app_instances do |t|
+    create_table :app_instances do |t|
       t.string :hr_id, null: false, index: { unique: true }
-      t.string :hr_account_id,  index: true
+      t.string :hr_account_id, index: true
       t.string :hr_location_id, index: true
       t.string :hr_catalog_id
       t.string :hr_customer_list_id
-      t.string :hr_access_token, null: false
+      t.string :access_token, null: false
     end
 
-    add_foreign_key :hr_app_instances, :hr_accounts, primary_key: :hr_id
-    add_foreign_key :hr_app_instances, :hr_locations, primary_key: :hr_id
+    add_foreign_key :app_instances, :accounts, primary_key: :hr_id, column: :hr_account_id
+    add_foreign_key :app_instances, :locations, primary_key: :hr_id, column: :hr_location_id
 
-    create_table :hr_users do |t|
-      t.string   :hr_id,            null: false, index: { unique: true }
-      t.json     :hr_api_data,      null: false
-      t.string   :hr_access_token,  null: false
-      t.datetime :refreshed_at,     null: false
+    create_table :users do |t|
+      t.string :hr_id, null: false, index: { unique: true }
+      t.string :access_token, null: false
+      t.string :email
+      t.string :first_name
+      t.string :last_name
+      t.string :locales, array: true
+      t.datetime :refreshed_at, null: false
     end
 
-    create_table :hr_user_app_instances do |t|
-      t.string    :hr_user_id,         index: true
-      t.string    :hr_app_instance_id, index: true
-      t.datetime  :refreshed_at, null: false
+    create_table :user_app_instances do |t|
+      t.string :hr_user_id, index: true
+      t.string :hr_app_instance_id, index: true
+      t.datetime :refreshed_at, null: false
     end
 
-    add_foreign_key :hr_user_app_instances, :hr_users, primary_key: :hr_id
-    add_foreign_key :hr_user_app_instances, :hr_app_instances, primary_key: :hr_id
-    add_index :hr_user_app_instances, %i[hr_user_id hr_app_instance_id refreshed_at], name: :index_user_app_instances, using: :btree
+    add_foreign_key :user_app_instances, :users, primary_key: :hr_id, column: :hr_user_id
+    add_foreign_key :user_app_instances, :app_instances, primary_key: :hr_id, column: :hr_app_instance_id
+    add_index :user_app_instances, %i[hr_user_id hr_app_instance_id refreshed_at], name: :index_user_app_instances, using: :btree
   end
 end

data/db/migrate/20200829091933_add_locations_timezone.rb

@@ -0,0 +1,5 @@
+class AddLocationsTimezone < ActiveRecord::Migration[6.0]
+  def change
+    add_column :locations, :timezone, :string, null: false, default: Time.zone.name
+  end
+end

data/db/migrate/20210120142001_add_api_json_column.rb

@@ -0,0 +1,6 @@
+class AddApiJsonColumn < ActiveRecord::Migration[6.0]
+  def change
+    add_column :accounts, :api_data, :json, after: :hr_id
+    add_column :locations, :api_data, :json, after: :hr_id
+  end
+end

data/db/migrate/20210121150253_remove_replaced_by_api_data_columns.rb

@@ -0,0 +1,9 @@
+class RemoveReplacedByApiDataColumns < ActiveRecord::Migration[6.0]
+  def change
+    remove_column :accounts, :name
+    remove_column :accounts, :currency
+
+    remove_column :locations, :name
+    remove_column :locations, :timezone
+  end
+end

data/db/migrate/20210419120038_add_catalogs_and_customer_lists.rb

@@ -0,0 +1,28 @@
+class AddCatalogsAndCustomerLists < ActiveRecord::Migration[6.0]
+  def change
+    create_table :catalogs do |t|
+      t.string :hr_id, null: false
+      t.json :api_data
+      t.datetime :refreshed_at, null: false
+      t.index :hr_id, unique: true
+    end
+
+    create_table :customer_lists do |t|
+      t.string :hr_id, null: false
+      t.json :api_data
+      t.datetime :refreshed_at, null: false
+      t.index :hr_id, unique: true
+    end
+
+    AppInstance.distinct.pluck(:hr_catalog_id).each do |hr_catalog_id|
+      Catalog.create!(hr_id: hr_catalog_id, refreshed_at: Date.new(2000)) if hr_catalog_id.present?
+    end
+
+    AppInstance.distinct.pluck(:hr_customer_list_id).each do |hr_customer_list_id|
+      CustomerList.create!(hr_id: hr_customer_list_id, refreshed_at: Date.new(2000)) if hr_customer_list_id.present?
+    end
+
+    add_foreign_key :app_instances, :catalogs, primary_key: :hr_id, column: :hr_catalog_id
+    add_foreign_key :app_instances, :customer_lists, primary_key: :hr_id, column: :hr_customer_list_id
+  end
+end

data/lib/hubrise_app/spec_support.rb

@@ -0,0 +1,3 @@
+class HubriseApp::SpecSupport
+  FACTORIES_PATH = File.expand_path("../../spec/factories", __dir__)
+end

data/lib/hubrise_app/version.rb

@@ -1,3 +1,3 @@
 module HubriseApp
-  VERSION = "0.1.2".freeze
+  VERSION = "1.1.1".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hubrise_app
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 1.1.1
 platform: ruby
 authors:
 - Antoine Monnier
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-05-31 00:00:00.000000000 Z
+date: 2022-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hubrise_client
@@ -17,26 +17,26 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 2.0.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.0.0
+        version: 2.0.6
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: 5.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">"
       - !ruby/object:Gem::Version
         version: 5.2.2
 description: 
@@ -45,36 +45,62 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- MIT-LICENSE
+- LICENSE
 - README.md
 - Rakefile
 - app/assets/config/hubrise_app_manifest.js
 - app/assets/stylesheets/hubrise_app/application.css
 - app/controllers/hubrise_app/application_controller.rb
+- app/controllers/hubrise_app/application_controller/app_instance_methods.rb
+- app/controllers/hubrise_app/application_controller/hubrise_gateway_methods.rb
+- app/controllers/hubrise_app/application_controller/session_methods.rb
 - app/controllers/hubrise_app/callback_controller.rb
+- app/controllers/hubrise_app/callback_controller/action_disconnect.rb
+- app/controllers/hubrise_app/callback_controller/action_event.rb
 - app/controllers/hubrise_app/oauth_controller.rb
+- app/controllers/hubrise_app/oauth_controller/action_authorize_callback.rb
+- app/controllers/hubrise_app/oauth_controller/action_connect_callback.rb
+- app/controllers/hubrise_app/oauth_controller/action_login_callback.rb
+- app/controllers/hubrise_app/sessions_controller.rb
+- app/lib/hubrise_app/config_loader.rb
 - app/lib/hubrise_app/hubrise_gateway.rb
+- app/lib/hubrise_app/refresher/account.rb
+- app/lib/hubrise_app/refresher/app_instance.rb
+- app/lib/hubrise_app/refresher/base.rb
+- app/lib/hubrise_app/refresher/catalog.rb
+- app/lib/hubrise_app/refresher/customer_list.rb
+- app/lib/hubrise_app/refresher/location.rb
+- app/lib/hubrise_app/refresher/user.rb
+- app/lib/hubrise_app/services/assign_app_instance.rb
 - app/lib/hubrise_app/services/connect_app_instance.rb
-- app/lib/hubrise_app/services/disconnect_app_instance.rb
-- app/lib/hubrise_app/services/handle_event.rb
-- app/lib/hubrise_app/services/override/connect_app_instance.rb
-- app/lib/hubrise_app/services/override/disconnect_app_instance.rb
-- app/lib/hubrise_app/services/override/handle_event.rb
-- app/models/concerns/hubrise_app/hr_api_resource.rb
+- app/lib/hubrise_app/services/resolve_app_instance.rb
+- app/models/account.rb
+- app/models/app_instance.rb
+- app/models/catalog.rb
+- app/models/customer_list.rb
+- app/models/hubrise_app/account_base.rb
+- app/models/hubrise_app/app_instance_base.rb
 - app/models/hubrise_app/application_record.rb
-- app/models/hubrise_app/hr_account.rb
-- app/models/hubrise_app/hr_app_instance.rb
-- app/models/hubrise_app/hr_location.rb
-- app/models/hubrise_app/hr_user.rb
-- app/models/hubrise_app/hr_user_app_instance.rb
-- app/models/hubrise_app/override/hr_user.rb
+- app/models/hubrise_app/catalog_base.rb
+- app/models/hubrise_app/customer_list_base.rb
+- app/models/hubrise_app/location_base.rb
+- app/models/hubrise_app/user_app_instance_base.rb
+- app/models/hubrise_app/user_base.rb
+- app/models/location.rb
+- app/models/user.rb
+- app/models/user_app_instance.rb
 - app/views/hubrise_app/application/root.html.haml
 - app/views/layouts/hubrise_app/application.html.erb
 - config/initializers/hubrise_app_config.rb
 - config/routes.rb
 - db/migrate/20190116155419_create_base_tables.rb
+- db/migrate/20200829091933_add_locations_timezone.rb
+- db/migrate/20210120142001_add_api_json_column.rb
+- db/migrate/20210121150253_remove_replaced_by_api_data_columns.rb
+- db/migrate/20210419120038_add_catalogs_and_customer_lists.rb
 - lib/hubrise_app.rb
 - lib/hubrise_app/engine.rb
+- lib/hubrise_app/spec_support.rb
 - lib/hubrise_app/version.rb
 homepage: https://github.com/HubRise/ruby-app
 licenses:
@@ -95,8 +121,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Rails engine to bootstrap a HubRise-based application

data/MIT-LICENSE

@@ -1,20 +0,0 @@
-Copyright 2019 nsave
-
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
-
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/app/lib/hubrise_app/services/disconnect_app_instance.rb

@@ -1,6 +0,0 @@
-class HubriseApp::Services::DisconnectAppInstance
-  prepend HubriseApp::Services::Override::DisconnectAppInstance
-
-  class << self; delegate :run, to: :new; end
-  def run(hr_app_instance); end
-end

data/app/lib/hubrise_app/services/handle_event.rb

@@ -1,22 +0,0 @@
-class HubriseApp::Services::HandleEvent
-  prepend HubriseApp::Services::Override::HandleEvent
-
-  class << self; delegate :run, to: :new; end
-
-  def run(hr_app_instance, params)
-    case params["resource_type"]
-    when "customer"
-      handle_customer_event(hr_app_instance, params)
-    when "order"
-      handle_order_event(hr_app_instance, params)
-    else
-      raise "Cannot handle hubrise event with type: " + params["resource_type"].to_s
-    end
-  end
-
-  protected
-
-  def handle_customer_event(hr_app_instance, params);  end
-
-  def handle_order_event(hr_app_instance, params);     end
-end

data/app/lib/hubrise_app/services/override/connect_app_instance.rb

@@ -1 +0,0 @@
-module HubriseApp::Services::Override::ConnectAppInstance; end

data/app/lib/hubrise_app/services/override/disconnect_app_instance.rb

@@ -1 +0,0 @@
-module HubriseApp::Services::Override::DisconnectAppInstance; end

data/app/lib/hubrise_app/services/override/handle_event.rb

@@ -1 +0,0 @@
-module HubriseApp::Services::Override::HandleEvent; end

data/app/models/concerns/hubrise_app/hr_api_resource.rb

@@ -1,29 +0,0 @@
-module HubriseApp::HrApiResource
-  REFRESH_THRESHOLD = 1.day
-
-  def stale?
-    refreshed_at.nil? || Time.now - refreshed_at > REFRESH_THRESHOLD
-  end
-
-  def refresh_via_api_client(api_client)
-    return unless stale?
-
-    refresh_with(self.class.fetch_hr_attrs(api_client, hr_id))
-  end
-
-  def refresh_with(attrs)
-    update!(attrs.merge(refreshed_at: Time.now))
-  end
-
-  def self.included(base)
-    base.extend(ClassMethods)
-  end
-
-  module ClassMethods
-    def refresh_or_create_via_api_client(api_client, hr_id)
-      hr_record = find_or_initialize_by(hr_id: hr_id)
-      hr_record.refresh_via_api_client(api_client)
-      hr_record
-    end
-  end
-end

data/app/models/hubrise_app/hr_account.rb

@@ -1,19 +0,0 @@
-class HubriseApp::HrAccount < HubriseApp::ApplicationRecord
-  self.table_name = :hr_accounts
-
-  include HubriseApp::HrApiResource
-  class << self
-    def fetch_hr_attrs(api_client, hr_id)
-      raise if api_client.account_id != hr_id
-
-      {
-        hr_id: hr_id,
-        hr_api_data: if api_client.location_id
-                       api_client.get_location(api_client.location_id).data["account"]
-                     else
-                       api_client.get_account(hr_id).data
-                     end.except("id")
-      }
-    end
-  end
-end

data/app/models/hubrise_app/hr_app_instance.rb

@@ -1,35 +0,0 @@
-module HubriseApp
-  class HrAppInstance < HubriseApp::ApplicationRecord
-    self.table_name = :hr_app_instances
-
-    belongs_to :hr_account,   optional: true, primary_key: :hr_id
-    belongs_to :hr_location,  optional: true, primary_key: :hr_id
-
-    def api_client
-      HubriseApp::HubriseGateway.build_api_client(
-        access_token: hr_access_token,
-        app_instance_id: hr_id,
-        account_id: hr_account_id,
-        location_id: hr_location_id,
-        catalog_id: hr_catalog_id,
-        customer_list_id: hr_customer_list_id
-      )
-    end
-
-    def self.refresh_or_create_via_api_client(api_client)
-      hr_app_instance = find_or_initialize_by(hr_id: api_client.app_instance_id)
-      hr_account      = api_client.account_id   && HrAccount.refresh_or_create_via_api_client(api_client, api_client.account_id) # rubocop:disable Metrics/LineLength
-      hr_location     = api_client.location_id  && HrLocation.refresh_or_create_via_api_client(api_client, api_client.location_id) # rubocop:disable Metrics/LineLength
-
-      hr_app_instance.update!(
-        hr_account: hr_account,
-        hr_location: hr_location,
-        hr_access_token: api_client.access_token,
-        hr_catalog_id: api_client.catalog_id,
-        hr_customer_list_id: api_client.customer_list_id
-      )
-
-      hr_app_instance
-    end
-  end
-end

data/app/models/hubrise_app/hr_location.rb

@@ -1,11 +0,0 @@
-class HubriseApp::HrLocation < HubriseApp::ApplicationRecord
-  self.table_name = :hr_locations
-
-  include HubriseApp::HrApiResource
-  def self.fetch_hr_attrs(api_client, hr_id)
-    {
-      hr_id: hr_id,
-      hr_api_data: api_client.get_location(hr_id).data.except("id")
-    }
-  end
-end

data/app/models/hubrise_app/hr_user.rb

@@ -1,35 +0,0 @@
-module HubriseApp
-  class HrUser < HubriseApp::ApplicationRecord
-    include HubriseApp::Override::HrUser
-
-    self.table_name = :hr_users
-
-    has_many :hr_user_app_instances, -> { fresh }, primary_key: :hr_id
-    has_many :hr_app_instances, through: :hr_user_app_instances
-
-    include HubriseApp::HrApiResource
-    class << self
-      def fetch_hr_attrs(api_client)
-        data = api_client.get_user.data
-        {
-          hr_id: data.delete("id"),
-          hr_api_data: data,
-          hr_access_token: api_client.access_token
-        }
-      end
-
-      def refresh_or_create_via_api_client(api_client)
-        hr_attrs  = fetch_hr_attrs(api_client)
-        hr_user   = find_or_initialize_by(hr_id: hr_attrs[:hr_id])
-        hr_user.refresh_with(hr_attrs)
-        hr_user
-      end
-    end
-
-    def assign_hr_app_instance(hr_app_instance)
-      hr_user_app_instance = HrUserAppInstance.find_or_initialize_by(hr_app_instance_id: hr_app_instance.hr_id,
-                                                                     hr_user_id: hr_id)
-      hr_user_app_instance.refresh!
-    end
-  end
-end