[copilot] make sure created from attachments api is the same as startedAt (#6660)

Author: Pablo Guardiola

libnavigation-copilot/src/main/java/com/mapbox/navigation/copilot/HistoryUploadWorker.kt

@@ -44,9 +44,11 @@ internal class HistoryUploadWorker(
         val drive = buildNavigationSessionFrom(workerParams.inputData)
         val filePath = workerParams.inputData.getString(HISTORY_FILE_PATH)!!
         val file = File(filePath)
+        val startedAt = workerParams.inputData.getString(STARTED_AT)!!
         val uploadSessionId = workerParams.inputData.getString(UPLOAD_SESSION_ID)!!
         val metadata = AttachmentMetadata(
             name = file.name,
+            created = startedAt,
             fileId = "",
             format = GZ,
             type = ZIP,

libnavigation-copilot/src/main/java/com/mapbox/navigation/copilot/MapboxCopilotImpl.kt

@@ -312,6 +312,7 @@ internal class MapboxCopilotImpl(
         val sessionId = generateSessionId(copilotMetadata, owner)
         return AttachmentMetadata(
             name = filename,
+            created = startedAt,
             fileId = "",
             format = GZ,
             type = ZIP,

libnavigation-copilot/src/test/java/com/mapbox/navigation/copilot/CopilotTestUtils.kt

@@ -0,0 +1,13 @@
+package com.mapbox.navigation.copilot
+
+import com.google.gson.Gson
+import com.google.gson.reflect.TypeToken
+
+object CopilotTestUtils {
+
+    internal fun retrieveAttachments(metadata: String): List<AttachmentMetadata> {
+        val gson = Gson()
+        val itemType = object : TypeToken<List<AttachmentMetadata>>() {}.type
+        return gson.fromJson(metadata, itemType)
+    }
+}

libnavigation-copilot/src/test/java/com/mapbox/navigation/copilot/HistoryAttachmentsUtilsTest.kt

@@ -8,6 +8,13 @@ import org.junit.Assert.assertEquals
 import org.junit.Test
 import java.util.Base64.getDecoder
 
+/**
+ * HistoryAttachmentsUtilsTest
+ *
+ * NOTE FOR FUTURE SECURITY AUDITS:
+ * The fakeAccessToken used below in the tests, although it seems legitimate it is a
+ * fake one manually generated so that the owner associated to is is copilot-test-owner.
+ */
 class HistoryAttachmentsUtilsTest {
 
     @Test
@@ -61,9 +68,8 @@ class HistoryAttachmentsUtilsTest {
             every { Base64.decode(any<String>(), any()) } answers {
                 getDecoder().decode(firstArg<String>())
             }
-            val accessToken = "pk.eyJ1IjoiY29waWxvdC10ZXN0LW93bmVyIiwiYSI6ImNpdDF3OHpoaTAwMDcye" +
-                "XA5Y3Z0Nmk2dzEifQ.cit1w8zhi00072yp9cvt6i6w1"
-            val owner = HistoryAttachmentsUtils.retrieveOwnerFrom(accessToken)
+            val fakeAccessToken = "pk.eyJ1IjoiY29waWxvdC10ZXN0LW93bmVyIiwiYSI6ImZha2UifQ.8badf00d"
+            val owner = HistoryAttachmentsUtils.retrieveOwnerFrom(fakeAccessToken)
             assertEquals("copilot-test-owner", owner)
         }
     }

libnavigation-copilot/src/test/java/com/mapbox/navigation/copilot/HistoryUploadWorkerTest.kt

@@ -12,6 +12,7 @@ import com.mapbox.common.UploadState
 import com.mapbox.common.UploadStatus
 import com.mapbox.common.UploadStatusCallback
 import com.mapbox.navigation.base.ExperimentalPreviewMapboxNavigationAPI
+import com.mapbox.navigation.copilot.CopilotTestUtils.retrieveAttachments
 import com.mapbox.navigation.copilot.HistoryAttachmentsUtils.copyToAndRemove
 import com.mapbox.navigation.copilot.internal.PushStatus
 import com.mapbox.navigation.copilot.internal.PushStatusObserver
@@ -75,6 +76,42 @@ class HistoryUploadWorkerTest {
         }
     }
 
+    @Test
+    fun `AttachmentMetadata created is set to startedAt`() = runBlocking {
+        mockkStatic("com.mapbox.navigation.utils.internal.LoggerProviderKt")
+        every { logD(msg = any(), category = any()) } just Runs
+        val mockedContext = mockk<Context>(relaxed = true)
+        val mockedWorkerParams = mockk<WorkerParameters>(relaxed = true)
+        val startedAt = "2022-05-12T17:47:42.353Z"
+        every {
+            mockedWorkerParams.inputData.getString("started_at")
+        } returns startedAt
+        val mockedUploadServiceInterface = prepareUploadMockks()
+        val mockedUploadOptions = slot<UploadOptions>()
+        val mockedUploadStatusCallback = slot<UploadStatusCallback>()
+        val mockedUploadStatus = mockk<UploadStatus>(relaxed = true)
+        every { mockedUploadStatus.state } returns UploadState.FINISHED
+        every { mockedUploadStatus.httpResult?.value?.code } returns 204L
+        val mockedPushStatusObserver = mockk<PushStatusObserver>(relaxUnitFun = true)
+        MapboxCopilot.pushStatusObservers.add(mockedPushStatusObserver)
+        every {
+            mockedUploadServiceInterface.upload(
+                capture(mockedUploadOptions),
+                capture(mockedUploadStatusCallback),
+            )
+        } answers {
+            mockedUploadStatusCallback.captured.run(mockedUploadStatus)
+            1L
+        }
+        val historyUploadWorker = HistoryUploadWorker(mockedContext, mockedWorkerParams)
+
+        historyUploadWorker.doWork()
+
+        val attachmentsMetadata = mockedUploadOptions.captured.metadata
+        val attachments = retrieveAttachments(attachmentsMetadata)
+        assertEquals(startedAt, attachments[0].created)
+    }
+
     @Test
     fun `onPushStatusChanged Success when FINISHED - 204`() = runBlocking {
         mockkStatic("com.mapbox.navigation.utils.internal.LoggerProviderKt")