diff options
author | 2020-10-05 08:05:08 -0700 | |
---|---|---|
committer | 2020-10-05 12:23:48 -0700 | |
commit | b93c97d7a105b46839ad7ec7628ef6593d2b06e4 (patch) | |
tree | 153509bef218e6ffed564bb0721adc2558cbaa5a | |
parent | 25993a64d460bcf62949fa9dd1c710253afa6e2b (diff) |
Migrate existing Build.VERSION.SDK_INT checks to SdkLevelUtil
Bug: 154742276
Test: atest FrameworksWifiApiTests
Change-Id: I9f2273331e1ea465651c47947319138228c0ffdf
-rw-r--r-- | wifi/tests/src/android/net/wifi/SoftApCapabilityTest.java | 13 | ||||
-rw-r--r-- | wifi/tests/src/android/net/wifi/SoftApConfigurationTest.java | 8 |
2 files changed, 10 insertions, 11 deletions
diff --git a/wifi/tests/src/android/net/wifi/SoftApCapabilityTest.java b/wifi/tests/src/android/net/wifi/SoftApCapabilityTest.java index b9c640cc3980..3c935058787b 100644 --- a/wifi/tests/src/android/net/wifi/SoftApCapabilityTest.java +++ b/wifi/tests/src/android/net/wifi/SoftApCapabilityTest.java @@ -16,10 +16,11 @@ package android.net.wifi; -import android.os.Build; +import android.net.wifi.util.SdkLevelUtil; import android.os.Parcel; import static org.junit.Assert.assertEquals; +import static org.junit.Assume.assumeTrue; import androidx.test.filters.SmallTest; @@ -92,14 +93,12 @@ public class SoftApCapabilityTest { @Test(expected = IllegalArgumentException.class) public void testGetSupportedChannelListWithInvalidBand() { + assumeTrue(SdkLevelUtil.isAtLeastS()); + long testSoftApFeature = SoftApCapability.SOFTAP_FEATURE_CLIENT_FORCE_DISCONNECT | SoftApCapability.SOFTAP_FEATURE_ACS_OFFLOAD; SoftApCapability capability = new SoftApCapability(testSoftApFeature); - if (Build.VERSION.SDK_INT > Build.VERSION_CODES.R) { - capability.getSupportedChannelList( - SoftApConfiguration.BAND_2GHZ | SoftApConfiguration.BAND_5GHZ); - } else { - throw new IllegalArgumentException("API doesn't support in current SDK version"); - } + capability.getSupportedChannelList( + SoftApConfiguration.BAND_2GHZ | SoftApConfiguration.BAND_5GHZ); } } diff --git a/wifi/tests/src/android/net/wifi/SoftApConfigurationTest.java b/wifi/tests/src/android/net/wifi/SoftApConfigurationTest.java index 254434b81f8f..7d7d2ba4c880 100644 --- a/wifi/tests/src/android/net/wifi/SoftApConfigurationTest.java +++ b/wifi/tests/src/android/net/wifi/SoftApConfigurationTest.java @@ -21,7 +21,7 @@ import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.assertNull; import android.net.MacAddress; -import android.os.Build; +import android.net.wifi.util.SdkLevelUtil; import android.os.Parcel; import androidx.test.filters.SmallTest; @@ -81,7 +81,7 @@ public class SoftApConfigurationTest { assertThat(original.getChannel()).isEqualTo(0); assertThat(original.isHiddenSsid()).isEqualTo(false); assertThat(original.getMaxNumberOfClients()).isEqualTo(0); - if (Build.VERSION.SDK_INT > Build.VERSION_CODES.R) { + if (SdkLevelUtil.isAtLeastS()) { assertThat(original.getMacRandomizationSetting()) .isEqualTo(SoftApConfiguration.RANDOMIZATION_PERSISTENT); } @@ -137,7 +137,7 @@ public class SoftApConfigurationTest { .setClientControlByUserEnabled(true) .setBlockedClientList(testBlockedClientList) .setAllowedClientList(testAllowedClientList); - if (Build.VERSION.SDK_INT > Build.VERSION_CODES.R) { + if (SdkLevelUtil.isAtLeastS()) { originalBuilder.setMacRandomizationSetting(SoftApConfiguration.RANDOMIZATION_NONE); } SoftApConfiguration original = originalBuilder.build(); @@ -153,7 +153,7 @@ public class SoftApConfigurationTest { assertThat(original.isClientControlByUserEnabled()).isEqualTo(true); assertThat(original.getBlockedClientList()).isEqualTo(testBlockedClientList); assertThat(original.getAllowedClientList()).isEqualTo(testAllowedClientList); - if (Build.VERSION.SDK_INT > Build.VERSION_CODES.R) { + if (SdkLevelUtil.isAtLeastS()) { assertThat(original.getMacRandomizationSetting()) .isEqualTo(SoftApConfiguration.RANDOMIZATION_NONE); } |