eco-helpers 3.0.29 → 3.0.31

data/lib/eco/api/common/people/person_parser.rb

@@ -81,7 +81,7 @@ module Eco
 
           # Select Options
           def select_tables
-            return nil unless @schema
+            return unless @schema
 
             @select_tables ||= @schema.fields.select do |fld|
               fld.type == 'select'
@@ -226,6 +226,7 @@ module Eco
 
             to_h.merge!(parser.to_h)
             patched!
+
             self
           end
 
@@ -252,6 +253,7 @@ module Eco
             end
 
             patched!
+
             self
           end
           # @!endgroup

data/lib/eco/api/custom/parser.rb

@@ -5,7 +5,7 @@
 #
 #    active_when_any "some-field1", "some-field2" # optional
 #    parsing_phase     :final # optional, default is  (default :internal)
-#    serializing_phase :final # optional, it will be    (default :person)
+#    serializing_phase :final # optional, it will be  (default :person)
 #
 #    def parser(hash, deps)
 #      get_values(hash).map do |name|

data/lib/eco/api/organization/people.rb

@@ -127,6 +127,7 @@ module Eco
           id          = attr_value(object, 'id')
           external_id = attr_value(object, 'external_id')
           email       = attr_value(object, 'email')
+
           person(id: id, external_id: external_id, email: email, strict: strict)
         end
         # @!endgroup

data/lib/eco/api/session/config.rb

@@ -236,10 +236,15 @@ module Eco
           org['location_codes']
         end
 
-        def locations_mapper
+        def locations_mapper(insensitive: false)
           return @locations_mapper if instance_variable_defined?(:@locations_mapper)
+
           file = file_manager.newest(location_codes)
-          @locations_mapper = Eco::Data::Mapper.new(file_manager.load_json(file), internal: :first)
+          @locations_mapper = Eco::Data::Mapper.new(
+            file_manager.load_json(file),
+            internal:    :first,
+            insensitive: insensitive
+          )
         end
 
         def tagtree=(file)

data/lib/eco/api/usecases/default/locations/cli/codes_to_tags_cli.rb

@@ -2,7 +2,7 @@ class Eco::API::UseCases::Default::Locations::CodesToTagsCase
   # Class to define the CLI integration of a usecase anywhere it suits.
   class Cli < Eco::API::UseCases::Cli
     str  = "Adds a column 'ecoPortalTag' to the input CSV "
-    str << "with the tags that the location codes map to"
+    str << 'with the tags that the location codes map to'
     desc str
 
     callback do |_sess, options, _case|
@@ -10,8 +10,8 @@ class Eco::API::UseCases::Default::Locations::CodesToTagsCase
       options.deep_merge!(other: {file: {name: file, format: :csv}})
     end
 
-    add_option("-column", "Specify the input column header with the codes") do |options|
-      col_codes = SCR.get_arg("-column", with_param: true)
+    add_option('-column', 'Specify the input column header with the codes') do |options|
+      col_codes = SCR.get_arg('-column', with_param: true)
       options.deep_merge!(other: {file: {codes_column: col_codes}})
     end
   end

data/lib/eco/api/usecases/default/locations/cli/csv_to_tree_cli.rb

@@ -1,7 +1,7 @@
 class Eco::API::UseCases::Default::Locations::CsvToTree
   # Class to define the CLI integration of a usecase anywhere it suits.
   class Cli < Eco::API::UseCases::Cli
-    str  = "Creates a JSON file with the tagtree from a CSV file"
+    str  = 'Creates a JSON file with the tagtree from a CSV file'
     desc str
 
     callback do |_sess, options, _case|

data/lib/eco/api/usecases/default/locations/cli/tagtree_extract_cli.rb

@@ -1,33 +1,33 @@
 class Eco::API::UseCases::Default::Locations::TagtreeExtract
   # Class to define the CLI integration of a usecase anywhere it suits.
   class Cli < Eco::API::UseCases::Cli
-    desc "Converts a tagtree into a specific format"
+    desc 'Converts a tagtree into a specific format'
 
-    add_option("-include-archived", "Whether the archived nodes should be included") do |options|
+    add_option('-include-archived', 'Whether the archived nodes should be included') do |options|
       options.deep_merge!(output: {include: {archived: true}})
     end
 
-    add_option("-format", "Kind of extract (list | json | nodes | csv_tree | excel_tree | excel_nodes )") do |options|
-      format = SCR.get_arg("-format", with_param: true)
+    add_option('-format', 'Kind of extract (list | json | nodes | csv_tree | excel_tree | excel_nodes )') do |options|
+      format = SCR.get_arg('-format', with_param: true)
       options.deep_merge!(output: {format: format})
     end
 
