From 858fbdb7599dcfe444272d229f1d5a0a157a47b6 Mon Sep 17 00:00:00 2001
From: Jacky Wang <jiannan@google.com>
Date: Fri, 6 Dec 2024 21:55:48 +0800
Subject: [PATCH] [Catalyst] Make AirplaneModePreference final

NO_IFTTT=Catalyst only

Bug: 375925972
Flag: TEST_ONLY
Test: atest
Change-Id: Ieea0e0e770d93d8bbe6f1f7a608541d5ba42c9cc
---
 .../network/AirplaneModePreference.kt         |  4 +---
 .../network/AirplaneModePreferenceTest.kt     | 22 ++++++++-----------
 2 files changed, 10 insertions(+), 16 deletions(-)
 rename tests/{unit => robotests}/src/com/android/settings/network/AirplaneModePreferenceTest.kt (87%)

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<Resources>()
     private val mockPackageManager = mock<PackageManager>()
     private var mockTelephonyManager = mock<TelephonyManager>()
 
-    private val context =
-        object : ContextWrapper(ApplicationProvider.getApplicationContext()) {
+    private val context = ApplicationProvider.getApplicationContext<Context>()
+    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