eco-helpers 2.0.8 → 2.0.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 507156a88f88bc8227f255251d1f8861f486a23a0fe3b23e003706dfe708507b
-  data.tar.gz: '07952da6db5134dfecc5461d41ad4ce0e264c0eaaec3d9ec55d9d42b41b2620b'
+  metadata.gz: 726f5a5930889b9fd59b4acc4ce7b938e3c6011909c2f5a5bb1a105a36df4d9b
+  data.tar.gz: 804bc3ae796491543ec3615e9f11308333e7ea6fc9a10f943778f66922aa4a25
 SHA512:
-  metadata.gz: fcfa79dfa92123af03399391797013e7a23a032504570d8cc2cfcb890f8a93ece62a7acbbd43b2fbd70a4de20c28baa60635536d9b431ac260de2905db825332
-  data.tar.gz: 0e63f504d100e5647e9b48eceeebda57814d1dda0ce3d8c0147cb58ce7c40bf5e1225bc261d207d531af22611dc4fd0461461f70016647cda0eaf1147e653c12
+  metadata.gz: f2e1c852b4660ae1bdc3e94bab274f3d4b7168c6affef57315b370af102b71cff19cdb0c2de0b59899f82b7f2f8b0478688cf018f3c52f26c1f1d891ed19cd65
+  data.tar.gz: 032a93267eb45f7550fbdad7805d0f472a0ea1a98353b089147bd71de785cdeaa4933b233bdd22d4ab69f7a85bfa65ec1f0850fe82e446dcf4d5f503ba1f191b

data/CHANGELOG.md

@@ -1,24 +1,61 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.0.8] - 2021-03-xx
+## [2.0.13] - 2021-03-31
 
 ### Added
+  - Stats on the `Eco::API::Session::Batch`
+  - Allow to inherit and re-use the `Eco::API::UseCases::DefaultCase::HrisCase`
+
 ### Changed
+
 ### Fixed
-  - `Ecoportal::API::External::Person#identity` adjust behaviour
-  
-## [2.0.7] - 2021-03-09
+  - Ensure auto-loading for `Eco::API::UseCases`
+
+## [2.0.12] - 2021-03-29
+
+### Changed
+  - `Eco::CLI#run` it does include any defined states on the workflow
+
+
+## [2.0.11] - 2021-03-29
+
+### Fixed
+  - `Eco::API::Session::Batch:Errors#errors` changed `Array#filter` call to `Array#select` call
+
+## [2.0.10] - 2021-03-26
 
 ### Added
+  - `Eco::API::Common::People::PersonParser`
+    - `#defined_attrs`: list of all attribute defined parsers (regardless they belong to the model)
+  - `Eco::API::Organization::TagTree` methods
+    - `#total_depth` to know the total depth of a tree
+    - `#flat?` to know if the tagtree has only 1 level
+
 ### Changed
+  - `Eco::API::Common::People::PersonParser`
+    - Renamed `#all_attrs` to `#all_model_attrs`
+    - Ranamed `#defined_attrs` to `#defined_model_attrs`
+    - Ranamed `#undefined_attrs` to `#undefined_model_attrs`
+
+## [2.0.9] - 2021-03-19
+
+### Changed
+  - `Eco::API::Session::Batch:Errors#person_ref` moved to the public method
+
+## [2.0.8] - 2021-03-09
+
+### Fixed
+  - `Ecoportal::API::V1::Person#identity` adjust behavior
+
+## [2.0.7] - 2021-03-09
+
 ### Fixed
   - `Eco::API::Session::Batch::JobsGroups` and `Eco::API::Session::Batch::Jobs`: when new `Batch::Job`s are creating during launch, they remained unlaunched
     - this fix makes `#launch` method to iterate until there are no pending
 
 ## [2.0.6] - 2021-03-08
 
-### Added
 ### Changed
   - `Eco::API::Session::Batch:Errors` moved some methods to be private
 ### Fixed
@@ -26,46 +63,33 @@ All notable changes to this project will be documented in this file.
 
 ## [2.0.5] - 2021-02-24
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Common::People::PersonParser#symbol_keys` to use `Array#select`
     - as `Array#filter` was introduced in ruby `2.5.5`
 
 ## [2.0.4] - 2021-02-23
 
-### Added
 ### Changed
   - `Eco::API::Common::People::PersonParser#defined_attrs` to include virtual attributes
 