-    add_option("-node-attrs", "Attributes to be dumped (i.e. name|id ). Default: id") do |options|
-      attrs = SCR.get_arg("-node-attrs", with_param: true)
+    add_option('-node-attrs', 'Attributes to be dumped (i.e. name|id ). Default: id') do |options|
+      attrs = SCR.get_arg('-node-attrs', with_param: true)
       options.deep_merge!(output: {attrs: attrs.split('|').map(&:to_sym)})
     end
 
-    add_option("-with-ancestors", "Include full path of ancestors in tree csv/excel_tree") do |options|
+    add_option('-with-ancestors', 'Include full path of ancestors in tree csv/excel_tree') do |options|
       options.deep_merge!(output: {include: {ancestors: true}})
     end
 
-    add_option("-indent", "Custom indentation (i.e. \"++\")") do |options|
-      indent = SCR.get_arg("-indent", with_param: true) || ""
+    add_option('-indent', 'Custom indentation (i.e. \"++\")') do |options|
+      indent = SCR.get_arg('-indent', with_param: true) || ''
       options.deep_merge!(output: {indent: indent})
     end
 
-    add_option("-file-as-source", "Use a file as locations input (rather than the live locations)") do |options|
-      file = SCR.get_file("-file-as-source", required: true, should_exist: true)
+    add_option('-file-as-source', 'Use a file as locations input (rather than the live locations)') do |options|
+      file = SCR.get_file('-file-as-source', required: true, should_exist: true)
       options.deep_merge!(source: {file: file})
     end
   end

data/lib/eco/api/usecases/default/locations/cli/tagtree_paths_cli.rb

@@ -1,11 +1,11 @@
 class Eco::API::UseCases::Default::Locations::TagTreePaths
   # Class to define the CLI integration of a usecase anywhere it suits.
   class Cli < Eco::API::UseCases::Cli
-    str  = "Creates a CSV with the paths to each tag"
+    str  = 'Creates a CSV with the paths to each tag'
     desc str
 
-    add_option("-column", "Specify the input column header with the codes") do |options|
-      col_codes = SCR.get_arg("-column", with_param: true)
+    add_option('-column', 'Specify the input column header with the codes') do |options|
+      col_codes = SCR.get_arg('-column', with_param: true)
       options.deep_merge!(other: {file: {codes_column: col_codes}})
     end
   end

data/lib/eco/api/usecases/default/locations/cli/tagtree_upload_cli.rb

@@ -8,13 +8,13 @@ class Eco::API::UseCases::Default::Locations::TagtreeUpload
       options.deep_merge!(source: {file: file})
     end
 
-    add_option("-top-id", "To nest the upload under some existing node (i.e. top node)") do |options|
-      format = SCR.get_arg("-top-id", with_param: true)
+    add_option('-top-id', 'To nest the upload under some existing node (i.e. top node)') do |options|
+      format = SCR.get_arg('-top-id', with_param: true)
       options.deep_merge!(input: {top_id: format})
     end
 
-    add_option("-structure-id", "Target structure id") do |options|
-      id = SCR.get_arg("-structure-id", with_param: true)
+    add_option('-structure-id', 'Target structure id') do |options|
+      id = SCR.get_arg('-structure-id', with_param: true)
       options.deep_merge!(source: {structure_id: id})
     end
   end

data/lib/eco/api/usecases/default/meta/cli/graphql_schema_cli.rb

@@ -1,14 +1,14 @@
 class Eco::API::UseCases::Default::Meta::GraphQLSchema
   class Cli < Eco::API::UseCases::Cli
-    desc "Retrieves and stores the GraphQL Schema"
+    desc 'Retrieves and stores the GraphQL Schema'
 
     callback do |_sess, options, _case|
       file = SCR.get_arg(cli_name, with_param: true)
       options.deep_merge!(output: {file: {name: file}})
     end
 
-    add_option("-format", "Kind of extract (definition | json | yaml )") do |options|
-      format = SCR.get_arg("-format", with_param: true)
+    add_option('-format', 'Kind of extract (definition | json | yaml )') do |options|
+      format = SCR.get_arg('-format', with_param: true)
       options.deep_merge!(output: {format: format})
     end
   end

data/lib/eco/api/usecases/default/meta/graphql_schema.rb

