eco-helpers 3.0.26 → 3.0.28

data/lib/eco/api/usecases/ooze_samples/register_export_case.rb

@@ -15,7 +15,7 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
 
   include Eco::API::UseCases::OozeSamples::Helpers
 
-  name "register-export-case"
+  name 'register-export-case'
   type :other
 
   attr_reader :session, :options, :usecase
@@ -58,29 +58,29 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   # @param ooze_id [String]
   # @return [Ecoportal::API::V2::Page]
   def build_full_ooze(ooze_id) # rubocop:disable Metrics/AbcSize
-    return unless (page = ooze(ooze_id))
+    return      unless (page = ooze(ooze_id))
     return page unless page.is_a?(Ecoportal::API::V2::Pages::PageStage)
 
     pending_stage_ids = page.stages.ordered.map(&:id) - [page.current_stage_id]
     state    = page.state
     secs_doc = page.sections.doc
     flds_doc = page.components.doc
-    hsecs    = secs_doc.each_with_object({}) {|sec, h| h[sec["id"]] = sec}
-    hflds    = flds_doc.each_with_object({}) {|fld, h| h[fld["id"]] = fld}
+    hsecs    = secs_doc.each_with_object({}) {|sec, h| h[sec['id']] = sec}
+    hflds    = flds_doc.each_with_object({}) {|fld, h| h[fld['id']] = fld}
 
     pending_stage_ids.each do |id|
       next unless (other_stage = stage(id, ooze: page))
 
-      state = "inprogress" unless other_stage.state == "complete"
+      state = 'inprogress' unless other_stage.state == 'complete'
       other_stage.sections.doc.each do |sec_doc|
-        unless hsecs.key?(id = sec_doc["id"])
+        unless hsecs.key?(id = sec_doc['id'])
           hsecs[id] = sec_doc
           secs_doc << sec_doc
         end
       end
 
       other_stage.components.doc.each do |comp_doc|
-        unless hflds.key?(id = comp_doc["id"])
+        unless hflds.key?(id = comp_doc['id'])
           hflds[id] = comp_doc
           flds_doc << comp_doc
         end
@@ -93,21 +93,27 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   end
 
   def batched_search_results
-    raise "Missing block. It yields in slices of #{self.class.batch_size} results" unless block_given?
+    msg = "Missing block. It yields in slices of #{self.class.batch_size} results"
+    raise msg unless block_given?
+
     results_preview
     results = []
+
     apiv2.registers.search(register_id, search_options) do |page_result|
       results << page_result
+
       if results.length >= self.class.batch_size
         yield(results)
         results = []
       end
     end
+
     yield(results) unless results.empty?
   end
 
   def ooze(ooze_id = nil, stage_id: nil)
     return target unless ooze_id
+
     apiv2.pages.get(ooze_id, stage_id: stage_id).tap do |ooze|
       if ooze
         new_target(ooze)
@@ -143,21 +149,22 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   end
 
   def default_proceed
-    "Y"
+    'Y'
   end
 
   def results_preview
     apiv2.registers.search(register_id, search_options.merge(only_first: true)).tap do |search_results|
       str_results = "Total target entries: #{search_results.total} (out of #{search_results.total_before_filtering})"
       session.prompt_user(
-        "Do you want to proceed (y/N):",
+        'Do you want to proceed (y/N):',
         explanation: str_results,
         default:     default_proceed,
         timeout:     10
       ) do |res|
-        next if res.upcase.start_with?("Y")
-        puts "..."
-        log(:info) { "Aborting script..." }
+        next if res.upcase.start_with?('Y')
+
+        puts '...'
+        log(:info) { 'Aborting script...' }
         exit(0)
       end
     end
@@ -165,20 +172,21 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
 
   def search_options
     @search_options ||= {}.tap do |opts|
-      opts.merge!(sort: "created_at")
-      opts.merge!(dir:  "asc")
+      opts.merge!(sort: 'created_at')
+      opts.merge!(dir:  'asc')
       opts.merge!(query:   conf_search) if conf_search
       opts.merge!(filters: conf_filters)
     end
   end
 
   def conf_filters
-    return filters if respond_to?(:filters)
+    return filters if respond_to?(:filters, true)
+
     []
   end
 
   def conf_search
