fix: support datafile download and events upload for API-16

.travis.yml

@@ -8,6 +8,7 @@ env:
  - ADB_INSTALL_TIMEOUT=5 # minutes
  - ANDROID_API=29 # api is same as gradle file
  matrix:
+ - EMULATOR_API=19
  - EMULATOR_API=21
  - EMULATOR_API=22
 android:

android-sdk/src/androidTest/java/com/optimizely/ab/android/sdk/OptimizelyClientEngineTest.java

@@ -34,7 +34,6 @@
 
 @RunWith(AndroidJUnit4.class)
 public class OptimizelyClientEngineTest {
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB_MR2)
     @Test
     public void testGetClientEngineFromContextAndroidTV() {
         Context context = mock(Context.class);
@@ -44,7 +43,6 @@ public void testGetClientEngineFromContextAndroidTV() {
         assertEquals(EventBatch.ClientEngine.ANDROID_TV_SDK, OptimizelyClientEngine.getClientEngineFromContext(context));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB_MR2)
     @Test
     public void testGetClientEngineFromContextAndroid() {
         Context context = mock(Context.class);

android-sdk/src/androidTest/java/com/optimizely/ab/android/sdk/OptimizelyManagerTest.java

@@ -23,8 +23,8 @@
 import android.content.pm.PackageManager;
 import android.os.Build;
 
-import androidx.annotation.RequiresApi;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
+import androidx.test.filters.SdkSuppress;
 import androidx.test.platform.app.InstrumentationRegistry;
 
 import com.optimizely.ab.android.datafile_handler.DatafileHandler;
@@ -356,7 +356,6 @@ public void stop() {
         assertNull(optimizelyManager.getOptimizelyStartListener());
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void injectOptimizely() {
         Context context = mock(Context.class);
@@ -377,7 +376,6 @@ public void injectOptimizely() {
 
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void injectOptimizelyWithDatafileLisener() {
         Context context = mock(Context.class);
@@ -398,7 +396,7 @@ public void injectOptimizelyWithDatafileLisener() {
     }
 
     @Test
-    @RequiresApi(api = Build.VERSION_CODES.LOLLIPOP)
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void injectOptimizelyNullListener() {
         Context context = mock(Context.class);
         PackageManager packageManager = mock(PackageManager.class);
@@ -442,7 +440,6 @@ public void injectOptimizelyNullListener() {
         assertEquals(optimizelyManager.getDatafileConfig().toJSONString(), intent2.getStringExtra(DatafileService.EXTRA_DATAFILE_CONFIG));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void injectOptimizelyHandlesInvalidDatafile() {
         Context context = mock(Context.class);
@@ -472,7 +469,6 @@ public void injectOptimizelyHandlesInvalidDatafile() {
         assertFalse(optimizelyManager.getOptimizely().isValid());
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void injectOptimizelyDoesNotDuplicateCallback() {
         Context context = mock(Context.class);

android-sdk/src/main/java/com/optimizely/ab/android/sdk/OptimizelyManager.java

@@ -398,7 +398,6 @@ private String safeLoadResource(Context context, @RawRes final Integer datafileR
 
     DatafileLoadedListener getDatafileLoadedListener(final Context context, @RawRes final Integer datafileRes) {
         return new DatafileLoadedListener() {
-            @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
             @Override
             public void onDatafileLoaded(@Nullable String datafile) {
                 if (datafile != null && !datafile.isEmpty()) {
@@ -516,7 +515,6 @@ private void startDatafileHandler(Context context) {
         });
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     void injectOptimizely(@NonNull final Context context, final @NonNull UserProfileService userProfileService,
                           @NonNull final String datafile) {
 

android-sdk/src/test/java/com/optimizely/ab/android/sdk/OptimizelyManagerDatafileServiceConnectionTest.java

@@ -18,7 +18,6 @@
 
 import android.content.Context;
 import android.os.Build;
-import androidx.annotation.RequiresApi;
 
 import org.junit.Before;
 import org.junit.Test;
@@ -55,7 +54,6 @@ public void setup() {
         datafileServiceConnection = new DatafileServiceConnection(optimizelyManager.getDatafileConfig(), context, optimizelyManager.getDatafileLoadedListener(context,null));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void onServiceConnected() {
         DatafileService.LocalBinder binder = mock(DatafileService.LocalBinder.class);
@@ -72,7 +70,6 @@ public void onServiceConnected() {
         verify(service).getDatafile(eq(sameString), any(DatafileLoader.class), any(DatafileLoadedListener.class));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void onServiceConnectedNullServiceFromBinder() {
         DatafileService.LocalBinder binder = mock(DatafileService.LocalBinder.class);

android-sdk/src/test/java/com/optimizely/ab/android/sdk/OptimizelyManagerOptlyActivityLifecycleCallbacksTest.java

@@ -18,7 +18,6 @@
 
 import android.app.Activity;
 import android.os.Build;
-import androidx.annotation.RequiresApi;
 
 import org.junit.Before;
 import org.junit.Test;
@@ -38,13 +37,11 @@ public class OptimizelyManagerOptlyActivityLifecycleCallbacksTest {
     @Mock Activity activity;
     private OptimizelyManager.OptlyActivityLifecycleCallbacks optlyActivityLifecycleCallbacks;
 
-    @RequiresApi(api = Build.VERSION_CODES.ICE_CREAM_SANDWICH)
     @Before
     public void setup() {
         optlyActivityLifecycleCallbacks = new OptimizelyManager.OptlyActivityLifecycleCallbacks(optimizelyManager);
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.ICE_CREAM_SANDWICH)
     @Test
     public void onActivityStopped() {
         optlyActivityLifecycleCallbacks.onActivityStopped(activity);

build.gradle

@@ -75,26 +75,23 @@ task clean(type: Delete) {
 
 task cleanAllModules () {
     logger.info("Running clean for all modules")
+    dependsOn(':android-sdk:clean', ':event-handler:clean',
+            ':user-profile:clean', ':shared:clean', ':datafile-handler:clean')
 }
 
-cleanAllModules.dependsOn(':android-sdk:clean', ':event-handler:clean',
-        ':user-profile:clean', ':shared:clean', ':datafile-handler:clean')
-
 task testAllModules () {
     logger.info("Running android tests for all modules")
+    dependsOn('testAllModulesTravis', ':test-app:connectedAndroidTest')
 }
 
 task testAllModulesTravis () {
     logger.info("Running android tests for Travis")
+    dependsOn(':android-sdk:connectedAndroidTest', ':android-sdk:test',
+            ':event-handler:connectedAndroidTest', ':event-handler:test',
+            ':datafile-handler:connectedAndroidTest', ':datafile-handler:test',
+            ':user-profile:connectedAndroidTest', ':shared:connectedAndroidTest')
 }
 
-testAllModulesTravis.dependsOn(':android-sdk:connectedAndroidTest', ':android-sdk:test',
-        ':event-handler:connectedAndroidTest', ':event-handler:test',
-        ':datafile-handler:connectedAndroidTest', ':datafile-handler:test',
-        ':user-profile:connectedAndroidTest', ':shared:connectedAndroidTest')
-
-testAllModules.dependsOn('testAllModulesTravis', ':test-app:connectedAndroidTest')
-
 // Publish to MavenCentral
 
 task ship() {

datafile-handler/src/androidTest/java/com/optimizely/ab/android/datafile_handler/DatafileLoaderTest.java

@@ -21,6 +21,7 @@
 
 import androidx.annotation.RequiresApi;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
+import androidx.test.filters.SdkSuppress;
 import androidx.test.platform.app.InstrumentationRegistry;
 
 import com.optimizely.ab.android.shared.Cache;
@@ -57,7 +58,6 @@
 /**
  * Tests for {@link DatafileLoader}
  */
-@RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
 @RunWith(AndroidJUnit4.class)
 public class DatafileLoaderTest {
 
@@ -147,6 +147,8 @@ public void noCacheAndLoadFromCDNFails() {
     }
 
     @Test
+    // flacky with lower API
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void warningsAreLogged() throws IOException {
         final ExecutorService executor = Executors.newSingleThreadExecutor();
         Cache cache = mock(Cache.class);
@@ -172,6 +174,8 @@ public void warningsAreLogged() throws IOException {
     }
 
     @Test
+    // flacky with lower API
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void debugLogged() throws IOException {
         final ExecutorService executor = Executors.newSingleThreadExecutor();
         Cache cache = mock(Cache.class);

datafile-handler/src/androidTest/java/com/optimizely/ab/android/datafile_handler/DatafileReschedulerTest.java

@@ -81,7 +81,6 @@ public void receivedActionBootCompleted() {
         verify(logger).info("Received intent with action {}", Intent.ACTION_BOOT_COMPLETED);
     }
 
-    @TargetApi(Build.VERSION_CODES.HONEYCOMB_MR1)
     @Test
     public void receivedActionMyPackageReplaced() {
         Context context = InstrumentationRegistry.getInstrumentation().getTargetContext();

datafile-handler/src/androidTest/java/com/optimizely/ab/android/datafile_handler/DatafileServiceTest.java

@@ -71,7 +71,6 @@ public void setup() {
         executor = Executors.newSingleThreadExecutor();
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     @Ignore
     public void testBinding() throws TimeoutException {
@@ -99,7 +98,6 @@ public void testBinding() throws TimeoutException {
         assertTrue(datafileService.isBound());
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void testValidStart() throws TimeoutException {
         Context context = InstrumentationRegistry.getInstrumentation().getTargetContext();
@@ -119,7 +117,6 @@ public void testValidStart() throws TimeoutException {
         assertEquals(val, START_FLAG_REDELIVERY);
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     @Ignore
     public void testNullIntentStart() throws TimeoutException {
@@ -139,7 +136,6 @@ public void testNullIntentStart() throws TimeoutException {
         verify(logger).warn("Data file service received a null intent");
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     @Ignore
     public void testNoProjectIdIntentStart() throws TimeoutException {
@@ -159,7 +155,6 @@ public void testNoProjectIdIntentStart() throws TimeoutException {
         verify(logger).warn("Data file service received an intent with no project id extra");
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     @Ignore
     public void testUnbind() throws TimeoutException {
@@ -180,7 +175,6 @@ public void testUnbind() throws TimeoutException {
         verify(logger).info("All clients are unbound from data file service");
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     @Ignore
     public void testIntentExtraData(){
@@ -209,7 +203,6 @@ public void testIntentExtraData(){
         assertTrue(intent2.getComponent().getShortClassName().contains("DatafileService"));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void testGetDatafileUrl(){
         // HARD-CODING link here to make sure we don't unintentionally mess up the datafile version
@@ -218,8 +211,7 @@ public void testGetDatafileUrl(){
         String datafileUrl = new DatafileConfig("1", null).getUrl();
         assertEquals("https://cdn.optimizely.com/json/1.json", datafileUrl);
     }
-    @
-            RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
+
     @Test
     public void testGetDatafileEnvironmentUrl(){
         // HARD-CODING link here to make sure we don't unintentionally mess up the datafile version

datafile-handler/src/main/java/com/optimizely/ab/android/datafile_handler/DatafileClient.java

@@ -16,6 +16,8 @@
 
 package com.optimizely.ab.android.datafile_handler;
 
+import android.os.Build;
+
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 

datafile-handler/src/main/java/com/optimizely/ab/android/datafile_handler/DatafileLoader.java

@@ -80,7 +80,6 @@ private void saveDownloadTime(String url) {
         storage.saveLong(url + datafileDownloadTime, time);
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     public void getDatafile(@NonNull String datafileUrl, @Nullable DatafileLoadedListener datafileLoadedListener) {
         if (!allowDownload(datafileUrl, datafileLoadedListener)) {
             return;

datafile-handler/src/main/java/com/optimizely/ab/android/datafile_handler/DatafileService.java

@@ -54,7 +54,6 @@ public class DatafileService extends Service {
      * @hide
      * @see Service#onStartCommand(Intent, int, int)
      */
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Override
     public int onStartCommand(Intent intent, int flags, int startId) {
         if (intent != null) {
@@ -111,7 +110,6 @@ public void stop() {
         stopSelf();
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     public void getDatafile(String datafileUrl, DatafileLoader datafileLoader, DatafileLoadedListener loadedListener) {
         datafileLoader.getDatafile(datafileUrl, loadedListener);
     }

datafile-handler/src/main/java/com/optimizely/ab/android/datafile_handler/DatafileServiceConnection.java

@@ -62,7 +62,6 @@ public DatafileServiceConnection(@NonNull DatafileConfig datafileConfig, @NonNul
      *
      * @see ServiceConnection#onServiceConnected(ComponentName, IBinder)
      */
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Override
     public void onServiceConnected(ComponentName className,
                                    IBinder service) {

event-handler/src/androidTest/java/com/optimizely/ab/android/event_handler/EventDAOTest.java

@@ -49,7 +49,6 @@ public class EventDAOTest {
     private Logger logger;
     private Context context;
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Before
     public void setupEventDAO() {
         logger = mock(Logger.class);

event-handler/src/androidTest/java/com/optimizely/ab/android/event_handler/EventDispatcherTest.java

@@ -21,6 +21,7 @@
 import android.content.Intent;
 import android.os.Build;
 import androidx.annotation.RequiresApi;
+import androidx.test.filters.SdkSuppress;
 import androidx.test.platform.app.InstrumentationRegistry;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 
@@ -62,7 +63,6 @@ public class EventDispatcherTest {
     private Context context;
     private Client client;
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Before
     public void setup() {
         context = InstrumentationRegistry.getInstrumentation().getTargetContext();
@@ -82,6 +82,7 @@ public void tearDown() {
     }
 
     @Test
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void handleIntentSchedulesWhenEventsLeftInStorage() throws IOException {
         Event event1 = new Event(new URL("http://www.foo1.com"), "");
         Event event2 = new Event(new URL("http://www.foo2.com"), "");
@@ -108,6 +109,7 @@ public void handleIntentSchedulesWhenEventsLeftInStorage() throws IOException {
     }
 
     @Test
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void handleIntentSchedulesWhenNewEventFailsToSend() throws IOException {
         Event event = new Event(new URL("http://www.foo.com"), "");
 
@@ -127,6 +129,7 @@ public void handleIntentSchedulesWhenNewEventFailsToSend() throws IOException {
     }
 
     @Test
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void unschedulesServiceWhenNoEventsToFlush() {
         Intent mockIntent = mock(Intent.class);
         eventDispatcher.dispatch(mockIntent);
@@ -135,6 +138,7 @@ public void unschedulesServiceWhenNoEventsToFlush() {
     }
 
     @Test
+    @SdkSuppress(minSdkVersion = Build.VERSION_CODES.LOLLIPOP)
     public void handleMalformedURL() throws MalformedURLException {
         String url= "foo";
 

event-handler/src/androidTest/java/com/optimizely/ab/android/event_handler/EventSQLiteOpenHelperTest.java

@@ -54,7 +54,6 @@ public void teardown() {
         context.deleteDatabase(String.format(EventSQLiteOpenHelper.DB_NAME, "1"));
     }
 
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Test
     public void onCreateMakesTables() {
         EventSQLiteOpenHelper eventSQLiteOpenHelper =

event-handler/src/main/java/com/optimizely/ab/android/event_handler/EventDAO.java

@@ -60,7 +60,6 @@ private EventDAO(@NonNull EventSQLiteOpenHelper dbHelper, @NonNull Logger logger
      * @param logger where to log errors and warnings.
      * @return a new instance of EventDAO.
      */
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     static EventDAO getInstance(@NonNull Context context, @NonNull String projectId, @NonNull Logger logger) {
         EventSQLiteOpenHelper sqLiteOpenHelper = new EventSQLiteOpenHelper(context, projectId, null, EventSQLiteOpenHelper.VERSION, LoggerFactory.getLogger(EventSQLiteOpenHelper.class));
         return new EventDAO(sqLiteOpenHelper, logger);

event-handler/src/main/java/com/optimizely/ab/android/event_handler/EventIntentService.java

@@ -57,7 +57,6 @@ public EventIntentService() {
      * Create the event dispatcher {@link EventDispatcher}
      * @see IntentService#onCreate()
      */
-    @RequiresApi(api = Build.VERSION_CODES.HONEYCOMB)
     @Override
     public void onCreate() {
         super.onCreate();