-
Notifications
You must be signed in to change notification settings - Fork 743
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
try porting mmseqs/search to nf-test #7231
Conversation
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.
Hopefully this will make the tests run!
Co-authored-by: Jim Downie <[email protected]>
Now I still get: │ Test [1f546857] 'sarscov2 - test-search' FAILED (6.297s) │
│ │
│ Assertion failed: │
│ │
│ 2 of 2 assertions failed │
│ │
│ Nextflow stdout: │
│ │
│ ERROR ~ Error executing process > 'MMSEQS_CREATEDB_QUERY (test_query)' │
│ │
│ Caused by: │
│ Process `MMSEQS_CREATEDB_QUERY (test_query)` terminated with an error exit status (132) │
│ │
│ │
│ Command executed: │
│ │
│ if [ "false" == "true" ]; then │
│ gzip -c -d contigs.fasta > contigs.fasta │
│ fi │
│ │
│ mkdir -p test_query │
│ │
│ mmseqs \ │
│ createdb \ │
│ contigs.fasta \ │
│ test_query/test_query \ │
│ \ │
│ --compressed 1 │
│ │
│ cat <<-END_VERSIONS > versions.yml │
│ "MMSEQS_CREATEDB_QUERY": │
│ mmseqs: $(mmseqs | grep 'Version' | sed 's/MMseqs2 Version: //') │
│ END_VERSIONS │
│ │
│ Command exit status: │
│ 132 │
│ │
│ Command output: │
│ (empty) │
│ │
│ Command error: │
│ .command.sh: line 13: 35 Illegal instruction mmseqs createdb contigs.fasta test_query/test_query --compressed 1 │
│ │
│ Work dir: │
│ /Users/famke/02-nf-core/modules/.nf-test/tests/1f5468570864d0273649d793a99b787b/work/cc/491340106cd3202f039939bdafd8fa │
│ │
│ Container: │
│ quay.io/biocontainers/mmseqs2:16.747c6--pl5321h6a68c12_0 │
│ │
│ Tip: you can try to figure out what's wrong by changing to the process work dir and showing the script file named `.command.sh` │
│ │
│ -- Check '/Users/famke/02-nf-core/modules/.nf-test/tests/1f5468570864d0273649d793a99b787b/meta/nextflow.log' file for details │
│ Nextflow stderr: |
@famosab I think I will take some of your initial work here into the removing teh compress module PR from Jasmin, as I guess the snapshot might change due to the removal of teh compression |
Just close it if its obsolete :) |
I was finishing this off here before James started so I finished it off (letting him know). |
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.
I don't see anything problematic, maybe just a comment why just checking for file names
@SPPearce I'm going to force merge this as your tests are passing but easysearch is fixed in the other PR |
PR checklist
Closes #XXX
Related to #7211
versions.yml
file.label
nf-core modules test <MODULE> --profile docker
nf-core modules test <MODULE> --profile singularity
nf-core modules test <MODULE> --profile conda
nf-core subworkflows test <SUBWORKFLOW> --profile docker
nf-core subworkflows test <SUBWORKFLOW> --profile singularity
nf-core subworkflows test <SUBWORKFLOW> --profile conda
I tried porting but I always run into the following error: