nf-core · drpatelh · Apr 9, 2021 · Mar 29, 2021 · Mar 29, 2021 · Apr 8, 2021
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * Strip values from `nf-core launch` web response which are False and have no default in the schema [[#976](https://github.com/nf-core/tools/issues/976)]
 * Try to fix the fix for the automated sync when we submit too many PRs at once [[#970](https://github.com/nf-core/tools/issues/970)]
 * Added `--conda-name` flag to `nf-core modules create` command to allow sidestepping questionary [[#988](https://github.com/nf-core/tools/issues/988)]
+* Update `modules` commands to use new test tag format `tool/subtool`
 
 ### Template
 

diff --git a/nf_core/module-template/tests/test.yml b/nf_core/module-template/tests/test.yml
@@ -5,7 +5,7 @@
   tags:
     - {{ tool }}
     {%- if subtool %}
-    - {{ tool_name }}
+    - {{ tool }}/{{ subtool }}
     {%- endif %}
   files:
     - path: output/{{ tool }}/test.bam

diff --git a/nf_core/modules/create.py b/nf_core/modules/create.py
@@ -113,7 +113,7 @@ def create(self):
         self.tool_dir = self.tool
 
         if self.subtool:
-            self.tool_name = f"{self.tool}_{self.subtool}"
+            self.tool_name = f"{self.tool}/{self.subtool}"
             self.tool_dir = os.path.join(self.tool, self.subtool)
 
         # Check existance of directories early for fast-fail

diff --git a/nf_core/modules/lint.py b/nf_core/modules/lint.py
@@ -519,6 +519,7 @@ def lint_module_tests(self):
         # Lint the test.yml file
         try:
             with open(self.test_yml, "r") as fh:
+                # TODO: verify that the tags are correct
                 test_yml = yaml.safe_load(fh)
             self.passed.append(("test_yml_exists", "Test `test.yml` exists", self.test_yml))
         except FileNotFoundError:

diff --git a/nf_core/modules/test_yml_builder.py b/nf_core/modules/test_yml_builder.py
@@ -170,7 +170,7 @@ def build_single_test(self, entry_point):
             mod_name_parts = self.module_name.split("/")
             tag_defaults = []
             for idx in range(0, len(mod_name_parts)):
-                tag_defaults.append("_".join(mod_name_parts[: idx + 1]))
+                tag_defaults.append("/".join(mod_name_parts[: idx + 1]))
             tag_defaults.append(entry_point.replace("test_", ""))
             # Remove duplicates
             tag_defaults = list(set(tag_defaults))