@@ -3,11 +3,11 @@ class Eco::API::UseCases::Default::Meta::GraphQLSchema < Eco::API::Common::Loade
 
   require_relative 'cli/graphql_schema_cli'
 
-  OUT_FOLDER      = "sftp".freeze
+  OUT_FOLDER      = 'sftp'.freeze
   OUT_TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
-  OUT_FILENAME    = "graphql_schema".freeze
+  OUT_FILENAME    = 'graphql_schema'.freeze
 
-  name "graphql-schema"
+  name 'graphql-schema'
   type :other
 
   def main(*_args)

data/lib/eco/api/usecases/default/people/amend/cli/clean_unknown_tags_cli.rb

@@ -5,11 +5,11 @@ class Eco::API::UseCases::Default::People::Amend::CleanUnknownTags
     str << 'It will preserve standard register tags of most common registers (i.e. EVENT, RISK).'
     desc str
 
-    add_option("-clean-register-tags", "Will also try to remove register tags") do |options|
+    add_option('-clean-register-tags', 'Will also try to remove register tags') do |options|
       options.deep_merge!(usecase: {clean_register_tags: true})
     end
 
-    add_option("-clean-archived", "Will also try to remove archived tags") do |options|
+    add_option('-clean-archived', 'Will also try to remove archived tags') do |options|
       options.deep_merge!(usecase: {clean_archived: true})
     end
   end

data/lib/eco/api/usecases/default/people/amend/cli/reinvite_sync_cli.rb

@@ -1,6 +1,6 @@
 class Eco::API::UseCases::Default::People::Amend::ReinviteSyncCase
   class Cli < Eco::API::UseCases::Default::People::Amend::ReinviteTransCase::Cli
-    str = "Re-sends invites to target users that have not accepted the invite as yet"
+    str = 'Re-sends invites to target users that have not accepted the invite as yet'
     desc str
 
     cli_name '-reinvite-from'

data/lib/eco/api/usecases/default/people/amend/cli/reinvite_trans_cli.rb

@@ -1,10 +1,10 @@
 class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase
   class Cli < Eco::API::UseCases::Cli
-    str = "Re-sends invites to all filtered users that have not accepted the invite as yet"
+    str = 'Re-sends invites to all filtered users that have not accepted the invite as yet'
     desc str
 
-    str = "If also send an invite email to those that have accepted the invite"
-    add_option("-force", str) do |options|
+    str = 'If also send an invite email to those that have accepted the invite'
+    add_option('-force', str) do |options|
       options.deep_merge!(force: {invite: true})
     end
   end

data/lib/eco/api/usecases/default/people/amend/cli/restore_db_cli.rb

@@ -1,25 +1,25 @@
 class Eco::API::UseCases::Default::People::Amend::RestoreDBCase
   class Cli < Eco::API::UseCases::Cli
-    str = "Restores the people manager by using a backup.json file"
+    str = 'Restores the people manager by using a backup.json file'
     desc str
 
     callback do |input, session, _opts, _case|
-      unless input && input.is_a?(Eco::API::Organization::People)
-        file  = SCR.get_file(cli_name, required: true)
-        input = Eco::API::Organization::People.new(JSON.parse(File.read(file)))
-        session.log(:info) {
-          "Source DB: loaded #{input.length} entries."
-        }
-      end
+      next if input.is_a?(Eco::API::Organization::People)
+
+      file  = SCR.get_file(cli_name, required: true)
+      input = Eco::API::Organization::People.new(JSON.parse(File.read(file)))
+      session.log(:info) {
+        "Source DB: loaded #{input.length} entries."
+      }
     end
 
-    str = "If it should DELETE people that do not exist in the backup file"
-    add_option("-include-delete", str) do |options|
+    str = 'If it should DELETE people that do not exist in the backup file'
+    add_option('-include-delete', str) do |options|
       options.deep_merge!(include: {delete: true})
     end
 
-    str = "If it should CREATE people that do not exist in the people manager"
-    add_option("-include-create", str) do |options|
+    str = 'If it should CREATE people that do not exist in the people manager'
+    add_option('-include-create', str) do |options|
       options.deep_merge!(include: {create: true})
     end
   end

data/lib/eco/api/usecases/default/people/amend/refresh_case.rb

@@ -1,9 +1,9 @@
 class Eco::API::UseCases::Default::People::Amend::RefreshCase < Eco::API::Common::Loaders::UseCase
-  name "refresh"
+  name 'refresh'
   type :transform
 
   def main(people, session, _options, usecase)
