eco-helpers 2.4.6 → 2.4.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c4f7620deee8791ef8b6bfa5552e6040657575fa2ee071efdf7ed76e588a998
-  data.tar.gz: a870243619d2b8842ef97525676097330b76a0686f4793f7c6720e4c658173aa
+  metadata.gz: facf3e640f8af36beba49a9da6eb7c72cb70b4a07ef93602ee70898cce31c356
+  data.tar.gz: 304bb16fdefd0db6ad63b524a654aed990375a24630565d59cfb6fa4470e94da
 SHA512:
-  metadata.gz: 5eef75b2a425b14b6bd17db7d58c153752ad121a708665f27736f9e4457c2eca5f06ec5af1344fe941478419c3271d0debdb3b09fda826eecd06414bf9a06405
-  data.tar.gz: 50c9b2e722457a10eae9da4723816f906efb816b82a485562779251ad80dea4c737cc0bd572c5e80eff3b9e2f6b3b909245bf65b766781f7cbdb63a8d5d68daf
+  metadata.gz: 6d4b1882f5c042ceeeee46bfb6e1604204092a108409c882df2f82e125259cd11551ce90e48c40cf5c7cdf50286eda42b624c9cd24d9e1ea13d4a8bb3203d9b2
+  data.tar.gz: 91061e3b0d8caafc59fbd77ca726cf84c83d459eb1bad420932ad3482842d57acd1a38a35373a82078638b6c7e41e570a6c7f2adb84a319a38c61509555d2bc9

data/CHANGELOG.md

@@ -1,12 +1,34 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.4.7] - 2023-04-xx
+## [2.4.9] - 2023-04-xx
 
 ### Added
 ### Changed
 ### Fixed
 
+## [2.4.8] - 2023-04-03
+
+### Added
+  - `Eco::API::Organization::TagTree#ancestors` returns the ancestor nodes.
+
+## [2.4.7] - 2023-04-03
+
+### Added
+  - `Eco::Data::Hashes::ArrayDiff#diffs?`
+  - `Eco::API::Organization::TagTree` added methods:
+    - `#source` => to expose the original doc
+    - `#all_nodes` => to implement the block call
+
+### Changed
+  - Upgraded `ecoportal-api-graphql` **gem**
+
+### Fixed
+  - `Eco::API::Session::Config::Api` **new `graphql` api** was missing `host`
+  - `Eco::API::UseCases::DefaultCases::CsvToTree::Helper`
+    - `#nodes_from_csv` assumed unique top level tag and not present
+    - This fix completes this feature (as top level tags were not there)
+
 ## [2.4.6] - 2023-04-02
 
 ### Added

data/eco-helpers.gemspec

@@ -32,7 +32,7 @@ Gem::Specification.new do |spec|
 
   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.3', '< 0.4'
+  spec.add_dependency 'ecoportal-api-graphql', '>= 0.3.5', '< 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'

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

@@ -13,6 +13,8 @@ module Eco
         attr_reader :depth, :path
         attr_reader :enviro
 
+        attr_reader :source
+
         include Enumerable
 
         # @example Node format:
@@ -42,20 +44,20 @@ module Eco
           if @source.is_a?(Array)
             @id        = id
             @name      = name
-            @row_nodes = @source
+            @raw_nodes = @source
           else
             @id        = @source.values_at('tag', 'id').compact.first&.upcase
             @name      = @source['name']
-            @archived  = @source.fetch('archived', false)
-            @archived_token = @source['archived_token']
-            @weight    = @source.fetch('weight', _weight)
-            @row_nodes = @source['nodes'] || []
+            @archived  = @source['archived'] || false
+            @archived_token   = @source['archived_token']
+            @source['weight'] = @weight = @source['weight'] || _weight
+            @raw_nodes = @source['nodes'] || []
           end
 
           @path  = path || []
           @path.push(@id) unless top?
 
-          @nodes = @row_nodes.map.with_index do |cnode, idx|
+          @nodes = @raw_nodes.map.with_index do |cnode, idx|
             TagTree.new(cnode, depth: depth + 1, path: @path.dup, parent: self, _weight: idx, enviro: @enviro)
           end
 
@@ -82,12 +84,27 @@ module Eco
 
         # All actual nodes of this tree
         # @note order is that of the parent to child relationships
-        # @return [Array[]]
+        # @return [Array<TagTree>]
         def all_nodes(&block)
