eco-helpers 2.0.35 → 2.0.39

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b4d6a312dda42afc3e6ba2becccf69d42121024d9b81e2c37276719089521971
-  data.tar.gz: a3536e8fba79c1600135b4c4fd523a9c3a5262d605c39c751a3dc278d096b96e
+  metadata.gz: 84c964d986dd27f75c533dfcca194ca69354eeda318fe751f40db3f1276c4d69
+  data.tar.gz: 958a277abf32af125c768b17e5b6bf0791c6136658ebb1e0183f973b4b4b4839
 SHA512:
-  metadata.gz: 8fcb26823bc3b6e3d35522c13d410094ea056269c14dea3b47bcaf16991ff36cde9537507a6a21c952aa15f590fb1f5b1fc594e55d08f04782a8e2802c695d0c
-  data.tar.gz: fb2768c6e345da3dec034e7793876ecf25ae1cff76a059d98a38d9328c57884efb2a2f0b449eb28aea686ba074e477875fd8ba30201773170935f1c766fd45ae
+  metadata.gz: c704901e864a071f11b62a8e2ec4916f3cfb63fc6ebd87a92391339dfb559afe1e1f062c0234e2ffaa9a754a99eec978969447cdb19bbe49aa629aac040d7d80
+  data.tar.gz: 98348a3777cb58849a007b3e4487db3f270190c595c6dc8384e2b28a838f203332fe746a2429adb69cc32b3af62fb6048376e3606587a9ea9df548708321251a

data/CHANGELOG.md

@@ -1,6 +1,48 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## [2.0.39] - 2021-09-07
+
+### Added
+### Changed
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase`
+    - Moved helpers from `Eco::API::UseCases::OozeSamples::OozeUpdateCase`
+
+### Fixed
+  - `Eco::API::UseCases::OozeSamples::RegisterUpdateCase`
+    - fix launch condition to be `Y`
+    - fix `search` method call
+  - Changed the `workflow` to prevent re-cache of people when `people` was not used
+
+## [2.0.38] - 2021-09-07
+
+### Added
+  - `Eco::CSV::Table` some more helpers to manage input `csv`s
+    - `#transform_headers` that allows to pass a block to decide the new name
+    - `#merge_same_header_names` that allows to deal with value of columns with same header name
+    - `#duplicated_header_names` that identifies repeated header names
+
+### Changed
+  - Changed the `workflow` to prevent re-cache of people when `people` was not used
+
+## [2.0.37] - 2021-09-03
+
+### Added
+### Changed
+
+### Fixed
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase`
+    - `#update_ooze` only count dry run for changed pages (to have better feedack)
+    - stop downloading the people manager
+  - `Eco::API::UseCases::OozeSamples::RegisterUpdateCase`
+    - prevent queuing non page objects
+
+## [2.0.36] - 2021-08-31
+
+### Fixed
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase`
+    - `#update_ooze` only count dry run for changed pages (to have better feedack)
+
 ## [2.0.35] - 2021-08-30
 
 ### Added

data/eco-helpers.gemspec

@@ -31,7 +31,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
 
   spec.add_dependency 'ecoportal-api', '>= 0.8.3', '< 0.9'
-  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.12', '< 0.9'
+  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.14', '< 0.9'
   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/usecases/ooze_samples/ooze_base_case.rb

@@ -12,6 +12,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   DRY_COUNT  = 5
 
   def main(session, options, usecase)
+    options[:end_get] = false
     raise "You need to inherit from this class ('#{self.class}') and call super with a block" unless block_given?
     @session = session; @options = options; @usecase = usecase
     @target  = nil
@@ -25,6 +26,48 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
     update_ooze(ooze)
   end
 