-    search if respond_to?(:search)
+    search if respond_to?(:search, true)
   end
 
   def register_id

data/lib/eco/api/usecases/ooze_samples/register_migration_case.rb

@@ -47,9 +47,9 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
       @csv = []
       super(session, options, usecase, &block)
     else
-      CSV.open(out_csv_filename, "w") do |csv|
+      CSV.open(out_csv_filename, 'w') do |csv|
         @csv = csv
-        csv << ["src_page_id", "dst_page_id"]
+        csv << %w[src_page_id dst_page_id]
         super(session, options, usecase, &block)
       end
       log(:info) { "File '#{out_csv_filename}' has been created." }
@@ -64,11 +64,11 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   #   # More custom logics
   # end
 
-  def process_ooze
+  def process_ooze # rubocop:disable Metrics/AbcSize
     with_target_stage(self.class::SOURCE_STAGE) do |source|
       draft_reference = object_reference(source)
       creating_new_page(draft_reference, template_id: elf.class::TEMPLATE_ID) do |draft|
-        draft.base_tags << ["IMPORTED", "MIGRATED"]
+        draft.base_tags << %w[IMPORTED MIGRATED]
 
         pairing_tracking = copy_pairings(source, draft)
 
@@ -78,17 +78,24 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
           yield(draft, source, fields_tracker: pairing_tracking)
         end
 
-        if msg = pairing_tracking.report_src_unpaired(only_present: !self.class::REPORT_SRC_EMTPY, only_indexed: !self.class::REPORT_SRC_DEINDEX)
-          log(:warn) { msg }
-        end
-
-        if msg = pairing_tracking.report_multi_pairs(only_present: !self.class::REPORT_SRC_EMTPY, only_indexed: !self.class::REPORT_SRC_DEINDEX, dst_exclude_ximport: !self.class::REPORT_DST_XIMPORT)
-          log(:warn) { msg }
-        end
-
-        if msg = pairing_tracking.report_dst_unpaired(only_required: false, exclude_ximport: !self.class::REPORT_DST_XIMPORT)
-          log(:warn) { msg }
-        end
+        msg = pairing_tracking.report_src_unpaired(
+          only_present: !self.class::REPORT_SRC_EMTPY,
+          only_indexed: !self.class::REPORT_SRC_DEINDEX
+        )
+        log(:warn) { msg } if msg
+
+        msg = pairing_tracking.report_multi_pairs(
+          only_present:        !self.class::REPORT_SRC_EMTPY,
+          only_indexed:        !self.class::REPORT_SRC_DEINDEX,
+          dst_exclude_ximport: !self.class::REPORT_DST_XIMPORT
+        )
+        log(:warn) { msg } if msg
+
+        msg = pairing_tracking.report_dst_unpaired(
+          only_required:   false,
+          exclude_ximport: !self.class::REPORT_DST_XIMPORT
+        )
+        log(:warn) { msg } if msg
       end
     end
   end
@@ -101,8 +108,18 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
       src_excluded: excluded_fields(src),
       dst_excluded: excluded_fields(dst)
     }
-    copy_generic_paired_fields(src, dst, **params.merge(exclude_ximport: self.class::EXCLUDE_XIMPORT), &after_copy).tap do |pairing_tracking|
-      copy_mapped_fields(src, dst, **params.merge(fields_tracker: pairing_tracking), &after_copy)
+    copy_generic_paired_fields(
+      src,
+      dst,
+      **params.merge(exclude_ximport: self.class::EXCLUDE_XIMPORT),
+      &after_copy
+    ).tap do |pairs_track|
+      copy_mapped_fields(
+        src,
+        dst,
+        **params.merge(fields_tracker: pairs_track),
+        &after_copy
+      )
     end
   end
 
@@ -110,7 +127,8 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
     return [] unless respond_to?(:excluded_field_hooks, true)
     excluded_field_hooks.each_with_object([]) do |(type, hooks), excluded|
       hooks.each do |hook|
-        type = (type == "all")? nil : type
+        type = nil if type == 'all'
+
         excluded.push(*with_fields(entry, label: to_regex(hook), type: type))
       end
     end
@@ -125,7 +143,8 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   def copy_mapped_fields(src, dst, **kargs, &after_copy)
     return [] unless field_maps?
     field_maps.each do |type, field_hooks|
