From 5734bbe4e7e509b81c6670c8a40225997593ddef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Parm=C3=A9nides=20GV?= <parmegv@sdf.org>
Date: Fri, 27 Mar 2015 13:40:12 +0100
Subject: Just coded, not tested

---
 .../test/testConfigurationWizard.java              | 72 +++++++++++++++++++++-
 .../test/testDashboardIntegration.java             |  2 +-
 2 files changed, 71 insertions(+), 3 deletions(-)

diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
index 7a19409b..c01e21c6 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
@@ -1,6 +1,14 @@
 package se.leap.bitmaskclient.test;
 
+import android.app.Instrumentation;
+import android.content.Intent;
+import android.graphics.drawable.Drawable;
 import android.test.*;
+import android.view.ActionProvider;
+import android.view.ContextMenu;
+import android.view.MenuItem;
+import android.view.SubMenu;
+import android.view.View;
 import android.widget.*;
 
 import com.robotium.solo.*;
@@ -97,14 +105,74 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
         assertTrue("Provider details dialog did not appear", solo.waitForActivity(AboutActivity.class));
     }
 
-    protected void toDashboard(String provider) {
+    protected void toDashboardAnonymously(String provider) {
         selectProvider(provider);
         useAnonymously();
     }
 
     private void useAnonymously() {
         String text = solo.getString(R.string.use_anonymously_button);
-        solo.clickOnText(text);
+        clickAndWaitForDashboard(text);
+    }
+
+    protected void toDashboardRegistered(String provider) {
+        selectProvider(provider);
+        useRegistered();
+    }
+
+    private void useRegistered() {
+        String text = solo.getString(R.string.signup_or_login_button);
+        clickAndWaitForDashboard(text);
+        login();
+    }
+
+    private void login() {
+        long milliseconds_to_log_in = 40 * 1000;
+        logIn("parmegvtest10", "holahola2");
+        solo.waitForDialogToClose(milliseconds_to_log_in);
+        assertSuccessfulLogin();
+    }
+
+    private void logIn(String username, String password) {
+        solo.clickOnActionBarItem(R.id.login_button);
+        solo.enterText(0, username);
+        solo.enterText(1, password);
+        solo.clickOnText("Log In");
+        solo.waitForDialogToClose();
+    }
+
+    private void assertSuccessfulLogin() {
+        String message = solo.getString(R.string.succesful_authentication_message);
+        assertTrue(solo.waitForText(message));
+    }
+
+    private void clickAndWaitForDashboard(String click_text) {
+        solo.clickOnText(click_text);
         solo.waitForText(solo.getString(R.string.title_activity_dashboard));
     }
+
+    public void testEveryProvider() {
+        toDashboardAnonymously("demo.bitmask.net");
+        connectAndComeBack();
+
+        toDashboardRegistered("calyx.net");
+        connectAndComeBack();
+
+        toDashboardRegistered("riseup.net");
+        connectAndComeBack();
+
+    }
+
+    private void connectAndComeBack() {
+        solo.clickOnView(solo.getView(R.id.eipSwitch));
+        if(!solo.waitForText(solo.getString(R.string.eip_state_connected)))
+            fail();
+
+        solo.clickOnView(solo.getView(R.id.eipSwitch));
+        if(!solo.waitForText(solo.getString(R.string.eip_state_not_connected)))
+            fail();
+
+        solo.clickOnMenuItem(solo.getString(R.string.switch_provider_menu_option));
+
+    }
 }
diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
index ee1917e2..8af20dea 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
@@ -27,7 +27,7 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
         ConnectionManager.setMobileDataEnabled(true, context);
         solo.unlockScreen();
         if (solo.searchText(solo.getString(R.string.configuration_wizard_title)))
-            new testConfigurationWizard(solo).toDashboard("demo.bitmask.net");
+            new testConfigurationWizard(solo).toDashboardAnonymously("demo.bitmask.net");
     }
 
     @Override
-- 
cgit v1.2.3


From 76396cda955c3681db9aa86224f465e3fd2ebf4b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Parm=C3=A9nides=20GV?= <parmegv@sdf.org>
Date: Mon, 30 Mar 2015 17:53:50 +0200
Subject: testEveryProvider() coded and working.

Calyx fails.
---
 app/build.gradle                                   |  2 +-
 .../test/testConfigurationWizard.java              | 25 ++------
 .../test/testDashboardIntegration.java             | 68 ++++++++++++++++++++--
 3 files changed, 71 insertions(+), 24 deletions(-)

diff --git a/app/build.gradle b/app/build.gradle
index 4d3c5b9c..4441cafa 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -39,7 +39,7 @@ android {
 }
 
 dependencies {
-  androidTestCompile 'com.jayway.android.robotium:robotium-solo:5.2.1'
+  androidTestCompile 'com.jayway.android.robotium:robotium-solo:5.3.1'
   compile 'com.jakewharton:butterknife:6.0.0+'
   provided 'com.squareup.dagger:dagger-compiler:1.2.2+'
   compile 'com.github.pedrovgs:renderers:1.3+'
diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
index c01e21c6..efb843f7 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
@@ -40,7 +40,7 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
 
     @Override
     protected void tearDown() throws Exception {
-        solo.finishOpenedActivities();
+
     }
 
     public void testListProviders() {
@@ -115,6 +115,11 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
         clickAndWaitForDashboard(text);
     }
 
+    private void clickAndWaitForDashboard(String click_text) {
+        solo.clickOnText(click_text);
+        assertTrue(solo.waitForActivity(Dashboard.class, 5000));
+    }
+
     protected void toDashboardRegistered(String provider) {
         selectProvider(provider);
         useRegistered();
@@ -146,23 +151,6 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
         assertTrue(solo.waitForText(message));
     }
 
-    private void clickAndWaitForDashboard(String click_text) {
-        solo.clickOnText(click_text);
-        solo.waitForText(solo.getString(R.string.title_activity_dashboard));
-    }
-
-    public void testEveryProvider() {
-        toDashboardAnonymously("demo.bitmask.net");
-        connectAndComeBack();
-
-        toDashboardRegistered("calyx.net");
-        connectAndComeBack();
-
-        toDashboardRegistered("riseup.net");
-        connectAndComeBack();
-
-    }
-
     private void connectAndComeBack() {
         solo.clickOnView(solo.getView(R.id.eipSwitch));
         if(!solo.waitForText(solo.getString(R.string.eip_state_connected)))
@@ -173,6 +161,5 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
             fail();
 
         solo.clickOnMenuItem(solo.getString(R.string.switch_provider_menu_option));
-
     }
 }
diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
index 8af20dea..bcabc46c 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
@@ -2,6 +2,7 @@ package se.leap.bitmaskclient.test;
 
 import android.content.*;
 import android.test.*;
+import android.widget.*;
 
 import com.robotium.solo.*;
 
@@ -108,6 +109,7 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
 
     public void testLogInAndOut() {
         long milliseconds_to_log_in = 40 * 1000;
+        solo.clickOnActionBarItem(R.id.login_button);
         logIn("parmegvtest1", " S_Zw3'-");
         solo.waitForDialogToClose(milliseconds_to_log_in);
         assertSuccessfulLogin();
@@ -116,7 +118,6 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
     }
 
     private void logIn(String username, String password) {
-        solo.clickOnActionBarItem(R.id.login_button);
         solo.enterText(0, username);
         solo.enterText(1, password);
         solo.clickOnText("Log In");
@@ -124,8 +125,7 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
     }
 
     private void assertSuccessfulLogin() {
-        String message = solo.getString(R.string.succesful_authentication_message);
-        assertTrue(solo.waitForText(message));
+        assertTrue(solo.waitForText("is logged in"));
     }
 
     private void logOut() {
@@ -149,11 +149,71 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
     }
 
     public void testSwitchProvider() {
+	tapSwitchProvider();
+        solo.goBack();
+    }
+
+    private void tapSwitchProvider() {
         solo.clickOnMenuItem(solo.getString(R.string.switch_provider_menu_option));
         solo.waitForActivity(ConfigurationWizard.class);
-        solo.goBack();
     }
 
+    public void testEveryProvider() {
+	changeProvider("demo.bitmask.net");
+	connectVpn();
+	disconnectVpn();
+
+	changeProvider("calyx.net");
+	connectVpn();
+	disconnectVpn();
+	
+	changeProvider("riseup.net");
+	connectVpn();
+	disconnectVpn();
+    }
+
+    private void changeProvider(String provider) {
+	tapSwitchProvider();
+        solo.clickOnText(provider);
+	useRegistered();
+    }
+
+    private void connectVpn() {
+	Switch vpn_switch = (Switch)solo.getView(R.id.eipSwitch);
+	assertFalse(vpn_switch.isChecked());
+	
+	solo.clickOnView(vpn_switch);
+        turningEipOn();
+    }
+
+    private void disconnectVpn() {
+	Switch vpn_switch = (Switch)solo.getView(R.id.eipSwitch);
+	assertTrue(vpn_switch.isChecked());
+
+	solo.clickOnView(vpn_switch);
+	solo.clickOnText("Yes");
+        turningEipOff();
+	
+    }
+
+    private void useRegistered() {
+        String text = solo.getString(R.string.signup_or_login_button);
+        clickAndWaitForDashboard(text);
+        login();
+    }
+
+    private void clickAndWaitForDashboard(String click_text) {
+        solo.clickOnText(click_text);
+        assertTrue(solo.waitForActivity(Dashboard.class, 5000));
+    }
+
+    private void login() {
+        long milliseconds_to_log_in = 40 * 1000;
+        logIn("parmegvtest10", "holahola2");
+        solo.waitForDialogToClose(milliseconds_to_log_in);
+        assertSuccessfulLogin();
+    }
+    
     /*public void testReboot() {
         runAdbCommand("shell am broadcast -a android.intent.action.BOOT_COMPLETED");
     }*/