-### Fixed
-
 ## [2.0.3] - 2021-02-23
 
 ### Added
   - `Eco::API::UseCases::OozeSamples::OozeUpdateCase`: added integration to update page instances (non-templates)
 
-### Changed
 ### Fixed
   - `Eco::API::Policies::DefaultPolicies::UserAccess` complete default policy code.
 
 ## [2.0.2] - 2021-02-22
 
-### Added
 ### Changed
   - upgraded dependencies with `ecoportal-api`
 
-### Fixed
-
-
 ## [2.0.1] - 2021-02-22
 
-### Added
 ### Changed
   - upgraded dependencies with `ecoportal-api-oozes` to `ecoportal-api-v2`
 
-### Fixed
-
 ## [1.5.15] - 2021-02-17
 
 ### Added
@@ -79,7 +103,6 @@ All notable changes to this project will be documented in this file.
     - `Eco::API::MicroCases#people_search`
     - `Eco::API::MicroCases#refresh`
 
-### Changed
 ### Fixed
   - `Eco::API::Policies::DefaultPolicies::UserAccess` typos in default api policy
 
@@ -101,11 +124,8 @@ All notable changes to this project will be documented in this file.
   - `Eco::API::Common::People::PersonEntry` add error log when wrong email error is detected
     - previously it would have crashed
 
-### Fixed
-
 ## [1.5.13] - 2021-02-01
 
-### Added
 ### Changed
   - upgraded dependency with `ecoportal-api-oozes`
 
@@ -121,23 +141,16 @@ All notable changes to this project will be documented in this file.
 ### Changed
   - upgraded `ecoportal-api` dependency
 
-### Fixed
-
-
 ## [1.5.11] - 2021-01-25
 
 ### Added
   - `Eco::API::Organization::TagTree#subtag?` to check if the tag is in any subtree.
 
-### Changed
-
 ### Fixed
   - `Eco::API::MicroCases#set_supervisor` shouldn't set it if the entry does not have it.
 
 ## [1.5.10] - 2021-01-19
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Session::Batch::Errors#print` show the row number of the input data.
 
@@ -146,14 +159,8 @@ All notable changes to this project will be documented in this file.
 ### Added
   - `Eco::API::Organization::TagTree#subtags` to get all the tags but those of the highest level.
 
-### Changed
-### Fixed
-
-
 ## [1.5.8] - 2021-01-05
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Session::Batch::Jobs#job` shouldn't be calling the post-launch callback function on creation.
   - `Eco::API::Session#new_job` should include a `&block` parameter.
@@ -161,15 +168,11 @@ All notable changes to this project will be documented in this file.
 
 ## [1.5.7] - 2020-12-17
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Sesssion#parse_attribute` was not using phase argument
 
 ## [1.5.6] - 2020-12-04
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::UseCases::DefaultCases::RestoreDBCase` fixed typo and slightly improved
   - fixed some back-end errors when chaining usecases
@@ -177,8 +180,6 @@ All notable changes to this project will be documented in this file.
 
 ## [1.5.5] - 2020-12-03
 
-### Added
-### Changed
 ### Fixed
   - rubies previous to `2.5` do not have `yield_self`
 
@@ -186,14 +187,11 @@ All notable changes to this project will be documented in this file.
 
 ### Added
   - update `ecoportal-api` dependency
-### Changed
 ### Fixed
   - `Eco::API::MicroCases#people_refresh` typo
 
 ## [1.5.3] - 2020-11-30
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Session::Batch::RequestStats#blanked_value?` better blank detection
   - `Eco::API::MicroCases#with_each_starter` rectified typo
@@ -334,14 +332,11 @@ All notable changes to this project will be documented in this file.
 
 ## [1.4.2] - 2020-07-23
 
-### Added
-### Changed
 ### Fixed
   - preserve backtrace on logging
 
 ## [1.4.0] - 2020-07-14
 
-### Added
 ### Changed
   - change abilities to align with ecoPortal release `1.5.0`
   - remove some patches on `ecoportal-api`
@@ -351,22 +346,17 @@ All notable changes to this project will be documented in this file.
 
 ## [1.3.19] - 2020-07-23
 
-### Added
-### Changed
 ### Fixed
   - preserve backtrace on logging
 
 ## [1.3.18] - 2020-07-08
 
