Skip to content

Commit

Permalink
FI-3367 Patch execute rollup (#550)
Browse files Browse the repository at this point in the history
* use result summarizer in execute

* tweak execute help docs
  • Loading branch information
Shaumik-Ashraf authored Oct 28, 2024
1 parent 7f1431a commit 39042af
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 3 additions & 1 deletion lib/inferno/apps/cli/execute.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
require_relative '../../utils/verify_runnable'
require_relative '../../utils/persist_inputs'
require_relative 'execute/console_outputter'
require_relative '../../result_summarizer'

module Inferno
module CLI
Expand Down Expand Up @@ -63,7 +64,8 @@ def run(options)
outputter.print_results(options, results)
outputter.print_end_message(options)

exit(0) if results.all? { |result| result.result == 'pass' }
# TODO: respect customized rollups
exit(0) if Inferno::ResultSummarizer.new(results).summarize == 'pass'

# exit(1) is for Thor failures
# exit(2) is for shell builtin failures
Expand Down
2 changes: 2 additions & 0 deletions lib/inferno/apps/cli/main.rb
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,8 @@ def version
Examples:
(These examples only work from within the inferno_core directory).
`bundle exec inferno execute --suite dev_validator \
--inputs "url:https://hapi.fhir.org/baseR4" \
patient_id:1234321`
Expand Down

0 comments on commit 39042af

Please sign in to comment.