Fix CLI params normalization

Signed-off-by: Ben Sherman <[email protected]>
nextflow-io · Jan 9, 2025 · 57f7e58 · 57f7e58
1 parent 884f745
commit 57f7e58
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy
@@ -685,7 +685,7 @@ class CmdRun extends CmdBase implements HubOptions {
             addParam((Map)nested, key.substring(p+1), value, path, fullKey)
         }
         else {
-            params.put(key.replaceAll(DOT_ESCAPED,'.'), parseParamValue(value))
+            addParam0(params, key.replaceAll(DOT_ESCAPED,'.'), parseParamValue(value))
         }
     }
 

diff --git a/modules/nextflow/src/test/groovy/nextflow/cli/CmdRunTest.groovy b/modules/nextflow/src/test/groovy/nextflow/cli/CmdRunTest.groovy
@@ -99,16 +99,16 @@ class CmdRunTest extends Specification {
 
         when:
         def params = [:]
-        CmdRun.addParam0(params, 'alphaBeta', 1)
-        CmdRun.addParam0(params, 'alpha-beta', 10)
+        CmdRun.addParam(params, 'alphaBeta', '1')
+        CmdRun.addParam(params, 'alpha-beta', '10')
         then:
         params['alphaBeta'] == 10
         !params.containsKey('alpha-beta')
 
         when:
         params = [:]
-        CmdRun.addParam0(params, 'aaa-bbb-ccc', 1)
-        CmdRun.addParam0(params, 'aaaBbbCcc', 10)
+        CmdRun.addParam(params, 'aaa-bbb-ccc', '1')
+        CmdRun.addParam(params, 'aaaBbbCcc', '10')
         then:
         params['aaaBbbCcc'] == 10
         !params.containsKey('aaa-bbb-ccc')
-Original file line number
+Diff line change
@@ Expand Up / @@ -685,7 +685,7 @@ class CmdRun extends CmdBase implements HubOptions { @@
                 addParam((Map)nested, key.substring(p+1), value, path, fullKey)
             }
             else {
-                params.put(key.replaceAll(DOT_ESCAPED,'.'), parseParamValue(value))
+                addParam0(params, key.replaceAll(DOT_ESCAPED,'.'), parseParamValue(value))
             }
         }
@@ Expand Down @@