Merge "ConnectivityServiceTest: fix testNetworkInfoOfTypeNone"

This commit is contained in:
Treehugger Robot
2017-07-19 07:54:45 +00:00
committed by Gerrit Code Review

View File

@@ -881,6 +881,7 @@ public class ConnectivityServiceTest extends AndroidTestCase {
} }
private void verifyNoNetwork() { private void verifyNoNetwork() {
waitForIdle();
// Test getActiveNetworkInfo() // Test getActiveNetworkInfo()
assertNull(mCm.getActiveNetworkInfo()); assertNull(mCm.getActiveNetworkInfo());
// Test getActiveNetwork() // Test getActiveNetwork()
@@ -3277,7 +3278,7 @@ public class ConnectivityServiceTest extends AndroidTestCase {
ConditionVariable broadcastCV = waitForConnectivityBroadcasts(1); ConditionVariable broadcastCV = waitForConnectivityBroadcasts(1);
verifyNoNetwork(); verifyNoNetwork();
MockNetworkAgent lowpanNetwork = new MockNetworkAgent(TRANSPORT_WIFI_AWARE); MockNetworkAgent wifiAware = new MockNetworkAgent(TRANSPORT_WIFI_AWARE);
assertNull(mCm.getActiveNetworkInfo()); assertNull(mCm.getActiveNetworkInfo());
Network[] allNetworks = mCm.getAllNetworks(); Network[] allNetworks = mCm.getAllNetworks();
@@ -3291,19 +3292,19 @@ public class ConnectivityServiceTest extends AndroidTestCase {
final TestNetworkCallback callback = new TestNetworkCallback(); final TestNetworkCallback callback = new TestNetworkCallback();
mCm.registerNetworkCallback(request, callback); mCm.registerNetworkCallback(request, callback);
// Bring up lowpan. // Bring up wifi aware network.
lowpanNetwork.connect(false, false); wifiAware.connect(false, false);
callback.expectAvailableCallbacks(lowpanNetwork); callback.expectAvailableCallbacks(wifiAware);
assertNull(mCm.getActiveNetworkInfo()); assertNull(mCm.getActiveNetworkInfo());
assertNull(mCm.getActiveNetwork()); assertNull(mCm.getActiveNetwork());
// TODO: getAllNetworkInfo is dirty and returns a non-empty array rght from the start // TODO: getAllNetworkInfo is dirty and returns a non-empty array right from the start
// of this test. Fix it and uncomment the assert below. // of this test. Fix it and uncomment the assert below.
//assertEmpty(mCm.getAllNetworkInfo()); //assertEmpty(mCm.getAllNetworkInfo());
// Disconnect lowpan. // Disconnect wifi aware network.
lowpanNetwork.disconnect(); wifiAware.disconnect();
callback.expectCallback(CallbackState.LOST, lowpanNetwork); callback.expectCallbackLike((info) -> info.state == CallbackState.LOST, TIMEOUT_MS);
mCm.unregisterNetworkCallback(callback); mCm.unregisterNetworkCallback(callback);
verifyNoNetwork(); verifyNoNetwork();