chore: Use assertAll in all tests

nf-core · Oct 13, 2024 · e310487 · e310487
1 parent 55b7bf9
commit e310487
Show file tree

Hide file tree

Showing 4 changed files with 34 additions and 17 deletions.
diff --git a/modules/local/grohmm/parametertuning/tests/main.nf.test b/modules/local/grohmm/parametertuning/tests/main.nf.test
@@ -28,7 +28,10 @@ nextflow_process {
 
         then {
             // FIXME Broken test
-            assert process.failed
+            assertAll(
+                { assert process.failed },
+                { assert snapshot(process.out).match() }
+            )
         }
 
     }

diff --git a/modules/local/grohmm/transcriptcalling/tests/main.nf.test b/modules/local/grohmm/transcriptcalling/tests/main.nf.test
@@ -27,8 +27,10 @@ nextflow_process {
         }
 
         then {
-            // FIXME Broken test
-            assert process.failed
+            assertAll(
+                { assert process.failed },
+                { assert snapshot(process.out).match() }
+            )
         }
 
     }

diff --git a/subworkflows/local/grohmm/tests/main.nf.test b/subworkflows/local/grohmm/tests/main.nf.test
@@ -27,8 +27,10 @@ nextflow_workflow {
         }
 
         then {
-            assert workflow.success
-            assert snapshot(workflow.out).match()
+            assertAll(
+                { assert workflow.success },
+                { assert snapshot(workflow.out).match() }
+            )
         }
 
     }
@@ -55,8 +57,10 @@ nextflow_workflow {
         }
 
         then {
-            assert workflow.success
-            assert snapshot(workflow.out).match()
+            assertAll(
+                { assert workflow.success },
+                { assert snapshot(workflow.out).match() }
+            )
         }
 
     }
@@ -84,8 +88,10 @@ nextflow_workflow {
         }
 
         then {
-            assert workflow.success
-            assert snapshot(workflow.out).match()
+            assertAll(
+                { assert workflow.success },
+                { assert snapshot(workflow.out).match() }
+            )
         }
 
     }

diff --git a/workflows/tests/aligner/bwa.nf.test b/workflows/tests/aligner/bwa.nf.test
@@ -71,10 +71,13 @@ nextflow_pipeline {
         }
 
         then {
-            assert workflow.success
-            assert snapshot(
-                workflow.trace.tasks().size(),
-            ).match()
+            assertAll(
+                { assert workflow.success },
+                { assert snapshot(
+                    workflow.trace.tasks().size(),
+                ).match()
+                }
+            )
         }
     }
 
@@ -91,10 +94,13 @@ nextflow_pipeline {
         }
 
         then {
-            assert workflow.success
-            assert snapshot(
-                workflow.trace.tasks().size(),
-            ).match()
+            assertAll(
+                { assert workflow.success },
+                { assert snapshot(
+                    workflow.trace.tasks().size(),
+                ).match()
+                }
+            )
         }
 
     }