-
Notifications
You must be signed in to change notification settings - Fork 1k
feat(maltextract): add stub block and migrate to topic channel versions #11419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Changes from all commits
8c1cce0
0b2c53a
0ab1c13
2c165bf
3032f4c
5f4cbad
9cd1fbc
e1d5cfb
e2e986f
d8f55ef
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -37,9 +37,36 @@ nextflow_process { | |||||
| assertAll( | ||||||
| { assert process.success }, | ||||||
| { assert snapshot( | ||||||
| process.out.versions, | ||||||
| path("${process.out.results.get(0).get(1)}/ScanSummary.txt") | ||||||
| ).match() }, | ||||||
| path("${process.out.results.get(0).get(1)}/ScanSummary.txt"), | ||||||
| path("${process.out.results.get(0).get(1)}/error.txt").text == "", | ||||||
| path("${process.out.results.get(0).get(1)}/log.txt"), | ||||||
| process.out.versions_maltextract | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This probably needs to be
Suggested change
to pass linting. |
||||||
| ).match() } | ||||||
| ) | ||||||
| } | ||||||
| } | ||||||
|
|
||||||
| test("test_maltextract - stub") { | ||||||
|
|
||||||
| options "-stub" | ||||||
|
|
||||||
| when { | ||||||
| process { | ||||||
| """ | ||||||
| input[0] = [ [], // meta map | ||||||
| file(params.modules_testdata_base_path + 'delete_me/malt/test.rma6') | ||||||
| ] | ||||||
| input[1] = file(params.modules_testdata_base_path + 'genomics/sarscov2/genome/db/maltextract/taxon_list.txt') | ||||||
| input[2] = [] | ||||||
| """ | ||||||
| } | ||||||
| } | ||||||
|
|
||||||
| then { | ||||||
| assertAll( | ||||||
| { assert process.success }, | ||||||
| { assert snapshot(sanitizeOutput(process.out)).match() }, | ||||||
| { assert snapshot(process.out.versions_maltextract).match("versions") } | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don't need this given that
Suggested change
|
||||||
| ) | ||||||
| } | ||||||
| } | ||||||
|
|
||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,15 +1,61 @@ | ||
| { | ||
| "test_maltextract - stub": { | ||
| "content": [ | ||
| { | ||
| "results": [ | ||
| [ | ||
| [], | ||
| [] | ||
| ] | ||
| ], | ||
| "versions_maltextract": [ | ||
| [ | ||
| "MALTEXTRACT", | ||
| "maltextract", | ||
| "1.7" | ||
| ] | ||
| ] | ||
| } | ||
| ], | ||
| "timestamp": "2026-04-29T22:34:21.854841", | ||
| "meta": { | ||
| "nf-test": "0.9.5", | ||
| "nextflow": "25.10.4" | ||
| } | ||
| }, | ||
| "test_maltextract": { | ||
| "content": [ | ||
| "ScanSummary.txt:md5,209fb35426f6a459db0f4af0dc544b10", | ||
| true, | ||
| "log.txt:md5,7d1a25809c8c92b550a43773fc78cfd4", | ||
| [ | ||
| [ | ||
| "MALTEXTRACT", | ||
| "maltextract", | ||
| "1.7" | ||
| ] | ||
| ] | ||
| ], | ||
| "timestamp": "2026-04-29T22:33:46.095165", | ||
| "meta": { | ||
| "nf-test": "0.9.5", | ||
| "nextflow": "25.10.4" | ||
| } | ||
| }, | ||
| "versions": { | ||
| "content": [ | ||
| [ | ||
| "versions.yml:md5,4d87de5def1287321effde6545901cf6" | ||
| ], | ||
| "ScanSummary.txt:md5,209fb35426f6a459db0f4af0dc544b10" | ||
| [ | ||
| "MALTEXTRACT", | ||
| "maltextract", | ||
| "1.7" | ||
| ] | ||
| ] | ||
| ], | ||
| "timestamp": "2026-04-29T22:34:21.854841", | ||
| "meta": { | ||
| "nf-test": "0.8.4", | ||
| "nextflow": "24.04.2" | ||
| }, | ||
| "timestamp": "2024-07-01T08:19:20.696046683" | ||
| "nf-test": "0.9.5", | ||
| "nextflow": "25.10.4" | ||
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is unrelated.