inferno_core 0.1.0 → 0.1.1.pre

data/lib/inferno/repositories/session_data.rb

@@ -54,7 +54,7 @@ module Inferno
         return nil if params[:value].blank?
 
         case params[:type]&.to_s
-        when 'text', 'textarea'
+        when 'text', 'textarea', 'radio'
           params[:value].to_s
         when 'oauth_credentials'
           credentials =

data/lib/inferno/result_summarizer.rb

@@ -0,0 +1,40 @@
+module Inferno
+  # @private
+  # This class takes an array of results and determines the overall result. This
+  # is used to determine the result of a TestGroup/TestSuite based on the
+  # results of it's children.
+  class ResultSummarizer
+    attr_reader :results
+
+    def initialize(results)
+      @results = results
+    end
+
+    def summarize
+      prioritized_result_strings.find { |result_string| unique_result_strings.include? result_string }
+    end
+
+    private
+
+    def prioritized_result_strings
+      Entities::Result::RESULT_OPTIONS
+    end
+
+    def required_results
+      @required_results ||= results.select(&:required?)
+    end
+
+    def all_optional_results?
+      required_results.blank?
+    end
+
+    def results_for_summary
+      all_optional_results? ? results : required_results
+    end
+
+    def unique_result_strings
+      @unique_result_strings ||=
+        results_for_summary.map(&:result).uniq
+    end
+  end
+end

data/lib/inferno/test_runner.rb

@@ -1,3 +1,4 @@
+require_relative './result_summarizer'
 require_relative './utils/markdown_formatter'
 
 module Inferno
@@ -120,7 +121,9 @@ module Inferno
 
       children = parent.children
       child_results = results_repo.current_results_for_test_session_and_runnables(test_session.id, children)
-      return if children.length != child_results.length
+      required_children = children.select(&:required?)
+      required_results = child_results.select(&:required?)
+      return if required_children.length != required_results.length
 
       old_result = results_repo.current_result_for_test_session(test_session.id, parent.reference_hash)&.result
       new_result = roll_up_result(child_results)
@@ -170,11 +173,7 @@ module Inferno
     end
 
     def roll_up_result(results)
-      result_priority = Entities::Result::RESULT_OPTIONS
-      unique_results = results.map(&:result).uniq
-      result_priority.find do |result|
-        unique_results.include? result
-      end
+      ResultSummarizer.new(results).summarize
     end
   end
 end

data/lib/inferno/version.rb

@@ -1,3 +1,3 @@
 module Inferno
-  VERSION = '0.1.0'.freeze
+  VERSION = '0.1.1.pre'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: inferno_core
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1.pre
 platform: ruby
 authors:
 - Stephen MacVicar
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-10 00:00:00.000000000 Z
+date: 2022-01-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -520,6 +520,7 @@ files:
 - lib/inferno/entities.rb
 - lib/inferno/entities/attributes.rb
 - lib/inferno/entities/entity.rb
+- lib/inferno/entities/has_runnable.rb
 - lib/inferno/entities/header.rb
 - lib/inferno/entities/message.rb
 - lib/inferno/entities/request.rb
@@ -557,6 +558,7 @@ files:
 - lib/inferno/repositories/test_suites.rb
 - lib/inferno/repositories/tests.rb
 - lib/inferno/repositories/validate_runnable_reference.rb
+- lib/inferno/result_summarizer.rb
 - lib/inferno/spec_support.rb
 - lib/inferno/test_runner.rb
 - lib/inferno/utils/markdown_formatter.rb
@@ -591,9 +593,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubygems_version: 3.1.6
 signing_key: