From 0a87a1685ed440a389595f62f5d6eab35c57caaa Mon Sep 17 00:00:00 2001
From: Charlotte Lu <charlottelu@google.com>
Date: Thu, 11 Apr 2024 19:51:25 +0800
Subject: [PATCH] [WIFI-Enterprise] Add Server name field.

Test: Unit Test
Fix: 333855426
Change-Id: Idfcb5f4efd36257188b04e2be7c7dc1127681b06
---
 res/xml/wifi_network_details_fragment2.xml    |  3 +
 .../details/WifiNetworkDetailsFragment.java   |  3 +
 .../ServerNamePreferenceController.kt         | 56 +++++++++++
 .../ServerNamePreferenceControllerTest.kt     | 92 +++++++++++++++++++
 4 files changed, 154 insertions(+)
 create mode 100644 src/com/android/settings/wifi/details2/ServerNamePreferenceController.kt
 create mode 100644 tests/spa_unit/src/com/android/settings/wifi/details2/ServerNamePreferenceControllerTest.kt

diff --git a/res/xml/wifi_network_details_fragment2.xml b/res/xml/wifi_network_details_fragment2.xml
index eacff88bdd9..518b65d8401 100644
--- a/res/xml/wifi_network_details_fragment2.xml
+++ b/res/xml/wifi_network_details_fragment2.xml
@@ -126,6 +126,9 @@
     <PreferenceCategory
             android:key="ip_details_category"
             android:title="@string/wifi_setup_detail">
+        <com.android.settings.spa.preference.ComposePreference
+            android:key="server_name"
+            settings:controller="com.android.settings.wifi.details2.ServerNamePreferenceController"/>
         <Preference
                 android:key="type"
                 android:title="@string/wifi_type_title"
diff --git a/src/com/android/settings/wifi/details/WifiNetworkDetailsFragment.java b/src/com/android/settings/wifi/details/WifiNetworkDetailsFragment.java
index 06b3e5d8dd4..eb0b40a8b98 100644
--- a/src/com/android/settings/wifi/details/WifiNetworkDetailsFragment.java
+++ b/src/com/android/settings/wifi/details/WifiNetworkDetailsFragment.java
@@ -56,6 +56,7 @@ import com.android.settings.wifi.WifiDialog2;
 import com.android.settings.wifi.WifiUtils;
 import com.android.settings.wifi.details2.AddDevicePreferenceController2;
 import com.android.settings.wifi.details2.CertificateDetailsPreferenceController;
+import com.android.settings.wifi.details2.ServerNamePreferenceController;
 import com.android.settings.wifi.details2.WifiAutoConnectPreferenceController2;
 import com.android.settings.wifi.details2.WifiDetailPreferenceController2;
 import com.android.settings.wifi.details2.WifiMeteredPreferenceController2;
@@ -129,6 +130,8 @@ public class WifiNetworkDetailsFragment extends RestrictedDashboardFragment impl
                 .setWifiEntryKey(wifiEntryKey);
         use(CertificateDetailsPreferenceController.class)
                 .setWifiEntry(mNetworkDetailsTracker.getWifiEntry());
+        use(ServerNamePreferenceController.class)
+                .setWifiEntry(mNetworkDetailsTracker.getWifiEntry());
     }
 
     @Override
