search

inferno-framework / bulk-data-test-kit

Inferno Test Kit for servers implementing the Bulk Data Access Implementation Guide
Apache License 2.0
0 stars 0 forks source link

Standardize validator env var #25

Closed dehall closed 4 months ago

dehall commented 4 months ago

Summary

Follow-on to #21 . Standardizes the environment variables used for the different test suites, now we can just use the default FHIR_RESOURCE_VALIDATOR_URL and remove the url line from the validator definition block

Testing Guidance

There should be no visible impact to the env var change - sessions should start up and work when running via ruby or via docker

Jammjammjamm commented 4 months ago

This seems like an intermittent failure unrelated to this branch.

dehall commented 4 months ago

I confirmed the issue locally and via validator.fhir.org, I think the error message from tx.fhir.org is breaking the parsing somewhere because of a null character: "Error from http://tx.fhir.org/r4: The attribute name \u0000 is illegal at Line 580, Column 20" (it's not encoded like that in the actual response, it's a true null character)