Update artifactTypes to use uppercase

androidx · May 23, 2024 · d3948a9 · d3948a9
1 parent db40894
commit d3948a9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/AndroidXCI/lib/src/main/kotlin/dev/androidx/ci/testRunner/TestRunnerService.kt b/AndroidXCI/lib/src/main/kotlin/dev/androidx/ci/testRunner/TestRunnerService.kt
@@ -354,10 +354,10 @@ interface TestRunnerService {
         val resourceType: String
     ) {
         companion object ResourceType {
-            const val LOGCAT = "logcat"
-            const val PNG = "png"
-            const val TEXTPROTO = "textproto"
-            const val PROTOBINARY = "pb"
+            const val LOGCAT = "LOGCAT"
+            const val PNG = "PNG"
+            const val TEXTPROTO = "TEXTPROTO"
+            const val PROTOBINARY = "PB"
         }
     }
 

diff --git a/AndroidXCI/lib/src/main/kotlin/dev/androidx/ci/testRunner/TestRunnerServiceImpl.kt b/AndroidXCI/lib/src/main/kotlin/dev/androidx/ci/testRunner/TestRunnerServiceImpl.kt
@@ -30,6 +30,7 @@ import dev.androidx.ci.testRunner.vo.DeviceSetup
 import dev.androidx.ci.testRunner.vo.RemoteApk
 import dev.androidx.ci.testRunner.vo.UploadedApk
 import java.io.InputStream
+import java.util.Locale
 
 /**
  * Real implementation of [TestRunnerService].
@@ -208,7 +209,7 @@ internal class TestRunnerServiceImpl internal constructor(
                     getTestResultFiles(visitor).addTestCaseArtifact(
                         testIdentifier,
                         ResultFileResourceImpl(visitor),
-                        "logcat"
+                        "LOGCAT"
                     )
                 }
             }
@@ -261,7 +262,7 @@ internal class TestRunnerServiceImpl internal constructor(
                     }.add(
                         TestRunnerService.TestCaseArtifact(
                             ResultFileResourceImpl(visitor),
-                            visitor.fileName.substringAfterLast(".")
+                            visitor.fileName.substringAfterLast(".").uppercase(Locale.getDefault())
                         )
                     )
                 }