-    update = session.new_job("main", "update", :update, usecase)
+    update = session.new_job('main', 'update', :update, usecase)
     people.each {|person| update.add(person)}
   end
 end

data/lib/eco/api/usecases/default/people/amend/reinvite_sync_case.rb

@@ -1,6 +1,6 @@
 module Eco::API::UseCases::Default::People::Amend
   class ReinviteSyncCase < ReinviteTransCase
-    name "reinvite"
+    name 'reinvite'
     type :sync
 
     require_relative 'cli/reinvite_sync_cli'

data/lib/eco/api/usecases/default/people/amend/reinvite_trans_case.rb

@@ -1,5 +1,5 @@
 class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase < Eco::API::Common::Loaders::UseCase
-  name "reinvite"
+  name 'reinvite'
   type :transform
 
   require_relative 'cli/reinvite_trans_cli'
@@ -11,7 +11,7 @@ class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase < Eco::API::
   private
 
   def reinvite(users)
-    invite = session.new_job("main", "invite", :update, usecase, :account)
+    invite = session.new_job('main', 'invite', :update, usecase, :account)
     users.each do |person|
       if force_invite?
         person.account.force_send_invites = true

data/lib/eco/api/usecases/default/people/treat/analyse_people_case.rb

@@ -97,7 +97,7 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
 
     session.process_case(
       'to-csv',
-      type:   :export,
+      type:    :export,
       data:    data,
       options: opts.merge(options.slice(:export))
     )
@@ -109,6 +109,7 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
 
   def field_similarity
     return :name unless use_field?
+
     use_field_proc
   end
 

data/lib/eco/api/usecases/default/people/treat/cli/analyse_people_cli.rb

@@ -1,65 +1,65 @@
 class Eco::API::UseCases::Default::People::Treat::Analyse
   class Cli < Eco::API::UseCases::Cli
-    str = "Provides a set of tools to analyse a set of people (i.e. detect duplicates)"
+    str = 'Provides a set of tools to analyse a set of people (i.e. detect duplicates)'
     desc str
 
     callback do |_people, _session, options|
-      options.deep_merge!(output: {file: "people_analysis.txt"}) unless options.dig(:output, :file)
+      options.deep_merge!(output: {file: 'people_analysis.txt'}) unless options.dig(:output, :file)
       #unless options.dig(:usecase, :analyse_people, :use_field)
       #  options.deep_merge!(usecase: {analyse_people: {use_field: :name}})
       #end
     end
 
-    add_option("-to", "Specify the output file.") do |options|
-      file = SCR.get_file("-to", required: true, should_exist: false)
+    add_option('-to', 'Specify the output file.') do |options|
+      file = SCR.get_file('-to', required: true, should_exist: false)
       options.deep_merge!(output: {file: file})
     end
 
-    str = "Generates a list of people with possible duplicates."
-    add_option("-identify-duplicates", str) do |options|
+    str = 'Generates a list of people with possible duplicates.'
+    add_option('-identify-duplicates', str) do |options|
       options.deep_merge!(usecase: {analyse_people: {identify_duplicates: true}})
     end
 
-    str = "Works with -identify-duplicates. Sets field to be used in the comparison."
-    add_option("-use-field", str) do |options|
-      expression = SCR.get_arg("-use-field", with_param: true)
+    str = 'Works with -identify-duplicates. Sets field to be used in the comparison.'
+    add_option('-use-field', str) do |options|
+      expression = SCR.get_arg('-use-field', with_param: true)
       options.deep_merge!(usecase: {analyse_people: {use_field: expression}})
     end
 
-    str = "Works with -identify-duplicates. Adds an additional layer of comparison."
-    add_option("-facet-field", str) do |options|
-      expression = SCR.get_arg("-facet-field", with_param: true)
+    str = 'Works with -identify-duplicates. Adds an additional layer of comparison.'
+    add_option('-facet-field', str) do |options|
+      expression = SCR.get_arg('-facet-field', with_param: true)
       options.deep_merge!(usecase: {analyse_people: {facet_field: expression}})
     end
 
-    str = "Works with -identify-duplicates. Skips the rearrangement stage."
-    add_option("-only-screening", str) do |options|
+    str = 'Works with -identify-duplicates. Skips the rearrangement stage.'
+    add_option('-only-screening', str) do |options|
       options.deep_merge!(usecase: {analyse_people: {only_screening: true}})
     end
 
