eco-helpers 2.4.8 → 2.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: facf3e640f8af36beba49a9da6eb7c72cb70b4a07ef93602ee70898cce31c356
-  data.tar.gz: 304bb16fdefd0db6ad63b524a654aed990375a24630565d59cfb6fa4470e94da
+  metadata.gz: b15015d62d69b57bfaef2e33cb9800786806f1796621f8a7c3fdc570bec81c99
+  data.tar.gz: 046b93c466ef6999e97c2d17584d4e760c3d6592783448c4fed3aed2edd3a0d2
 SHA512:
-  metadata.gz: 6d4b1882f5c042ceeeee46bfb6e1604204092a108409c882df2f82e125259cd11551ce90e48c40cf5c7cdf50286eda42b624c9cd24d9e1ea13d4a8bb3203d9b2
-  data.tar.gz: 91061e3b0d8caafc59fbd77ca726cf84c83d459eb1bad420932ad3482842d57acd1a38a35373a82078638b6c7e41e570a6c7f2adb84a319a38c61509555d2bc9
+  metadata.gz: '079b89d982758de246c8f81fcc5e8853fdcc75c217f1b7e417408b9234a8a942aa874e77705fe62c0863699bf72dcf666297894b6febdb3f47bcb740af2d438e'
+  data.tar.gz: 7feaa9e6022432a50c3fab188283e6af6566a8fe9de7b84fd3dcfba4584c64fd02cc5d926f3f50db8a44f139138844438345222d941b7407a6f4369b257ca070

data/CHANGELOG.md

@@ -1,12 +1,55 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.4.9] - 2023-04-xx
+## [2.5.2] - 2023-06-xx
 
 ### Added
 ### Changed
 ### Fixed
 
+## [2.5.1] - 2023-06-06
+
+### Added
+  - `Eco::API::Common::Session:SFTP#host` method
+  - New options to control csv headers check (`Eco::API::Common::People::DefaultParsers::CSVParser`)
+  - `Eco::API::Common::Loaders::CaseBase` add to the setter the `@usecase`
+    - Implemented via `Eco::API::UseCases::UseCase`
+
+### Changed
+  - Dry out dependency loading.
+    - `ecoportal-api` **gem** remains as eager loading.
+  - Patch `Ecoportal::API::V1::Person` with `include`
+  - Upgraded `ecoportal-api-graphql` **gem**
+  - `Eco::API::Common::People::PersonEntryAttributeMapper`
+    - Switched reversed attr defs warning to `info` logging level.
+  - `Eco::API::Common::DefaultParsers::SelectParser`
+    - When there is dependency with the hash of options, it should return nil when the option is unknown.
+
+### Fixed
+  - Tidied up file encoding when getting content.
+    - This fixes `Eco::CSV::read`
+  - `Eco::API::Common::People::DefaultParsers::CSVParser` made some refactor.
+
+## [2.4.9] - 2023-04-17
+
+### Added
+  - `Eco::API::UseCases#source_object` to retrieve the original use case definition object
+    - This allows the `Eco::API::UseCases::UseCase` object to access the object that defined the usecase and initialize `@session`.
+    - It now can know what `type` of use case created the `@callback`
+    - Next steps with this may be play a bit with the `aritity` of the `@callback` block, allow parsers to skip the serializer definition (so the default one runs), etc.
+  - `Eco::API::Common::Loaders::Base` the above feature allowed to initialize the `@options`, and therefore...
+    - `#options` method **added** (based on `@options` instance var)
+
+### Changed
+  - `Eco::API::Common::Loaders::Base`
+    - `#session` method is based on `@session` instance var, which allows to override without having to redefine `session` (`attr_reader :session` is no longer necessary)
+  - Softened max version of **gem** dependencies
+  - Upgraded self-managed **gem** dependencies
+
+### Fixed
+  - `Eco::API::Common::ClassAutoLoader::autoload_children`
+    - Skip `singleton_classes`
+
 ## [2.4.8] - 2023-04-03
 
 ### Added

data/eco-helpers.gemspec

@@ -24,24 +24,24 @@ Gem::Specification.new do |spec|
   #spec.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
