LineageOS/android_packages_modules_Connectivity
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Updating timeout for Per-App CTS

Browse Source 
Increasing the timeout values for waiting for callbacks for the per-app
CTS tests to match the timeout value used in the waitForAvailable()
helper method.

Bug: 191230288
Test: atest CtsNetTestCases:android.net.cts.ConnectivityManagerTest
Change-Id: I49e5a99f18a5651370e6914f30be659134b537ff
This commit is contained in:
James Mattis
2021-06-16 19:11:45 -07:00
parent 27586f8210
commit 4dad9710e1
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tests/cts/net/src/android/net/cts/ConnectivityManagerTest.java
View File

@@ -2197,7 +2197,8 @@ public class ConnectivityManagerTest {
// Validate when setting unmetered to metered, unmetered is lost and replaced by the // Validate when setting unmetered to metered, unmetered is lost and replaced by the
// network with the TEST transport. // network with the TEST transport.
setWifiMeteredStatusAndWait(ssid, true /* isMetered */); setWifiMeteredStatusAndWait(ssid, true /* isMetered */);
defaultCallback.expectCallback(CallbackEntry.LOST, wifiNetwork); defaultCallback.expectCallback(CallbackEntry.LOST, wifiNetwork,
NETWORK_CALLBACK_TIMEOUT_MS);
waitForAvailable(defaultCallback, tnt.getNetwork()); waitForAvailable(defaultCallback, tnt.getNetwork());
// Depending on if this device has cellular connectivity or not, multiple available // Depending on if this device has cellular connectivity or not, multiple available
// callbacks may be received. Eventually, metered Wi-Fi should be the final available // callbacks may be received. Eventually, metered Wi-Fi should be the final available
@@ -2207,7 +2208,8 @@ public class ConnectivityManagerTest {
} finally { } finally {
// Validate that removing the test network will fallback to the default network. // Validate that removing the test network will fallback to the default network.
runWithShellPermissionIdentity(tnt::teardown); runWithShellPermissionIdentity(tnt::teardown);
defaultCallback.expectCallback(CallbackEntry.LOST, tnt.getNetwork()); defaultCallback.expectCallback(CallbackEntry.LOST, tnt.getNetwork(),
NETWORK_CALLBACK_TIMEOUT_MS);
waitForAvailable(defaultCallback); waitForAvailable(defaultCallback);
setWifiMeteredStatusAndWait(ssid, oldMeteredValue); setWifiMeteredStatusAndWait(ssid, oldMeteredValue);
@@ -2243,7 +2245,8 @@ public class ConnectivityManagerTest {
waitForAvailable(systemDefaultCallback, wifiNetwork); waitForAvailable(systemDefaultCallback, wifiNetwork);
} finally { } finally {
runWithShellPermissionIdentity(tnt::teardown); runWithShellPermissionIdentity(tnt::teardown);
defaultCallback.expectCallback(CallbackEntry.LOST, tnt.getNetwork()); defaultCallback.expectCallback(CallbackEntry.LOST, tnt.getNetwork(),
NETWORK_CALLBACK_TIMEOUT_MS);
// This network preference should only ever use the test network therefore available // This network preference should only ever use the test network therefore available
// should not trigger when the test network goes down (e.g. switch to cellular). // should not trigger when the test network goes down (e.g. switch to cellular).