inferno_core 0.0.3 → 0.0.7

data/lib/inferno/dsl/request_storage.rb

@@ -38,13 +38,14 @@ module Inferno
 
       # TODO: do a check in the test runner
       def named_request(name)
-        requests.find { |request| request.name == name.to_sym }
+        requests.find { |request| request.name == self.class.config.request_name(name.to_sym) }
       end
 
-      # @api private
+      # @private
       def store_request(direction, name = nil, &block)
         response = block.call
 
+        name = self.class.config.request_name(name)
         request =
           if response.is_a? FHIR::ClientReply
             Entities::Request.from_fhir_client_reply(
@@ -60,40 +61,60 @@ module Inferno
         request
       end
 
-      # @api private
+      # @private
       def load_named_requests
         requests_repo = Inferno::Repositories::Requests.new
         self.class.named_requests_used.map do |request_name|
-          request = requests_repo.find_named_request(test_session_id, request_name)
-          raise StandardError, "Unable to find '#{request_name}' request" if request.nil?
+          request_alias = self.class.config.request_name(request_name)
+          request = requests_repo.find_named_request(test_session_id, request_alias)
+          raise StandardError, "Unable to find '#{request_alias}' request" if request.nil?
 
           requests << request
         end
       end
 
       module ClassMethods
-        # @api private
+        # @private
         def named_requests_made
           @named_requests_made ||= []
         end
 
-        # @api private
+        # @private
         def named_requests_used
           @named_requests_used ||= []
         end
 
         # Specify the named requests made by a test
         #
-        # @param *names [Symbol] one or more Symbols
-        def makes_request(*names)
-          named_requests_made.concat(names)
+        # @param identifiers [Symbol] one or more request identifiers
+        def makes_request(*identifiers)
+          named_requests_made.concat(identifiers).uniq!
+          identifiers.each do |identifier|
+            config.add_request(identifier)
+          end
+        end
+
+        # Specify the name for a request received by a test
+        #
+        # @param identifier [Symbol]
+        def receives_request(identifier)
+          config.add_request(identifier)
+          @incoming_request_name = identifier
+        end
+
+        # @private
+        def incoming_request_name
+          @incoming_request_name
         end
 
         # Specify the named requests used by a test
         #
-        # @param *names [Symbol] one or more Symbols
-        def uses_request(*names)
-          named_requests_used.concat(names)
+        # @param identifiers [Symbol] one or more request identifiers
+        def uses_request(*identifiers)
+          named_requests_used.concat(identifiers).uniq!
+          identifiers.each do |identifier|
+            config.add_request(identifier)
+          end
         end
       end
     end

data/lib/inferno/dsl/results.rb

@@ -49,6 +49,55 @@ module Inferno
       def omit_if(test, message = '')
         raise Exceptions::OmitException, message if test
       end
+
+      # Halt execution of the current test and wait for execution to resume.
+      #
+      # @see Inferno::DSL::Runnable#resume_test_route
+      # @example
+      #   resume_test_route :get, '/launch' do
+      #     request.query_parameters['iss']
+      #   end
+      #
+      #   test do
+      #     input :issuer
+      #     receives_request :launch
+      #
+      #     run do
+      #       wait(
+      #         identifier: issuer,
+      #         message: "Wating to receive a request with an issuer of #{issuer}"
+      #       )
+      #     end
+      #   end
+      # @param identifier [String] An identifier which can uniquely identify
+      #   this test run based on an incoming request. This is necessary so that
+      #   the correct test run can be resumed.
+      # @param message [String]
+      # @param timeout [Integer] Number of seconds to wait for an incoming
+      #   request
+      def wait(identifier:, message: '', timeout: 300)
+        identifier(identifier)
+        wait_timeout(timeout)
+
+        raise Exceptions::WaitException, message
+      end
+
+      def identifier(identifier = nil)
+        @identifier ||= identifier
+      end
+
+      def wait_timeout(timeout = nil)
+        @wait_timeout ||= timeout
+      end
+
+      # Halt execution of the current test. This provided for testing purposes
+      # and should not be used in real tests.
+      #
+      # @param message [String]
+      # @private
+      def cancel(message = '')
+        raise Exceptions::CancelException, message
+      end
     end
   end
 end

data/lib/inferno/dsl/resume_test_route.rb

@@ -0,0 +1,89 @@
+require 'hanami-controller'
+
+module Inferno
+  module DSL
+    # A base class for creating routes to resume test execution upon receiving
+    # an incoming request.
+    # @private
+    # @see Inferno::DSL::Runnable#resume_test_route
+    class ResumeTestRoute
+      include Hanami::Action
+      include Import[
+                requests_repo: 'repositories.requests',
+                results_repo: 'repositories.results',
+                test_runs_repo: 'repositories.test_runs',
+                tests_repo: 'repositories.tests'
+              ]
+
+      def self.call(params)
+        new.call(params)
+      end
+
+      # The incoming request
+      #
+      # @return [Inferno::Entities::Request]
+      def request
+        @request ||= Inferno::Entities::Request.from_rack_env(@params.env)
+      end
+
+      # @private
+      def test_run
+        @test_run ||=
+          test_runs_repo.find_latest_waiting_by_identifier(test_run_identifier)
+      end
+
+      # @private
+      def waiting_result
+        @waiting_result ||= results_repo.find_waiting_result(test_run_id: test_run.id)
+      end
+
+      # @private
+      def update_result
+        results_repo.pass_waiting_result(waiting_result.id)
+      end
+
+      # @private
+      def persist_request
+        requests_repo.create(
+          request.to_hash.merge(
+            test_session_id: test_run.test_session_id,
+            result_id: waiting_result.id,
+            name: test.config.request_name(test.incoming_request_name)
+          )
+        )
+      end
+
+      # @private
+      def redirect_route
+        "/test_sessions/#{test_run.test_session_id}##{waiting_group_id}"
+      end
+
+      # @private
+      def test
+        @test ||= tests_repo.find(waiting_result.test_id)
+      end
+
+      # @private
+      def waiting_group_id
+        test.parent.id
+      end
+
+      # @private
+      def call(_params)
+        if test_run.nil?
+          status(500, "Unable to find test run with identifier '#{test_run_identifier}'.")
+          return
+        end
+
+        test_runs_repo.mark_as_no_longer_waiting(test_run.id)
+
+        update_result
+        persist_request
+
+        Jobs.perform(Jobs::ResumeTestRun, test_run.id)
+
+        redirect_to redirect_route
+      end
+    end
+  end
+end

data/lib/inferno/dsl/runnable.rb

@@ -1,3 +1,7 @@
+require_relative 'configurable'
+require_relative 'resume_test_route'
+require_relative '../utils/markdown_formatter'
+
 module Inferno
   module DSL
     # This module contains the DSL for defining child entities in the test
@@ -5,14 +9,17 @@ module Inferno
     module Runnable
       attr_accessor :parent
 
+      include Inferno::Utils::MarkdownFormatter
+
       # When a class (e.g. TestSuite/TestGroup) uses this module, set it up
       # so that subclassing it works correctly.
       # - add the subclass to the relevant repository when it is created
       # - copy the class instance variables from the superclass
       # - add a hook to the subclass so that its subclasses do the same
-      # @api private
+      # @private
       def self.extended(extending_class)
         super
+        extending_class.extend Configurable
 
         extending_class.define_singleton_method(:inherited) do |subclass|
           copy_instance_variables(subclass)
@@ -34,14 +41,16 @@ module Inferno
       # classes. When inheriting from a Runnable class, these class instance
       # variables need to be copied. Any child Runnable classes will themselves
       # need to be subclassed so that their parent can be updated.
-      # @api private
+      # @private
       def copy_instance_variables(subclass)
         instance_variables.each do |variable|
-          next if [:@id, :@groups, :@tests, :@parent, :@children].include?(variable)
+          next if [:@id, :@groups, :@tests, :@parent, :@children, :@test_count, :@config].include?(variable)
 
           subclass.instance_variable_set(variable, instance_variable_get(variable).dup)
         end
 
+        subclass.config(config)
+
         child_types.each do |child_type|
           new_children = send(child_type).map do |child|
             Class.new(child).tap do |subclass_child|
@@ -54,12 +63,13 @@ module Inferno
         end
       end
 
-      # @api private
+      # @private
       def add_self_to_repository
         repository.insert(self)
       end
 
       # An instance of the repository for the class using this module
+      # @private
       def repository
         nil
       end
@@ -67,7 +77,7 @@ module Inferno
       # This method defines a child entity. Classes using this module should
       # alias the method name they wish to use to define child entities to this
       # method.
-      # @api private
+      # @private
       def define_child(*args, &block)
         hash_args = process_args(args)
 
@@ -87,7 +97,7 @@ module Inferno
         klass
       end
 
-      # @api private
+      # @private
       def process_args(args)
         hash_args =
           if args[0].is_a? Hash
@@ -103,13 +113,13 @@ module Inferno
         hash_args
       end
 
-      # @api private
+      # @private
       def child_metadata(metadata = nil)
         @child_metadata = metadata if metadata
         @child_metadata
       end
 
-      # @api private
+      # @private
       def create_child_class(hash_args)
         superclass_id = hash_args.delete :from
 
@@ -122,18 +132,18 @@ module Inferno
         Class.new(superclass)
       end
 
-      # @api private
+      # @private
       def configure_child_class(klass, hash_args) # rubocop:disable Metrics/CyclomaticComplexity
-        inputs.each do |input_definition|
-          next if klass.inputs.include? input_definition
+        inputs.each do |name|
+          next if klass.inputs.any? { |klass_input_name| klass_input_name == name }
 
-          klass.input input_definition
+          klass.input name
         end
 
-        outputs.each do |output_definition|
-          next if klass.outputs.include? output_definition
+        outputs.each do |output_name|
+          next if klass.outputs.include? output_name
 
-          klass.output output_definition
+          klass.output output_name
         end
 
         new_fhir_client_definitions = klass.instance_variable_get(:@fhir_client_definitions) || {}
@@ -152,8 +162,14 @@ module Inferno
         end
         klass.instance_variable_set(:@http_client_definitions, new_http_client_definitions)
 
+        klass.config(config)
+
         hash_args.each do |key, value|
-          klass.send(key, *value)
+          if value.is_a? Array
+            klass.send(key, *value)
+          else
+            klass.send(key, value)
+          end
         end
 
         klass.children.each do |child_class|
@@ -162,11 +178,15 @@ module Inferno
         end
       end
 
-      # @api private
+      # @private
       def handle_child_definition_block(klass, &block)
         klass.class_eval(&block) if block_given?
       end
 
+      # Set/Get a runnable's id
+      #
+      # @param new_id [String,Symbol]
+      # @return [String,Symbol] the id
       def id(new_id = nil)
         return @id if new_id.nil? && @id.present?
 
@@ -182,53 +202,93 @@ module Inferno
         @id = "#{prefix}#{@base_id}"
       end
 
+      # Set/Get a runnable's title
+      #
+      # @param new_title [String]
+      # @return [String] the title
       def title(new_title = nil)
         return @title if new_title.nil?
 
         @title = new_title
       end
 
+      # Set/Get a runnable's description
+      #
+      # @param new_description [String]
+      # @return [String] the description
       def description(new_description = nil)
         return @description if new_description.nil?
 
-        @description = new_description
+        @description = format_markdown(new_description)
       end
 
       # Define inputs
       #
-      # @param inputs [Symbol]
+      # @param identifier [Symbol] identifier for the input
+      # @param other_identifiers [Symbol] array of symbols if specifying multiple inputs
+      # @param input_definition [Hash] options for input such as type, description, or title
+      # @option input_definition [String] :title Human readable title for input
+      # @option input_definition [String] :description Description for the input
+      # @option input_definition [String] :type text | textarea
+      # @option input_definition [String] :default The default value for the input
+      # @option input_definition [Boolean] :optional Set to true to not require input for test execution
       # @return [void]
       # @example
-      #   input :patient_id, :bearer_token
-      def input(*input_definitions)
-        inputs.concat(input_definitions)
+      #   input :patientid, title: 'Patient ID', description: 'The ID of the patient being searched for',
+      #                     default: 'default_patient_id'
+      # @example
+      #   input :textarea, title: 'Textarea Input Example', type: 'textarea', optional: true
+      def input(identifier, *other_identifiers, **input_definition)
+        if other_identifiers.present?
+          [identifier, *other_identifiers].compact.each do |input_identifier|
+            inputs << input_identifier
+            config.add_input(input_identifier)
+          end
+        else
+          inputs << identifier
+          config.add_input(identifier, input_definition)
+        end
       end
 
       # Define outputs
       #
-      # @param output_definitions [Symbol]
+      # @param output_list [Symbol]
       # @return [void]
       # @example
       #   output :patient_id, :bearer_token
-      def output(*output_definitions)
-        outputs.concat(output_definitions)
+      def output(*output_list)
+        output_list.each do |output_identifier|
+          outputs << output_identifier
+          config.add_output(output_identifier)
+        end
       end
 
-      # @api private
+      # @private
       def default_id
         to_s
       end
 
-      # @api private
+      # @private
       def inputs
         @inputs ||= []
       end
 
-      # @api private
+      # @private
+      def input_definitions
+        config.inputs.slice(*inputs)
+      end
+
+      # @private
+      def output_definitions
+        config.outputs.slice(*outputs)
+      end
+
+      # @private
       def outputs
         @outputs ||= []
       end
 
+      # @private
       def child_types
         return [] if ancestors.include? Inferno::Entities::Test
         return [:groups] if ancestors.include? Inferno::Entities::TestSuite
@@ -236,6 +296,7 @@ module Inferno
         [:groups, :tests]
       end
 
+      # @private
       def children
         @children ||= []
       end
@@ -245,6 +306,100 @@ module Inferno
 
         @validator_url = url
       end
+
+      # @private
+      def suite
+        return self if ancestors.include? Inferno::Entities::TestSuite
+
+        parent.suite
+      end
+
+      # Create a route which will resume a test run when a request is received
+      #
+      # @see Inferno::DSL::Results#wait
+      # @example
+      #   resume_test_route :get, '/launch' do
+      #     request.query_parameters['iss']
+      #   end
+      #
+      #   test do
+      #     input :issuer
+      #     receives_request :launch
+      #
+      #     run do
+      #       wait(
+      #         identifier: issuer,
+      #         message: "Wating to receive a request with an issuer of #{issuer}"
+      #       )
+      #     end
+      #   end
+      #
+      # @param method [Symbol] the HTTP request type (:get, :post, etc.) for the
+      #   incoming request
+      # @param path [String] the path for this request. The route will be served
+      #   with a prefix of `/custom/TEST_SUITE_ID` to prevent path conflicts.
+      #   [Any of the path options available in Hanami
+      #   Router](https://github.com/hanami/router/tree/f41001d4c3ee9e2d2c7bb142f74b43f8e1d3a265#a-beautiful-dsl)
+      #   can be used here.
+      # @yield This method takes a block which must return the identifier
+      #   defined when a test was set to wait for the test run that hit this
+      #   route. The block has access to the `request` method which returns a
+      #   {Inferno::Entities::Request} object with the information for the
+      #   incoming request.
+      def resume_test_route(method, path, &block)
+        route_class = Class.new(ResumeTestRoute) do
+          define_method(:test_run_identifier, &block)
+          define_method(:request_name, -> { options[:name] })
+        end
+
+        route(method, path, route_class)
+      end
+
+      # Create a route to handle a request
+      #
+      # @param method [Symbol] the HTTP request type (:get, :post, etc.) for the
+      #   incoming request. `:all` will accept all HTTP request types.
+      # @param path [String] the path for this request. The route will be served
+      #   with a prefix of `/custom/TEST_SUITE_ID` to prevent path conflicts.
+      #   [Any of the path options available in Hanami
+      #   Router](https://github.com/hanami/router/tree/f41001d4c3ee9e2d2c7bb142f74b43f8e1d3a265#a-beautiful-dsl)
+      #   can be used here.
+      # @param handler [#call] the route handler. This can be any Rack
+      #   compatible object (e.g. a `Proc` object, a [Sinatra
+      #   app](http://sinatrarb.com/)) as described in the [Hanami Router
+      #   documentation.](https://github.com/hanami/router/tree/f41001d4c3ee9e2d2c7bb142f74b43f8e1d3a265#mount-rack-applications)
+      def route(method, path, handler)
+        Inferno.routes << { method: method, path: path, handler: handler, suite: suite }
+      end
+
+      # @private
+      def test_count
+        @test_count ||= children&.reduce(0) { |sum, child| sum + child.test_count } || 0
+      end
+
+      # @private
+      def required_inputs(prior_outputs = [])
+        required_inputs = inputs.select do |input|
+          !input_definitions[input][:optional] && !prior_outputs.include?(input)
+        end
+        required_inputs.map! { |input_identifier| input_definitions[input_identifier][:name] }
+        children_required_inputs = children.flat_map { |child| child.required_inputs(prior_outputs) }
+        prior_outputs.concat(outputs)
+        (required_inputs + children_required_inputs).flatten.uniq
+      end
+
+      # @private
+      def missing_inputs(submitted_inputs)
+        submitted_inputs = [] if submitted_inputs.nil?
+
+        required_inputs.map(&:to_s) - submitted_inputs.map { |input| input[:name] }
+      end
+
+      def user_runnable?
+        @user_runnable ||= parent.nil? ||
+                           !parent.respond_to?(:run_as_group?) ||
+                           (parent.user_runnable? && !parent.run_as_group?)
+      end
     end
   end
 end

data/lib/inferno/dsl.rb

@@ -1,6 +1,5 @@
 require_relative 'dsl/assertions'
 require_relative 'dsl/fhir_client'
-require_relative 'dsl/fhir_manipulation'
 require_relative 'dsl/fhir_validation'
 require_relative 'dsl/http_client'
 require_relative 'dsl/results'
@@ -14,8 +13,7 @@ module Inferno
       FHIRClient,
       HTTPClient,
       Results,
-      FHIRValidation,
-      FHIRManipulation
+      FHIRValidation
     ].freeze
 
     EXTENDABLE_DSL_MODULES = [

data/lib/inferno/entities/header.rb

@@ -2,13 +2,20 @@ module Inferno
   module Entities
     # A `Header` represents an HTTP request/response header
     #
-    # @attr_reader [String] id of the header
-    # @attr_reader [String] request_id index of the HTTP request
-    # @attr_reader [String] name header name
-    # @attr_reader [String] value header value
-    # @attr_reader [String] type request/response
-    # @attr_reader [Time] created_at
-    # @attr_reader [Time] updated_at
+    # @!attribute id
+    #   @return [String] id of the header
+    # @!attribute request_id
+    #   @return [String] index of the HTTP request
+    # @!attribute name
+    #   @return [String] header name
+    # @!attribute value
+    #   @return [String] header value
+    # @!attribute type
+    #   @return [String] request/response
+    # @!attribute created_at
+    #   @return [Time]
+    # @!attribute updated_at
+    #   @return [Time]
     class Header < Entity
       ATTRIBUTES = [:id, :request_id, :name, :type, :value, :created_at, :updated_at].freeze
 

data/lib/inferno/entities/message.rb

@@ -2,12 +2,22 @@ module Inferno
   module Entities
     # A `Message` represents a message generated during a test.
     #
-    # @attr_reader [String] id of the message
-    # @attr_reader [String] index of the message. Used for ordering.
-    # @attr_reader [String] result_id
-    # @attr_reader [Inferno::Entities::Result] result
-    # @attr_reader [String] type
-    # @attr_reader [String] message
+    # @!attribute id
+    #   @return [String] id of the message
+    # @!attribute index
+    #   @return [String] index of the message. Used for ordering.
+    # @!attribute result_id
+    #   @return [String]
+    # @!attribute result
+    #   @return [Inferno::Entities::Result]
+    # @!attribute type
+    #   @return [String]
+    # @!attribute message
+    #   @return [String]
+    # @!attribute created_at
+    #   @return [Time]
+    # @!attribute updated_at
+    #   @return [Time]
     class Message < Entity
       ATTRIBUTES = [:id, :index, :message, :result_id, :result, :type, :created_at, :updated_at].freeze
       TYPES = ['error', 'warning', 'info'].freeze