-  spec.add_development_dependency "bundler",   ">= 2.4.8",  "< 2.5"
-  spec.add_development_dependency "rspec",     ">= 3.10.0", "< 3.11"
-  spec.add_development_dependency "rake",      ">= 13.0.3", "< 13.1"
-  spec.add_development_dependency "yard",      ">= 0.9.26", "< 0.10"
-  spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
+  spec.add_development_dependency "bundler",   ">= 2.4.8",  "< 3"
+  spec.add_development_dependency "rspec",     ">= 3.10.0", "< 4"
+  spec.add_development_dependency "rake",      ">= 13.0.3", "< 14"
+  spec.add_development_dependency "yard",      ">= 0.9.26", "< 1"
+  spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 4"
 
-  spec.add_dependency 'ecoportal-api', '>= 0.9.3', '< 0.10'
-  spec.add_dependency 'ecoportal-api-v2', '>= 1.1.1', '< 1.2'
-  spec.add_dependency 'ecoportal-api-graphql', '>= 0.3.5', '< 0.4'
+  spec.add_dependency 'ecoportal-api', '>= 0.9.4', '< 0.10'
+  spec.add_dependency 'ecoportal-api-v2', '>= 1.1.2', '< 1.2'
+  spec.add_dependency 'ecoportal-api-graphql', '>= 0.3.7', '< 0.4'
   spec.add_dependency 'aws-sdk-s3',    '>= 1.83.0', '< 2'
   spec.add_dependency 'aws-sdk-ses',   '>= 1.36.0', '< 2'
-  spec.add_dependency 'dotenv',        '>= 2.7.6', '< 2.8'
-  spec.add_dependency 'net-sftp',      '>= 3.0.0', '< 3.1'
-  spec.add_dependency 'hashdiff',      '>= 1.0.1', '< 1.1'
-  spec.add_dependency 'fuzzy_match',   '>= 2.1.0', '< 2.2'
-  spec.add_dependency 'amatch',        '>= 0.4.0', '< 0.5'
-  spec.add_dependency 'jaro_winkler',  '>= 1.5.4', '< 1.6'
-  spec.add_dependency 'roo',           '>= 2.8.3', '< 2.9'
-  spec.add_dependency 'roo-xls',       '>= 1.2.0', '< 1.3'
-  spec.add_dependency 'docx',          '>= 0.6.2', '< 0.7'
+  spec.add_dependency 'dotenv',        '>= 2.7.6',  '< 3'
+  spec.add_dependency 'net-sftp',      '>= 3.0.0',  '< 4'
+  spec.add_dependency 'hashdiff',      '>= 1.0.1',  '< 1.1'
+  spec.add_dependency 'fuzzy_match',   '>= 2.1.0',  '< 2.2'
+  spec.add_dependency 'amatch',        '>= 0.4.0',  '< 0.5'
+  spec.add_dependency 'jaro_winkler',  '>= 1.5.4',  '< 1.6'
+  spec.add_dependency 'roo',           '>= 2.8.3',  '< 2.9'
+  spec.add_dependency 'roo-xls',       '>= 1.2.0',  '< 1.3'
+  spec.add_dependency 'docx',          '>= 0.6.2',  '< 0.7'
 end

data/lib/eco/api/common/class_auto_loader.rb

@@ -80,8 +80,14 @@ module Eco
           return false if pending_children.empty?
           @loading_children = true
           pending_children.each do |klass|
-            @child = klass.new(object)
-            autoloaded_children.push(klass)
+            begin
+              @child = klass.new(object)
+            rescue TypeError => e
+              # Can't create from this class (must be the singleton class)
+              # Just ignore
+            ensure
+              autoloaded_children.push(klass)
+            end
           end
           @loading_children = false
           true
@@ -102,7 +108,6 @@ module Eco
         def new_classes
           ObjectSpace.each_object(::Class).to_a - known_classes
         end
-
       end
     end
   end

data/lib/eco/api/common/loaders/base.rb

@@ -36,7 +36,11 @@ module Eco
           private
 
           def session
-            ASSETS.session
+            @session ||= ASSETS.session
+          end
+
+          def options
+            @options ||= {}
           end
 
           def config

data/lib/eco/api/common/loaders/case_base.rb

@@ -3,7 +3,6 @@ module Eco
     module Common
       module Loaders
         class CaseBase < Loaders::Base
-
           class << self
             attr_writer :name, :type
 
@@ -19,13 +18,13 @@ module Eco
             def name_only_once!
               raise "You have already declared #{self} or you are trying to give it a name twice" if @name
             end
-
           end
 
+          attr_reader :usecase
+
           def name
             self.class.name
           end
-
         end
       end
     end

data/lib/eco/api/common/people/default_parsers/csv_parser.rb

@@ -3,7 +3,8 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
 
   def parser(data, deps)
     Eco::CSV.parse(data, headers: true, skip_blanks: true).tap do |table|
-      check_headers(table) if deps[:check_headers] && !options.dig(:input, :skip_header_check)
+      require_headers!(table)
+      check_headers(table) if deps[:check_headers] && check_headers?
     end.each_with_object([]) do |row, arr_hash|
       row_hash = row.headers.uniq.each_with_object({}) do |attr, hash|
         next if attr.to_s.strip.empty?
@@ -26,6 +27,26 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
 
   private
 
+  def abort(message)
+    logger.error(message)
+    exit(1)
+  end
+
+  def require_headers!(table)
+    headers = table.headers
+    abort("Missing headers in CSV") unless headers && !headers.empty?
+    empty = []
+    with_value = headers.each_with_index do |header, idx|
+      empty << idx if header.to_s.strip.empty?
+    end
+    abort("Empty headers in column(s): #{empty.join(', ')}") unless empty.empty?
+    true
+  end
+
+  def check_headers?
+    !options.dig(:input, :header_check, :skip)
+  end
+
   def options
     ASSETS.cli.options
   end
@@ -43,29 +64,54 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
   end
 
   def check_headers(table)
-    headers = table.headers
-    missing = missing_headers(headers)
-    unknown = unknown_headers(headers)
-    unless missing.empty? && unknown.empty?
-      msg  = "Detected possible HEADER ISSUES !!!\n"
-      msg << "There might be Missing or Wrong HEADER names in the CSV file:\n"
-      msg << "  * UNKNOWN (or not used?): #{unknown}\n" unless unknown.empty?
-      msg << "  * MISSING DIRECT: #{missing[:direct]}\n" unless (missing[:direct] || []).empty?
-      unless (data =  missing[:indirect] || []).empty?
-        msg << "  * MISSING INDIRECT:\n"
-        data.each do |ext, info|
-          msg <<  "    - '#{ext}' => "
-          msg << (info[:attrs] || {}).map do |status, attrs|
-            if status == :inactive
-              "makes inactive: #{attrs}"
-            elsif status == :active
-              "there could be missing info in: #{attrs}"
-            end
-          end.compact.join("; ") + "\n"
-        end
+    headers  = table.headers
+    unmatch  = []
+    unmatch  = unmatched_headers(headers) if options.dig(:input, :header_check, :order)
+    missing  = missing_headers(headers)
+    unknown  = unknown_headers(headers)
+    criteria = [unknown, missing[:direct], missing[:indirect], unmatch]
+    return if criteria.all?(&:empty?)
+
+    msg  = "Detected possible HEADER ISSUES !!!\n"
+
+    # requires exact match
+    unless unmatch.empty?
+      msg << "CSV headers do NOT exactly match the expected:\n"
+      msg << "  * Expected: #{known_headers}\n"
+      expected, given = unmatch.first
+      msg << "  * First unmatch => Given: '#{given}' where expected '#{expected}'\n"
+      missed = known_headers - headers
+      unless missed.empty?
+        msg << "  * Missing headers:\n"
+        msg << "    - #{missed.join("\n    - ")}\n"
       end
-      logger.warn(msg)
-      sleep(2)
+    end
+    msg << "Missing or Wrong HEADER names in the CSV file:\n"
+    msg << "  * UNKNOWN (or not used?): #{unknown}\n" unless unknown.empty?
+    msg << "  * MISSING HEADER: #{missing[:direct]}\n" unless missing[:direct].empty?
+    unless (data = missing[:indirect]).empty?
+      msg << "  * MISSING INDIRECTLY:\n"
+      data.each do |ext, info|
+        msg <<  "    - '#{ext}' => "
+        msg << (info[:attrs] || {}).map do |status, attrs|
+          if status == :inactive
+            "makes inactive: #{attrs}"
+          elsif status == :active
+            "there could be missing info in: #{attrs}"
+          end
+        end.compact.join("; ") + "\n"
+      end
+    end
+    logger.warn(msg)
+    if options.dig(:input, :header_check, :must_be_valid)
+      abort("There were issues identified on the CSV header names. Aborting...")
+    end
+    sleep(2)
+  end
+
+  def unmatched_headers(headers)
+    known_headers.zip(headers).reject do |(expected, given)|
+      expected == given
     end
   end
 
