From 61ea59ab30a7571eb4b5c7f65fa22b4dfb129750 Mon Sep 17 00:00:00 2001
From: Zoey Chen <zoeychen@google.com>
Date: Mon, 9 Nov 2020 22:49:11 +0800
Subject: [PATCH] [Provider Model] Implement Calls & SMS fragment test case

New test case: NetworkProviderCallsSmsFragmentTest
Modify the key of preference

Bug: 172053880
Test: manual & atest -c NetworkProviderCallsSmsFragmentTest
Change-Id: I218aaf1194898e70b2118428c1ea5a389d85d036
---
 res/xml/network_provider_calls_sms.xml        |  8 +-
 .../NetworkProviderCallsSmsFragment.java      | 14 ++-
 .../NetworkProviderCallsSmsFragmentTest.java  | 87 +++++++++++++++++++
 3 files changed, 101 insertions(+), 8 deletions(-)
 create mode 100644 tests/unit/src/com/android/settings/network/NetworkProviderCallsSmsFragmentTest.java

diff --git a/res/xml/network_provider_calls_sms.xml b/res/xml/network_provider_calls_sms.xml
index b68b48fc0a8..2fa2eb79446 100644
--- a/res/xml/network_provider_calls_sms.xml
+++ b/res/xml/network_provider_calls_sms.xml
@@ -17,11 +17,11 @@
 <PreferenceScreen
     xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:settings="http://schemas.android.com/apk/res-auto"
-    android:key="calls_and_sms_screen"
+    android:key="provider_model_calls_and_sms_screen"
     android:title="@string/calls_and_sms">
 
     <ListPreference
-        android:key="calls_preference"
+        android:key="provider_model_calls_preference"
         android:title="@string/calls_preference_title"
         settings:controller="com.android.settings.network.telephony.CallsDefaultSubscriptionController"
         android:order="10"
@@ -29,7 +29,7 @@
         />
 
     <ListPreference
-        android:key="sms_preference"
+        android:key="provider_model_sms_preference"
         android:title="@string/sms_preference_title"
         settings:controller="com.android.settings.network.telephony.SmsDefaultSubscriptionController"
         android:order="15"
@@ -45,7 +45,7 @@
         />
 
     <com.android.settingslib.widget.FooterPreference
-        android:key="calls_sms_footer"
+        android:key="provider_model_calls_sms_footer"
         android:title="@string/calls_sms_footnote"
         android:selectable="false"
         settings:searchable="false"
diff --git a/src/com/android/settings/network/NetworkProviderCallsSmsFragment.java b/src/com/android/settings/network/NetworkProviderCallsSmsFragment.java
index 58b2872c28a..3339cdb2bc7 100644
--- a/src/com/android/settings/network/NetworkProviderCallsSmsFragment.java
+++ b/src/com/android/settings/network/NetworkProviderCallsSmsFragment.java
@@ -20,6 +20,8 @@ import android.app.settings.SettingsEnums;
 import android.content.Context;
 import android.os.UserManager;
 
+import androidx.annotation.VisibleForTesting;
+
 import com.android.settings.R;
 import com.android.settings.Utils;
 import com.android.settings.dashboard.DashboardFragment;
@@ -35,10 +37,14 @@ import java.util.List;
 
 @SearchIndexable(forTarget = SearchIndexable.ALL & ~SearchIndexable.ARC)
 public class NetworkProviderCallsSmsFragment extends DashboardFragment {
-    private static final String LOG_TAG = "NetworkProviderCallsSmsFragment";
-    private static final String KEY_PREFERENCE_CATEGORY_CALLING = "provider_model_calling_category";
-    private static final String KEY_PREFERENCE_CALLS= "calls_preference";
-    private static final String KEY_PREFERENCE_SMS = "sms_preference";
+    @VisibleForTesting
+    static final String LOG_TAG = "NetworkProviderCallsSmsFragment";
+    @VisibleForTesting
+    static final String KEY_PREFERENCE_CATEGORY_CALLING = "provider_model_calling_category";
+    @VisibleForTesting
+    static final String KEY_PREFERENCE_CALLS= "provider_model_calls_preference";
+    @VisibleForTesting
+    static final String KEY_PREFERENCE_SMS = "provider_model_sms_preference";
 
     private NetworkProviderWifiCallingPreferenceController
             mNetworkProviderWifiCallingPreferenceController;
diff --git a/tests/unit/src/com/android/settings/network/NetworkProviderCallsSmsFragmentTest.java b/tests/unit/src/com/android/settings/network/NetworkProviderCallsSmsFragmentTest.java
new file mode 100644
index 00000000000..8327d223d98
--- /dev/null
+++ b/tests/unit/src/com/android/settings/network/NetworkProviderCallsSmsFragmentTest.java
@@ -0,0 +1,87 @@
+/*
+ * Copyright (C) 2020 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.network;
+
+import static com.google.common.truth.Truth.assertThat;
+
+import static org.mockito.Mockito.spy;
+
+import android.app.Instrumentation;
+import android.content.Context;
+import android.os.Looper;
+import android.provider.SearchIndexableResource;
+import android.util.FeatureFlagUtils;
+
+import androidx.test.annotation.UiThreadTest;
+import androidx.test.core.app.ApplicationProvider;
+import androidx.test.ext.junit.runners.AndroidJUnit4;
+import androidx.test.platform.app.InstrumentationRegistry;
+
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.MockitoAnnotations;
+
+import java.util.ArrayList;
+import java.util.List;
+
+@RunWith(AndroidJUnit4.class)
+public class NetworkProviderCallsSmsFragmentTest {
+
+    private Context mContext;
+    private NetworkProviderCallsSmsFragment mNetworkProviderCallsSmsFragment;
+    private List<String> mPreferenceKeyList;
+
+    @Before
+    @UiThreadTest
+    public void setUp() {
+        MockitoAnnotations.initMocks(this);
+        mContext = spy(ApplicationProvider.getApplicationContext());
+
+        if (Looper.myLooper() == null) {
+            Looper.prepare();
+        }
+
+        mNetworkProviderCallsSmsFragment = new NetworkProviderCallsSmsFragment();
+    }
+
+    @Test
+    @UiThreadTest
+    public void isPageSearchEnabled_providerModelEnable_shouldIncludeFragmentXml() {
+        FeatureFlagUtils.setEnabled(mContext, FeatureFlagUtils.SETTINGS_PROVIDER_MODEL, true);
+        mPreferenceKeyList =
+                NetworkProviderCallsSmsFragment.SEARCH_INDEX_DATA_PROVIDER
+                        .getNonIndexableKeys(mContext);
+        assertThat(mPreferenceKeyList).doesNotContain(
+                NetworkProviderCallsSmsFragment.KEY_PREFERENCE_CALLS);
+        assertThat(mPreferenceKeyList).doesNotContain(
+                NetworkProviderCallsSmsFragment.KEY_PREFERENCE_SMS);
+    }
+
+    @Test
+    @UiThreadTest
+    public void isPageSearchEnabled_providerModelDisable_shouldNotIncludeFragmentXml() {
+        FeatureFlagUtils.setEnabled(mContext, FeatureFlagUtils.SETTINGS_PROVIDER_MODEL, false);
+        mPreferenceKeyList =
+                NetworkProviderCallsSmsFragment.SEARCH_INDEX_DATA_PROVIDER
+                        .getNonIndexableKeys(mContext);
+        assertThat(mPreferenceKeyList).contains(NetworkProviderCallsSmsFragment
+                .KEY_PREFERENCE_CALLS);
+        assertThat(mPreferenceKeyList).contains(NetworkProviderCallsSmsFragment
+                .KEY_PREFERENCE_SMS);
+    }
+}