Core Telemetry Events: switch dispatcher from IMMEDIATE to QUEUED (#6622)

RingerJK · web-flow · commit eaffcea0fdec · 2022-11-18T13:39:02.000+01:00
diff --git a/libnavigation-metrics/src/main/java/com/mapbox/navigation/metrics/MapboxMetricsReporter.kt b/libnavigation-metrics/src/main/java/com/mapbox/navigation/metrics/MapboxMetricsReporter.kt
@@ -119,7 +119,7 @@ object MapboxMetricsReporter : MetricsReporter {
                 return
             }
             eventsService.sendEvent(
-                Event(EventPriority.IMMEDIATE, metricEvent.toValue(), null)
+                Event(EventPriority.QUEUED, metricEvent.toValue(), null)
             ) {
                 if (it != null) {
                     logE("Failed to send event ${metricEvent.metricName}: $it", LOG_CATEGORY)
diff --git a/libnavigation-metrics/src/test/java/com/mapbox/navigation/metrics/MapboxMetricsReporterTest.kt b/libnavigation-metrics/src/test/java/com/mapbox/navigation/metrics/MapboxMetricsReporterTest.kt
@@ -118,7 +118,7 @@ class MapboxMetricsReporterTest {
             MapboxMetricsReporter.addEvent(metricEvent)
 
             assertTrue(slotEvent.isCaptured)
-            assertEquals(EventPriority.IMMEDIATE, slotEvent.captured.priority)
+            assertEquals(EventPriority.QUEUED, slotEvent.captured.priority)
             assertEquals(metricEvent.toValue(), slotEvent.captured.attributes)
             assertEquals(null, slotEvent.captured.deferredOptions)
         }

Original file line number	Diff line number	Diff line change
`@@ -119,7 +119,7 @@ object MapboxMetricsReporter : MetricsReporter {`
`119`	`119`	`return`
`120`	`120`	`}`
`121`	`121`	`eventsService.sendEvent(`
`122`		`- Event(EventPriority.IMMEDIATE, metricEvent.toValue(), null)`
	`122`	`+ Event(EventPriority.QUEUED, metricEvent.toValue(), null)`
`123`	`123`	`) {`
`124`	`124`	`if (it != null) {`
`125`	`125`	`logE("Failed to send event ${metricEvent.metricName}: $it", LOG_CATEGORY)`
Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ class MapboxMetricsReporterTest {`
`118`	`118`	`MapboxMetricsReporter.addEvent(metricEvent)`
`119`	`119`
`120`	`120`	`assertTrue(slotEvent.isCaptured)`
`121`		`- assertEquals(EventPriority.IMMEDIATE, slotEvent.captured.priority)`
	`121`	`+ assertEquals(EventPriority.QUEUED, slotEvent.captured.priority)`
`122`	`122`	`assertEquals(metricEvent.toValue(), slotEvent.captured.attributes)`
`123`	`123`	`assertEquals(null, slotEvent.captured.deferredOptions)`
`124`	`124`	`}`