Add assumptions for Spoon tests to be ignored on API 21+

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/Assumptions.java

@@ -0,0 +1,28 @@
+package com.jraska.falcon.sample;
+
+import android.content.Context;
+import android.content.pm.PackageManager;
+import android.os.Build;
+import android.support.v4.content.ContextCompat;
+import org.junit.Assume;
+
+import static android.Manifest.permission.WRITE_EXTERNAL_STORAGE;
+import static org.hamcrest.Matchers.is;
+import static org.hamcrest.Matchers.lessThan;
+
+public final class Assumptions {
+  public static void assumeNoCI() {
+    Assume.assumeThat("Test should not run on CI server.", isContinuousIntegration(), is(false));
+  }
+
+  public static boolean isContinuousIntegration() {
+    return BuildConfig.CI_BUILD;
+  }
+
+  public static void assumeSpoonPermissions() {
+    // TODO: 28/07/16 PULL WELCOME: grant permissions to test application on API 21+
+    Assume.assumeThat("Spoon cannot take screenshots on API 21+" +
+            " without granted WRITE_EXTERNAL_STORAGE permission",
+        Build.VERSION.SDK_INT, lessThan(Build.VERSION_CODES.LOLLIPOP));
+  }
+}

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/EspressoSpoonTest.java

@@ -7,6 +7,7 @@
 import android.support.test.runner.AndroidJUnit4;
 import com.jraska.falcon.FalconSpoon;
 import org.junit.After;
+import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -18,10 +19,9 @@
 import static android.support.test.espresso.Espresso.onView;
 import static android.support.test.espresso.action.ViewActions.click;
 import static android.support.test.espresso.assertion.ViewAssertions.matches;
-import static android.support.test.espresso.matcher.ViewMatchers.isDisplayed;
-import static android.support.test.espresso.matcher.ViewMatchers.withId;
-import static android.support.test.espresso.matcher.ViewMatchers.withText;
-import static com.jraska.falcon.sample.CICheck.assumeNoCI;
+import static android.support.test.espresso.matcher.ViewMatchers.*;
+import static com.jraska.falcon.sample.Assumptions.assumeNoCI;
+import static com.jraska.falcon.sample.Assumptions.assumeSpoonPermissions;
 import static com.jraska.falcon.sample.asserts.BitmapFileAssert.assertThatFile;
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -43,6 +43,12 @@ public class EspressoSpoonTest {
 
   //region Setup Methods
 
+  @Before
+  public void before() throws Exception {
+    assumeNoCI();
+    assumeSpoonPermissions();
+  }
+
   @After
   public void after() throws Exception {
     for (File screenshot : takenScreenshots) {
@@ -56,8 +62,6 @@ public void after() throws Exception {
 
   @Test
   public void dialogTakenInScreenshot() throws Exception {
-    assumeNoCI();
-
     SampleActivity activity = _activityRule.getActivity();
 
     File screenshotWithoutDialogFile = FalconSpoon.screenshot(activity, "No_dialog");

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/FalconDialogInOnCreateTest.java

@@ -12,7 +12,7 @@
 import static android.support.test.espresso.assertion.ViewAssertions.matches;
 import static android.support.test.espresso.matcher.ViewMatchers.isDisplayed;
 import static android.support.test.espresso.matcher.ViewMatchers.withText;
-import static com.jraska.falcon.sample.CICheck.assumeNoCI;
+import static com.jraska.falcon.sample.Assumptions.assumeNoCI;
 import static com.jraska.falcon.sample.asserts.BitmapAssert.assertThatBitmap;
 
 public class FalconDialogInOnCreateTest {

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/FalconFloatingWindowTest.java

@@ -13,7 +13,7 @@
 import static android.support.test.espresso.Espresso.onView;
 import static android.support.test.espresso.action.ViewActions.click;
 import static android.support.test.espresso.matcher.ViewMatchers.withId;
-import static com.jraska.falcon.sample.CICheck.assumeNoCI;
+import static com.jraska.falcon.sample.Assumptions.assumeNoCI;
 import static com.jraska.falcon.sample.asserts.BitmapAssert.assertThatBitmap;
 import static org.assertj.core.api.Assertions.assertThat;
 

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/FalconSpoonTest.java

@@ -4,12 +4,14 @@
 import android.support.test.runner.AndroidJUnit4;
 import com.jraska.falcon.FalconSpoon;
 import org.junit.After;
+import org.junit.Before;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
 import java.io.File;
 
+import static com.jraska.falcon.sample.Assumptions.assumeSpoonPermissions;
 import static com.jraska.falcon.sample.asserts.BitmapFileAssert.assertThatFile;
 import static org.assertj.core.api.Assertions.assertThat;
 
@@ -31,6 +33,11 @@ public class FalconSpoonTest {
 
   //region Setup Methods
 
+  @Before
+  public void before() throws Exception {
+    assumeSpoonPermissions();
+  }
+
   @After
   public void after() throws Exception {
     if (_screenshotFile != null) {

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/FalconTest.java

@@ -15,7 +15,7 @@
 import static android.support.test.espresso.Espresso.onView;
 import static android.support.test.espresso.action.ViewActions.click;
 import static android.support.test.espresso.matcher.ViewMatchers.withId;
-import static com.jraska.falcon.sample.CICheck.assumeNoCI;
+import static com.jraska.falcon.sample.Assumptions.assumeNoCI;
 import static com.jraska.falcon.sample.asserts.BitmapAssert.assertThatBitmap;
 import static com.jraska.falcon.sample.asserts.BitmapFileAssert.assertThatFile;
 import static org.assertj.core.api.Assertions.assertThat;

falcon-sample/src/androidTest/java/com/jraska/falcon/sample/asserts/BitmapAssert.java

@@ -113,8 +113,10 @@ private static int decideBitmapTraversalStep(Bitmap bitmap) {
       return 1;
     } else if (countOfPixels < 1024 * 768) {
       return 2;
-    } else {
+    } else if (countOfPixels < 1600 * 1200) {
       return 4;
+    } else {
+      return 8;
     }
   }
 }