Merge branch 'master' of github.com:ooni/probe-android into feat/pr-n…

…umber-version-codes
ooni · Jan 25, 2024 · de8c9b8 · de8c9b8
2 parents de08458 + 6af25fd
commit de8c9b8
Show file tree

Hide file tree

Showing 7 changed files with 88 additions and 122 deletions.
diff --git a/app/build.gradle b/app/build.gradle
@@ -13,8 +13,8 @@ android {
  applicationId 'org.openobservatory.ooniprobe'
  minSdk libs.versions.minSdk.get().toInteger()
  targetSdk libs.versions.targetSdk.get().toInteger()
- versionName '3.8.5.1'
- versionCode 108
+ versionName '3.8.6'
+ versionCode 110
  testInstrumentationRunner "org.openobservatory.ooniprobe.TestAndroidJUnitRunner"
  buildConfigField 'String', 'OONI_API_BASE_URL', '"https://api.ooni.io/"'
  buildConfigField 'String', 'NOTIFICATION_SERVER', '"https://countly.ooni.io"'

diff --git a/...rc/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/CircumventionTest.java b/...rc/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/CircumventionTest.java
@@ -113,41 +113,4 @@ public void testBlockedTor() {
  onView(withId(R.id.authorities)).check(matches(withText(formattedAuthorities)));
  assertMeasurementRuntimeAndNetwork(measurement, testResult.network);
  }
-
- @Test
- public void testSuccessfulRiseUpVPN() {
- // Arrange
- Result testResult = ResultFactory.createAndSave(new CircumventionSuite(), 3, 0);
- Measurement measurement = testResult.getMeasurement("riseupvpn");
-
- // Act
- launchDetails(testResult.id);
- onView(withText("RiseupVPN Test")).check(matches(isDisplayed())).perform(click());
-
- // Assert
- assertMeasurementOutcome(true);
- onView(withId(R.id.bootstrap_value)).check(matches(withText(TEST_RESULTS_AVAILABLE)));
- onView(withId(R.id.openvpn_value)).check(matches(withText(TEST_RESULTS_AVAILABLE)));
- onView(withId(R.id.bridges_value)).check(matches(withText(TEST_RESULTS_AVAILABLE)));
- assertMeasurementRuntimeAndNetwork(measurement, testResult.network);
- }
-
- @Test
- public void testBlockedRiseUpVPN() {
- // Arrange
- Result testResult = ResultFactory.createAndSave(new CircumventionSuite(), 0, 3);
- Measurement measurement = testResult.getMeasurement("riseupvpn");
-
- // Act
- launchDetails(testResult.id);
- onView(withText("RiseupVPN Test")).check(matches(isDisplayed())).perform(click());
-
- // Assert
- assertMeasurementOutcome(false);
- onView(withId(R.id.bootstrap_value)).check(matches(withText("Blocked")));
- onView(withId(R.id.openvpn_value)).check(matches(withText("1 blocked")));
- onView(withId(R.id.bridges_value)).check(matches(withText("1 blocked")));
- assertMeasurementRuntimeAndNetwork(measurement, testResult.network);
- }
-
 }
diff --git a/app/src/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/PerformanceTest.java b/app/src/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/PerformanceTest.java
@@ -1,5 +1,18 @@
 package org.openobservatory.ooniprobe.ui.resultdetails;
 
+import static androidx.test.espresso.Espresso.onData;
+import static androidx.test.espresso.Espresso.onView;
+import static androidx.test.espresso.action.ViewActions.click;
+import static androidx.test.espresso.action.ViewActions.swipeLeft;
+import static androidx.test.espresso.assertion.ViewAssertions.matches;
+import static androidx.test.espresso.matcher.ViewMatchers.isDisplayed;
+import static androidx.test.espresso.matcher.ViewMatchers.withId;
+import static androidx.test.espresso.matcher.ViewMatchers.withText;
+import static org.hamcrest.Matchers.allOf;
+import static org.hamcrest.Matchers.anything;
+import static org.hamcrest.Matchers.containsString;
+
+import androidx.test.espresso.assertion.ViewAssertions;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
 
 import com.schibsted.spain.barista.rule.flaky.AllowFlaky;
@@ -20,19 +33,6 @@
 import org.openobservatory.ooniprobe.test.test.Ndt;
 import org.openobservatory.ooniprobe.utils.FormattingUtils;
 
-import static androidx.test.espresso.Espresso.onView;
-import static androidx.test.espresso.action.ViewActions.click;
-import static androidx.test.espresso.action.ViewActions.swipeLeft;
-import static androidx.test.espresso.assertion.ViewAssertions.matches;
-import static androidx.test.espresso.contrib.RecyclerViewActions.actionOnItemAtPosition;
-import static androidx.test.espresso.contrib.RecyclerViewActions.scrollToPosition;
-import static androidx.test.espresso.matcher.ViewMatchers.isDisplayed;
-import static androidx.test.espresso.matcher.ViewMatchers.withId;
-import static androidx.test.espresso.matcher.ViewMatchers.withText;
-import static org.hamcrest.Matchers.allOf;
-import static org.hamcrest.Matchers.containsString;
-import static org.openobservatory.ooniprobe.ui.utils.RecyclerViewMatcher.withRecyclerView;
-
 @RunWith(AndroidJUnit4.class)
 public class PerformanceTest extends MeasurementAbstractTest {
 
@@ -112,35 +112,34 @@ public void testListOfMeasurements() {
  launchDetails(testResult.id);
 
  // Assert
- onView(withId(R.id.recyclerView))
- .perform(scrollToPosition(0));
-
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(0, R.id.data1))
- .check(matches(withText(download + " " + downloadUnity)));
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(0, R.id.data2))
- .check(matches(withText(upload + " " + uploadUnity)));
-
- onView(withId(R.id.recyclerView))
- .perform(scrollToPosition(1));
-
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(1, R.id.data1))
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(0)
+ .onChildView(withId(R.id.data1))
+ .check(ViewAssertions.matches(withText(download + " " + downloadUnity)));
+
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(0)
+ .onChildView(withId(R.id.data2))
+ .check(matches(withText(upload + " " + uploadUnity)));
+
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(1)
+ .onChildView(withId(R.id.data1))
  .check(matches(withText(videoQuality)));
 
- onView(withId(R.id.recyclerView))
- .perform(scrollToPosition(2));
-
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(2, R.id.data1))
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(2)
+ .onChildView(withId(R.id.data1))
  .check(matches(withText(notDetected)));
 
- onView(withId(R.id.recyclerView))
- .perform(scrollToPosition(3));
-
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(3, R.id.data1))
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(3)
+ .onChildView(withId(R.id.data1))
  .check(matches(withText(notDetected)));
  }
 
@@ -174,7 +173,7 @@ public void testNDT() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(0, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(0).perform(click());
 
  // Assert
  onView(withText(download + downloadUnity)).check(matches(isDisplayed()));
@@ -202,7 +201,7 @@ public void testStreaming() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(1, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(1).perform(click());
 
  // Assert
  onView(withText(containsString(videoQuality))).check(matches(isDisplayed()));
@@ -219,7 +218,7 @@ public void testRequestLine() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(2, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(2).perform(click());
 
  // Assert
  onView(withText(getResourceString(R.string.TestResults_Details_Middleboxes_HTTPInvalidRequestLine_NotFound_Hero_Title))).check(matches(isDisplayed()));
@@ -234,7 +233,7 @@ public void testRequestLineDetection() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(2, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(2).perform(click());
 
  // Assert
  onView(withText(getResourceString(R.string.TestResults_Details_Middleboxes_HTTPInvalidRequestLine_Found_Hero_Title))).check(matches(isDisplayed()));
@@ -249,7 +248,7 @@ public void testFieldManipulation() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(3, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(3).perform(click());
 
  // Assert
  onView(withText(getResourceString(R.string.TestResults_Details_Middleboxes_HTTPHeaderFieldManipulation_NotFound_Hero_Title))).check(matches(isDisplayed()));
@@ -264,7 +263,7 @@ public void testFieldManipulationDetection() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(3, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(3).perform(click());
 
  // Assert
  onView(withText(getResourceString(R.string.TestResults_Details_Middleboxes_HTTPHeaderFieldManipulation_Found_Hero_Title))).check(matches(isDisplayed()));

diff --git a/app/src/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/WebsitesTest.java b/app/src/androidTest/java/org/openobservatory/ooniprobe/ui/resultdetails/WebsitesTest.java
@@ -18,6 +18,7 @@
 import java.util.List;
 import java.util.concurrent.TimeUnit;
 
+import static androidx.test.espresso.Espresso.onData;
 import static androidx.test.espresso.Espresso.onView;
 import static androidx.test.espresso.action.ViewActions.click;
 import static androidx.test.espresso.assertion.ViewAssertions.matches;
@@ -27,6 +28,7 @@
 import static androidx.test.espresso.matcher.ViewMatchers.isRoot;
 import static androidx.test.espresso.matcher.ViewMatchers.withId;
 import static androidx.test.espresso.matcher.ViewMatchers.withText;
+import static org.hamcrest.Matchers.anything;
 import static org.hamcrest.Matchers.containsString;
 import static org.openobservatory.ooniprobe.ui.utils.RecyclerViewMatcher.withRecyclerView;
 import static org.openobservatory.ooniprobe.ui.utils.ViewMatchers.waitPartialText;
@@ -64,12 +66,11 @@ public void checkListOfMeasurementsTest() {
 
  // Assert
  for (int i = 0; i < measurements.size(); i++) {
- onView(withId(R.id.recyclerView))
- .perform(scrollToPosition(i));
-
- onView(withRecyclerView(R.id.recyclerView)
- .atPositionOnView(i, R.id.text))
- .check(matches(withText(containsString(measurements.get(i).getUrlString()))));
+ onData(anything())
+ .inAdapterView(withId(R.id.recyclerView))
+ .atPosition(i)
+ .onChildView(withId(R.id.text))
+ .check(matches(withText(containsString(measurements.get(i).getUrlString()))));
  }
 
  }
@@ -83,7 +84,7 @@ public void testSucceed() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(0, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(0).perform(click());
 
  // Assert
  onView(withText(headerOutcome)).check(matches(isDisplayed()));
@@ -99,7 +100,7 @@ public void testBlocked() {
 
  // Act
  launchDetails(testResult.id);
- onView(withId(R.id.recyclerView)).perform(actionOnItemAtPosition(0, click()));
+ onData(anything()).inAdapterView(withId(R.id.recyclerView)).atPosition(0).perform(click());
 
  // Assert
  onView(withText(headerOutcome)).check(matches(isDisplayed()));