[telemetry] Remove WAKE_LOCK permission

We no longer want to have to specify the WAKE_LOCK permission, so we need to replace JobIntentService with Worker and WorkManager, which doesn't have this dependency.

app/build.gradle

@@ -38,6 +38,7 @@ dependencies {
     implementation dependenciesList.supportAppcompat
     implementation dependenciesList.supportConstraintLayout
     implementation dependenciesList.gmsLocation
+    implementation dependenciesList.workManager
 
     debugImplementation dependenciesList.leakCanary
     releaseImplementation dependenciesList.leakCanaryNoOp

gradle/dependencies.gradle

@@ -20,7 +20,8 @@ ext {
             espressoVersion     : '3.1.0',
             archLifecycleVersion: "1.1.0",
             assertJ             : "3.11.1",
-            leakCanaryVersion   : "1.6.3"
+            leakCanaryVersion   : "1.6.3",
+            workManagerVersion  : "2.6.0",
     ]
 
     pluginVersion = [
@@ -58,6 +59,7 @@ ext {
             supportAppcompat       : "androidx.appcompat:appcompat:${version.androidXVersion}",
             supportConstraintLayout: "androidx.constraintlayout:constraintlayout:${version.constraintLayout}",
             supportCoreUtils       : "androidx.legacy:legacy-support-core-utils:${version.androidXVersion}",
+            workManager            : "androidx.work:work-runtime:${version.workManagerVersion}",
 
             // instrumentation test
             testRunner             : "androidx.test.ext:junit:${version.testRunnerVersion}",

libtelemetry/build.gradle

@@ -49,6 +49,7 @@ android {
 
 dependencies {
     implementation dependenciesList.androidCore
+    implementation dependenciesList.workManager
     okhttp3Implementation dependenciesList.okhttp3
     okhttp4Implementation dependenciesList.okhttp4
     implementation dependenciesList.gson

libtelemetry/src/androidTest/java/com/mapbox/android/telemetry/errors/ErrorReporterWorkerInstrumentationTest.java

@@ -0,0 +1,22 @@
+package com.mapbox.android.telemetry.errors;
+
+import androidx.test.platform.app.InstrumentationRegistry;
+import androidx.work.ExistingWorkPolicy;
+import androidx.work.WorkManager;
+
+import org.junit.Test;
+
+import com.mapbox.android.telemetry.MapboxTelemetryConstants;
+
+public class ErrorReporterWorkerInstrumentationTest {
+
+  @Test
+  public void enqueueWork() {
+    WorkManager.getInstance(InstrumentationRegistry.getInstrumentation().getTargetContext())
+      .enqueueUniqueWork(
+        MapboxTelemetryConstants.ACTION_TOKEN_CHANGED,
+        ExistingWorkPolicy.KEEP,
+        ErrorReporterWorker.createWorkRequest());
+    // TODO: verify work is executed
+  }
+}

libtelemetry/src/main/AndroidManifest.xml

@@ -3,8 +3,6 @@
     xmlns:android="http://schemas.android.com/apk/res/android"
     package="com.mapbox.android.telemetry">
 
-    <!--Required for CrashReporterJobIntentService on API levels below 25-->
-    <uses-permission android:name="android.permission.WAKE_LOCK"/>
     <uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
     <uses-permission android:name="android.permission.ACCESS_WIFI_STATE"/>
     <uses-permission android:name="android.permission.INTERNET"/>
@@ -15,8 +13,6 @@
             android:authorities="${applicationId}.mapboxtelemetryinitprovider"
             android:exported="false"
             android:initOrder="100"/>
-        <service android:name=".errors.ErrorReporterJobIntentService"
-            android:permission="android.permission.BIND_JOB_SERVICE" android:exported="false"/>
 
         <service android:name=".MapboxTelemetryService" android:exported="false"/>
 

libtelemetry/src/main/java/com/mapbox/android/telemetry/errors/ErrorReporterWorker.java

@@ -0,0 +1,43 @@
+package com.mapbox.android.telemetry.errors;
+
+import android.content.Context;
+import android.util.Log;
+
+import androidx.annotation.NonNull;
+
+import androidx.work.Constraints;
+import androidx.work.NetworkType;
+import androidx.work.OneTimeWorkRequest;
+import androidx.work.Worker;
+import androidx.work.WorkerParameters;
+
+/**
+ * This is a background job that sends error events to the telemetry endpoint
+ * at startup.
+ */
+public final class ErrorReporterWorker extends Worker {
+  private static final String LOG_TAG = "ErrorReportWorker";
+
+  public ErrorReporterWorker(@NonNull Context context, @NonNull WorkerParameters workerParams) {
+    super(context, workerParams);
+  }
+
+  @NonNull
+  @Override
+  public Result doWork() {
+    Log.d(LOG_TAG, "doWork");
+    try {
+      ErrorReporterEngine.sendReports(getApplicationContext());
+    } catch (Throwable throwable) {
+      // TODO: log silent crash
+      Log.e(LOG_TAG, throwable.toString());
+    }
+    return Result.success();
+  }
+
+  public static OneTimeWorkRequest createWorkRequest() {
+    return new OneTimeWorkRequest.Builder(ErrorReporterWorker.class)
+      .setConstraints(new Constraints.Builder().setRequiredNetworkType(NetworkType.CONNECTED).build())
+      .build();
+  }
+}

libtelemetry/src/main/java/com/mapbox/android/telemetry/errors/TokenChangeBroadcastReceiver.java

@@ -6,7 +6,10 @@
 import android.content.IntentFilter;
 import androidx.annotation.NonNull;
 import androidx.localbroadcastmanager.content.LocalBroadcastManager;
+import androidx.work.ExistingWorkPolicy;
+import androidx.work.WorkManager;
 import android.util.Log;
+
 import com.mapbox.android.telemetry.MapboxTelemetryConstants;
 
 /**
@@ -31,7 +34,11 @@ public static void register(@NonNull Context context) {
   public void onReceive(Context context, Intent intent) {
     try {
       // Start background job
-      ErrorReporterJobIntentService.enqueueWork(context);
+      WorkManager.getInstance(context)
+        .enqueueUniqueWork(
+          MapboxTelemetryConstants.ACTION_TOKEN_CHANGED,
+          ExistingWorkPolicy.KEEP,
+          ErrorReporterWorker.createWorkRequest());
       // Unregister receiver - we need it once at startup
       LocalBroadcastManager.getInstance(context).unregisterReceiver(this);
     } catch (Throwable throwable) {