@@ -82,27 +128,28 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
     ext_present = known_headers_present(int_head) | ext
     ext_miss    = known_headers - ext_present
 
-    ext_miss.each_with_object({}) do |ext, missing|
-      next unless int = fields_mapper.to_internal(ext)
-      if all_internal_attrs.include?(int)
-        missing[:direct] ||= []
-        missing[:direct] << ext
-      end
-      related_attrs_requirements = required_attrs.values.select do |req|
-        dep       = req.dependant?(int)
-        affects   = dep && !int_head.include?(int)
-        in_header = int_head.include?(req.attr)
-        affects || (dep && !in_header)
-      end
-      next if related_attrs_requirements.empty?
-      missing[:indirect] ||= {}
-      data = missing[:indirect][ext] = {}
-      data[:int]   = int
-      data[:attrs] = {}
-      related_attrs_requirements.each_with_object(data[:attrs]) do |req, attrs|
-        status = req.active?(*int_head) ? :active : :inactive
-        attrs[status] ||= []
-        attrs[status] << req.attr
+    {
+      direct:   [],
+      indirect: {}
+    }.tap do |missing|
+      ext_miss.each do |ext|
+        next unless int = fields_mapper.to_internal(ext)
+        missing[:direct] << ext if all_internal_attrs.include?(int)
+        related_attrs_requirements = required_attrs.values.select do |req|
+          dep       = req.dependant?(int)
+          affects   = dep && !int_head.include?(int)
+          in_header = int_head.include?(req.attr)
+          affects || (dep && !in_header)
+        end
+        next if related_attrs_requirements.empty?
+        data = missing[:indirect][ext] = {}
+        data[:int]   = int
+        data[:attrs] = {}
+        related_attrs_requirements.each_with_object(data[:attrs]) do |req, attrs|
+          status = req.active?(*int_head) ? :active : :inactive
+          attrs[status] ||= []
+          attrs[status] << req.attr
+        end
       end
     end
   end
@@ -114,8 +161,11 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
     end
   end
 
+  # Scopes what internal attrs appear in headers as they are
   def internal_present_or_active(headers, inactive_requirements = {})
-    hint = headers & all_internal_attrs
+    # internal attrs that are not being mapped
+    int_all = all_internal_attrs.reject {|i| fields_mapper.external?(i)}
+    hint = headers & int_all
     hext = headers - hint
     int_present = hint + hext.map {|e| fields_mapper.to_internal(e)}.compact
 
@@ -142,8 +192,9 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
     int_present
   end
 
+  # The csv header names as expected
   def known_headers
-    @known_headers ||= fields_mapper.list(:external).compact
+    @known_headers ||= fields_mapper.list(:external).compact.uniq
   end
 
   def fields_mapper
@@ -167,5 +218,4 @@ class Eco::API::Common::People::DefaultParsers::CSVParser < Eco::API::Common::Lo
   def person_parser
     session.entry_factory.person_parser
   end
-
 end

data/lib/eco/api/common/people/default_parsers/select_parser.rb

@@ -11,9 +11,9 @@ module Eco
                   options_hash = deps[:select_hashes][deps["attr"]]
                   next value if !options_hash
                   if value.is_a?(Array)
-                    value.map { |v| options_hash[v&.downcase.strip] || v }
+                    value.map { |v| options_hash[v&.downcase.strip]  }.compact
                   elsif value
-                    options_hash[value&.downcase.strip] || value
+                    options_hash[value&.downcase.strip]
                   end
                 end.def_serializer do |value|
                   value

data/lib/eco/api/common/people/default_parsers/xls_parser.rb

@@ -64,5 +64,4 @@ class Eco::API::Common::People::DefaultParsers::XLSParser < Eco::API::Common::Lo
     require 'roo-xls'
     already_required = true
   end
-
 end

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

@@ -86,10 +86,10 @@ module Eco
           # @option options [Boolean] :check_headers signals if the `csv` file headers should be expected.
           # @return [Eco::API::Common::People::Entries] collection of `Eco::API::Common::People::PersonEntry`.
           def entries(data: (no_data = true; nil), file: (no_file = true; nil), format: (no_format = true; nil), **options)
