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

Merge changes I0789d9bd,Idf4cf74a

Browse Source 
* changes:
  Disable ignoring validation on roam just after boot.
  Minor cleanups for ignore validation failures after roam tests.
This commit is contained in:
Treehugger Robot
2022-10-25 02:15:29 +00:00
committed by Gerrit Code Review
parent 9b9e2911bc 580d0d53be
commit a40faea28d
2 changed files with 80 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

98
tests/unit/java/com/android/server/ConnectivityServiceTest.java
View File

@@ -16782,23 +16782,25 @@ public class ConnectivityServiceTest {
assertThrows(NullPointerException.class, () -> mService.unofferNetwork(null));
}
public void doTestIgnoreValidationAfterRoam(final boolean enabled) throws Exception {
assumeFalse(SdkLevel.isAtLeastT());
doReturn(enabled ? 5000 : -1).when(mResources)
public void doTestIgnoreValidationAfterRoam(int resValue, final boolean enabled)
throws Exception {
doReturn(resValue).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
final String bssid1 = "AA:AA:AA:AA:AA:AA";
final String bssid2 = "BB:BB:BB:BB:BB:BB";
mCellNetworkAgent = new TestNetworkAgentWrapper(TRANSPORT_CELLULAR);
mCellNetworkAgent.connect(true);
NetworkCapabilities wifiNc1 = new NetworkCapabilities()
.addCapability(NET_CAPABILITY_INTERNET)
.addCapability(NET_CAPABILITY_NOT_VPN)
.addCapability(NET_CAPABILITY_NOT_RESTRICTED)
.addCapability(NET_CAPABILITY_TRUSTED)
.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)
.addTransportType(TRANSPORT_WIFI)
.setTransportInfo(new WifiInfo.Builder().setBssid("AA:AA:AA:AA:AA:AA").build());
NetworkCapabilities wifiNc2 = new NetworkCapabilities()
.addCapability(NET_CAPABILITY_INTERNET)
.addCapability(NET_CAPABILITY_NOT_VCN_MANAGED)
.addTransportType(TRANSPORT_WIFI)
.setTransportInfo(new WifiInfo.Builder().setBssid("BB:BB:BB:BB:BB:BB").build());
.setTransportInfo(new WifiInfo.Builder().setBssid(bssid1).build());
NetworkCapabilities wifiNc2 = new NetworkCapabilities(wifiNc1)
.setTransportInfo(new WifiInfo.Builder().setBssid(bssid2).build());
final LinkProperties wifiLp = new LinkProperties();
wifiLp.setInterfaceName(WIFI_IFNAME);
mWiFiNetworkAgent = new TestNetworkAgentWrapper(TRANSPORT_WIFI, wifiLp, wifiNc1);
@@ -16862,6 +16864,7 @@ public class ConnectivityServiceTest {
wifiNetworkCallback.expectCapabilitiesWithout(NET_CAPABILITY_PARTIAL_CONNECTIVITY,
mWiFiNetworkAgent);
}
mCm.unregisterNetworkCallback(wifiNetworkCallback);
// Wi-Fi roams from wifiNc1 to wifiNc2, and now becomes really invalid. If validation
// failures after roam are not ignored, this will cause cell to become the default network.
@@ -16872,39 +16875,76 @@ public class ConnectivityServiceTest {
mWiFiNetworkAgent.setNetworkInvalid(false /* isStrictMode */);
mCm.reportNetworkConnectivity(mWiFiNetworkAgent.getNetwork(), false);
if (!enabled) {
if (enabled) {
// Network validation failed, but the result will be ignored.
assertTrue(mCm.getNetworkCapabilities(mWiFiNetworkAgent.getNetwork()).hasCapability(
NET_CAPABILITY_VALIDATED));
mWiFiNetworkAgent.setNetworkValid(false);
// Behavior of after config_validationFailureAfterRoamIgnoreTimeMillis
ConditionVariable waitForValidationBlock = new ConditionVariable();
doReturn(50).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
// Wi-Fi roaming from wifiNc2 to wifiNc1.
mWiFiNetworkAgent.setNetworkCapabilities(wifiNc1, true);
mWiFiNetworkAgent.setNetworkInvalid(false);
waitForValidationBlock.block(150);
mCm.reportNetworkConnectivity(mWiFiNetworkAgent.getNetwork(), false);
mDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent);
} else {
mDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent);
return;
}
// Network validation failed, but the result will be ignored.
assertTrue(mCm.getNetworkCapabilities(mWiFiNetworkAgent.getNetwork()).hasCapability(
NET_CAPABILITY_VALIDATED));
mWiFiNetworkAgent.setNetworkValid(false);
// Wi-Fi is still connected and would become the default network if cell were to
// disconnect. This assertion ensures that the switch to cellular was not caused by
// Wi-Fi disconnecting (e.g., because the capability change to wifiNc2 caused it
// to stop satisfying the default request).
mCellNetworkAgent.disconnect();
mDefaultNetworkCallback.expectCallback(CallbackEntry.LOST, mCellNetworkAgent);
mDefaultNetworkCallback.expectAvailableCallbacksUnvalidated(mWiFiNetworkAgent);
// Behavior of after config_validationFailureAfterRoamIgnoreTimeMillis
ConditionVariable waitForValidationBlock = new ConditionVariable();
doReturn(50).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
// Wi-Fi roaming from wifiNc2 to wifiNc1.
mWiFiNetworkAgent.setNetworkCapabilities(wifiNc1, true);
mWiFiNetworkAgent.setNetworkInvalid(false);
waitForValidationBlock.block(150);
mCm.reportNetworkConnectivity(mWiFiNetworkAgent.getNetwork(), false);
mDefaultNetworkCallback.expectAvailableCallbacksValidated(mCellNetworkAgent);
mCm.unregisterNetworkCallback(wifiNetworkCallback);
}
@Test
public void testIgnoreValidationAfterRoamDisabled() throws Exception {
doTestIgnoreValidationAfterRoam(false /* enabled */);
doTestIgnoreValidationAfterRoam(-1, false /* enabled */);
}
@Test
public void testIgnoreValidationAfterRoamEnabled() throws Exception {
doTestIgnoreValidationAfterRoam(true /* enabled */);
final boolean enabled = !SdkLevel.isAtLeastT();
doTestIgnoreValidationAfterRoam(5_000, enabled);
}
@Test
public void testShouldIgnoreValidationFailureAfterRoam() {
// Always disabled on T+.
assumeFalse(SdkLevel.isAtLeastT());
NetworkAgentInfo nai = fakeWifiNai(new NetworkCapabilities());
// Enabled, but never roamed.
doReturn(5_000).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
assertEquals(0, nai.lastRoamTime);
assertFalse(mService.shouldIgnoreValidationFailureAfterRoam(nai));
// Roamed recently.
nai.lastRoamTime = SystemClock.elapsedRealtime() - 500 /* ms */;
assertTrue(mService.shouldIgnoreValidationFailureAfterRoam(nai));
// Disabled due to invalid setting (maximum is 10 seconds).
doReturn(15_000).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
assertFalse(mService.shouldIgnoreValidationFailureAfterRoam(nai));
// Disabled.
doReturn(-1).when(mResources)
.getInteger(R.integer.config_validationFailureAfterRoamIgnoreTimeMillis);
assertFalse(mService.shouldIgnoreValidationFailureAfterRoam(nai));
}
@Test
public void testLegacyTetheringApiGuardWithProperPermission() throws Exception {
final String testIface = "test0";