eco-helpers 2.0.38 → 2.0.39

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e84f7c290837d063ed13e63165dadaf272219105227d90b983b9af16d338fe5
-  data.tar.gz: 38ea16433f7a5689cfdcbde0b3bd4f1065e8adb9e742493c097a87c684b97b86
+  metadata.gz: 84c964d986dd27f75c533dfcca194ca69354eeda318fe751f40db3f1276c4d69
+  data.tar.gz: 958a277abf32af125c768b17e5b6bf0791c6136658ebb1e0183f973b4b4b4839
 SHA512:
-  metadata.gz: cef2930cc5941db9baab1a726fd04fb6186b57d9498cb8112589b2b12e40e53ae061ca2b68f062a197111ba85f47acafcf512f4aa3559e043b924412b23ba038
-  data.tar.gz: e79992f8ed299014a4c30b62c18773c1741a1961c4777560bbd137a60adc265920a3d35b18085256b42ce6cf467b9d4a950ba06e28adf50770a71aa94b3236ad
+  metadata.gz: c704901e864a071f11b62a8e2ec4916f3cfb63fc6ebd87a92391339dfb559afe1e1f062c0234e2ffaa9a754a99eec978969447cdb19bbe49aa629aac040d7d80
+  data.tar.gz: 98348a3777cb58849a007b3e4487db3f270190c595c6dc8384e2b28a838f203332fe746a2429adb69cc32b3af62fb6048376e3606587a9ea9df548708321251a

data/CHANGELOG.md

@@ -1,6 +1,19 @@
 # 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
@@ -12,8 +25,6 @@ All notable changes to this project will be documented in this file.
 ### Changed
   - Changed the `workflow` to prevent re-cache of people when `people` was not used
 
-### Fixed
-
 ## [2.0.37] - 2021-09-03
 
 ### 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.13', '< 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

@@ -26,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

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

@@ -129,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)
@@ -142,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

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.38"
+  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.38
+  version: 2.0.39
 platform: ruby
 authors:
 - Oscar Segura
@@ -136,7 +136,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.13
+        version: 0.8.14
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -146,7 +146,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.13
+        version: 0.8.14
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'