Improve error message on IOS

release_notes.md

@@ -1,5 +1,6 @@
 ## next (unreleased)
-- [#952](https://github.com/Flank/flank/pull/952) Fix version printing on Flank release
+- [#937](https://github.com/Flank/flank/pull/968) Improve error message on iOS when test or xctestrun-file not found ([sloox](https://github.com/Sloox))
+- [#952](https://github.com/Flank/flank/pull/952) Fix version printing on Flank release ([sloox](https://github.com/Sloox))
 - [#950](https://github.com/Flank/flank/pull/950) Fix crash when --legacy-junit-result set. ([adamfilipow92](https://github.com/adamfilipow92))
 - [#948](https://github.com/Flank/flank/pull/948) Increment retry tries and change sync tag for jfrogSync. ([piotradamczyk5](https://github.com/piotradamczyk5))
 - [#946](https://github.com/Flank/flank/pull/946) Added tests for flank scripts. ([piotradamczyk5](https://github.com/piotradamczyk5))

test_runner/src/main/kotlin/ftl/args/ValidateIosArgs.kt

@@ -7,6 +7,7 @@ import ftl.util.IncompatibleTestDimensionError
 fun IosArgs.validate() {
     assertXcodeSupported()
     assertDevicesSupported()
+    assertTestTypes()
     assertMaxTestShards()
 }
 private fun IosArgs.assertMaxTestShards() { this.maxTestShards
@@ -17,6 +18,10 @@ private fun IosArgs.assertMaxTestShards() { this.maxTestShards
         "max-test-shards must be >= ${IArgs.AVAILABLE_PHYSICAL_SHARD_COUNT_RANGE.first} and <= ${IArgs.AVAILABLE_PHYSICAL_SHARD_COUNT_RANGE.last}, or -1. But current is $maxTestShards"
     )
 }
+private fun IosArgs.assertTestTypes() {
+    if (xctestrunFile.isBlank() or xctestrunZip.isBlank())
+        throw FlankConfigurationError("Both of following options must be specified [test, xctestrun-file].")
+}
 private fun IosArgs.assertXcodeSupported() = when {
     xcodeVersion == null -> Unit
     IosCatalog.supportedXcode(xcodeVersion, project) -> Unit

test_runner/src/test/kotlin/ftl/args/IosArgsTest.kt

@@ -916,6 +916,51 @@ IosArgs
         val args = IosArgs.load(yaml)
         assertEquals(IArgs.AVAILABLE_PHYSICAL_SHARD_COUNT_RANGE.last, args.maxTestShards)
     }
+
+    @Test(expected = FlankConfigurationError::class)
+    fun `verify appropriate error message when test and xctestrun not set`() {
+        val yaml = """
+        gcloud:
+        flank:
+          max-test-shards: -1
+        """.trimIndent()
+        IosArgs.load(yaml)
+    }
+
+    @Test(expected = FlankConfigurationError::class)
+    fun `verify appropriate error message when xctestrun not set`() {
+        val yaml = """
+        gcloud:
+          test: $testPath
+        flank:
+          max-test-shards: -1
+        """.trimIndent()
+        IosArgs.load(yaml)
+    }
+
+    @Test(expected = FlankConfigurationError::class)
+    fun `verify appropriate error message when test not set`() {
+        val yaml = """
+        gcloud:
+          xctestrun-file: $testPath
+        flank:
+          max-test-shards: -1
+        """.trimIndent()
+        IosArgs.load(yaml)
+    }
+
+    @Test
+    fun `verify no error message when test and xctestrun-file set`() {
+        val yaml = """
+        gcloud:
+          test: $testPath
+          xctestrun-file: $testPath
+        flank:
+          max-test-shards: -1
+        """.trimIndent()
+        IosArgs.load(yaml)
+    }
 }
 
-private fun IosArgs.Companion.load(yamlData: String, cli: IosRunCommand? = null): IosArgs = load(StringReader(yamlData), cli)
+private fun IosArgs.Companion.load(yamlData: String, cli: IosRunCommand? = null): IosArgs =
+    load(StringReader(yamlData), cli)