profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/Jammjammjamm/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.

alex-barstow/group-project-fall-2016 3

see README for details

Jammjammjamm/health-cards 0

Health Cards Framework: implementation guide and supporting material

Jammjammjamm/inferno 0

Source code for the Inferno Testing Tool

Jammjammjamm/smart-app-launch 0

SMART on FHIR App Launch Protocol

Jammjammjamm/smart-dev-sandbox 0

Docker based sandbox for smart apps

Jammjammjamm/smart-launcher 0

Launcher for SMART apps

Jammjammjamm/treebank_data 0

Perseus Treebank Data

projecttacoma/deqm-test-kit 0

A set of tests for DEQM's operations and resources

PullRequestReviewEvent

pull request commentinferno-community/inferno-core

FI-1310 Lock input

I would expect locking an input to just prevent the value from being changed by the user. It could be locked to default value or a value set by a previous test. We use this in program for things like preventing the user from changing the access token received during the launch.

Screen Shot 2021-09-15 at 12 23 09 PM

I haven't looked too closely yet, but the changes in this PR should be enough to allow us to set an input as locked on an imported test.

Jack-Fraser16

comment created time in a day

PullRequestReviewEvent

Pull request review commentinferno-community/inferno-core

Auto-create a session if only one suite.

              primary_key [:id]       +      index [:test_session_id, :test_group_id], :name=>:results_session_group

These need to be added via a migration.

arscan

comment created time in a day

PullRequestReviewEvent

push eventprojecttacoma/deqm-test-kit

Stephen MacVicar

commit sha ff224850dbed86e0e25ba01ee343e3ec589ba559

fix linting error

view details

push time in 2 days

Pull request review commentprojecttacoma/deqm-test-kit

Update inferno core

 services:     # Update this path to match your directory structure     volumes:       - ./lib/deqm_test_kit/igs:/home/igs+  fhir_validator_app:

If it's not useful to have the standalone validator app, you can remove this. If it is useful, you can find it at http://localhost/validator.

Jammjammjamm

comment created time in 2 days

PullRequestReviewEvent

PR opened projecttacoma/deqm-test-kit

Update inferno core

Summary

Update inferno_core to 0.0.5.

New behavior

Tests now run asynchronously by default. Improvements to input handling in the UI.

Inputs have more configuration options now:

input :url,
  title: 'URL',
  description: 'Insert url of FHIR server',
  default: 'https://inferno.healthit.gov/reference-server/r4'

Code changes

Testing guidance

Make sure everything still works

+227 -49

0 comment

14 changed files

pr created time in 2 days

create barnchprojecttacoma/deqm-test-kit

branch : update-inferno-core

created branch time in 2 days

Pull request review commentinferno-community/inferno-core

Fi 1311 add input require optional

 def route(method, path, handler)       def test_count         @test_count ||= children&.reduce(0) { |sum, child| sum + child.test_count } || 0       end++      def contained_required_inputs(prior_outputs = [])

This name confuses me. I think this method should be required_inputs. I'd also remove get_ from get_missing_inputs, as I don't think it really adds anything.

czh-orz

comment created time in 3 days

PullRequestReviewEvent

Pull request review commentinferno-community/inferno-core

Fi 1311 add input require optional

 def route(method, path, handler)       def test_count         @test_count ||= children&.reduce(0) { |sum, child| sum + child.test_count } || 0       end++      def contained_required_inputs(prior_outputs = [])+        required_inputs = inputs.select do |input|+          !input_definitions[input][:optional] && !prior_outputs.include?(input)+        end+        children_required_inputs = children.map { |child| child.contained_required_inputs(prior_outputs) }.flatten
        children_required_inputs = children.flat_map { |child| child.contained_required_inputs(prior_outputs) }
czh-orz

comment created time in 3 days

PullRequestReviewEvent

delete branch inferno-community/inferno-template

delete branch : add-run-scripts

delete time in 6 days

push eventinferno-community/inferno-template

Stephen MacVicar

commit sha 5a9a52558a9d7b5edb401d51d8100500d5b322f5

add scripts and update readme (#5)

view details

push time in 6 days

PR merged inferno-community/inferno-template

Add scripts and update readme

Add setup and run scripts to simplify running inferno.

+10 -5

2 comments

3 changed files

Jammjammjamm

pr closed time in 6 days

pull request commentinferno-community/inferno-template

Add scripts and update readme

.sh are somewhat platform specific (not Windows, except WSL). I suspect many of our users will be windows. Do we want to try to support non-wsl windows or just point people to wsl always? I just bring it up in case this is the first platform-specific stuff we are introducing.

I don't have a good answer right now for what our windows support strategy is, but I think we are unlikely to want to support non-wsl windows.

Jammjammjamm

comment created time in 6 days

PR opened inferno-community/inferno-template

Reviewers
Add scripts and update readme

Add setup and run scripts to simplify running inferno.

+10 -5

0 comment

3 changed files

pr created time in 6 days

create barnchinferno-community/inferno-template

branch : add-run-scripts

created branch time in 6 days

delete branch inferno-community/inferno-template

delete branch : add-validator

delete time in 6 days

push eventinferno-community/inferno-template

Jack Fraser

commit sha 3e8d3640657b14b80cbd98f8449e63c0364a4b90

FI-1285: Add validator app (#4) * Adding nginx.conf file to Config folder, altered the docker-compose.yml, and updated the README to reflect that the validator app can now be accessed from within the template repo. * Update README.md

view details

push time in 6 days

PR merged inferno-community/inferno-template

FI-1285: Add validator app

Adding nginx.conf file to Config folder, altered the docker-compose.yml, and updated the README to reflect that the validator app can now be accessed from within the template repo.

+119 -3

0 comment

3 changed files

Jack-Fraser16

pr closed time in 6 days

PullRequestReviewEvent

create barnchonc-healthit/inferno-program

branch : fix-pg-gemfile

created branch time in 6 days

PullRequestReviewEvent

Pull request review commentonc-healthit/inferno-program

FI-1316 Call bundle.next_bundle to get next page of search result

 def fetch_all_bundled_resources(reply, reply_handler = nil)           resources += bundle&.entry&.map { |entry| entry&.resource }           next_bundle_link = bundle&.link&.find { |link| link.relation == 'next' }&.url           reply_handler&.call(reply)+           break if next_bundle_link.blank?            reply = @client.raw_read_url(next_bundle_link)           error_message = "Could not resolve next bundle. #{next_bundle_link}"           assert_response_ok(reply, error_message)-          assert_valid_json(reply.body, error_message)

So now a server would pass if the next link in a bundle didn't return a body?

yunwwang

comment created time in 7 days

PullRequestReviewEvent
PullRequestReviewEvent