-### Added
-### Changed
 ### Fixed
   - the `update` case was missing the code to use the `default_usergroup`
 
 
 ## [1.3.17] - 2020-07-06
 
-### Added
 ### Changed
   - the `hris` case should not only include as `leavers` those that have account, but anyone that leaves
     * as we could have active people with no account
@@ -389,7 +379,6 @@ All notable changes to this project will be documented in this file.
 ### Added
   - default usecase to export to `csv` (`-detailed`) now includes `"Supervisor Name"` column
 ### Changed
-### Fixed
 
 ## [1.3.14] - 2020-06-10
 
@@ -397,33 +386,24 @@ All notable changes to this project will be documented in this file.
   - `Eco::API::Common::People::SupervisorHelpers` now has its methods as class methods
 ### Changed
   - upgraded `ecoportal-api` gem dependency to minimum version `0.5.6`
-### Fixed
 
 
 ## [1.3.13] - 2020-05-29
 
-### Added
-### Changed
 ### Fixed
   - `Eco::API::Organization::TagTree#tag?` to accept `nil` by returning `false`
   - `Eco::API::Common::People::DefaultParsers::DateParser` will parse to `Date` class
     * it was parsing to `Time` class, while the native gem `ecoportal-api` parses as `Date` ([reference](https://gitlab.com/ecoPortal/ecoportal-api/-/blob/master/lib/ecoportal/api/v1/schema_field.rb))
 ## [1.3.12] - 2020-05-19
 
-### Added
 ### Changed
   - stop using `email` as `external_id` on `People#person` & `People#find`
     * this should result in more accurate searches when using `:strict` options
-### Fixed
-
 
 ## [1.3.11] - 2020-05-12
 
-### Added
 ### Changed
   - remove popping up comments on `Eco::API::Organization::PolicyGroups#`
-### Fixed
-
 
 ## [1.3.10] - 2020-05-12
 
@@ -432,14 +412,11 @@ All notable changes to this project will be documented in this file.
   - `config.people.default_usergroup`, when defined, will have effect on usecases: `update` (this case was missing the change)
     * on account creation, if the input file did not specify `policy_group_ids`
 
-### Changed
 ### Fixed
   - `upsert`, `hris` and `create` usecases: fixed condition for use of default_usergroup
 
 ## [1.3.9] - 2020-05-12
 
-### Added
-### Changed
 ### Fixed
 
 - `usecase` callback was not receiving `usecase` paramater

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

@@ -8,7 +8,7 @@ module Eco
             # @return [Symbol] the `type` of usecase (i.e. `:sync`, `:transform`, `:import`, `:other`)
             def type(value = nil)
               unless value
-                return @type || raise("You should specify a type of case [:sync, :transform, :import, :other] for #{self.class}")
+                return @type || raise("You should specify a type of case [:sync, :transform, :import, :other] for #{self}")
               end
               @type = value
             end

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

@@ -256,7 +256,7 @@ module Eco
           # @param mapped_entry [Hash] that with **internal** names but **external** values and types.
           # @return [Hash] with **external** names, values and types.
           def _external_serializing(mapped_entry)
-            target_attrs    = @emap.all_attrs | @emap.aliased_attrs
+            target_attrs    = @emap.all_model_attrs | @emap.aliased_attrs
             rest_keys       = mapped_entry.keys - target_attrs
             target_attrs   -= ["send_invites"]
             external_entry  = target_attrs.each_with_object({}) do |attr, hash|

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

@@ -6,14 +6,14 @@ module Eco
         # @attr_reader core_attrs [Array<String>] core attributes that are present in the person entry.
         # @attr_reader details_attrs [Array<String>] schema details attributes that are present in the person entry.
         # @attr_reader account_attrs [Array<String>] account attributes that are present in the person entry.
-        # @attr_reader all_attrs [Array<String>] all the attrs that are present in the person entry.
+        # @attr_reader all_model_attrs [Array<String>] all the attrs that are present in the person entry.
         # @attr_reader internal_attrs [Array<String>] all the internally named attributes that the person entry has.
         # @attr_reader aliased_attrs [Array<String>] only those internal attributes present in the person entry that have an internal/external name mapping.
         # @attr_reader direct_attrs [Array<String>] only those internal attributes present in the person entry that do **not** have an internal/external name mapping.
         class PersonEntryAttributeMapper
           @@cached_warnings = {}
 
-          attr_reader :core_attrs, :details_attrs, :account_attrs, :all_attrs
+          attr_reader :core_attrs, :details_attrs, :account_attrs, :all_model_attrs
           attr_reader :internal_attrs, :aliased_attrs, :direct_attrs
 
           # Helper class tied to `PersonEntry` that allows to track which attributes of a person entry are present
@@ -41,14 +41,14 @@ module Eco
               init_attr_trackers
             else  # SERIALIZING
               @person          = data
-              @internal_attrs  = @person_parser.all_attrs
+              @internal_attrs  = @person_parser.all_model_attrs
               @aliased_attrs   = @attr_map.list(:internal)
             end
 
             @core_attrs    = @person_parser.target_attrs_core(@internal_attrs)
             @details_attrs = @person_parser.target_attrs_details(@internal_attrs)
             @account_attrs = @person_parser.target_attrs_account(@internal_attrs)
-            @all_attrs     = @core_attrs | @account_attrs | @details_attrs
+            @all_model_attrs = @core_attrs | @account_attrs | @details_attrs
           end
 
           # To know if currently the object is in parse or serialize mode.
@@ -98,7 +98,7 @@ module Eco
               end
             end
 
-            return nil unless @person_parser.all_attrs.include?(attr)
+            return nil unless @person_parser.all_model_attrs.include?(attr)
           end
 
           # Serializing helper also used to do a reverse mapping when parsing:
@@ -146,9 +146,9 @@ module Eco
           # when parsing:
           def init_attr_trackers
             # (def) all internal attributes we can expect
-            def_all_attrs     = @person_parser.all_attrs
+            def_all_attrs     = @person_parser.all_model_attrs
             # (def) internal attrs with no aliasing nor parser definition (expected to be direct)
-            def_unlinked      = @person_parser.undefined_attrs.select { |attr| !to_external(attr) }
+            def_unlinked      = @person_parser.undefined_model_attrs.select { |attr| !to_external(attr) }
             # (def) those with parser or alias:
             def_linked        = def_all_attrs - def_unlinked
 
@@ -168,7 +168,7 @@ module Eco
             # (data) virtual internal attrs (the internal names of those virtual attrs)
             data_vi_int_aliased = data_vi_ext_alias.map do |attr|
               # to_internal(attr) can't be used here, becauase virtual fields would get filtered out,
-              # as they are not recognized by @parser.all_attrs.include?(attr)
+              # as they are not recognized by @parser.all_model_attrs.include?(attr)
               @attr_map.to_internal(attr)
             end.compact
 

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

@@ -7,7 +7,7 @@ module Eco
         #
         # @attr_reader schema [Ecoportal::API::V1::PersonSchema, nil] schema of person details that this parser will be based upon.
         # @attr_reader details_attrs [Array<String>] internal names of schema details attributes.
-        # @attr_reader all_attrs [Array<String>] all the internal name attributes, including _core_, _account_ and _details_.
+        # @attr_reader all_model_attrs [Array<String>] all the internal name attributes, including _core_, _account_ and _details_.
         class PersonParser
           extend  Eco::API::Common::ClassAutoLoader
           autoloads_children_of     "Eco::API::Common::Loaders::Parser"
@@ -19,8 +19,7 @@ module Eco
           FORMAT        = [:csv, :xml, :json]
 
           attr_reader :schema
-          attr_reader :details_attrs, :all_attrs
-          attr_reader :defined_attrs
+          attr_reader :details_attrs, :all_model_attrs
           attr_reader :patch_version
 
           # @example Example of usage:
@@ -46,7 +45,7 @@ module Eco
               @details_attrs  = @schema&.fields.map { |fld| fld.alt_id }
             end
 
-            @all_attrs = CORE_ATTRS + ACCOUNT_ATTRS + @details_attrs
+            @all_model_attrs = CORE_ATTRS + ACCOUNT_ATTRS + @details_attrs
             self.class.autoload_children(self)
           end
 
@@ -60,6 +59,11 @@ module Eco
 
           # @!group Scopping attributes (identifying, presence & active)
 
+          # All the internal name attributes, including _core_, _account_ and _details_.
+          def all_attrs(include_defined_parsers: false)
+            all_model_attrs | defined_model_attrs
+          end
+
           # Scopes `source_attrs` using the _**core** attributes_.
           # @note use this helper to know which among your attributes are **core** ones.
           # @param source_attrs [Array<String>]
@@ -93,14 +97,21 @@ module Eco
             @parsers.keys
           end
 
+          # Returns a list of all the internal attributes that have a parser defined.
+          # @note These attributes do not necessarily belong to the model. They could be virtual attributes
+          # @return [Array<String>] list of all attribute defined parsers.
+          def defined_attrs
+            defined_list - symbol_keys
+          end
+
           # Returns a list of all the internal attributes of the model that have a parser defined.
           # @note
           #  - it excludes any parser that is not in the model, such as type parsers (i.e. `:boolean`, `:multiple`)
           #  - the list is sorted according `CORE_ATTRS` + `ACCOUNT_ATTRS` + schema attrs
-          # @return [Array<String>] list of all attribute defined parsers.
-          def defined_attrs
+          # @return [Array<String>] list of all attribute defined parsers in the model.
+          def defined_model_attrs
             defined     = @parsers.keys
-            defined     = (all_attrs | defined) & defined
+            defined     = (all_model_attrs | defined) & defined
             defined - symbol_keys
           end
 
@@ -118,7 +129,7 @@ module Eco
           # @param process [Symbol] either `:parse` or `:serialize`, depending if we want to parse or serialize the `attr`.
           # @return [Array<String>] list of all attribute defined parsers that should be active for the given `source_data`.
           def active_attrs(source_data, phase = :any, process: :parse)
-            defined_attrs.select do |attr|
+            defined_model_attrs.select do |attr|
               if process == :serialize
                 @parsers[attr].serializer_active?(phase)
               else
@@ -130,8 +141,8 @@ module Eco
           # Returns a list of all the internal attributes of the model that do **not** have a parser defined.
           # @note it excludes any parser that is **not** in the model, such as type parsers (i.e. :boolean, :multiple)
           # @return [Array<String>] list of all attributes without a defined parser.
-          def undefined_attrs
-            all_attrs - defined_attrs
+          def undefined_model_attrs
+            all_model_attrs - defined_model_attrs
           end
 
           # @param attr [String] internal name of an attribute.
@@ -238,7 +249,7 @@ module Eco
           end
 
           def valid_attr?(attr)
-            attr.is_a?(String) && (!@schema || @all_attrs.include?(attr))
+            attr.is_a?(String) && (!@schema || @all_model_attrs.include?(attr))
           end
 
           def valid_type?(attr)

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

@@ -4,7 +4,7 @@ module Eco
 
       # Provides helpers to deal with tagtrees.
       class TagTree
-        attr_reader :tag, :nodes
+        attr_reader :tag, :nodes, :children_count
         attr_reader :depth, :path
         attr_reader :enviro
 
@@ -37,6 +37,7 @@ module Eco
 
           nodes = @source.is_a?(Array) ? @source : @source.dig('nodes') || []
           @nodes = nodes.map {|cnode| TagTree.new(cnode, depth: @depth + 1, path: @path.dup, enviro: @enviro)}
+          @children_count = @nodes.count
 
           init_hashes
         end
@@ -46,6 +47,21 @@ module Eco
           @has_tags.empty?
         end
 
+        # @return [Integer] the highest `depth` of all the children.
+        def total_depth
+          @total_depth ||= if children_count > 0
+              deepest_node = nodes.max_by {|node| node.total_depth}
+              deepest_node.depth
+            else
+              depth
+            end
+        end
+
+        # @return [Integer] if there's only top level.
+        def flat?
+          self.total_depth <= 0
+        end
+
         # Gets all the tags of the current node tree.
         # @note
         #   - this will include the upper level tag(s) as well
@@ -147,7 +163,8 @@ module Eco
         # * take the deepest tag (the one that is further down in the tree)
         # * if there are different options (several nodes at the same depth):
         #    * take the common node between them (i.e. you have Hamilton and Auckland -> take New Zealand)
-        #    * if there's no common node between them, take the `first` (unless they are at top level of the tree)
+        #    * if there's no common node between them, take the `first`, unless they are at top level of the tree
+        #    * to the above, take the `first` also on top level, but only if there's 1 level for the entire tree
         # @param [Array<String>] values list of tags.
         # @return [String] default tag.
         def default_tag(*values)
@@ -169,7 +186,7 @@ module Eco
             common      = nodes.reduce(self.tags.reverse) {|com, cnode| com & cnode.path.reverse}
             default_tag = common.first if common.length > 0 && depth > 0
           end
-          default_tag = nodes.first&.tag if !default_tag && depth > 0
+          default_tag = nodes.first&.tag  if !default_tag && ( (depth > 0) || flat?)
           default_tag
         end
 

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

@@ -105,10 +105,15 @@ module Eco
           iterations = (data.length.to_f / per_page).ceil
 
           Eco::API::Session::Batch::Status.new(enviro, queue: data, method: method).tap do |status|
+            start_time  = Time.now
+            start_slice = Time.now; slice = []
             data.each_slice(per_page) do |slice|
-              msg = "starting batch '#{method}' iteration #{iteration}/#{iterations}, with #{slice.length} entries of #{data.length} -- #{done} done"
+              msg  = "starting batch '#{method}' iteration #{iteration}/#{iterations},"
+              msg += " with #{slice.length} entries of #{data.length} -- #{done} done"
+              msg += " (last: #{str_stats(start_slice, slice.length)}; total: #{str_stats(start_time, done)})"
               logger.info(msg) unless silent
 
+              start_slice = Time.now
               people_api.batch do |batch|
                 slice.each do |person|
                   batch.public_send(method, person) do |response|
@@ -118,12 +123,23 @@ module Eco
                 end
               end # next batch
 
-              iteration += 1
-              done += slice.length
+              iteration  += 1
+              done       += slice.length
             end # next slice
           end
         end
 
+        def str_stats(start, count)
+          now     = Time.now
+          secs    = (now - start).round(3)
+          if secs > 0.0
+            per_sec = (count.to_f / secs).round(2)
+            "#{secs}s -> #{per_sec} people/s"
+          else
+            " -- "
+          end
+        end
+
       end
     end
   end

data/lib/eco/api/session/batch/errors.rb

@@ -111,6 +111,11 @@ module Eco
           end
           # @!endgroup
 
+          def person_ref(entry)
+            row_str = (row = get_row(entry)) ? "(row: #{row}) " : nil
+            "#{row_str}(id: '#{get_attr(entry, :id)}') '#{get_attr(entry, :name)}' ('#{get_attr(entry, :external_id)}': '#{get_attr(entry, :email)}')"
+          end
+
           private
 
           # Input entries that got **error** response from the _Server_.
@@ -119,7 +124,7 @@ module Eco
           #   - please, observe that this can only happen if there were repeated entries in the `source_queue`
           # @return [Array<Hash>, Array<Ecoportal::API::V1::Person>, Array<Ecoportal::API::Internal::Person>]
           def entries
-            queue.filter.with_index do |query, i|
+            queue.select.with_index do |query, i|
               unless response = status[i]
                 msg  = "Error: query with no response. You might have duplicated entries in your queue.\n"
                 msg += "Queue length: #{queue.length}; Queue elements class: #{queue.first.class}\n"
@@ -136,11 +141,6 @@ module Eco
             end
           end
 
-          def person_ref(entry)
-            row_str = (row = get_row(entry)) ? "(row: #{row}) " : nil
-            "#{row_str}(id: '#{get_attr(entry, :id)}') '#{get_attr(entry, :name)}' ('#{get_attr(entry, :external_id)}': '#{get_attr(entry, :email)}')"
-          end
-
           def get_attr(entry, attr)
             if entry.respond_to?(attr.to_sym)
               entry.public_send(attr.to_sym)

data/lib/eco/api/session/batch/feedback.rb

@@ -45,25 +45,22 @@ module Eco
           # @!group Pure feedback methods
 
           # Slightly modifies the behaviour of `Ecoportal::API::Common::BaseModel#as_update`, so schema details fields show the `alt_id`
+          # It also fixes possible patch updates that are incomplete or unnecessary.
           # @note for better feedback
           # @param entry [Hash, Ecoportal::API::V1::Person, Ecoportal::API::Internal::Person]
-          def as_update(entry)
+          # @param add_feedback [Boolean] if `true` it tweak the hash update with additional data.
+          def as_update(entry, add_feedback: true)
             case
             when entry.is_a?(Hash)
               hash = entry
             else #entry.is_a?(Ecoportal::API::V1::Person)
               if only_ids?
-                hash = {
-                  "id"          => entry.id,
-                  "external_id" => entry.external_id,
-                  "email"       => entry.email
-                }
                 hash = entry.as_json.slice("id", "external_id", "email")
               else
                 hash = entry.as_update
-                #if entry.details
-                if hash["details"]
-                  if hfields = hash.dig("details", "fields")
+
+                if add_feedback && details = hash["details"]
+                  if hfields = details["fields"]
                     hfields.each do |fld|
                       fld.merge!("alt_id" => entry.details.get_field(fld["id"]).alt_id)
                     end

data/lib/eco/api/session/batch/job.rb

@@ -169,7 +169,8 @@ module Eco
 
             unless simulate
               if pqueue.length > 0
-                backup_update(requests)
+                req_backup = pqueue.map {|e| as_update(e, add_feedback: false)}
+                backup_update(req_backup)
                 session.batch.launch(pqueue, method: type).tap do |job_status|
                   @status     = job_status
                   status.root = self
@@ -217,7 +218,7 @@ module Eco
             [].tap do |msg|
               subjobs.map {|subjob| msg << subjob.summary}
             end.join("\n")
-          end          
+          end
 
           def as_update(*args)
             feedback.as_update(*args)

data/lib/eco/api/usecases.rb

@@ -30,6 +30,7 @@ module Eco
         @usecases      = {}
         @cache_init    = false
         @cases_by_name = {}
+        init_caches
       end
 
       def length
@@ -79,6 +80,7 @@ module Eco
 
       # @return [Eco::API::UseCases] a copy of instance object of `self`.
       def dup
+        init_caches
         self.class.new.merge(self)
       end
 

data/lib/eco/api/usecases/default_cases/hris_case.rb

@@ -2,18 +2,17 @@ class Eco::API::UseCases::DefaultCases::HrisCase < Eco::API::Common::Loaders::Us
   name "hris"
   type :sync
 
+  attr_reader :creation, :update, :supers, :leavers
+
   def main(entries, people, session, options, usecase)
     micro    = session.micro
-    creation = session.new_job("main", "create",  :create, usecase)
-    update   = session.new_job("main", "update",  :update, usecase)
-    supers   = session.new_job("post", "supers",  :update, usecase, :core)
-    leavers  = session.new_job("post", "leavers", :update, usecase, :account)
+    @creation = session.new_job("main", "create",  :create, usecase)
+    @update   = session.new_job("main", "update",  :update, usecase)
+    @supers   = session.new_job("post", "supers",  :update, usecase, :core)
+    @leavers  = session.new_job("post", "leavers", :update, usecase, :account)
 
     micro.with_each_leaver(entries, people, options) do |person|
-      leavers.add(person) do |person|
-        person.supervisor_id = nil
-        person.account       = nil if person.account
-      end
+      leavers.add(person, &method(:leavers_callback))
     end
 
     micro.with_each(entries, people, options) do |entry, person|
@@ -24,4 +23,11 @@ class Eco::API::UseCases::DefaultCases::HrisCase < Eco::API::Common::Loaders::Us
     end
   end
 
+  private
+
+  def leavers_callback(person)
+    person.supervisor_id = nil
+    person.account       = nil if person.account
+  end
+
 end

data/lib/eco/cli.rb

@@ -22,16 +22,17 @@ module Eco
 
     def run(session:)
       io = Eco::API::UseCases::BaseIO.new(session: session, options: options)
-      #session.workflow.run(io: io)
-      session.workflow(io: io) do |wf, io|
-        io = wf.run(:options, io: io)
-        io = wf.run(:load, io: io)
-        io = wf.run(:usecases, io: io)
-        io = wf.run(:launch_jobs, io: io)
-        io = wf.run(:post_launch, io: io)
-        io = wf.run(:end, io: io)
-        io = wf.run(:close, io: io)
-      end
+      session.workflow(io: io).run(io: io)
+      #session.workflow(io: io) do |wf, io|
+      #  io = wf.run(:options, io: io)
+      #  io = wf.run(:load, io: io)
+      #  io = wf.run(:usecases, io: io)
+      #  io = wf.run(:launch_jobs, io: io)
+      #  io = wf.run(:post_launch, io: io)
+      #  io = wf.run(:report, io: io)
+      #  io = wf.run(:end, io: io)
+      #  io = wf.run(:close, io: io)
+      #end
     end
 
   end

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.8"
+  VERSION = "2.0.13"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.8
+  version: 2.0.13
 platform: ruby
 authors:
 - Oscar Segura