Adds constraint to verify wifi, charging and vpn status

ooni · May 4, 2024 · 2fd1804 · 2fd1804
1 parent 497f68e
commit 2fd1804
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/app/src/main/java/org/openobservatory/ooniprobe/common/service/ServiceUtil.java b/app/src/main/java/org/openobservatory/ooniprobe/common/service/ServiceUtil.java
@@ -98,8 +98,16 @@ public static void startRunTestServiceUnattended(Application app) {
 	 * Application context used to start `RunTestService`.
 	 */
     public static void startRunTestServiceNetworkChanged(Application app) {
+        app.getServiceComponent().inject(d);
+
+        boolean isVPNInUse = ReachabilityManager.isVPNinUse(app);
+
+        OONICheckInConfig config = app.getOONICheckInConfig();
+
+        if (!d.generateAutoRunServiceSuite.shouldStart(config.isOnWiFi(),config.isCharging(), isVPNInUse)) {
+            return;
+        }
 
-		// TODO: (aanorbel) - Enforce constraints required before tests are run.
 		//  Tag tests with identifier required to set Settings$Options#software_name
         ArrayList<AbstractSuite> testSuites = new ArrayList<>();
         testSuites.add(new WebsitesSuite());