-            fatal("You should at least use data: or file:, but not both") if no_data == no_file
+            fatal("You should at least use data: or file:, but not both")         if no_data == no_file
             fatal("You must specify a valid format: (symbol) when you use file.") if file && no_format
-            fatal("Format should be a Symbol. Given '#{format}'") if format && !format.is_a?(Symbol)
-            fatal("There is no parser/serializer for format ':#{format.to_s}'") unless no_format || @person_parser.defined?(format)
+            fatal("Format should be a Symbol. Given '#{format}'")                 if format && !format.is_a?(Symbol)
+            fatal("There is no parser/serializer for format ':#{format.to_s}'")   unless no_format || @person_parser.defined?(format)
 
             options.merge!(content:  data)     unless no_data
             options.merge!(file:     file)     unless no_file
@@ -113,9 +113,8 @@ module Eco
             # Get content only when it's not :xls
             # note: even if content was provided, file takes precedence
             if (format != :xls) && file
-              content = get_file_content(file, encoding)
+              content = get_file_content(file, encoding: encoding)
             end
-            #content = get_file_content(file, format, encoding) if (format != :xls) && file
 
             case content
             when Hash
@@ -129,15 +128,16 @@ module Eco
               case sample
               when Hash, Array, ::CSV::Row
                 Eco::CSV::Table.new(content).to_array_of_hashes
+              when NilClass
+                abort("There is NO input data")
               else
-                logger.error("Input content 'Array' of '#{sample.class}' is not supported.")
+                abort("Input content 'Array' of '#{sample.class}' is not supported.")
               end
             else
               if file && format == :xls
                 person_parser.parse(format, file)
               else
-                logger.error("Could not obtain any data out of these: #{kargs}. Given content: '#{content.class}'")
-                exit(1)
+                abort("Could not obtain any data out of these: #{kargs}. Given content: '#{content.class}'")
               end
             end.tap do |out_array|
               start_from_two = (format == :csv) || format == :xls
@@ -189,6 +189,11 @@ module Eco
 
           private
 
+          def abort(message)
+            logger.error(message)
+            exit(1)
+          end
+
           def fatal(msg)
             logger.fatal(msg)
             raise msg

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

@@ -11,7 +11,7 @@ module Eco
           # @return [RequiredAttrs]
           def required_attrs
             @required_attrs ||= @dependencies[:required_attrs]
-            #@required_attrs ||= RequiredAttrs.new(attr, :unkown, [attr])
+            #@required_attrs ||= RequiredAttrs.new(attr, :unknown, [attr])
           end
 
           # @see Eco::Language::Models::ParserSerializer#def_parser

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

@@ -133,7 +133,7 @@ module Eco
                 attr = @attr_map.to_internal(value.strip)
               when @attr_map.internal?(value) || @attr_map.internal?(value.strip) || @attr_map.internal?(value.strip.downcase)
                 unless cached_warning("external", "reversed", value)
-                  logger.warn("The mapper [external, internal] attribute names may be declared reversedly for EXTERNAL attribute: '#{value}'")
+                  logger.info("The mapper [external, internal] attribute names may be declared reversedly for EXTERNAL attribute: '#{value}'")
                 end
               end
             end
@@ -172,7 +172,7 @@ module Eco
                 attr = @attr_map.to_external(value.strip)
               when @attr_map.external?(value) || @attr_map.external?(value.strip) || @attr_map.external?(value.strip.downcase)
                 unless cached_warning("internal", "reversed", value)
-                  logger.warn("The mapper [external, internal] attribute names may be declared reversedly for INTERNAL attribute: '#{value}'")
+                  logger.info("The mapper [external, internal] attribute names may be declared reversedly for INTERNAL attribute: '#{value}'")
                 end
               end
             end

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

@@ -174,7 +174,7 @@ module Eco
 
           # Helper to define and associate a parser/serializer to a type or attribute.
           # @raise [Exception] if trying to define a parser/serializer for:
-          #   - an unkown attribute (`String`)
+          #   - an unknown attribute (`String`)
           #   - an unrecognized type or format (`Symbol`)
           # @param attr [String] type (`Symbol`) or attribute (`String`) to define the parser/serializer to.
           # @param dependencies [Hash] dependencies to be used when calling the parser/serializer.

data/lib/eco/api/common/session/base_session.rb

