Skip to content
This repository was archived by the owner on Jul 24, 2020. It is now read-only.

Provide more test data when running verify_tool.#103

Merged
jmchilton merged 6 commits intogalaxyproject:masterfrom
jmchilton:external_tool_testing
Mar 12, 2018
Merged

Provide more test data when running verify_tool.#103
jmchilton merged 6 commits intogalaxyproject:masterfrom
jmchilton:external_tool_testing

Conversation

@jmchilton
Copy link
Member

@jmchilton jmchilton commented Mar 7, 2018

This is information Planemo generates from Galaxy's XUnit report and inserts back into the result JSON, with this change I think one can simply run planemo test_reports output.json on the result of the driver script here to generate Planemo-style reports for tool test runs.

pip install galaxy-lib
galaxy-tool-test -u http://localhost:8080 -a dccd838ad8f46637e0d7661d849806a1 -k dccd838ad8f46637e0d7661d849806a1 -t inputs_as_json -j output.json
planemo test_reports --test_output output.html --test_output_markdown output.md output.json

Requires some small changes to Planemo's test_reports command contained within galaxyproject/planemo#799.

This is information Planemo generates from Galaxy's XUnit report and inserts back into the result JSON, with this change I think one can simply run ``planemo test_reports result.json`` on the result of the driver script here to generate Planemo-style reports for tool test runs.

```
pip install galaxy-lib
galaxy-tool-test -u http://localhost:8080 -a dccd838ad8f46637e0d7661d849806a1 -k dccd838ad8f46637e0d7661d849806a1 -t inputs_as_json -j output.json
planemo test_reports --test_output output.html --test_output_markdown output.md output.json
```

Requires some small changes to Planemo's test_reports command contained within galaxyproject/planemo#781.
HISTORY.rst Outdated
---------------------


* Add missing markupsafe dependency (thanks to @mbargull).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the acknowledgement, but no need to do it twice -- already done in 18.5.5 😉

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch - thanks.

@jmchilton jmchilton merged commit ee3cef5 into galaxyproject:master Mar 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants