Rework the background task service shutdown procedure to give more he…

…adroom against the 5s ANR timeout

bugsnag-android-core/src/main/java/com/bugsnag/android/BackgroundTaskService.kt

@@ -158,18 +158,13 @@ internal class BackgroundTaskService(
         internalReportExecutor.shutdownNow()
         defaultExecutor.shutdownNow()
 
-        // shutdown the error/session executors first, waiting for existing tasks to complete.
-        // If a request fails it may perform IO to persist the payload for delivery next launch,
-        // which would submit tasks to the IO executor - therefore it's critical to
-        // shutdown the IO executor last.
+        // Wait a little while for these ones to shut down
         errorExecutor.shutdown()
         sessionExecutor.shutdown()
+        ioExecutor.shutdown()
 
         errorExecutor.awaitTerminationSafe()
         sessionExecutor.awaitTerminationSafe()
-
-        // shutdown the IO executor last, waiting for any existing tasks to complete
-        ioExecutor.shutdown()
         ioExecutor.awaitTerminationSafe()
     }
 

bugsnag-android-core/src/test/java/com/bugsnag/android/LaunchCrashDeliveryTest.kt

@@ -140,7 +140,7 @@ class LaunchCrashDeliveryTest {
             payload: EventPayload,
             deliveryParams: DeliveryParams
         ): DeliveryStatus {
-            Thread.sleep(3000)
+            Thread.sleep(2000)
             count.getAndIncrement()
             return DeliveryStatus.DELIVERED
         }