nf-test for pilon #5293
nf-test for pilon #5293
Conversation
|
I think
|
| tag "pilon" | ||
|
|
||
|
|
||
| test("Test PILON") { |
There was a problem hiding this comment.
Change test name to something more specific, there is no standard (yet) but something like pilon - bam - paired should do
| ) | ||
| } | ||
|
|
||
| } |
There was a problem hiding this comment.
I see that the module has multiple modes, is it possible to add tests for the others? And a stub test as well, although I see the module currently lacks a stub
| process { | ||
| """ | ||
| input[0] = [ [ id:'test', single_end:false ], // meta map | ||
| file(params.test_data['homo_sapiens']['genome']['genome_fasta'], checkIfExists: true) |
There was a problem hiding this comment.
Apparently the new guideline is to use URLs instead of the test_data object, so please change the tests accordingly
There was a problem hiding this comment.
Hi, I just started so can you point to the guidelines? I found some info (but not technical details) here?
There was a problem hiding this comment.
Refer to #5281 for an example, I believe it's not documented yet, but the documentation team is working on it
There was a problem hiding this comment.
Changing to the test-data path is not required currently, just optional.
|
Please add a stub and test for the stub as mentioned by @itrujnara |
replaced pytest with nf-test for pilon