more wip

mobile-dev-inc · Aug 15, 2024 · d089f0c · d089f0c
1 parent 2b29c53
commit d089f0c
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 12 deletions.
diff --git a/maestro-cli/src/main/java/maestro/cli/report/TestDebugReporter.kt b/maestro-cli/src/main/java/maestro/cli/report/TestDebugReporter.kt
@@ -4,6 +4,7 @@ import com.fasterxml.jackson.annotation.JsonInclude
 import com.fasterxml.jackson.annotation.JsonProperty
 import com.fasterxml.jackson.databind.JsonMappingException
 import com.fasterxml.jackson.databind.ObjectMapper
+import com.fasterxml.jackson.module.kotlin.jacksonObjectMapper
 import maestro.MaestroException
 import maestro.TreeNode
 import maestro.cli.runner.CommandStatus
@@ -33,19 +34,15 @@ import kotlin.io.path.exists
 object TestDebugReporter {
 
     private val logger = LoggerFactory.getLogger(TestDebugReporter::class.java)
-    private val mapper = ObjectMapper()
+    private val mapper = jacksonObjectMapper()
+                .setSerializationInclusion(JsonInclude.Include.NON_NULL)
+                .setSerializationInclusion(JsonInclude.Include.NON_EMPTY)
+                .writerWithDefaultPrettyPrinter()
 
     private var debugOutputPath: Path? = null
     private var debugOutputPathAsString: String? = null
     private var flattenDebugOutput: Boolean = false
 
-    init {
-
-        // json
-        mapper.setSerializationInclusion(JsonInclude.Include.NON_NULL)
-        mapper.setSerializationInclusion(JsonInclude.Include.NON_EMPTY)
-    }
-
     fun saveFlow(flowName: String, debugOutput: FlowDebugOutput, aiOutput: FlowAIOutput?, path: Path) {
         // TODO(bartekpacia): Potentially accept a single "FlowPersistentOutput object
 
@@ -78,11 +75,18 @@ object TestDebugReporter {
             it.screenshot.copyTo(file)
         }
 
-        // AI output
-        aiOutput?.let {
+        // AI screenshots
+        aiOutput?.outputs?.forEach { output ->
+            val screenshotFilename = output.screenshotPath.name
+            val screenshotFile = File(path.absolutePathString(), screenshotFilename)
+            output.screenshotPath.copyTo(screenshotFile)
+        }
+
+        // AI JSON output
+        aiOutput?.let { flowAiOutput ->
             val filename = "ai-(${flowName.replace("/", "_")}).json"
             val file = File(path.absolutePathString(), filename)
-            mapper.writeValue(file, it)
+            mapper.writeValue(file, flowAiOutput)
         }
     }
 

diff --git a/maestro-cli/src/main/java/maestro/cli/runner/MaestroCommandRunner.kt b/maestro-cli/src/main/java/maestro/cli/runner/MaestroCommandRunner.kt
@@ -87,7 +87,9 @@ object MaestroCommandRunner {
         }
 
         fun writeAIscreenshot(buffer: Buffer): File {
-            val out = File.createTempFile("ai-screenshot-${System.currentTimeMillis()}", ".png")
+            val out = File
+                .createTempFile("ai-screenshot-${System.currentTimeMillis()}", ".png")
+                .also { it.deleteOnExit() }
             out.outputStream().use { it.write(buffer.readByteArray()) }
             return out
         }