Merge "Dynamic summary text for network/connected device screen."
This commit is contained in:
committed by
Android (Google) Code Review
commit
f76dafea55
@@ -17,12 +17,15 @@ package com.android.settings.connecteddevice;
|
||||
|
||||
import android.content.Context;
|
||||
import android.content.pm.PackageManager;
|
||||
import android.nfc.NfcAdapter;
|
||||
import android.nfc.NfcManager;
|
||||
import android.provider.SearchIndexableResource;
|
||||
|
||||
import com.android.settings.bluetooth.BluetoothMasterSwitchPreferenceController;
|
||||
import com.android.settings.nfc.NfcPreferenceController;
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.SettingsRobolectricTestRunner;
|
||||
import com.android.settings.TestConfig;
|
||||
import com.android.settings.dashboard.SummaryLoader;
|
||||
import com.android.settings.nfc.NfcPreferenceController;
|
||||
import com.android.settings.testutils.XmlTestUtils;
|
||||
import com.android.settingslib.drawer.CategoryKey;
|
||||
|
||||
@@ -36,12 +39,17 @@ import org.robolectric.annotation.Config;
|
||||
|
||||
import java.util.List;
|
||||
|
||||
import static android.content.Context.NFC_SERVICE;
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.verify;
|
||||
import static org.mockito.Mockito.verifyZeroInteractions;
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
@RunWith(SettingsRobolectricTestRunner.class)
|
||||
@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
|
||||
public class ConnectedDeviceDashboardFragmentTest {
|
||||
|
||||
@Mock
|
||||
Context mContext;
|
||||
|
||||
@@ -65,7 +73,8 @@ public class ConnectedDeviceDashboardFragmentTest {
|
||||
@Test
|
||||
public void testSearchIndexProvider_shouldIndexResource() {
|
||||
final List<SearchIndexableResource> indexRes =
|
||||
mFragment.SEARCH_INDEX_DATA_PROVIDER.getXmlResourcesToIndex(mContext, true /* enabled */);
|
||||
mFragment.SEARCH_INDEX_DATA_PROVIDER.getXmlResourcesToIndex(mContext,
|
||||
true /* enabled */);
|
||||
|
||||
assertThat(indexRes).isNotNull();
|
||||
assertThat(indexRes.get(0).xmlResId).isEqualTo(mFragment.getPreferenceScreenResId());
|
||||
@@ -74,7 +83,8 @@ public class ConnectedDeviceDashboardFragmentTest {
|
||||
@Test
|
||||
public void testSearchIndexProvider_NoNfc_KeyAdded() {
|
||||
when(mManager.hasSystemFeature(PackageManager.FEATURE_NFC)).thenReturn(false);
|
||||
final List<String> keys = mFragment.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(mContext);
|
||||
final List<String> keys = mFragment.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(
|
||||
mContext);
|
||||
|
||||
assertThat(keys).isNotNull();
|
||||
assertThat(keys).contains(NfcPreferenceController.KEY_TOGGLE_NFC);
|
||||
@@ -84,7 +94,8 @@ public class ConnectedDeviceDashboardFragmentTest {
|
||||
@Test
|
||||
public void testSearchIndexProvider_NFC_KeyNotAdded() {
|
||||
when(mManager.hasSystemFeature(PackageManager.FEATURE_NFC)).thenReturn(true);
|
||||
final List<String> keys = mFragment.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(mContext);
|
||||
final List<String> keys = mFragment.SEARCH_INDEX_DATA_PROVIDER.getNonIndexableKeys(
|
||||
mContext);
|
||||
|
||||
assertThat(keys).isNotNull();
|
||||
assertThat(keys).doesNotContain(NfcPreferenceController.KEY_TOGGLE_NFC);
|
||||
@@ -103,4 +114,43 @@ public class ConnectedDeviceDashboardFragmentTest {
|
||||
|
||||
assertThat(keys).containsAllIn(niks);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testSummaryProvider_hasNfc_shouldReturnNfcSummary() {
|
||||
final NfcManager nfcManager = mock(NfcManager.class);
|
||||
final SummaryLoader summaryLoader = mock(SummaryLoader.class);
|
||||
|
||||
when(mContext.getApplicationContext()).thenReturn(mContext);
|
||||
when(mContext.getSystemService(NFC_SERVICE)).thenReturn(nfcManager);
|
||||
when(nfcManager.getDefaultAdapter()).thenReturn(mock(NfcAdapter.class));
|
||||
|
||||
SummaryLoader.SummaryProvider provider =
|
||||
new ConnectedDeviceDashboardFragment.SummaryProvider(mContext, summaryLoader);
|
||||
|
||||
provider.setListening(false);
|
||||
|
||||
verifyZeroInteractions(summaryLoader);
|
||||
|
||||
provider.setListening(true);
|
||||
|
||||
verify(mContext).getString(R.string.connected_devices_dashboard_summary);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testSummaryProvider_noNfc_shouldReturnNoNfcSummary() {
|
||||
final SummaryLoader summaryLoader = mock(SummaryLoader.class);
|
||||
|
||||
when(mContext.getApplicationContext()).thenReturn(mContext);
|
||||
|
||||
SummaryLoader.SummaryProvider provider =
|
||||
new ConnectedDeviceDashboardFragment.SummaryProvider(mContext, summaryLoader);
|
||||
|
||||
provider.setListening(false);
|
||||
|
||||
verifyZeroInteractions(summaryLoader);
|
||||
|
||||
provider.setListening(true);
|
||||
|
||||
verify(mContext).getString(R.string.connected_devices_dashboard_no_nfc_summary);
|
||||
}
|
||||
}
|
||||
|
@@ -19,15 +19,18 @@ import android.content.Context;
|
||||
import android.provider.SearchIndexableResource;
|
||||
import android.view.Menu;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.SettingsRobolectricTestRunner;
|
||||
import com.android.settings.TestConfig;
|
||||
import com.android.settings.connecteddevice.ConnectedDeviceDashboardFragment;
|
||||
import com.android.settings.dashboard.SummaryLoader;
|
||||
import com.android.settings.testutils.XmlTestUtils;
|
||||
import com.android.settingslib.drawer.CategoryKey;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.mockito.Mock;
|
||||
import org.mockito.MockitoAnnotations;
|
||||
import org.robolectric.RuntimeEnvironment;
|
||||
import org.robolectric.annotation.Config;
|
||||
import org.robolectric.shadows.ShadowApplication;
|
||||
@@ -38,16 +41,23 @@ import java.util.List;
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
import static org.mockito.Matchers.any;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.times;
|
||||
import static org.mockito.Mockito.verify;
|
||||
import static org.mockito.Mockito.verifyZeroInteractions;
|
||||
import static org.mockito.Mockito.when;
|
||||
|
||||
@RunWith(SettingsRobolectricTestRunner.class)
|
||||
@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION)
|
||||
public class NetworkDashboardFragmentTest {
|
||||
|
||||
@Mock
|
||||
private Context mContext;
|
||||
|
||||
private NetworkDashboardFragment mFragment;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
MockitoAnnotations.initMocks(this);
|
||||
mFragment = new NetworkDashboardFragment();
|
||||
}
|
||||
|
||||
@@ -89,4 +99,58 @@ public class NetworkDashboardFragmentTest {
|
||||
|
||||
assertThat(keys).containsAllIn(niks);
|
||||
}
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testSummaryProvider_hasMobileAndHotspot_shouldReturnMobileSummary() {
|
||||
final MobileNetworkPreferenceController mobileNetworkPreferenceController =
|
||||
mock(MobileNetworkPreferenceController.class);
|
||||
final TetherPreferenceController tetherPreferenceController =
|
||||
mock(TetherPreferenceController.class);
|
||||
|
||||
final SummaryLoader summaryLoader = mock(SummaryLoader.class);
|
||||
final SummaryLoader.SummaryProvider provider =
|
||||
new NetworkDashboardFragment.SummaryProvider(mContext, summaryLoader,
|
||||
mobileNetworkPreferenceController, tetherPreferenceController);
|
||||
|
||||
provider.setListening(false);
|
||||
|
||||
verifyZeroInteractions(summaryLoader);
|
||||
|
||||
when(mobileNetworkPreferenceController.isAvailable()).thenReturn(true);
|
||||
when(tetherPreferenceController.isAvailable()).thenReturn(true);
|
||||
|
||||
provider.setListening(true);
|
||||
|
||||
verify(mContext).getString(R.string.wifi_settings_title);
|
||||
verify(mContext).getString(R.string.network_dashboard_summary_data_usage);
|
||||
verify(mContext).getString(R.string.network_dashboard_summary_hotspot);
|
||||
verify(mContext).getString(R.string.network_dashboard_summary_mobile);
|
||||
verify(mContext, times(3)).getString(R.string.join_many_items_middle, null, null);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testSummaryProvider_noMobileOrHotspot_shouldReturnSimpleSummary() {
|
||||
final MobileNetworkPreferenceController mobileNetworkPreferenceController =
|
||||
mock(MobileNetworkPreferenceController.class);
|
||||
final TetherPreferenceController tetherPreferenceController =
|
||||
mock(TetherPreferenceController.class);
|
||||
|
||||
final SummaryLoader summaryLoader = mock(SummaryLoader.class);
|
||||
final SummaryLoader.SummaryProvider provider =
|
||||
new NetworkDashboardFragment.SummaryProvider(mContext, summaryLoader,
|
||||
mobileNetworkPreferenceController, tetherPreferenceController);
|
||||
|
||||
provider.setListening(false);
|
||||
|
||||
verifyZeroInteractions(summaryLoader);
|
||||
|
||||
when(mobileNetworkPreferenceController.isAvailable()).thenReturn(false);
|
||||
when(tetherPreferenceController.isAvailable()).thenReturn(false);
|
||||
|
||||
provider.setListening(true);
|
||||
|
||||
verify(mContext).getString(R.string.wifi_settings_title);
|
||||
verify(mContext).getString(R.string.network_dashboard_summary_data_usage);
|
||||
verify(mContext).getString(R.string.join_many_items_middle, null, null);
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user