@@ -12,7 +12,7 @@ module Eco
           alias_method :fm, :file_manager
 
           include Session::Helpers
-          
+
           def initialize(e)
             raise "Expected object Eco::API::Common::Session::Environment. Given: #{e.class}" unless e.is_a?(Environment)
             self.environment = e
@@ -72,7 +72,6 @@ module Eco
             logger.fatal(msg)
             raise msg
           end
-
         end
       end
     end

data/lib/eco/api/common/session/environment.rb

@@ -24,9 +24,8 @@ module Eco
           end
 
           def mailer
-            if mailer?
-              @mailer      ||= Eco::API::Common::Session::Mailer.new(enviro: self)
-            end
+            return nil unless mailer?
+            @mailer ||= Eco::API::Common::Session::Mailer.new(enviro: self)
           end
 
           def mailer?
@@ -34,9 +33,8 @@ module Eco
           end
 
           def sftp
-            if sftp?
-              @sftp        ||= Eco::API::Common::Session::SFTP.new(enviro: self)
-            end
+            return nil unless sftp?
+            @sftp ||= Eco::API::Common::Session::SFTP.new(enviro: self)
           end
 
           def sftp?
@@ -44,9 +42,8 @@ module Eco
           end
 
           def s3uploader
-            if s3uploader?
-              @s3uploader  ||= Eco::API::Common::Session::S3Uploader.new(enviro: self)
-            end
+            return nil unless s3uploader?
+            @s3uploader  ||= Eco::API::Common::Session::S3Uploader.new(enviro: self)
           end
 
           def s3uploader?
@@ -56,7 +53,6 @@ module Eco
           def api(version: nil)
             config.api(logger, version: version)
           end
-
         end
       end
     end

data/lib/eco/api/common/session/helpers/prompt_user.rb

@@ -1,31 +1,31 @@
-require 'timeout'
 module Eco
   module API
     module Common
       module Session
         module Helpers
           module PromptUser
-
+            # Prompts user for input with option for default on timeout.
             def prompt_user(question, default:, explanation: "", timeout: nil)
-              response = if config.run_mode_remote?
-                          default
-                        else
-                          puts explanation
-                          print "#{question} "
-                          if timeout
-                            begin
-                              Timeout::timeout(timeout) { STDIN.gets.chop }
-                            rescue Timeout::Error
-                              default
-                            end
-                          else
-                            STDIN.gets.chop
-                          end
-                        end
+              require 'timeout'
+              response = \
+                if config.run_mode_remote?
+                  default
+                else
+                  puts explanation
+                  print "#{question} "
+                  if timeout
+                    begin
+                      Timeout::timeout(timeout) { STDIN.gets.chop }
+                    rescue Timeout::Error
+                      default
+                    end
+                  else
+                    STDIN.gets.chop
+                  end
+                end
               return response unless block_given?
               yield(response)
             end
-
           end
         end
       end

data/lib/eco/api/common/session/logger.rb

@@ -57,8 +57,8 @@ module Eco
             return log(:fatal, *args, &block)
           end
 
-          def unkown(msg, &block)
-            return log(:unkown, *args, &block)
+          def unknown(msg, &block)
+            return log(:unknown, *args, &block)
           end
 
           def << (msg)

data/lib/eco/api/common/session/mailer.rb

@@ -1,5 +1,3 @@
-require 'aws-sdk-ses'
-
 module Eco
   module API
     module Common
@@ -42,6 +40,7 @@ module Eco
           private
 
           def ses
+            require 'aws-sdk-ses'
             begin
               @ses ||= Aws::SES::Client.new(
                 access_key_id:     fetch_access_key_id,
@@ -98,7 +97,6 @@ module Eco
           def fetch_message_id_domain
             config.mailer.message_id_domain
           end
-
         end
       end
     end

data/lib/eco/api/common/session/s3_uploader.rb

@@ -1,5 +1,3 @@
-require 'aws-sdk-s3'
-
 module Eco
   module API
     module Common
@@ -75,6 +73,7 @@ module Eco
           end
 
           def bucket
+            require 'aws-sdk-s3'
             begin
               @bucket ||= Aws::S3::Resource.new(
                 access_key_id:     fetch_access_key_id,
@@ -128,7 +127,6 @@ module Eco
           def fetch_region
             config.s3storage.region || ENV['AWS_REGION']
           end
-
         end
       end
     end