Re-structure nf-test test files

[adamrtalbot]

Changes:

• Move nf-test files to the same directory as the tool (modules/nf-core/tool/subtool/main.nf.test{,.snap})
• Will reduce need for separate test folder
• Automatically imported with module into pipeline by nf-core/tools

Module structure is now:

modules/
├─ tests/
│  ├─ main.nf.test
│  ├─ main.nf.test.snap
├─ main.nf
├─ nextflow.config
├─ meta.yml

PR checklist

relates to #2341

• This comment contains a description of changes (with reason).
• If you've fixed a bug or added code that should be tested, add tests!
• If you've added a new tool - have you followed the module conventions in the contribution docs
• If necessary, include test data in your PR.
• Remove all TODO statements.
• Emit the versions.yml file.
• Follow the naming conventions.
• Follow the parameters requirements.
• Follow the input/output options guidelines.
• Add a resource label
• Use BioConda and BioContainers if possible to fulfil software requirements.
• Ensure that the test works with either Docker / Singularity. Conda CI tests can be quite flaky:
  • PROFILE=docker pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=singularity pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware
  • PROFILE=conda pytest --tag <MODULE> --symlink --keep-workflow-wd --git-aware

[edmundmiller]

I kinda love this.

Worse case it's easy to move back too.

Outta make finding the files really simple. We could add other test files.

I feel like the changes y'all is so simple we'll be able to get rid of it, we just have to wait for the inspiration of how...

[SPPearce]

modules/
├─ tests/
│  ├─ main.nf.test
│  ├─ main.nf.test.snap
├─ main.nf
├─ nextflow.config
├─ meta.yml

Is the nextflow.config only for subworkflows as currently? We don't have configs for modules as far as I'm aware.

[adamrtalbot]

modules/
├─ tests/
│  ├─ main.nf.test
│  ├─ main.nf.test.snap
├─ main.nf
├─ nextflow.config
├─ meta.yml

Is the nextflow.config only for subworkflows as currently? We don't have configs for modules as far as I'm aware.

Yes. For now...nextflow-io/nextflow#4205