+  protected
+  
+  def add_field(name, type, section, after:  nil, before: nil, side: :left)
+    unless section.is_a?(Ecoportal::API::V2::Page::Section)
+      raise "You need to specify a section for a new field. Given: #{section.class}"
+    end
+    target.components.add(label: name, type: type) do |field|
+      section.add_component(field, after: after, before: before, side: side)
+    end.tap do |field|
+      yield(field) if block_given?
+    end
+  end
+
+  def with_fields(type: nil, label: nil)
+    flds = target.components
+    flds = flds.get_by_type(type) if type
+    flds = flds.select do |fld|
+      value = (label == :unnamed) ? nil : label
+      !label || same_string?(fld.label, value)
+    end.each do |field|
+      yield(field) if block_given?
+    end
+  end
+
+  def with_sections(type: nil, heading: nil)
+    secs = target.sections
+    secs = secs.get_by_type(type) if type
+    secs = secs.select do |sec|
+      value = (heading == :unnamed) ? nil : heading
+      !heading || same_string?(sec.heading, value)
+    end.each do |sec|
+      yield(sec) if block_given?
+    end
+  end
+
+  def with_stage(name:)
+    if stage = target.stages.get_by_name(name)
+      yield(stage) if block_given?
+    end
+    stage
+  end
+
   private
 
   # Hook method to use before the target is switched
@@ -84,7 +127,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
         end
       end
       backup_patch!(ooze)
-      exit(0) if dry_count > DRY_COUNT
+      exit(0) if dirty?(ooze) && dry_count > DRY_COUNT
     end
   end
 

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

@@ -4,44 +4,6 @@ class Eco::API::UseCases::OozeSamples::OozeUpdateCase < Eco::API::UseCases::Ooze
 
   private
 
-  def with_fields(type: nil, label: nil)
-    flds = target.components
-    flds = flds.get_by_type(type) if type
-    flds = flds.select do |fld|
-      value = (label == :unnamed) ? nil : label
-      !label || same_string?(fld.label, value)
-    end.each do |field|
-      yield(field) if block_given?
-    end
-  end
-
-  def with_sections(type: nil, heading: nil)
-    secs = target.sections
-    secs = secs.get_by_type(type) if type
-    secs = secs.select do |sec|
-      value = (heading == :unnamed) ? nil : heading
-      !heading || same_string?(sec.heading, value)
-    end.each do |sec|
-      yield(sec) if block_given?
-    end
-  end
-
-  def with_stage(name:)
-    if stage = target.stages.get_by_name(name)
-      yield(stage) if block_given?
-    end
-    stage
-  end
-
-  def add_field(name, type, section, after:  nil, side: :left)
-    raise "You need to specify a section for a new field. Given: #{section.class}" unless section.is_a?(Ecoportal::API::V2::Page::Section)
-    target.components.add(label: name, type: type) do |field|
-      section.add_component(field, after: after, side: side)
-    end.tap do |field|
-      yield(field) if block_given?
-    end
-  end
-
   def to_field(value)
     fld   = nil
     fld ||= value if value.is_a?(Ecoportal::API::V2::Page::Component)
@@ -50,5 +12,4 @@ class Eco::API::UseCases::OozeSamples::OozeUpdateCase < Eco::API::UseCases::Ooze
     #fld ||= value.reference if value.is_a?(Ecoportal::API::V2::Page::Force::Binding)
   end
 
-
 end

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

@@ -40,7 +40,7 @@ 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|
-      #queue(pending)
+      #enqueue(pending)
     end
   end
 
@@ -65,6 +65,9 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
 
   def enqueue(object)
     return unless object && object.respond_to?(:id)
+    unless object.is_a?(Ecoportal::API::V2::Page) or object.is_a?(Ecoportal::API::V2::Pages::PageStage)
+      raise "Queuing is just for entries. Expecting Page or PageStage. Given: #{object.class}"
+    end
     if elem = queue_get(object.id)
       msg = "Something is wrong. Native case 'RegisterUpdateCase' is "
       if (elem != object) && dirty?(elem)
@@ -126,7 +129,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
     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):", explanation: str_results, default: "N", timeout: 10) do |res|