-    str = "Works with -identify-duplicates. Re-adjust scores ignoring matching words."
-    add_option("-ignore-matching-words", str) do |options|
+    str = 'Works with -identify-duplicates. Re-adjust scores ignoring matching words.'
+    add_option('-ignore-matching-words', str) do |options|
       options.deep_merge!(usecase: {analyse_people: {ignore_matching_words: true}})
     end
 
-    str = "Works with -identify-duplicates. Re-adjust the comparing strings to do not have repeated words."
-    add_option("-unique-words", str) do |options|
+    str = 'Works with -identify-duplicates. Re-adjust the comparing strings to do not have repeated words.'
+    add_option('-unique-words', str) do |options|
       options.deep_merge!(usecase: {analyse_people: {unique_words: true}})
     end
 
-    add_option("-identify-unnamed", "Identifies all people with no names.") do |options|
+    add_option('-identify-unnamed', 'Identifies all people with no names.') do |options|
       options.deep_merge!(usecase: {analyse_people: {identify_unnamed: true}})
     end
 
-    str = "Generates a json file with all the people involved in the final results of the analysis."
-    add_option("-backup-people-results", str) do |options|
-      file = SCR.get_file("-backup-people-results", required: true, should_exist: false)
+    str = 'Generates a json file with all the people involved in the final results of the analysis.'
+    add_option('-backup-people-results', str) do |options|
+      file = SCR.get_file('-backup-people-results', required: true, should_exist: false)
       options.deep_merge!(usecase: {analyse_people: {backup_people: File.expand_path(file)}})
     end
 
-    str = "Genarates a CSV file with all people of the final results."
-    add_option("-to-csv", str) do |options|
-      file = SCR.get_file("-to-csv", required: true, should_exist: false) || "Results.csv"
+    str = 'Genarates a CSV file with all people of the final results.'
+    add_option('-to-csv', str) do |options|
+      file = SCR.get_file('-to-csv', required: true, should_exist: false) || 'Results.csv'
       options.deep_merge!(usecase: {analyse_people: {csv_file: File.expand_path(file)}})
     end
   end

data/lib/eco/api/usecases/default/people/treat/cli/org_data_convert_cli.rb

@@ -1,7 +1,7 @@
 class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase
   class Cli < Eco::API::UseCases::Cli
     str  = "Usage '#{cli_name} backup.json -restore-db-from'."
-    str << " Transforms an input .json file to the values of the destination environment "
+    str << ' Transforms an input .json file to the values of the destination environment '
     str << " (names missmatch won't solve: i.e. usergroups)"
     desc str
 
@@ -16,19 +16,19 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase
 
       unless options[:source_enviro]
         session.log(:error) {
-          "You need to specify a -source-enviro for the conversion to work out"
+          'You need to specify a -source-enviro for the conversion to work out'
         }
         exit(1)
       end
     end
 
-    str = "The defined -source-enviro API configuration that the backup file was generated from"
-    add_option("-source-enviro", str) do |options|
-      options.merge!(source_enviro: SCR.get_arg("-source-enviro", with_param: true))
+    str = 'The defined -source-enviro API configuration that the backup file was generated from'
+    add_option('-source-enviro', str) do |options|
+      options.merge!(source_enviro: SCR.get_arg('-source-enviro', with_param: true))
     end
 
-    str = "Prevents the script to crash when backup file has missing usergroups in the org"
-    add_option("-ignore-missing-policy-groups", str) do |options|
+    str = 'Prevents the script to crash when backup file has missing usergroups in the org'
+    add_option('-ignore-missing-policy-groups', str) do |options|
       options.deep_merge!(ignore: {missing: {policy_groups: true}})
     end
   end

data/lib/eco/api/usecases/default/people/treat/cli/supers_cyclic_identify_cli.rb

@@ -1,14 +1,13 @@
 class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify
   class Cli < Eco::API::UseCases::Cli
-    str = "Provides a set of tools to analyse a set of people (i.e. detect duplicates)"
-    desc str
+    desc 'Identifies Cyclic supervisor chains'
 
     # callback do |_people, _session, options|
-    #   options.deep_merge!(output: {file: "cyclic_supers.txt"}) unless options.dig(:output, :file)
+    #   options.deep_merge!(output: {file: 'cyclic_supers.txt'}) unless options.dig(:output, :file)
     # end
 
-    add_option("-to", "Specify the output file") do |options|
-      file = SCR.get_file("-to", required: true, should_exist: false)
+    add_option('-to', 'Specify the output file') do |options|
+      file = SCR.get_file('-to', required: true, should_exist: false)
       options.deep_merge!(output: {file: file})
     end
   end

