ADD CATPACK_DOWNLOAD

[jfy133]

A real test is not possible because they download >10GBs databases (NCBI Nr or GTDB)

closes #5586

PR checklist

Closes #XXX

• 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:
  • For modules:
    • nf-core modules test <MODULE> --profile docker
    • nf-core modules test <MODULE> --profile singularity
    • nf-core modules test <MODULE> --profile conda
  • For subworkflows:
    • nf-core subworkflows test <SUBWORKFLOW> --profile docker
    • nf-core subworkflows test <SUBWORKFLOW> --profile singularity
    • nf-core subworkflows test <SUBWORKFLOW> --profile conda

[jasmezz]

You want a review already, right? (Not sure because wrong channel)
Snapshots not passing yet 👀

[nvnieuwk]

One small thing, otherwise LGTM (I'll approve when that thing has been looked at :))

[jasmezz]

Please just update the broken link, otherwise LGTM!

[jfy133]

Please just update the broken link, otherwise LGTM!

What the poop?! I did fix it! Well have to look again...

[jfy133]

Fixed the thing @nvnieuwk I'll merge once tests pass as you said it's OK once I've done that and Jasmin has already given me a ✔️

@jasmezz I did update the link -> there is not 'format' for directory so I use data_1049 as this is called 'directory name' which is teh closest I could find.

The original link didn't work as I forgot to up the number ID part

[nvnieuwk]

Tests will probably fail otherwise

[jasmezz]

Ok I am fine with it if you are fine with it @jfy133 – I don't fully understand but the number ID wasn't updated in line 45, it still links to http://edamontology.org/data_3752 which doesn't exist?

[jfy133]

Ok I am fine with it if you are fine with it @jfy133 – I don't fully understand but the number ID wasn't updated in line 45, it still links to http://edamontology.org/data_3752 which doesn't exist?

I updated the wrong line 😮‍💨