inferno_core 0.6.11 → 0.6.13

data/lib/inferno/dsl/runnable.rb

@@ -39,6 +39,7 @@ module Inferno
       # @private
       VARIABLES_NOT_TO_COPY = [
         :@id, # New runnable will have a different id
+        :@database_id, # New runnable will have a different database_id
         :@parent, # New runnable unlikely to have the same parent
         :@all_children, # New subclasses have to be made for each child
         :@test_count, # Needs to be recalculated
@@ -164,22 +165,6 @@ module Inferno
           klass.output output_name
         end
 
-        new_fhir_client_definitions = klass.instance_variable_get(:@fhir_client_definitions) || {}
-        fhir_client_definitions.each do |name, definition|
-          next if new_fhir_client_definitions.include? name
-
-          new_fhir_client_definitions[name] = definition.dup
-        end
-        klass.instance_variable_set(:@fhir_client_definitions, new_fhir_client_definitions)
-
-        new_http_client_definitions = klass.instance_variable_get(:@http_client_definitions) || {}
-        http_client_definitions.each do |name, definition|
-          next if new_http_client_definitions.include? name
-
-          new_http_client_definitions[name] = definition.dup
-        end
-        klass.instance_variable_set(:@http_client_definitions, new_http_client_definitions)
-
         klass.config(config)
 
         klass.all_children.select!(&:required?) if hash_args.delete(:exclude_optional)
@@ -210,20 +195,20 @@ module Inferno
       def id(new_id = nil)
         return @id if new_id.nil? && @id.present?
 
-        prefix =
-          if parent
-            "#{parent.id}-"
-          else
-            ''
-          end
-
+        prefix = parent ? "#{parent.id}-" : ''
         @base_id = new_id || @base_id || default_id
+        @id = "#{prefix}#{@base_id}"
 
-        final_id = "#{prefix}#{@base_id}"
+        if @id.length > 255
+          hash = Digest::SHA1.hexdigest(@id)[0...10]
+          @database_id = "#{@id[0...244]}-#{hash}"
+        end
 
-        raise Exceptions::InvalidRunnableIdException, final_id if final_id.length > 255
+        @id
+      end
 
-        @id = final_id
+      def database_id
+        @database_id ||= id
       end
 
       # Set/Get a runnable's title
@@ -571,10 +556,10 @@ module Inferno
       end
 
       # @private
-      def all_requirements(suite_options = [])
-        children(suite_options).flat_map do |child|
-          child.verifies_requirements + child.all_requirements(suite_options)
-        end
+      def all_verified_requirements(suite_options = [])
+        verifies_requirements + children(suite_options).flat_map do |child|
+          child.all_verified_requirements(suite_options)
+        end.uniq
       end
 
       # @private

data/lib/inferno/dsl/suite_requirements.rb

@@ -9,33 +9,33 @@ module Inferno
       # @return [Array<Inferno::DSL::RequirementSet>]
       #
       # @example
-      # class Suite < Inferno::TestSuite
-      #   requirement_sets(
-      #     {
-      #       identifier: 'example-regulation-1',
-      #       title: 'Example Regulation 1',
-      #       actor: 'Provider' # Only include requirements for the 'Provider'
-      #                         # actor
-      #     },
-      #     {
-      #       identifier: 'example-ig-1',
-      #       title: 'Example Implementation Guide 1',
-      #       actor: 'Provider',
-      #       requirements: '2, 4-5' # Only include these specific requirements
-      #     },
-      #     {
-      #       identifier: 'example-ig-2',
-      #       title: 'Example Implementation Guide 2',
-      #       requirements: 'Referenced', # Only include requirements from this
-      #                                   # set that are referenced by other
-      #                                   # included requirements
-      #       actor: 'Server',
-      #       suite_options: {      # Only include these requirements if the ig
-      #         ig_version: '3.0.0' # version 3.0.0 suite option is selected
+      #   class Suite < Inferno::TestSuite
+      #     requirement_sets(
+      #       {
+      #         identifier: 'example-regulation-1',
+      #         title: 'Example Regulation 1',
+      #         actor: 'Provider' # Only include requirements for the 'Provider'
+      #                           # actor
+      #       },
+      #       {
+      #         identifier: 'example-ig-1',
+      #         title: 'Example Implementation Guide 1',
+      #         actor: 'Provider',
+      #         requirements: '2, 4-5' # Only include these specific requirements
+      #       },
+      #       {
+      #         identifier: 'example-ig-2',
+      #         title: 'Example Implementation Guide 2',
+      #         requirements: 'Referenced', # Only include requirements from this
+      #                                     # set that are referenced by other
+      #                                     # included requirements
+      #         actor: 'Server',
+      #         suite_options: {      # Only include these requirements if the ig
+      #           ig_version: '3.0.0' # version 3.0.0 suite option is selected
+      #         }
       #       }
-      #     }
-      #   )
-      # end
+      #     )
+      #   end
       def requirement_sets(*sets)
         @requirement_sets = sets.map { |set| RequirementSet.new(**set) } if sets.present?
 

data/lib/inferno/entities/requirement.rb

@@ -11,7 +11,7 @@ module Inferno
         :url,
         :requirement,
         :conformance,
-        :actor,
+        :actors,
         :sub_requirements,
         :conditionality,
         :not_tested_reason,
@@ -23,7 +23,9 @@ module Inferno
       def initialize(params)
         super(params, ATTRIBUTES)
 
-        self.requirement_set = id.split('@').first if requirement_set.blank? && id&.include?('@')
+        return unless requirement_set.blank? && (id&.include?('@') || id&.include?('#'))
+
+        self.requirement_set = id.split(/[@#]/).first
       end
 
       # Expand a comma-delimited list of requirement id references into an Array
@@ -35,12 +37,14 @@ module Inferno
       #   used if none is included in the `requirement_id_string`
       #
       # @example
-      # expand_requirement_ids('example-ig@1,3,5-7')
-      # # => ['example-ig@1','example-ig@3','example-ig@5','example-ig@6','example-ig@7']
-      # expand_requirement_ids('example-ig')
-      # # => []
-      # expand_requirement_ids('1,3,5-7', 'example-ig')
-      # # => ['example-ig@1','example-ig@3','example-ig@5','example-ig@6','example-ig@7']
+      #   expand_requirement_ids('example-ig@1,3,5-7')
+      #   # => ['example-ig@1','example-ig@3','example-ig@5','example-ig@6','example-ig@7']
+      #   expand_requirement_ids('example-ig')
+      #   # => []
+      #   expand_requirement_ids('1,3,5-7', 'example-ig')
+      #   # => ['example-ig@1','example-ig@3','example-ig@5','example-ig@6','example-ig@7']
+      #   expand_requirement_ids('example-ig#actor1')
+      #   # => [all requirements for actor1 from example-ig]
       def self.expand_requirement_ids(requirement_id_string, default_set = nil) # rubocop:disable Metrics/CyclomaticComplexity
         return [] if requirement_id_string.blank?
 
@@ -49,10 +53,16 @@ module Inferno
           .split(',')
           .map(&:strip)
           .flat_map do |requirement_string|
-            current_set, requirement_string = requirement_string.split('@') if requirement_string.include?('@')
+            if requirement_string.include? '@'
+              current_set, requirement_string = requirement_string.split('@')
+            elsif requirement_string.include? '#'
+              current_set, actor = requirement_string.split('#')
+            end
 
             requirement_ids =
-              if requirement_string.include? '-'
+              if actor.present?
+                return Repositories::Requirements.new.requirements_for_actor(current_set, actor).map(&:id)
+              elsif requirement_string.include? '-'
                 start_id, end_id = requirement_string.split('-')
                 if start_id.match?(/^\d+$/) && end_id.match?(/^\d+$/)
                   (start_id..end_id).to_a
@@ -67,6 +77,10 @@ module Inferno
           end
       end
 
+      def actor?(actor_to_check)
+        actors.any? { |actor| actor.casecmp? actor_to_check }
+      end
+
       def tested?
         not_tested_reason.blank?
       end

data/lib/inferno/entities/result.rb

@@ -71,6 +71,45 @@ module Inferno
       def waiting?
         result == 'wait'
       end
+
+      def inputs
+        input_json.present? ? JSON.parse(input_json) : []
+      end
+
+      def outputs
+        output_json.present? ? JSON.parse(output_json) : []
+      end
+
+      # Flags large inputs or outputs and replaces their values with a reference message.
+      #
+      # This method inspects either the `inputs` or `outputs` array and,
+      # for each item whose `value` exceeds the configured size threshold, sets `is_large: true`
+      # and replaces the `value` with a message pointing to the full content endpoint.
+      #
+      # @param io_type [String] Must be either `'inputs'` or `'outputs'`.
+      # @return [Array<Hash>] The mutated list of inputs or outputs.
+      def handle_large_io(io_type)
+        io_array = public_send(io_type)
+
+        io_array.each do |io|
+          next unless io_is_large?(io['value'])
+
+          io['is_large'] = true
+          io['value'] = <<~MESSAGE
+            #{io_type.singularize.capitalize} is too large to display, please visit
+            #{Inferno::Application['base_url']}/api/test_sessions/#{test_session_id}/results/#{id}/io/#{io_type}/#{io['name']}
+            for details
+          MESSAGE
+        end
+
+        io_array
+      end
+
+      # @private
+      def io_is_large?(io_value)
+        size_in_char = io_value.is_a?(String) ? io_value.length : io_value.to_json.length
+        size_in_char > ENV.fetch('MAX_IO_DISPLAY_CHAR', 10000).to_i
+      end
     end
   end
 end

data/lib/inferno/entities/test_kit.rb

@@ -149,6 +149,10 @@ module Inferno
           @options = suites.each_with_object({}) { |suite, hash| hash[suite.id] = suite.suite_options }
         end
 
+        def contains_test_suite?(test_suite_id)
+          suite_ids.map(&:to_sym).include? test_suite_id.to_sym
+        end
+
         # @private
         def add_self_to_repository
           repository.insert(self)