Add logs to NetworkCallbackTest to diagnose issues
Bug: 157391946 Test: NetworkCallbackTests Merged-In: I552a1ad05f97a2a9f83f5f7de77d2e7b64084ddc Change-Id: Ic08df1908828f1487253a2b5dfb14e46442942b8 (cherry picked from commit 446a3ef6c2f9671dd7509906dbb647e1304ade78, aosp/1318216)
This commit is contained in:
committed by
Chalard Jean
parent
fd4a967604
commit
5df6449e3e
@@ -130,14 +130,11 @@ public class NetworkCallbackTest extends AbstractRestrictBackgroundNetworkTestCa
|
||||
setLastCallback(CallbackState.CAPABILITIES, network, cap);
|
||||
}
|
||||
|
||||
public void expectLostCallback(Network expectedNetwork) {
|
||||
expectCallback(CallbackState.LOST, expectedNetwork, null);
|
||||
}
|
||||
|
||||
public Network expectAvailableCallbackAndGetNetwork() {
|
||||
final CallbackInfo cb = nextCallback(TEST_CALLBACK_TIMEOUT_MS);
|
||||
if (cb.state != CallbackState.AVAILABLE) {
|
||||
fail("Network is not available");
|
||||
fail("Network is not available. Instead obtained the following callback :"
|
||||
+ cb);
|
||||
}
|
||||
return cb.network;
|
||||
}
|
||||
@@ -152,7 +149,7 @@ public class NetworkCallbackTest extends AbstractRestrictBackgroundNetworkTestCa
|
||||
do {
|
||||
final CallbackInfo cb = nextCallback((int) (deadline - System.currentTimeMillis()));
|
||||
if (cb.state == CallbackState.BLOCKED_STATUS) {
|
||||
assertEquals(expectBlocked, (Boolean) cb.arg);
|
||||
assertEquals(expectBlocked, cb.arg);
|
||||
return;
|
||||
}
|
||||
} while (System.currentTimeMillis() <= deadline);
|
||||
@@ -165,10 +162,10 @@ public class NetworkCallbackTest extends AbstractRestrictBackgroundNetworkTestCa
|
||||
final NetworkCapabilities cap = (NetworkCapabilities) cb.arg;
|
||||
assertEquals(expectedNetwork, cb.network);
|
||||
assertEquals(CallbackState.CAPABILITIES, cb.state);
|
||||
if (hasCapability) {
|
||||
assertTrue(cap.hasCapability(capability));
|
||||
} else {
|
||||
assertFalse(cap.hasCapability(capability));
|
||||
if (hasCapability != cap.hasCapability(capability)) {
|
||||
fail("NetworkCapabilities callback "
|
||||
+ (hasCapability ? "missing expected" : "has unexpected")
|
||||
+ " capability. " + cb);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user