Conversation
adding EDAM description
| assertAll( | ||
| { assert process.success }, | ||
| { assert snapshot(process.out.versions).match() } | ||
| { assert snapshot(process.out.versions_ctree).match() } |
There was a problem hiding this comment.
| { assert snapshot(process.out.versions_ctree).match() } | |
| { assert snapshot(path(process.out.versions[0]).yaml).match() } |
with that you get the version actually printed, which should make it better to track for changes.
There was a problem hiding this comment.
Additionallly, how come you don't check anything else from process.out?
There was a problem hiding this comment.
hi @mashehu! I updated those lines as you suggested. also, I added some tests (to test more outputs of the module) and I noticed a small bug which I fixed
There was a problem hiding this comment.
I have just realised that the linting is not working due to assert snapshot(path(process.out.versions[0]).yaml).match(). I modified what you suggested writing versions_ctree instead of versions since that's the emit name in the main.nf and the test works, but the linting fails. Do you have any suggestion?
PR checklist
Closes #10334
Updating versions.yml output to solve issues and update to versions topic.
topic: versions- See version_topicslabelnf-core modules test <MODULE> --profile dockernf-core modules test <MODULE> --profile singularitynf-core modules test <MODULE> --profile condanf-core subworkflows test <SUBWORKFLOW> --profile dockernf-core subworkflows test <SUBWORKFLOW> --profile singularitynf-core subworkflows test <SUBWORKFLOW> --profile conda