Fix inspect command

Signed-off-by: Ben Sherman <[email protected]>
nextflow-io · Sep 15, 2023 · 5130c33 · 5130c33
1 parent 78cd98b
commit 5130c33
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 28 deletions.
diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/CmdInspect.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/CmdInspect.groovy
@@ -108,7 +108,18 @@ class CmdInspect {
 
         @Override
         void run() {
-            new CmdInspect(this).run()
+            final opts = new CmdRun.V1()
+            opts.launcher = launcher
+            opts.ansiLog = false
+            opts.preview = true
+            opts.args = args
+            opts.params = params
+            opts.paramsFile = paramsFile
+            opts.profile = profile
+            opts.revision = revision
+            opts.runConfig = runConfig
+
+            new CmdInspect(this).run(opts)
         }
 
     }
@@ -123,21 +134,10 @@ class CmdInspect {
     /* For testing purposes only */
     CmdInspect() {}
 
-    void run() {
+    void run(CmdRun.Options opts) {
         // configure quiet mode
         LoggerHelper.setQuiet(true)
         // setup the target run command
-        final opts = new CmdRun.V1()
-        opts.launcherCli = this.launcherCli
-        opts.launcherOptions = this.launcherOptions
-        opts.args = this.args
-        opts.profile = this.profile
-        opts.revision = this.revision
-        opts.runConfig = this.runConfig
-        opts.params = this.params
-        opts.paramsFile = this.paramsFile
-        opts.preview = true
-        opts.ansiLog = false
         final target = new CmdRun(opts)
         target.previewAction = this.&applyInspect
         // run it

diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/CmdRun.groovy
@@ -119,8 +119,6 @@ class CmdRun {
         String getLauncherCli()
         CliOptions getLauncherOptions()
 
-        void setLauncherCli(String launcherCli)
-        void setLauncherOptions(CliOptions launcherOptions)
         void setRunName(String runName)
     }
 
@@ -344,18 +342,14 @@ class CmdRun {
             args.size() > 1 ? args[1..-1] : []
         }
 
-        String launcherCli
-
-        CliOptions launcherOptions
-
         @Override
         String getLauncherCli() {
-            launcherCli ?: launcher.cliString
+            launcher.cliString
         }
 
         @Override
         CliOptions getLauncherOptions() {
-            launcherOptions ?: launcher.options
+            launcher.options
         }
 
         @Override

diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/v2/InspectCmd.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/v2/InspectCmd.groovy
@@ -110,7 +110,19 @@ class InspectCmd extends AbstractCmd implements CmdInspect.Options {
 
     @Override
     void run() {
-        new CmdInspect(this).run()
+        final opts = new RunCmd()
+        opts.launcher = launcher
+        opts.ansiLog = false
+        opts.preview = true
+        opts.pipeline = pipeline
+        opts.args = args
+        opts.params = params
+        opts.paramsFile = paramsFile
+        opts.profile = profile
+        opts.revision = revision
+        opts.runConfig = runConfig
+
+        new CmdInspect(this).run(opts)
     }
 
 }
diff --git a/modules/nextflow/src/main/groovy/nextflow/cli/v2/RunCmd.groovy b/modules/nextflow/src/main/groovy/nextflow/cli/v2/RunCmd.groovy
@@ -257,18 +257,26 @@ class RunCmd extends AbstractCmd implements CmdRun.Options, HubOptionsV2 {
         return pipelineParams
     }
 
-    String launcherCli
-
-    CliOptions launcherOptions
-
     @Override
     String getLauncherCli() {
-        launcherCli ?: launcher.cliString
+        launcher.cliString
     }
 
     @Override
     CliOptions getLauncherOptions() {
-        launcherOptions ?: launcher.options
+        launcher.options
+    }
+
+    void setLauncher(Launcher launcher) {
+        this.launcher = launcher
+    }
+
+    void setArgs(List<String> args) {
+        this.pipelineArgs = args
+    }
+
+    void setParams(Map<String,String> params) {
+        this.pipelineParams = params
     }
 
     @Override