diff --git a/src/com/android/settings/network/AirplaneModePreference.kt b/src/com/android/settings/network/AirplaneModePreference.kt index 4719a571a9c..164e3a9b513 100644 --- a/src/com/android/settings/network/AirplaneModePreference.kt +++ b/src/com/android/settings/network/AirplaneModePreference.kt @@ -45,14 +45,12 @@ import com.android.settingslib.metadata.PreferenceLifecycleProvider import com.android.settingslib.metadata.ReadWritePermit import com.android.settingslib.metadata.SensitivityLevel import com.android.settingslib.metadata.SwitchPreference -import com.android.settingslib.preference.SwitchPreferenceBinding import java.util.concurrent.Executors import java.util.concurrent.TimeUnit // LINT.IfChange -open class AirplaneModePreference : +class AirplaneModePreference : SwitchPreference(KEY, R.string.airplane_mode), - SwitchPreferenceBinding, PreferenceAvailabilityProvider, PreferenceLifecycleProvider, PreferenceRestrictionMixin { diff --git a/tests/unit/src/com/android/settings/network/AirplaneModePreferenceTest.kt b/tests/robotests/src/com/android/settings/network/AirplaneModePreferenceTest.kt similarity index 87% rename from tests/unit/src/com/android/settings/network/AirplaneModePreferenceTest.kt rename to tests/robotests/src/com/android/settings/network/AirplaneModePreferenceTest.kt index 8ae4a5fefe2..640ded90132 100644 --- a/tests/unit/src/com/android/settings/network/AirplaneModePreferenceTest.kt +++ b/tests/robotests/src/com/android/settings/network/AirplaneModePreferenceTest.kt @@ -16,13 +16,13 @@ package com.android.settings.network +import android.content.Context import android.content.ContextWrapper import android.content.pm.PackageManager import android.content.pm.PackageManager.FEATURE_LEANBACK import android.content.res.Resources import android.provider.Settings import android.telephony.TelephonyManager -import androidx.annotation.DrawableRes import androidx.preference.SwitchPreferenceCompat import androidx.test.core.app.ApplicationProvider import androidx.test.ext.junit.runners.AndroidJUnit4 @@ -37,14 +37,15 @@ import org.mockito.kotlin.mock import org.mockito.kotlin.stub @RunWith(AndroidJUnit4::class) -open class AirplaneModePreferenceTest { +class AirplaneModePreferenceTest { private val mockResources = mock() private val mockPackageManager = mock() private var mockTelephonyManager = mock() - private val context = - object : ContextWrapper(ApplicationProvider.getApplicationContext()) { + private val context = ApplicationProvider.getApplicationContext() + private val contextWrapper = + object : ContextWrapper(context) { override fun getResources(): Resources = mockResources override fun getPackageManager(): PackageManager = mockPackageManager @@ -56,19 +57,14 @@ open class AirplaneModePreferenceTest { } } - private var airplaneModePreference = - object : AirplaneModePreference() { - // TODO: Remove override - override val icon: Int - @DrawableRes get() = 0 - } + private val airplaneModePreference = AirplaneModePreference() @Test fun isAvailable_hasConfigAndNoFeatureLeanback_shouldReturnTrue() { mockResources.stub { on { getBoolean(anyInt()) } doReturn true } mockPackageManager.stub { on { hasSystemFeature(FEATURE_LEANBACK) } doReturn false } - assertThat(airplaneModePreference.isAvailable(context)).isTrue() + assertThat(airplaneModePreference.isAvailable(contextWrapper)).isTrue() } @Test @@ -76,7 +72,7 @@ open class AirplaneModePreferenceTest { mockResources.stub { on { getBoolean(anyInt()) } doReturn false } mockPackageManager.stub { on { hasSystemFeature(FEATURE_LEANBACK) } doReturn false } - assertThat(airplaneModePreference.isAvailable(context)).isFalse() + assertThat(airplaneModePreference.isAvailable(contextWrapper)).isFalse() } @Test @@ -84,7 +80,7 @@ open class AirplaneModePreferenceTest { mockResources.stub { on { getBoolean(anyInt()) } doReturn true } mockPackageManager.stub { on { hasSystemFeature(FEATURE_LEANBACK) } doReturn true } - assertThat(airplaneModePreference.isAvailable(context)).isFalse() + assertThat(airplaneModePreference.isAvailable(contextWrapper)).isFalse() } @Test