I think it is fine to just merge this (and add the PR number as a second reference in the changelog entry)

nf-core/modules#2847 broke some tests 💀

why would the rename break the tests? Is something not updated?

We install the subworkflow to an old version, as the module in this subworkflow has the old name when it tries to install it from nf-core/modules it's not found.
I think the module should also be installed to the old version but I'm not sure about how file renaming works when we clone the modules repo.

:shacking-fist: @maxulysse
So, updating the subworkflow should fix this?

It might actually be a bug on tools 🤔
We need the outdated subworkflow for this test as we check a change of module, this one is the only subworkflow that I could find with this change :(
Will try to find where are things breaking.

I actually corrected the subworkflow name

nf-core/modules#2847 broke some tests skull

How come? all my tests where ✔️

I think the problem is that we check if the module is available before installing it, and to do this we get the list of available modules from the last nf-core/modules repo version.
If I'm not wrong, this only affects name changes like this one when move from TOOL to TOOL/SUBTOOL, and only when modules are automatically installed from a subworkflow, not when installing them by command line.

Codecov Report

Merging #2177 (cab29c8) into dev (b728841) will increase coverage by 0.04%.
The diff coverage is 84.00%.

@@            Coverage Diff             @@
##              dev    #2177      +/-   ##
==========================================
+ Coverage   71.51%   71.55%   +0.04%     
==========================================
  Files          77       77              
  Lines        8338     8347       +9     
==========================================
+ Hits         5963     5973      +10     
+ Misses       2375     2374       -1     

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.