Merge "Moved wi-fi switch to hostside." into nyc-dev
am: af09243
* commit 'af092432df0c7ff1ebef69ec423c91b80e568e16':
Moved wi-fi switch to hostside.
Change-Id: I8c7ac6524da761a0977d17e5ee3abe84f8e2cb90
diff --git a/tests/cts/hostside/app/src/com/android/cts/net/hostside/AbstractRestrictBackgroundNetworkTestCase.java b/tests/cts/hostside/app/src/com/android/cts/net/hostside/AbstractRestrictBackgroundNetworkTestCase.java
index f33d434..41fd638 100644
--- a/tests/cts/hostside/app/src/com/android/cts/net/hostside/AbstractRestrictBackgroundNetworkTestCase.java
+++ b/tests/cts/hostside/app/src/com/android/cts/net/hostside/AbstractRestrictBackgroundNetworkTestCase.java
@@ -247,8 +247,9 @@
protected void assertDelayedShellCommand(String command, String expectedResult)
throws Exception {
final int maxTries = 5;
+ String result = "";
for (int i = 1; i <= maxTries; i++) {
- final String result = executeShellCommand(command).trim();
+ result = executeShellCommand(command).trim();
if (result.equals(expectedResult))
return;
Log.v(TAG, "Command '" + command + "' returned '" + result + " instead of '"
@@ -256,7 +257,7 @@
Thread.sleep(SECOND_IN_MS);
}
fail("Command '" + command + "' did not return '" + expectedResult + "' after " + maxTries
- + " attempts");
+ + " attempts. Last result: '" + result + "'");
}
protected void setMeteredNetwork() throws Exception {
@@ -265,6 +266,8 @@
if (metered) {
Log.d(TAG, "Active network already metered: " + info);
return;
+ } else {
+ Log.w(TAG, "Active network not metered: " + info);
}
final String netId = setWifiMeteredStatus(true);
assertTrue("Could not set wifi '" + netId + "' as metered ("
@@ -274,7 +277,10 @@
}
protected String setWifiMeteredStatus(boolean metered) throws Exception {
- mWfm.setWifiEnabled(true);
+ // We could call setWifiEnabled() here, but it might take sometime to be in a consistent
+ // state (for example, if one of the saved network is not properly authenticated), so it's
+ // better to let the hostside test take care of that.
+ assertTrue("wi-fi is disabled", mWfm.isWifiEnabled());
// TODO: if it's not guaranteed the device has wi-fi, we need to change the tests
// to make the actual verification of restrictions optional.
final String ssid = mWfm.getConnectionInfo().getSSID();
diff --git a/tests/cts/hostside/src/com/android/cts/net/HostsideNetworkTestCase.java b/tests/cts/hostside/src/com/android/cts/net/HostsideNetworkTestCase.java
index 08fb887..ab1b7d6 100644
--- a/tests/cts/hostside/src/com/android/cts/net/HostsideNetworkTestCase.java
+++ b/tests/cts/hostside/src/com/android/cts/net/HostsideNetworkTestCase.java
@@ -63,6 +63,8 @@
assertNotNull(mAbi);
assertNotNull(mCtsBuild);
+ assertTrue("device not connected to network", getDevice().checkConnectivity());
+
uninstallPackage(TEST_PKG, false);
installPackage(TEST_APK);
}