diff --git a/src/com/android/settings/wifi/details2/ServerNamePreferenceController.kt b/src/com/android/settings/wifi/details2/ServerNamePreferenceController.kt
new file mode 100644
index 00000000000..7a7f43e8ffe
--- /dev/null
+++ b/src/com/android/settings/wifi/details2/ServerNamePreferenceController.kt
@@ -0,0 +1,56 @@
+/*
+ * Copyright (C) 2024 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.settings.wifi.details2
+
+import android.content.Context
+import android.util.Log
+import androidx.compose.runtime.Composable
+import androidx.compose.ui.res.stringResource
+import com.android.settings.R
+import com.android.settings.spa.preference.ComposePreferenceController
+import com.android.settingslib.spa.widget.preference.Preference
+import com.android.settingslib.spa.widget.preference.PreferenceModel
+import com.android.wifi.flags.Flags
+import com.android.wifitrackerlib.WifiEntry
+
+class ServerNamePreferenceController(context: Context, preferenceKey: String) :
+    ComposePreferenceController(context, preferenceKey) {
+
+    private lateinit var wifiEntry: WifiEntry
+
+    fun setWifiEntry(entry: WifiEntry) {
+        wifiEntry = entry
+    }
+
+    override fun getAvailabilityStatus(): Int {
+        return if (Flags.androidVWifiApi() && wifiEntry.certificateInfo?.domain != null) AVAILABLE
+        else CONDITIONALLY_UNAVAILABLE
+    }
+
+    @Composable
+    override fun Content() {
+        ServerName()
+    }
+
+    @Composable
+    fun ServerName() {
+        Preference(object : PreferenceModel {
+            override val title = stringResource(R.string.server_name_title)
+            override val summary = { wifiEntry.certificateInfo?.domain ?: "" }
+        })
+    }
+}
\ No newline at end of file
diff --git a/tests/spa_unit/src/com/android/settings/wifi/details2/ServerNamePreferenceControllerTest.kt b/tests/spa_unit/src/com/android/settings/wifi/details2/ServerNamePreferenceControllerTest.kt
new file mode 100644
index 00000000000..4c0e3cf9767
--- /dev/null
+++ b/tests/spa_unit/src/com/android/settings/wifi/details2/ServerNamePreferenceControllerTest.kt
@@ -0,0 +1,92 @@
+/*
+ * Copyright (C) 2024 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.settings.wifi.details2
+
+import android.content.Context
+import android.platform.test.annotations.RequiresFlagsEnabled
+import androidx.compose.runtime.CompositionLocalProvider
+import androidx.compose.ui.platform.LocalContext
+import androidx.compose.ui.test.assertIsDisplayed
+import androidx.compose.ui.test.junit4.createComposeRule
+import androidx.compose.ui.test.onNodeWithText
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import com.android.settings.R
+import com.android.wifitrackerlib.WifiEntry
+import org.junit.Before
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.mockito.kotlin.any
+import org.mockito.kotlin.doNothing
+import org.mockito.kotlin.doReturn
+import org.mockito.kotlin.mock
+import org.mockito.kotlin.spy
+import org.mockito.kotlin.whenever
+
+@RunWith(AndroidJUnit4::class)
+class ServerNamePreferenceControllerTest {
+    @get:Rule
+    val composeTestRule = createComposeRule()
+
+    private val context: Context = spy(ApplicationProvider.getApplicationContext()) {
+        doNothing().whenever(mock).startActivity(any())
+    }
+    private val controller = ServerNamePreferenceController(context, TEST_KEY)
+
+    private val mockCertificateInfo = mock<WifiEntry.CertificateInfo> {
+        it.domain = DOMAIN
+    }
+
+    private val mockWifiEntry =
+        mock<WifiEntry> { on { certificateInfo } doReturn mockCertificateInfo }
+
+    @Before
+    fun setUp() {
+        controller.setWifiEntry(mockWifiEntry)
+    }
+
+    @Test
+    @RequiresFlagsEnabled(com.android.wifi.flags.Flags.FLAG_ANDROID_V_WIFI_API)
+    fun title_isDisplayed() {
+        composeTestRule.setContent {
+            CompositionLocalProvider(LocalContext provides context) {
+                controller.Content()
+            }
+        }
+
+        composeTestRule.onNodeWithText(context.getString(R.string.server_name_title))
+            .assertIsDisplayed()
+    }
+
+    @Test
+    @RequiresFlagsEnabled(com.android.wifi.flags.Flags.FLAG_ANDROID_V_WIFI_API)
+    fun summary_isDisplayed() {
+        composeTestRule.setContent {
+            CompositionLocalProvider(LocalContext provides context) {
+                controller.Content()
+            }
+        }
+
+        composeTestRule.onNodeWithText(DOMAIN).assertIsDisplayed()
+    }
+
+    private companion object {
+        const val TEST_KEY = "test_key"
+        const val DOMAIN = "domain"
+    }
+}
\ No newline at end of file