data/lib/eco/api/usecases/default/people/treat/cli/supers_hierachy_cli.rb

@@ -1,14 +1,14 @@
 class Eco::API::UseCases::Default::People::Treat::SupersHierarchy
   class Cli < Eco::API::UseCases::Cli
-    str = "Draws the Supervisors hiearchy in a file"
+    str = 'Draws the Supervisors hiearchy in a file'
     desc str
 
     # callback do |_people, _session, options|
-    #   options.deep_merge!(output: {file: "supers_hierarchy.txt"}) unless options.dig(:output, :file)
+    #   options.deep_merge!(output: {file: 'supers_hierarchy.txt'}) unless options.dig(:output, :file)
     # end
 
-    add_option("-to", "Specify the output file") do |options|
-      file = SCR.get_file("-to", required: true, should_exist: false)
+    add_option('-to', 'Specify the output file') do |options|
+      file = SCR.get_file('-to', required: true, should_exist: false)
       options.deep_merge!(output: {file: file})
     end
   end

data/lib/eco/api/usecases/default/people/treat/org_data_convert_case.rb

@@ -1,5 +1,5 @@
 class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API::Common::Loaders::UseCase
-  name "org-data-convert"
+  name 'org-data-convert'
   type :import
 
   require_relative 'cli/org_data_convert_cli'
@@ -45,7 +45,7 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
     return unless person.details
 
     person.details.as_json.tap do |details_doc|
-      details_doc["schema_id"] = session.schema.id
+      details_doc['schema_id'] = session.schema.id
       person.add_details(session.schema)
       person.details = details_doc
     end
@@ -85,7 +85,7 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
   def source_enviro
     @source_enviro ||= options[:source_enviro].tap do |enviro|
       next if enviro
-      error("A source environment is required to migrate data from")
+      error('A source environment is required to migrate data from')
     end
   end
 

data/lib/eco/api/usecases/default/people/treat/supers_cyclic_identify_case.rb

@@ -4,9 +4,9 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
 
   require_relative 'cli/supers_cyclic_identify_cli'
 
-  OUT_FOLDER      = "sftp".freeze
+  OUT_FOLDER      = 'sftp'.freeze
   OUT_TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
-  OUT_FILENAME    = "cyclic_supers".freeze
+  OUT_FILENAME    = 'cyclic_supers'.freeze
 
   def main(*_args)
     options[:end_get] = false
@@ -21,28 +21,28 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
 
   def save!(data)
     if data.empty?
-      log(:info) { "There were no cyclic supervisors identified!!" }
+      log(:info) { 'There were no cyclic supervisors identified!!' }
       return
     end
 
-    ext = File.extname(file).downcase.delete(".")
+    ext = File.extname(file).downcase.delete('.')
 
     case ext
-    when "txt"
+    when 'txt'
       create_file(data, file: output_filename(ext: 'txt'), format: :txt)
-    when "html"
-      log(:warn) { "html is still not supported" }
+    when 'html'
+      log(:warn) { 'html is still not supported' }
       exit(1)
       # create_file(data, file: file, format: :html)
-    when "json"
-      log(:warn) { "json is still not supported" }
+    when 'json'
+      log(:warn) { 'json is still not supported' }
       exit(1)
       # create_file(data, file: file, format: :json)
     end
   end
 
   def create_file(sets, file:, format: :txt)
-    File.open(file, "w") do |fd|
+    File.open(file, 'w') do |fd|
       fd << sets_to_str(sets, format: format)
     end
 
@@ -53,7 +53,7 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
 
   def sets_to_str(sets, format: :txt)
     raise "Required Array. Given: #{sets.class}" unless sets.is_a?(Array)
-    "".tap do |str|
+    ''.tap do |str|
       sets.each do |set|
         str << set_to_str(set, format: format)
       end
@@ -63,13 +63,14 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
   def set_to_str(set, lev: 0, format: :txt)
     raise "Required Array. Given: #{set.class}" unless set.is_a?(Array)
 
-    "".tap do |str|
+    ''.tap do |str|
       entry = set.shift
-      str << ("  " * lev)
-      str << lev&.positive? ? "+-#{lev}- " : ""
+      str << ('  ' * lev)
+      str << lev&.positive? ? "+-#{lev}- " : ''
       str << entry.name
       str << " (#{entry.external_id}|#{entry.email}|#{entry.id})\n"
       next unless set && !set.empty?
+
       str << set_to_str(set, lev: lev + 1, format: format)
     end
   end