-- 
cgit v1.2.3


From 9ddbecd55d29923543f224d773c5c16e98276332 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Parm=C3=A9nides=20GV?= <parmegv@sdf.org>
Date: Mon, 30 Mar 2015 18:16:13 +0200
Subject: Cleaning moved methods.

---
 .../test/testConfigurationWizard.java              | 51 ----------------------
 1 file changed, 51 deletions(-)

diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
index efb843f7..1fa4cf2f 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testConfigurationWizard.java
@@ -1,14 +1,6 @@
 package se.leap.bitmaskclient.test;
 
-import android.app.Instrumentation;
-import android.content.Intent;
-import android.graphics.drawable.Drawable;
 import android.test.*;
-import android.view.ActionProvider;
-import android.view.ContextMenu;
-import android.view.MenuItem;
-import android.view.SubMenu;
-import android.view.View;
 import android.widget.*;
 
 import com.robotium.solo.*;
@@ -119,47 +111,4 @@ public class testConfigurationWizard extends ActivityInstrumentationTestCase2<Co
         solo.clickOnText(click_text);
         assertTrue(solo.waitForActivity(Dashboard.class, 5000));
     }
-
-    protected void toDashboardRegistered(String provider) {
-        selectProvider(provider);
-        useRegistered();
-    }
-
-    private void useRegistered() {
-        String text = solo.getString(R.string.signup_or_login_button);
-        clickAndWaitForDashboard(text);
-        login();
-    }
-
-    private void login() {
-        long milliseconds_to_log_in = 40 * 1000;
-        logIn("parmegvtest10", "holahola2");
-        solo.waitForDialogToClose(milliseconds_to_log_in);
-        assertSuccessfulLogin();
-    }
-
-    private void logIn(String username, String password) {
-        solo.clickOnActionBarItem(R.id.login_button);
-        solo.enterText(0, username);
-        solo.enterText(1, password);
-        solo.clickOnText("Log In");
-        solo.waitForDialogToClose();
-    }
-
-    private void assertSuccessfulLogin() {
-        String message = solo.getString(R.string.succesful_authentication_message);
-        assertTrue(solo.waitForText(message));
-    }
-
-    private void connectAndComeBack() {
-        solo.clickOnView(solo.getView(R.id.eipSwitch));
-        if(!solo.waitForText(solo.getString(R.string.eip_state_connected)))
-            fail();
-
-        solo.clickOnView(solo.getView(R.id.eipSwitch));
-        if(!solo.waitForText(solo.getString(R.string.eip_state_not_connected)))
-            fail();
-
-        solo.clickOnMenuItem(solo.getString(R.string.switch_provider_menu_option));
-    }
 }
-- 
cgit v1.2.3


From 66673b5b60eb99d5682a440f6d90a08ccb1835b6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Parm=C3=A9nides=20GV?= <parmegv@sdf.org>
Date: Mon, 30 Mar 2015 18:37:39 +0200
Subject: Wait for the vpn certificate download to start vpn in test

---
 .../se/leap/bitmaskclient/test/testDashboardIntegration.java | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
index bcabc46c..d2fb9901 100644
--- a/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
+++ b/app/src/androidTest/java/se/leap/bitmaskclient/test/testDashboardIntegration.java
@@ -162,26 +162,28 @@ public class testDashboardIntegration extends ActivityInstrumentationTestCase2<D
 	changeProvider("demo.bitmask.net");
 	connectVpn();
 	disconnectVpn();
-
-	changeProvider("calyx.net");
-	connectVpn();
-	disconnectVpn();
 	
 	changeProvider("riseup.net");
 	connectVpn();
 	disconnectVpn();
+
+	changeProvider("calyx.net");
+	connectVpn();
+	disconnectVpn();
     }
 
     private void changeProvider(String provider) {
 	tapSwitchProvider();
         solo.clickOnText(provider);
 	useRegistered();
+	solo.waitForText("Downloading VPN certificate");
+	assertDisconnected();
     }
 
     private void connectVpn() {
 	Switch vpn_switch = (Switch)solo.getView(R.id.eipSwitch);
 	assertFalse(vpn_switch.isChecked());
-	
+
 	solo.clickOnView(vpn_switch);
         turningEipOn();
     }
-- 
cgit v1.2.3