feat(nf-google): fixes for failing tests

Signed-off-by: Tomiles <116039+tomiles@users.noreply.github.com>

Author: tomiles

plugins/nf-google/src/main/nextflow/cloud/google/batch/GoogleBatchScriptLauncher.groovy

@@ -58,11 +58,7 @@ class GoogleBatchScriptLauncher extends BashWrapperBuilder implements GoogleBatc
     protected GoogleBatchScriptLauncher() {}
 
     GoogleBatchScriptLauncher(TaskBean bean, Path remoteBinDir, BatchConfig batchConfig) {
-        // Unstaging is cross-device on Google Batch (gcsfuse-mounted work dir).
-        // `move` can fail with overlapping outputs or symlinked paths.
-        if( bean.stageOutMode == null )
-            bean.stageOutMode = 'copy'
-        super(bean, copyStrategyFor(bean, batchConfig))
+        super(defaultStageOutMode(bean), copyStrategyFor(bean, batchConfig))
         // keep track the google storage work dir
         this.remoteWorkDir = (CloudStoragePath) bean.workDir
         this.remoteBinDir = toContainerMount(remoteBinDir)
@@ -114,6 +110,14 @@ class GoogleBatchScriptLauncher extends BashWrapperBuilder implements GoogleBatc
                 : new SimpleFileCopyStrategy(bean)
     }
 
+    private static TaskBean defaultStageOutMode(TaskBean bean) {
+        // Unstaging is cross-device on Google Batch (gcsfuse-mounted work dir).
+        // `move` can fail with overlapping outputs or symlinked paths.
+        if( bean.stageOutMode == null )
+            bean.stageOutMode = 'copy'
+        return bean
+    }
+
     protected String headerScript(TaskBean bean) {
         def result = "NXF_CHDIR=${Escape.path(bean.workDir)}\n"
         if( remoteBinDir ) {

plugins/nf-google/src/main/nextflow/cloud/google/batch/client/BatchConfig.groovy

@@ -258,7 +258,7 @@ class BatchConfig implements ConfigScope {
     }
 
     static boolean isCliCopyTransport(String transport) {
-        return transport in CLI_COPY_TRANSPORTS
+        return transport != null && transport in CLI_COPY_TRANSPORTS
     }
 
     BatchRetryConfig getRetryConfig() { retry }