Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

Incorporate integration test suite and link with CI #3

Open
geetduggal opened this issue Oct 19, 2017 · 6 comments
Open

Incorporate integration test suite and link with CI #3

geetduggal opened this issue Oct 19, 2017 · 6 comments
Milestone

Comments

@geetduggal
Copy link
Contributor

No description provided.

@geetduggal geetduggal added this to the beta-release milestone Oct 19, 2017
@mr-c
Copy link

mr-c commented Oct 20, 2017

@geetduggal
Copy link
Contributor Author

Great thanks! I'm setting up a really basic framework right now integrated with out internal Drone servers and will hopefully be able to integrate the generic runner with our ./dx-cwl run-workflow or something to that affect.

@geetduggal
Copy link
Contributor Author

Basic Drone integration:

image

@geetduggal
Copy link
Contributor Author

@mr-c, thanks for all your assistance and input so far. I am starting work on ensuring there is a CWL runner for DNAnexus and I'd like to iteratively add conformance tests to the mix. I'd like to start with a subset of basic tests for our continuous integration and add more as we flesh out features. This way we can see if a new commit to our repo breaks functionality we know has previously worked.

I am currently looking at this file: https://github.com/common-workflow-language/common-workflow-language/blob/master/v1.0/conformance_test_v1.0.yaml. Would the easiest way to move forward is to create a new file within our dx-cwl repository that we call conformance_test_v1.0_dnanexus_subset.yaml or is there another method to run a subset of CWL conformance tests? Thanks in advance

@geetduggal
Copy link
Contributor Author

Also, as I generate the CWL output object document, I noticed, e.g. https://github.com/common-workflow-language/common-workflow-language/blob/master/v1.0/conformance_test_v1.0.yaml#L43 that a local file location is given. Do the conformance tests assume files are available in the local directory? I can def download them from our cloud assuming they're not too big.

@mr-c
Copy link

mr-c commented Nov 8, 2017

@geetduggal You are quite welcome!

You can call ./run-tests.sh with a particular test number; but it may be easier to use just your subset file. We'll be running the entire suite ourselves.

Well, we ship the conformance tests via Git so we wrote them with local references. The test data isn't very big; feel free to re-write the locations to use any identifier/URI that works for you all. Again, we'll be running them locally, so it would be nice if your cwl-runner equivalent can do the needful with local paths. Note: the line you linked to is on the output side, not the input side.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants