inferno_core 0.3.4 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c905fd9a450198c42daa6a7f3119277d1eeb1e39acb788856d90c3817719d02
-  data.tar.gz: e11d6fcfa8acb3c487c62d512b9210db18f5d7bba5c9148063587de8a3e78420
+  metadata.gz: 40fa8983d93a7fc191a7d03bdb78c0bd80e138cfe5245d377c15f28360fb6b03
+  data.tar.gz: 4aa592411c216c13bc55ebcef30ce2019522cc3575edc184fcb10593eb28239a
 SHA512:
-  metadata.gz: d62c7ebc7b9f9ac2024bfb933254cd54010b869fde72d15c77e935e7228096e2ba45844e3f8ba2f7b28f71315120d7a194b63d7e95675600e26efd1bbe822cc1
-  data.tar.gz: ed6b0d392219155cd2cd6865e01d9ba838c067f36bfdd86ac10e80e923b51ddf688cff4f8e8a67e62e53a1d28c151f1c76d5e791166e920ff9832676fa1239c0
+  metadata.gz: bb0f7e9aafced3a1181cd40d8638f4ac07fefc0239ab771b85ce22aaec3785b7328ca82969cd83b1617051201fa149272ba80a53a05789129e6ee35302cf3a1b
+  data.tar.gz: 5f14e0db2c0a2d3238198600a736790ea549ea11c91bb84578dda2e3de2270c31cb314b44972ec2fbc6635aa77b93f0ce4df2926d45c072745b19a1d6371b34f

data/lib/inferno/apps/web/controllers/test_sessions/create.rb

@@ -3,9 +3,13 @@ module Inferno
     module Controllers
       module TestSessions
         class Create < Controller
-          PARAMS = [:test_suite_id].freeze
+          PARAMS = [:test_suite_id, :suite_options].freeze
+
+          def call(raw_params)
+            query_params = raw_params.to_h
+            body_params = JSON.parse(request.body.string).symbolize_keys
+            params = query_params.merge(body_params)
 
-          def call(params)
             session = repo.create(create_params(params))
 
             repo.apply_preset(session.id, params[:preset_id]) if params[:preset_id].present?
@@ -21,7 +25,7 @@ module Inferno
           end
 
           def create_params(params)
-            params.to_h.slice(*PARAMS)
+            params.slice(*PARAMS)
           end
         end
       end

data/lib/inferno/apps/web/router.rb

@@ -51,6 +51,11 @@ module Inferno
             send(route[:method], path, to: route[:handler])
           end
         end
+
+        Inferno::Repositories::TestSuites.all.map { |suite| "/#{suite.id}" }.each do |suite_path|
+          Application['logger'].info("Registering suite route: #{suite_path}")
+          get suite_path, to: ->(_env) { [200, {}, [client_page]] }
+        end
       end
     end
   end

data/lib/inferno/apps/web/serializers/suite_option.rb

@@ -0,0 +1,13 @@
+module Inferno
+  module Web
+    module Serializers
+      class SuiteOption < Serializer
+        identifier :id
+        field :title, if: :field_present?
+        field :description, if: :field_present?
+        field :list_options, if: :field_present?
+        field :value, if: :field_present?
+      end
+    end
+  end
+end

data/lib/inferno/apps/web/serializers/test_group.rb

@@ -15,8 +15,12 @@ module Inferno
         field :user_runnable?, name: :user_runnable
         field :optional?, name: :optional
 
-        association :groups, name: :test_groups, blueprint: TestGroup
-        association :tests, blueprint: Test
+        field :test_groups do |group, options|
+          TestGroup.render_as_hash(group.groups(options[:suite_options]))
+        end
+        field :tests do |group, options|
+          Test.render_as_hash(group.tests(options[:suite_options]))
+        end
         field :available_inputs, name: :inputs, extractor: HashValueExtractor, blueprint: Input
         field :output_definitions, name: :outputs, extractor: HashValueExtractor
       end

data/lib/inferno/apps/web/serializers/test_session.rb

@@ -1,3 +1,4 @@
+require_relative 'suite_option'
 require_relative 'test_suite'
 
 module Inferno
@@ -7,10 +8,11 @@ module Inferno
         identifier :id
 
         field :test_suite_id
+        association :suite_options, blueprint: SuiteOption
 
-        association :test_suite, blueprint: TestSuite, view: :full
-        # association :test_runs, blueprint: TestRun
-        # association :results, blueprint: Result
+        field :test_suite do |session, _options|
+          TestSuite.render_as_hash(session.test_suite, view: :full, suite_options: session.suite_options)
+        end
       end
     end
   end

data/lib/inferno/apps/web/serializers/test_suite.rb

@@ -11,12 +11,15 @@ module Inferno
           field :input_instructions
           field :test_count
           field :version
+          association :suite_options, blueprint: SuiteOption
           association :presets, view: :summary, blueprint: Preset
         end
 
         view :full do
           include_view :summary
-          association :groups, name: :test_groups, blueprint: TestGroup
+          field :test_groups do |suite, options|
+            TestGroup.render_as_hash(suite.groups(options[:suite_options]))
+          end
           field :configuration_messages
           field :available_inputs, name: :inputs, extractor: HashValueExtractor, blueprint: Input
         end

data/lib/inferno/db/migrations/007_add_suite_options.rb

@@ -0,0 +1,5 @@
+Sequel.migration do
+  change do
+    add_column :test_sessions, :suite_options, String, text: true, size: 255
+  end
+end

data/lib/inferno/db/schema.rb

@@ -9,6 +9,7 @@ Sequel.migration do
       String :test_suite_id, :size=>255, :null=>false
       DateTime :created_at, :null=>false
       DateTime :updated_at, :null=>false
+      String :suite_options, :text=>true
       
       primary_key [:id]
     end

data/lib/inferno/dsl/input_output_handling.rb

@@ -24,10 +24,20 @@ module Inferno
           [identifier, *other_identifiers].compact.each do |input_identifier|
             inputs << input_identifier
             config.add_input(input_identifier)
+            children
+              .reject { |child| child.inputs.include? input_identifier }
+              .each do |child|
+                child.input(input_identifier)
+              end
           end
         else
           inputs << identifier
           config.add_input(identifier, input_params)
+          children
+            .reject { |child| child.inputs.include? identifier }
+            .each do |child|
+              child.input(identifier, **input_params)
+            end
         end
       end
 
@@ -47,10 +57,20 @@ module Inferno
           [identifier, *other_identifiers].compact.each do |output_identifier|
             outputs << output_identifier
             config.add_output(output_identifier)
+            children
+              .reject { |child| child.outputs.include? output_identifier }
+              .each do |child|
+                child.output(output_identifier)
+              end
           end
         else
           outputs << identifier
           config.add_output(identifier, output_definition)
+          children
+            .reject { |child| child.outputs.include? identifier }
+            .each do |child|
+              child.output(identifier, **output_definition)
+            end
         end
       end
 

data/lib/inferno/dsl/runnable.rb

@@ -379,7 +379,7 @@ module Inferno
       end
 
       # @private
-      def test_count(selected_suite_options = {})
+      def test_count(selected_suite_options = [])
         @test_counts ||= {}
 
         @test_counts[selected_suite_options] ||=
@@ -395,17 +395,23 @@ module Inferno
       end
 
       def required_suite_options(suite_option_requirements)
-        @suite_option_requirements = suite_option_requirements
+        @suite_option_requirements =
+          suite_option_requirements.map do |key, value|
+            DSL::SuiteOption.new(id: key, value: value)
+          end
       end
 
-      def children(selected_suite_options = nil)
+      def children(selected_suite_options = [])
         return all_children if selected_suite_options.blank?
 
         all_children.select do |child|
-          requirements = child.suite_option_requirements || {}
+          requirements = child.suite_option_requirements
 
-          # requirements are a subset of selected options or equal to selected options
-          selected_suite_options >= requirements
+          if requirements.blank?
+            true
+          else
+            requirements.all? { |requirement| selected_suite_options.include? requirement }
+          end
         end
       end
     end

data/lib/inferno/dsl/suite_option.rb

