E2-1930 Fixing a crash loop when receiving push notification (#366)

Author: milos1290

AndroidSDKCore/src/main/java/com/leanplum/internal/RequestOld.java

@@ -274,10 +274,15 @@ public Map<String, Object> createArgsDictionary() {
 
   private void saveRequestForLater(Map<String, Object> args) {
     try {
+
+      Context context = Leanplum.getContext();
+      if (context == null) {
+        return;
+      }
+
       requestSequenceRecorder.beforeWrite();
 
       synchronized (RequestOld.class) {
-        Context context = Leanplum.getContext();
         SharedPreferences preferences = context.getSharedPreferences(
             LEANPLUM, Context.MODE_PRIVATE);
         SharedPreferences.Editor editor = preferences.edit();

AndroidSDKFcm/src/main/java/com/leanplum/LeanplumPushFirebaseMessagingService.java

@@ -61,6 +61,9 @@ public void onNewToken(String token) {
   @Override
   public void onMessageReceived(RemoteMessage remoteMessage) {
     try {
+      if (Leanplum.getContext() == null) {
+        Leanplum.setApplicationContext(this);
+      }
       Map<String, String> messageMap = remoteMessage.getData();
       if (messageMap.containsKey(Constants.Keys.PUSH_MESSAGE_TEXT)) {
         LeanplumPushService.handleNotification(this, getBundle(messageMap));

AndroidSDKTests/src/test/java/com/leanplum/internal/RequestOldTest.java

@@ -21,6 +21,7 @@
 package com.leanplum.internal;
 
 import android.app.Application;
+import android.content.Context;
 
 import com.leanplum.Leanplum;
 import com.leanplum.__setup.LeanplumTestApp;
@@ -371,6 +372,34 @@ private List<Map<String, Object>> mockRequests(int requestSize) {
     return requests;
   }
 
+  @Test
+  public void testNotSendingIfContextIsNull() {
+    Context context = Leanplum.getContext();
+    Leanplum.setApplicationContext(null);
+
+    final Semaphore semaphore = new Semaphore(1);
+    semaphore.tryAcquire();
+
+    // Given a request.
+    Map<String, Object> params = new HashMap<>();
+    params.put("data1", "value1");
+    params.put("data2", "value2");
+    RequestOld request = new RequestOld(POST, Constants.Methods.START, params);
+    request.onError(new RequestOld.ErrorCallback() {
+      @Override
+      public void error(Exception e) {
+        assertNotNull(e);
+        semaphore.release();
+      }
+    });
+    request.setAppId("fskadfshdbfa", "wee5w4waer422323");
+
+    // When the request is sent.
+    request.sendIfConnected();
+
+    Leanplum.setApplicationContext(context);
+  }
+
   private static class ThreadRequestSequenceRecorder implements RequestSequenceRecorder {
     Instant beforeReadTime, afterReadTime, beforeWriteTime, afterWriteTime;
     final Semaphore writeSemaphore = new Semaphore(0);