fix: NPE on dumpShards

integration_tests/src/test/kotlin/integration/DumpShardsIT.kt

@@ -0,0 +1,51 @@
+package integration
+
+import FlankCommand
+import com.google.common.truth.Truth
+import flank.common.isWindows
+import org.junit.Assume
+import org.junit.Test
+import run
+
+class DumpShardsIT {
+    private val name = this::class.java.simpleName
+
+    @Test
+    fun `dump shards - android`() {
+        val name = "$name-android"
+        val result = FlankCommand(
+            flankPath = FLANK_JAR_PATH,
+            ymlPath = "$CONFIGS_PATH/dump_shards_android.yml",
+            params = androidRunCommands + "--dump-shards"
+        ).run(
+            workingDirectory = "./",
+            testSuite = name
+        )
+
+        assertExitCode(result, 0)
+
+        val resOutput = result.output.removeUnicode()
+        Truth.assertThat(resOutput).containsMatch(findInCompare(name))
+        assertNoOutcomeSummary(resOutput)
+    }
+
+    @Test
+    fun `dump shards - ios`() {
+        Assume.assumeFalse(isWindows)
+        val name = "$name-ios"
+        val result = FlankCommand(
+            flankPath = FLANK_JAR_PATH,
+            ymlPath = "$CONFIGS_PATH/dump_shards_ios.yml",
+            params = iosRunCommands + "--dump-shards"
+        ).run(
+            workingDirectory = "./",
+            testSuite = name
+        )
+
+        assertExitCode(result, 0)
+
+        val resOutput = result.output.removeUnicode()
+        Truth.assertThat(resOutput).containsMatch(findInCompare(name))
+        assertNoOutcomeSummary(resOutput)
+    }
+}

integration_tests/src/test/resources/cases/dump_shards_android.yml

@@ -0,0 +1,3 @@
+gcloud:
+  app: ../test_runner/src/test/kotlin/ftl/fixtures/tmp/apk/app-debug.apk
+  test: ../test_runner/src/test/kotlin/ftl/fixtures/tmp/apk/app-single-success-debug-androidTest.apk

integration_tests/src/test/resources/cases/dump_shards_ios.yml

@@ -0,0 +1,4 @@
+gcloud:
+  test: ../test_runner/src/test/kotlin/ftl/fixtures/tmp/ios/EarlGreyExample/EarlGreyExample.zip
+  xctestrun-file: ../test_runner/src/test/kotlin/ftl/fixtures/tmp/ios/EarlGreyExample/EarlGreyExampleSwiftTests.xctestrun
+

integration_tests/src/test/resources/compare/DumpShardsIT-android-compare

@@ -0,0 +1 @@
+  Saved 1 shards to android_shards.json

integration_tests/src/test/resources/compare/DumpShardsIT-ios-compare

@@ -0,0 +1 @@
+  Saved 1 shards to ios_shards.json

test_runner/src/main/kotlin/ftl/cli/firebase/test/android/AndroidRunCommand.kt

@@ -68,18 +68,22 @@ class AndroidRunCommand : CommonRunCommand(), Runnable {
 
         AndroidArgs.load(Paths.get(configPath), cli = this).apply {
             setupLogLevel()
-            logLn(this)
+
             outputReport.configure(toOutputReportConfiguration())
             outputReport.log(this)
             setCrashReportTag(
                 DEVICE_SYSTEM to "android",
                 TEST_TYPE to type?.name.orEmpty()
             )
             sendConfiguration()
-        }.validate().run {
+        }.validate().also { args ->
             runBlocking {
-                if (dumpShards) dumpShards()
-                else newTestRun()
+                if (dumpShards)
+                    args.dumpShards()
+                else {
+                    logLn(args)
+                    args.newTestRun()
+                }
             }
         }
     }

test_runner/src/main/kotlin/ftl/cli/firebase/test/ios/IosRunCommand.kt

@@ -68,14 +68,14 @@ class IosRunCommand : CommonRunCommand(), Runnable {
 
         IosArgs.load(Paths.get(configPath), cli = this).apply {
             setupLogLevel()
-            logLn(this)
             outputReport.configure(toOutputReportConfiguration())
             outputReport.log(this)
             setCrashReportTag(
                 DEVICE_SYSTEM to "ios",
                 TEST_TYPE to type?.name.orEmpty()
             )
             sendConfiguration()
+            if (dumpShards.not()) logLn(this)
         }.validate().run {
             if (dumpShards) dumpShards()
             else runBlocking { newTestRun() }

test_runner/src/main/kotlin/ftl/run/DumpShards.kt

@@ -71,7 +71,11 @@ fun saveShardChunks(
     )
     logLn("${FtlConstants.indent}Saved $size shards to $shardFilePath", OutputLogLevel.DETAILED)
 }
-private fun String.createDirectories() = File(this).parent.toFile().mkdirs()
+
+private fun String.createDirectories() {
+    if (this !in listOf(ANDROID_SHARD_FILE, IOS_SHARD_FILE))
+        File(this).parent.toFile().mkdirs()
+}
 
 private fun getGson(obfuscatedOutput: Boolean) =
     if (obfuscatedOutput) obfuscatePrettyPrinter else prettyPrint