-          [].tap do |all_nodes|
-            all_nodes.push(self) unless top?
+          [].tap do |out_nodes|
+            unless top?
+              out_nodes.push(self)
+              yield(self) if block_given?
+            end
             nodes.each do |node|
-              all_nodes.concat(node.all_nodes)
+              out_nodes.concat(node.all_nodes(&block))
+            end
+          end
+        end
+
+        # All the acenstor nodes of the current node
+        # @note it does not include the current node
+        # @return [Array<TagTree>] ancestors sorted from top to bottom.
+        def ancestors
+          [].tap do |ans|
+            unless parent.top?
+              ans << parent
+              ans.concat(parent.ancestors)
             end
           end
         end
@@ -109,8 +126,11 @@ module Eco
             nodes_json
           else
             {
-              "id"    => tag,
-              "nodes" => nodes_json
+              "id"        => tag,
+              "archived"  => archived,
+              "archived_token" => archived_token,
+              "weight"    => weight,
+              "nodes"     => nodes_json
             }
           end
         end

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

@@ -171,7 +171,7 @@ module Eco
             when :v2
               klass.new(user_key: user_key, org_key: external_key, host: host, logger: logger)
             when :graphql
-              klass.new(org_id: org_id, email: email, pass: pass)
+              klass.new(host: host, org_id: org_id, email: email, pass: pass)
             end.tap do |api|
               unless !api || log_connection?
                 @logger.info("Created api#{self.version(version)} connection on '#{name}' enviro, pointing to '#{host}' in '#{mode}' mode")

data/lib/eco/api/usecases/default_cases/csv_to_tree_case/helper.rb

@@ -25,11 +25,16 @@ class Eco::API::UseCases::DefaultCases::CsvToTree
           node = Node.new(i, *values)
           prev_node ||= node
 
-          if prev_node.raw_level <= node.raw_level
+          # If node is nested in prev_node or is a sibling thereof
+          if prev_node.raw_level <= node.raw_level\
+            # Make sure parent is among upper level tags
             node.set_high_levels(prev_node)
           else
             if parent_node = prev_nodes[node.raw_level - 1]
               node.set_high_levels(parent_node)
+            elsif node.raw_level == 1
+              # It is expected not to have parent
+              #puts "Node '#{node.raw_tag}' doesn't have parent, but it's top level tag"
             else
               raise "Node '#{node.raw_tag}' (#{node.row_num} row) doesn't have parent"
             end

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

@@ -44,7 +44,7 @@ class Eco::API::UseCases::DefaultCases::HrisCase < Eco::API::Common::Loaders::Us
       other_people  = people.group_by_schema.values_at(*other_schemas).map(&:to_a).flatten
       if other_people.length > 3
         msg  = "There are #{other_people.length} people in schemas other than #{active_schema.name}."
-        msg << " Please, use the filter option '-schema_id SchemaName' for the 'hris' case to only include those of that schema"
+        msg << " Please, use the filter option '-schema-id SchemaName' for the 'hris' case to only include those of that schema"
         msg << " in the current update. The HRIS case identifies people that are not in the file as leavers"
         msg << " (as it will remove the account of all the people of other schemas if they are not in the input file)."
         msg << "\n For example: -schema-id '#{active_schema.name.downcase}'"

data/lib/eco/data/hashes/array_diff.rb

@@ -56,15 +56,24 @@ module Eco
         #   - It also allows to know the original value
         # @return [Hash] where `key` is the key of the record, and `value` a `DiffResult` object
         def diffs
-          @diffs ||= paired_sources.each_with_object([]) do |(src1, src2), diffs|
+          @diffs ||= source_results.select do |res|
+            res.diff?
+          end
+        end
+
+        # @return [Boolean] wheter or not there are differences.
+        def diffs?
+          !diffs.empty?
+        end
+
+        def source_results
+          @source_results ||= paired_sources.each_with_object([]) do |(src1, src2), res|
             args = {
               key:            key,
               compare:        compared_attrs,
               case_sensitive: case_sensitive?
             }
-            diff_result_class.new(src1, src2, **args).tap do |res|
-              diffs << res if res.diff?
-            end
+            res << diff_result_class.new(src1, src2, **args)
           end
         end
 

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.4.6"
+  VERSION = "2.4.8"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.4.6
+  version: 2.4.8
 platform: ruby
 authors:
 - Oscar Segura
@@ -156,7 +156,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.3
+        version: 0.3.5
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.4'
@@ -166,7 +166,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.3.3
+        version: 0.3.5
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.4'