eco-helpers 2.0.36 → 2.0.40

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '069fe6b03a523420dd24aa933eeffb05c20153ace763deb228de9e978a2f0ef6'
-  data.tar.gz: 974d0af1009597161438cc9de087d42a73a503b6c0c8d50ab97c3030672e7180
+  metadata.gz: 48ff2a20387ce7665b56aebc5b9c4746088c4bdcbe1eb47cf97dd8464e8cea07
+  data.tar.gz: 2a63611643f60bd63f84e273a6fab267685b46dead2e01060e71f37a05398e22
 SHA512:
-  metadata.gz: 8c089ac0050ab88d68eda450060785d9a59870cfa87b3c0c958612cb85f8251d826c824aa9da93b1cffa99afa4e7877ebd42dec1ad0d161764fea7de29495fb7
-  data.tar.gz: f42d0a303f79775a71da9d13aa1639c8e0de85131590b84301fa1523e0293f87a943d9eaf65c22b97b73d71d667e889ae8bc0137861ac3811fed1a8a439de66f
+  metadata.gz: e421231ced6f863fd0f849f229ebba0ca242b666304613dce7d2084a891ac304de9e9002fb7a97212948a002cfe959351ce25752707daa74f63163b684f8332a
+  data.tar.gz: b4273a6a9e9de5fddc0bcd117ec0a467bcd57d3b904286f3130c36e6226b930e9b38bf6839867a09d79b0c284af17087b4bf0ea38b0edc69e024c64884097941

data/CHANGELOG.md

@@ -1,10 +1,51 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.0.36] - 2021-08-xx
+## [2.0.40] - 2021-09-xx
 
 ### Added
+  - Unique access point for `validation` via core case `Eco::API::UseCases::OozeSamples::OozeBaseCase#update_ooze`
+
 ### Changed
+### Fixed
+
+## [2.0.39] - 2021-09-28
+
+### 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)

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.19', '< 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
@@ -69,6 +112,11 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   def update_ooze(ooze = target)
     if !options[:simulate]
       return unless dirty?(ooze)
+      
+      ooze.validate.tap do |validation|
+        raise validation if validation.is_a?(String)
+      end
+
       apiv2.pages.update(ooze).tap do |response|
         if response.success?
           ooze.consolidate!

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.36"
+  VERSION = "2.0.40"
 end

metadata

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