WIP: Testing for training assembly intro workflow #775
WIP: Testing for training assembly intro workflow #775jmchilton wants to merge 2 commits intogalaxyproject:masterfrom
Conversation
Leverage ephemeris to build list of repositories needed for the workflow. xref http://galaxyproject.github.io/training-material/topics/assembly/tutorials/general-introduction/tutorial.html
1dcc92f to
f36fab2
Compare
|
@jmchilton can you try to use revison 7db5615f4621 from simons tool? We migrated it to IUC and updated it last year - not in the workflow as it seems. I guess a different workflow is easier, I will try to spin a very simple one up with text-processing for testing. Thanks! |
|
Hi, here is a very simple workflow with only text-processing and bedtools. This is used in one of the Galaxy introduction trainings on GTN. Example history with input data here: https://galaxy.bi.uni-freiburg.de/u/bgruening/h/for-john |
|
@bgruening Thanks for the workflow. Unfortunately this one also has short path problems. Planemo clearly needs some tooling for upgrading workflows - I'll create an issue for that. This workflow also used Grouping1 - which I need to think about how to incorporate into workflow testing - I guess Planemo should keep a mapping of non-toolshed IDs to tool paths to load as needed for workflows. |
|
@jmchilton can you please try again. I fixed the conda package hopefully. |
|
Tried locally and on Travis and it stills seems to be a problem: The requirement at that version seems to be 8.25. https://toolshed.g2.bx.psu.edu/repository?repository_id=2593fd36ae8011aa&changeset_revision=20344ce0c811 Here is the PR to update the build - https://github.com/bioconda/bioconda-recipes/pull/7999/files. I'm not sure what is going on here - maybe the error message is deceptive and there is some other issue |
|
I guess it could be any of the dependencies for any of the tools in the repository - there is more than just coreutils here I think. We maybe need to make this error message more clear. |
|
|
||
| @skip_if_environ("PLANEMO_SKIP_GALAXY_TESTS") | ||
| def test_workflow_test_assembly_intro_example(self): | ||
| """Test testing a simple GA workflow with Galaxy.""" |
There was a problem hiding this comment.
This test description should be tailored for this test.
|
Galaxy-Workflow-Cell_paper_tutorial_workflow.ga.txt @jmchilton hopefully this is better. |
|
@bgruening I haven't tried the new workflow yet but I did get my modified version of the last one working by adding a new option Unfortunately all these new tests are very lengthy to run and error prone. Planemo's test suite has grown unstable I think - I'm thinking maybe I'll create a new repository just for workflow test examples. We can place some of these simple tests there and then add more complex tests as well if we find them. This could also serve as a cleaner example of how to structure a .travis.yml file for planemo workflow testing with postgres and such. |
|
@jmchilton that is a good idea! We need for the training project a separate testing repo as well. Secrets are stored in travis ENVs and maintained by us. To the Maybe we can run before all these tests the planemo workflow test in the same repo to test planemo at first? |
|
Closing in lieu of trying to get #777 working |
Update ephemeris usage for a modern version to work with both .ga and .yml workflows.
Had to modify the workflow slightly to define a workflow output for testing.
xref http://galaxyproject.github.io/training-material/topics/assembly/tutorials/general-introduction/tutorial.html
This seems to be falling apart on a Conda problem Planemo can't really solve? Anyone want to trigger a rebuild of that with a newer Conda for me?
Maybe I'll try again with a different workflow?