-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix TES compliance tests #458
Conversation
# openapi-test-runner repo provides a default entry point script, but since TES uses basic auth it needs to be replaced to add approprite credentials | ||
# https://github.com/elixir-cloud-aai/openapi-test-runner | ||
openapi-test-runner report --server http://$tesuser:$tespassword@$teshostname/ --tag all --version 1.0.0 --output_path results |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i asume we are still on version tes 1.0.0, is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great point, yes, and I wonder if we could loop through each TES version (1.0.0, 1.1.0), run the openapi-test-runner
, and update the status of a badge on the readme that says "✔️ GA4GH TES 1.0.0", "🏗️ GA4GH TES 1.1.0"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we certainly can - but this is a bit more of a lift. Great opportunity for someone to pick this up and learn how these tests work (so I am not the only one with this knowledge). I'll make an issue to track this and happy to help if any road blocks are hit.
ever since the tes compliance test repo was renamed, the tes tests have not been working due to this change. This PR will fix that.
issues related to this:
#457
#397
#393