@@ -0,0 +1,40 @@
+require_relative '../entities/attributes'
+
+module Inferno
+  module DSL
+    class SuiteOption
+      ATTRIBUTES = [
+        :id,
+        :title,
+        :description,
+        :list_options,
+        :value
+      ].freeze
+
+      include Entities::Attributes
+
+      def initialize(raw_params)
+        params = raw_params.deep_symbolize_keys
+        bad_params = params.keys - ATTRIBUTES
+
+        raise Exceptions::UnknownAttributeException.new(bad_params, self.class) if bad_params.present?
+
+        params
+          .compact
+          .each { |key, value| send("#{key}=", value) }
+
+        self.id = id.to_sym if id.is_a? String
+      end
+
+      def ==(other)
+        id == other.id && value == other.value
+      end
+
+      def to_hash
+        self.class::ATTRIBUTES.each_with_object({}) do |attribute, hash|
+          hash[attribute] = send(attribute)
+        end.compact
+      end
+    end
+  end
+end

data/lib/inferno/entities/test.rb

@@ -145,10 +145,12 @@ module Inferno
         # Define outputs for this Test
         #
         # @param output_definitions [Symbol]
+        # @param _output_params [Hash] Unused parameter. Just makes method
+        #   signature compatible with `Inferno::DSL::InputOutputHandling.output`
         # @return [void]
         # @example
         #   output :patient_id, :bearer_token
-        def output(*output_definitions)
+        def output(*output_definitions, **_output_params)
           super
 
           output_definitions.each do |output|

data/lib/inferno/entities/test_group.rb

@@ -31,12 +31,12 @@ module Inferno
           Inferno::Repositories::TestGroups.new
         end
 
-        def groups
-          all_children.select { |child| child < Inferno::Entities::TestGroup }
+        def groups(options = nil)
+          children(options).select { |child| child < Inferno::Entities::TestGroup }
         end
 
-        def tests
-          all_children.select { |child| child < Inferno::Entities::Test }
+        def tests(options = nil)
+          children(options).select { |child| child < Inferno::Entities::Test }
         end
 
         # Methods to configure Inferno::DSL::Runnable

data/lib/inferno/entities/test_session.rb

@@ -37,6 +37,34 @@ module Inferno
 
       def initialize(params)
         super(params, ATTRIBUTES)
+
+        self.suite_options ||= []
+
+        test_suite.suite_options&.each do |option|
+          if suite_options.none? { |selected_option| selected_option.id == option.id }
+            suite_options << DSL::SuiteOption.new(id: option.id, value: option.list_options.first[:value])
+          end
+        end
+      end
+
+      def test_suite
+        @test_suite ||= Repositories::TestSuites.new.find(test_suite_id)
+      end
+
+      def to_hash
+        session_hash = (self.class::ATTRIBUTES - [:suite_options]).each_with_object({}) do |attribute, hash|
+          hash[attribute] = send(attribute)
+        end
+
+        session_hash[:suite_options] = suite_options&.map(&:to_hash) || []
+
+        session_hash.compact
+      end
+
+      def suite_options_hash
+        (suite_options || []).each_with_object({}) do |option, hash|
+          hash[option.id] = option.value
+        end
       end
     end
   end

data/lib/inferno/entities/test_suite.rb

@@ -1,5 +1,6 @@
 require_relative 'test_group'
 require_relative '../dsl/runnable'
+require_relative '../dsl/suite_option'
 require_relative '../repositories/test_groups'
 require_relative '../repositories/test_suites'
 
@@ -30,8 +31,8 @@ module Inferno
           Inferno::Repositories::TestSuites.new
         end
 
-        def groups
-          all_children.select { |child| child < Inferno::Entities::TestGroup }
+        def groups(options = nil)
+          children(options).select { |child| child < Inferno::Entities::TestGroup }
         end
 
         # Methods to configure Inferno::DSL::Runnable
@@ -86,11 +87,11 @@ module Inferno
         end
 
         def suite_option(identifier, **input_params)
-          suite_options[identifier] = input_params
+          suite_options << DSL::SuiteOption.new(input_params.merge(id: identifier))
         end
 
         def suite_options
-          @suite_options ||= {}
+          @suite_options ||= []
         end
       end
     end