-      type = (type == "all")? nil : type
+      type = nil if type == 'all'
+
       field_hooks.each do |hook_name|
         copy_hooked_fields(src, dst, hook: hook_name, type: type, **kargs, &after_copy)
       end
@@ -137,17 +156,15 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   end
 
   def callback(sym)
-    if callback?(sym)
-      method(callback_name(sym))
-    end
+    method(callback_name(sym)) if callback?(sym)
   end
 
   def callback_name(sym)
-    "#{sym}_callback".to_sym
+    :"#{sym}_callback"
   end
 
   def with_target_stage(name)
-    if stg = target_stage(name)
+    if (stg = target_stage(name))
       yield(stg)
     else
       log(:warn) { "Entry '#{target.id}' does not have stage '#{name}'" }

data/lib/eco/api/usecases/ooze_samples/register_update_case.rb

@@ -15,7 +15,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
     end
   end
 
-  name "register-update-case"
+  name 'register-update-case'
   type :other
 
   attr_reader :retrieved_oozes, :non_retrieved_oozes
@@ -33,7 +33,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
           process_ooze(&block)
         end
       elsif mode == :delegate
-        raise "You need to pass a block when running main in `:delegate` mode" unless block_given?
+        raise 'You need to pass a block when running main in `:delegate` mode' unless block_given?
         yield
       end
     end
@@ -42,7 +42,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
   end
 
   def process_ooze(_ooze = target)
-    raise "You need to define this method"
+    raise 'You need to define this method'
   end
 
   private
@@ -63,8 +63,8 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
 
   def new_target(object, warn_pending_changes: false)
     enqueue(object)
-    super(object, warn_pending_changes: warn_pending_changes) do |pending|
-      #enqueue(pending)
+    super do |pending|
+      # enqueue(pending)
     end
   end
 
@@ -79,7 +79,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
         end
 
         if (pending = queue_shift(page_result.id)) && dirty?(pending)
-          msg  = "Inconsistent search results. "
+          msg  = 'Inconsistent search results. '
           msg << "Launching update on '#{object_reference(pending)}' to be able to queue it back"
           log(:warn) { msg }
           update_ooze(pending)
@@ -186,7 +186,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
   end
 
   def backup_patch!(ooze = target)
-    return if (patch_doc(ooze) || {})["page"]
+    return if (patch_doc(ooze) || {})['page']
 
     log(:info) {
       "No changes to update for #{object_reference(ooze)}."
@@ -198,17 +198,17 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
       @total_search_oozes = search_results.total
 
       str_results = "Total target entries: #{search_results.total} (out of #{search_results.total_before_filtering})"
-      default_answer = session.config.run_mode_remote?? "Y" : "N"
+      default_answer = session.config.run_mode_remote?? 'Y' : 'N'
 
       session.prompt_user(
-        "Do you want to proceed (y/N):",
+        'Do you want to proceed (y/N):',
         explanation: str_results,
         default:     default_answer,
         timeout:     10
       ) do |res|
-        next if res.upcase.start_with?("Y")
-        puts "..."
-        log(:info) { "Aborting script..." }
+        next if res.upcase.start_with?('Y')
+        puts '...'
+        log(:info) { 'Aborting script...' }
         exit(0)
       end
     end
@@ -233,7 +233,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
 
   def kpis_message
     msg  = []
-    msg << "Run end:"
+    msg << 'Run end:'
     msg << "  * Search results: #{total_search_oozes}"
     msg << "  * Duplicated search results #{dupped_search_oozes}"
     msg << "  * Retrieved a total of #{retrieved_oozes}"
@@ -254,12 +254,13 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
   end
 
   def conf_filters
-    return filters if respond_to?(:filters)
+    return filters if respond_to?(:filters, true)
+
     []
   end
 
   def conf_search
-    search if respond_to?(:search)
+    search if respond_to?(:search, true)
   end
 
   def register_id

data/lib/eco/api/usecases/samples/drivers/cli/sftp_cli.rb

@@ -1,45 +1,45 @@
 class  Eco::API::UseCases::Samples::Drivers::Sftp
   class Cli < Eco::API::UseCases::Cli
-    desc "SFTP case: manages remote files that match a file pattern"
+    desc 'SFTP case: manages remote files that match a file pattern'
 
-    add_option("-file-pattern-const", "Constant that should be used as a pattern") do |options|
-      pattern_const = SCR.get_arg("-file-pattern-const", with_param: true)
+    add_option('-file-pattern-const', 'Constant that should be used as a pattern') do |options|
+      pattern_const = SCR.get_arg('-file-pattern-const', with_param: true)
       options.deep_merge!(sftp: {file_pattern_const: pattern_const})
     end
 
-    add_option("-list", "It lists all the files present in the remote folder for the current environment.") do |options|
+    add_option('-list', 'It lists all the files present in the remote folder for the current environment.') do |options|
       options.deep_merge!(sftp: {command: :list})
     end
 
-    add_option("-get-last", "It gets the last file present in the remote folder for the current environment.") do |options|
+    add_option('-get-last', 'It gets the last file present in the remote folder for the current environment.') do |options|
       options.deep_merge!(sftp: {command: :get_last})
     end
 
-    add_option("-get", "It gets all the files present in the remote folder for the current environment.") do |options|
+    add_option('-get', 'It gets all the files present in the remote folder for the current environment.') do |options|
       options.deep_merge!(sftp: {command: :get})
     end
 
-    add_option("-archive", "It moves all the files present in the remote folder to the archive subfolder.") do |options|
+    add_option('-archive', 'It moves all the files present in the remote folder to the archive subfolder.') do |options|
       options.deep_merge!(sftp: {command: :archive})
     end
 
-    add_option("-local-folder", "Local folder to pull the remote files to.") do |options|
-      local_folder = SCR.get_file("-local-folder", required: true, should_exist: true)
+    add_option('-local-folder', 'Local folder to pull the remote files to.') do |options|
+      local_folder = SCR.get_file('-local-folder', required: true, should_exist: true)
       options.deep_merge!(sftp: {local_folder: local_folder})
     end
 
-    add_option("-remote-subfolder", "Remote sub folder to pull the remote files from.") do |options|
-      remote_subfolder = SCR.get_arg("-remote-subfolder", with_param: true)
+    add_option('-remote-subfolder', 'Remote sub folder to pull the remote files from.') do |options|
+      remote_subfolder = SCR.get_arg('-remote-subfolder', with_param: true)
       options.deep_merge!(sftp: {remote_subfolder: remote_subfolder})
     end
 
-    add_option("-remote-target-folder", "Overrides -remote-subfolder and -remote-folder.") do |options|
-      remote_target_folder = SCR.get_arg("-remote-target-folder", with_param: true)
+    add_option('-remote-target-folder', 'Overrides -remote-subfolder and -remote-folder.') do |options|
+      remote_target_folder = SCR.get_arg('-remote-target-folder', with_param: true)
       options.deep_merge!(sftp: {remote_target_folder: remote_target_folder})
     end
 
-    add_option("-remote-folder", "Overrides -remote-folder (but not -remote-subfolder).") do |options|
-      remote_folder = SCR.get_arg("-remote-folder", with_param: true)
+    add_option('-remote-folder', 'Overrides -remote-folder (but not -remote-subfolder).') do |options|
+      remote_folder = SCR.get_arg('-remote-folder', with_param: true)
       options.deep_merge!(sftp: {remote_folder: remote_folder})
     end
   end

data/lib/eco/api/usecases/samples/drivers/cli/url_pull_cli.rb

@@ -1,14 +1,14 @@
 class Eco::API::UseCases::Samples::Drivers::UrlPull
   class Cli < Eco::API::UseCases::Cli
-    desc "Url Pull sample: pull the file from a specific URL"
+    desc 'Url Pull sample: pull the file from a specific URL'
 
-    add_option("-target-url-const", "Constant that should be used as a pattern") do |options|
-      url_const = SCR.get_arg("-target-url-const", with_param: true)
+    add_option('-target-url-const', 'Constant that should be used as a pattern') do |options|
+      url_const = SCR.get_arg('-target-url-const', with_param: true)
       options.deep_merge!(url_pull: {target_url_const: url_const})
     end
 
-    add_option("-local-folder", "Local folder to pull the remote files to.") do |options|
-      local_folder = SCR.get_file("-local-folder", required: true, should_exist: true)
+    add_option('-local-folder', 'Local folder to pull the remote files to.') do |options|
+      local_folder = SCR.get_file('-local-folder', required: true, should_exist: true)
       options.deep_merge!(url_pull: {local_folder: local_folder})
     end
   end

data/lib/eco/api/usecases/samples/drivers/sftp_sample.rb

@@ -5,8 +5,8 @@ class Eco::API::UseCases::Samples::Drivers::Sftp < Eco::API::Common::Loaders::Us
   name 'sftp'
   type :other
 
-  include Eco::API::UseCases::Lib::Sftp
-  include Eco::API::UseCases::Lib::FilePattern
+  include Eco::API::UseCases::Lib::Files::Sftp
+  include Eco::API::UseCases::Lib::Files::FilePattern
 
   def main(session, options, _usecase)
     options[:end_get] = false
@@ -37,7 +37,9 @@ class Eco::API::UseCases::Samples::Drivers::Sftp < Eco::API::Common::Loaders::Us
 
       file_names = files.map {|file| to_remote_path(file.name)}
 
-      puts "Getting the following files into the local folder '#{local_folder}':"
+      msg  = "Getting the following files into the local folder "
+      msg << "'#{local_folder || '.'}':"
+      puts msg
       puts file_names
 
       sftp.download(file_names, local_folder: local_folder)

data/lib/eco/api/usecases/use_case.rb

@@ -106,32 +106,32 @@ module Eco
 
             if dt
               @data = dt
-              singleton_class.attr_reader(:data) unless respond_to?(:data)
+              singleton_class.attr_reader(:data) unless respond_to?(:data, true)
 
               case mdl
               when :people
                 @people = dt
-                singleton_class.attr_reader(:people) unless respond_to?(:people)
+                singleton_class.attr_reader(:people) unless respond_to?(:people, true)
               when :contractors
                 @contractors = dt
-                singleton_class.attr_reader(:contractors) unless respond_to?(:contractors)
+                singleton_class.attr_reader(:contractors) unless respond_to?(:contractors, true)
               end
             end
 
             if ent # entries/input
               @input = ent
-              singleton_class.attr_reader(:input) unless respond_to?(:input)
+              singleton_class.attr_reader(:input) unless respond_to?(:input, true)
 
               # Below problematic code... usage has cases where entries = (gets assigned)
               # @entries = ent
-              # singleton_class.attr_reader(:entries) unless respond_to?(:entries)
+              # singleton_class.attr_reader(:entries) unless respond_to?(:entries, true)
             end
 
             # `self` is the use case itself (when used the Loader)
             next unless is_a?(Eco::API::Common::Loaders::CaseBase)
 
             @usecase = use_case_self
-            singleton_class.attr_reader(:usecase) unless respond_to?(:usecase)
+            singleton_class.attr_reader(:usecase) unless respond_to?(:usecase, true)
           end
 
           true

data/lib/eco/api/usecases/use_case_chain/chaining.rb

@@ -37,10 +37,10 @@ class Eco::API::UseCases::UseCaseChain
       msg << "Looks like a recursive cyclic chain 'use'"
       raise msg if @@num_chains >= MAX_CHAINS
 
-      msg = "A UseCase can only be chained with another UseCase"
+      msg = 'A UseCase can only be chained with another UseCase'
       raise msg if usecase && !usecase.is_a?(UseCase)
 
-      msg = "Missuse. Please use either parameter or block but not both"
+      msg = 'Missuse. Please use either parameter or block but not both'
       raise msg if block_given? && usecase
 
       usecase = block if block_given?
@@ -51,7 +51,7 @@ class Eco::API::UseCases::UseCaseChain
     protected
 
     def chain_to(usecase)
-      msg = "A UseCase can only be chained with another UseCase"
+      msg = 'A UseCase can only be chained with another UseCase'
       raise msg if usecase && !usecase.is_a?(UseCase)
 
       usecase.chain(self)
@@ -60,7 +60,7 @@ class Eco::API::UseCases::UseCaseChain
     def resolved_chains(use_group = nil)
       return @resolved_chains if @resolved_chains
 
-      msg  = "Only Eco::API::UseCases object can contain resolved chains. "
+      msg  = 'Only Eco::API::UseCases object can contain resolved chains. '
       msg << "Given: #{use_group.class}"
       raise msg if use_group && !use_group.is_a?(Eco::API::UseCases)
 
@@ -68,7 +68,7 @@ class Eco::API::UseCases::UseCaseChain
       @resolved_chains = @chains.map do |usecase|
         usecase = usecase.call(use_group) if usecase.respond_to?(:call)
 
-        msg = "A UseCase can only be chained with another UseCase"
+        msg = 'A UseCase can only be chained with another UseCase'
         raise msg if usecase && !usecase.is_a?(UseCase)
 
         usecase.resolved_chains(use_group) if usecase.respond_to?(:resolved_chains)
@@ -81,7 +81,7 @@ class Eco::API::UseCases::UseCaseChain
     private
 
     def reset_chains!
-      @chains = []
+      @chains          = []
       @resolved_chains = nil
     end
   end

data/lib/eco/api/usecases/use_case_chain.rb

@@ -8,7 +8,7 @@ module Eco
 
         def initialize(name = nil, root:, type: nil, model: nil, usecase: nil, &block)
           if usecase
-            msg  = "Expected Eco::API::UseCases::UseCase. "
+            msg  = 'Expected Eco::API::UseCases::UseCase. '
             msg << "Given #{usecase.class}"
             raise msg unless usecase.is_a?(Eco::API::UseCases::UseCase)
 
@@ -25,14 +25,14 @@ module Eco
         def root=(value)
           return super unless @resolved_chains # rubocop:disable Lint/ReturnInVoidContext
 
-          msg = "You cannot change root Eco::API::UseCases "
-          msg << "once the chains have been resolved"
+          msg = 'You cannot change root Eco::API::UseCases '
+          msg << 'once the chains have been resolved'
           raise msg
         end
 
         # @see Eco::API::UseCases::UseCase#launch
         def launch(io: nil, **kargs)
-          super(io: io, **kargs).tap do |uio|
+          super.tap do |uio|
             next if resolved_chains.empty?
 
             resolved_chains.each do |usecase|

data/lib/eco/api/usecases/use_case_io.rb

@@ -27,6 +27,7 @@ module Eco
         def new(usecase:, **kargs)
           default = { usecase: usecase }
           default.merge!(job: @job) if @job
+
           super(**default.merge(kargs))
         end
 
@@ -59,7 +60,7 @@ module Eco
         # @see Eco::API::UseCases::BaseIO#params
         # Same as its superclass but adding `usecase` and `job` parameters
         def params(keyed: false, all: false)
-          super(keyed: keyed, all: all).tap do |res|
+          super.tap do |res|
             if keyed
               res.merge!(usecase: usecase)
               res.merge!(job: @job) if @job

data/lib/eco/api/usecases.rb

@@ -4,17 +4,17 @@ module Eco
     class UseCases
       class UnknownCase < StandardError
         def initialize(msg = nil, case_name: nil, type: nil)
-          msg ||= "Unknown case"
-          msg += ". Case name '#{case_name}'" if case_name
-          msg += ". Case type '#{type}'" if type
+          msg ||= 'Unknown case'
+          msg  << ". Case name '#{case_name}'" if case_name
+          msg  << ". Case type '#{type}'" if type
           super(msg)
         end
       end
 
       class AmbiguousCaseReference < StandardError
         def initialize(msg = nil, case_name: nil)
-          msg ||= "You must specify type when there are multiple cases with same name"
-          msg += ". Case name '#{case_name}'" if case_name
+          msg ||= 'You must specify type when there are multiple cases with same name'
+          msg  << ". Case name '#{case_name}'" if case_name
           super(msg)
         end
       end
@@ -22,8 +22,8 @@ module Eco
       extend  Eco::API::Common::ClassAutoLoader
       include Enumerable
 
-      autoloads_children_of "Eco::API::Common::Loaders::UseCase"
-      autoload_namespace_ignore "Eco::API"
+      autoloads_children_of 'Eco::API::Common::Loaders::UseCase'
+      autoload_namespace_ignore 'Eco::API'
 
       def initialize
         @usecases      = {}
@@ -160,11 +160,11 @@ module Eco
       end
 
       def name(key)
-        key.to_s.split(":").first
+        key.to_s.split(':').first
       end
 
       def type
-        key.to_s.split(":").last&.to_sym
+        key.to_s.split(':').last&.to_sym
       end
     end
   end