diff --git a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java index 49e214beaa1ec..443543b89a66b 100644 --- a/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java +++ b/packages/SettingsLib/src/com/android/settingslib/wifi/AccessPoint.java @@ -1395,7 +1395,7 @@ public class AccessPoint implements Comparable { mConfig = new WifiConfiguration(); mConfig.SSID = AccessPoint.convertToQuotedString(ssid); - if (security == SECURITY_NONE || !getWifiManager().isEasyConnectSupported()) { + if (security == SECURITY_NONE) { mConfig.allowedKeyManagement.set(KeyMgmt.NONE); } else { mConfig.allowedKeyManagement.set(KeyMgmt.OWE); diff --git a/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java b/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java index 2a7050651c6d8..3818057e85359 100644 --- a/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java +++ b/packages/SettingsLib/tests/integ/src/com/android/settingslib/wifi/AccessPointTest.java @@ -1713,4 +1713,21 @@ public class AccessPointTest { .setScanResults(new ArrayList(Arrays.asList(scanResult))) .build(); } + + @Test + public void testGenerateOpenNetworkConfig_oweNotSupported_shouldGetCorrectSecurity() { + when(mMockContext.getSystemService(Context.WIFI_SERVICE)).thenReturn(mMockWifiManager); + AccessPoint oweAccessPoint = new TestAccessPointBuilder(mMockContext) + .setSecurity(AccessPoint.SECURITY_OWE).build(); + AccessPoint noneAccessPoint = new TestAccessPointBuilder(mMockContext) + .setSecurity(AccessPoint.SECURITY_NONE).build(); + + oweAccessPoint.generateOpenNetworkConfig(); + noneAccessPoint.generateOpenNetworkConfig(); + + assertThat(oweAccessPoint.getConfig().allowedKeyManagement.get(KeyMgmt.NONE)).isFalse(); + assertThat(oweAccessPoint.getConfig().allowedKeyManagement.get(KeyMgmt.OWE)).isTrue(); + assertThat(noneAccessPoint.getConfig().allowedKeyManagement.get(KeyMgmt.NONE)).isTrue(); + assertThat(noneAccessPoint.getConfig().allowedKeyManagement.get(KeyMgmt.OWE)).isFalse(); + } }