-        if res.upcase.start_with?("N")
+        unless res.upcase.start_with?("Y")
           puts "..."
           logger.info "Aborting script..."
           exit(0)
@@ -139,7 +142,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
     @search_options ||= {}.tap do |opts|
       opts.merge!(sort: "created_at")
       opts.merge!(dir:  "asc")
-      opts.merge!(query:   conf_filters) if conf_search
+      opts.merge!(query:   conf_search) if conf_search
       opts.merge!(filters: conf_filters)
     end
   end

data/lib/eco/cli/config/default/workflow.rb

@@ -101,19 +101,28 @@ ASSETS.cli.config do |config|
         get_people     = io.options.dig(:people, :get)
         partial_update = get_people && get_people.dig(:type) == :partial
         run_it         = !io.options[:dry_run] || io.options.dig(:post_launch, :run)
-        if run_it && partial_update
-          unless io.options[:dry_run]
+        refresh_data   = !io.options[:dry_run] && partial_update
+        if run_it
+          if refresh_data
             # get target people afresh
             people = io.session.micro.people_refresh(people: io.people, include_created: true)
             io     = io.base.new(people: people)
+          else
+            msg = "Although there are post_launch cases, data will not be refreshed before their run"
+            if io.options[:dry_run]
+              msg += ", because we are in dry-run (simulate)."
+            elsif !partial_update
+              msg += ", because it is not a partial update (-get-partial option not present)."
+            end
+            io.session.logger.warn(msg)
           end
         else
           wf_post.skip!
           msg = "Although there are post_launch cases, they will NOT be RUN"
-          if io.options[:dry_run]
-            msg+= ", because we are in dry-run (simulate)."
-          elsif !partial_update
+          if !partial_update
             msg+= ", because it is not a partial update (-get-partial)"
+          elsif io.options[:dry_run]
+            msg+= ", because we are in dry-run (simulate)."
           end
           io.session.logger.info(msg)
         end
@@ -146,8 +155,11 @@ ASSETS.cli.config do |config|
       get_people     = io.options.dig(:people, :get)
       partial_update = get_people && get_people.dig(:type) == :partial
       unless !io.options[:end_get] || io.options[:dry_run] || partial_update
-        people = io.session.micro.people_cache
-        io     = io.new(people: people)
+        # Prevent getting people when there were no use cases that used them
+        unless io.people
+          people = io.session.micro.people_cache
+          io     = io.new(people: people)
+        end
       end
       io
     end

data/lib/eco/csv/table.rb

@@ -15,6 +15,46 @@ module Eco
         end
       end
 
+      # @return [Eco::CSV::Table]
+      def transform_headers
+        header = self.headers
+        cols   = self.columns
+        cols.each do |col|
+          col[0] = yield(col.first)
+        end
+        columns_to_table(cols)
+      end
+
+      # @return [Eco::CSV::Table]
+      def merge_same_header_names
+        dups = self.duplicated_header_names
+        out_rows = self.map do |row|
+          row.to_a.each_with_object({}) do |(name, value), out|
+            if dups.include?(name) && out.key?(name)
+              if block_given?
+                yield(value, out[name], name)
+              else
+                # resolve
+                value || out[name]
+              end
+            elsif out.key?(name)
+              out[name]
+            else
+              value
+            end.tap do |final_value|
+              out[name] = final_value
+            end
+          end
+        end
+        self.class.new(out_rows)
+      end
+
+      # @return [Array<String>] list of duplicated header names
+      def duplicated_header_names
+        header = self.headers
+        header.select {|e| header.count(e) > 1}.uniq
+      end
+
       # @return [Eco::CSV::Table]
       def transform_values
         transformed_rows = rows.map do |row|

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.35"
+  VERSION = "2.0.39"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.35
+  version: 2.0.39
 platform: ruby
 authors:
 - Oscar Segura
@@ -136,7 +136,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.12
+        version: 0.8.14
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -146,7 +146,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.12
+        version: 0.8.14
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'