From 52ccb49fbe65c8e84a5d4fde1d8028b62a713204 Mon Sep 17 00:00:00 2001 From: jackqdyulei Date: Wed, 10 May 2017 14:57:16 -0700 Subject: [PATCH] Split BluetoothSettings into two pages This cl splits the BluetoothSettings into paired device page and pairing page, including small changes about: 1. Refactor the pages so they could get as much as static preference from xml file rather than dynamically add/remove them everytime. 2. Remove creating method in BluetoothDeviceNamePreferenceController and add it in xml file 3. Create BluetoothPairingDetail page, basically move the logic from BluetoothSettings. 4. Make pairing preference clickable and jump to BluetoothPairingDetail 5. Add and update bunch of tests Bug: 35877041 Test: RunSettingsRoboTests Change-Id: Ief9e9690c612f7b46c58e866e5cecc511af642c8 --- res/values/strings.xml | 5 + res/xml/bluetooth_pairing_detail.xml | 30 +++ res/xml/bluetooth_settings.xml | 15 +- ...uetoothDeviceNamePreferenceController.java | 5 + .../bluetooth/BluetoothPairingDetail.java | 194 +++++++++++++++ .../BluetoothPairingPreferenceController.java | 18 +- .../settings/bluetooth/BluetoothSettings.java | 225 ++++-------------- .../DeviceListPreferenceFragment.java | 73 ++++-- .../bluetooth/DevicePickerFragment.java | 10 +- ...randfather_not_implementing_index_provider | 1 + .../bluetooth/BluetoothPairingDetailTest.java | 133 +++++++++++ ...etoothPairingPreferenceControllerTest.java | 27 ++- .../bluetooth/BluetoothSettingsTest.java | 61 ++++- .../DeviceListPreferenceFragmentTest.java | 131 ++++++++++ 14 files changed, 704 insertions(+), 224 deletions(-) create mode 100644 res/xml/bluetooth_pairing_detail.xml create mode 100644 src/com/android/settings/bluetooth/BluetoothPairingDetail.java create mode 100644 tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingDetailTest.java create mode 100644 tests/robotests/src/com/android/settings/bluetooth/DeviceListPreferenceFragmentTest.java diff --git a/res/values/strings.xml b/res/values/strings.xml index ac76e9486fc..a4ae39c672c 100644 --- a/res/values/strings.xml +++ b/res/values/strings.xml @@ -378,6 +378,11 @@ Visible as ^1 to other devices + + Your devices + + Pair bluetooth device + Date & time diff --git a/res/xml/bluetooth_pairing_detail.xml b/res/xml/bluetooth_pairing_detail.xml new file mode 100644 index 00000000000..30eaf09b464 --- /dev/null +++ b/res/xml/bluetooth_pairing_detail.xml @@ -0,0 +1,30 @@ + + + + + + + + + + + + diff --git a/res/xml/bluetooth_settings.xml b/res/xml/bluetooth_settings.xml index 783a860429f..046295bcdc0 100644 --- a/res/xml/bluetooth_settings.xml +++ b/res/xml/bluetooth_settings.xml @@ -4,9 +4,9 @@ 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. @@ -16,6 +16,15 @@ + android:title="@string/bluetooth_settings"> + + + + + + diff --git a/src/com/android/settings/bluetooth/BluetoothDeviceNamePreferenceController.java b/src/com/android/settings/bluetooth/BluetoothDeviceNamePreferenceController.java index db4c3e0fd87..baab3fcf88b 100644 --- a/src/com/android/settings/bluetooth/BluetoothDeviceNamePreferenceController.java +++ b/src/com/android/settings/bluetooth/BluetoothDeviceNamePreferenceController.java @@ -77,6 +77,11 @@ public class BluetoothDeviceNamePreferenceController extends PreferenceControlle mLocalAdapter = localAdapter; } + @Override + public void displayPreference(PreferenceScreen screen) { + mPreference = screen.findPreference(KEY_DEVICE_NAME); + } + @Override public void onStart() { mContext.registerReceiver(mReceiver, diff --git a/src/com/android/settings/bluetooth/BluetoothPairingDetail.java b/src/com/android/settings/bluetooth/BluetoothPairingDetail.java new file mode 100644 index 00000000000..7e2978d8303 --- /dev/null +++ b/src/com/android/settings/bluetooth/BluetoothPairingDetail.java @@ -0,0 +1,194 @@ +/* + * Copyright (C) 2017 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.bluetooth; + +import static android.os.UserManager.DISALLOW_CONFIG_BLUETOOTH; + +import android.bluetooth.BluetoothAdapter; +import android.bluetooth.BluetoothDevice; +import android.content.Context; +import android.os.Bundle; +import android.support.annotation.VisibleForTesting; + +import com.android.internal.logging.nano.MetricsProto.MetricsEvent; +import com.android.settings.R; +import com.android.settings.core.PreferenceController; +import com.android.settings.search.Indexable; +import com.android.settingslib.bluetooth.BluetoothDeviceFilter; +import com.android.settingslib.bluetooth.CachedBluetoothDevice; +import com.android.settingslib.widget.FooterPreference; + +import java.util.ArrayList; +import java.util.List; + +/** + * BluetoothPairingDetail is a page to scan bluetooth devices and pair them. + */ +public class BluetoothPairingDetail extends DeviceListPreferenceFragment implements + Indexable { + private static final String TAG = "BluetoothPairingDetail"; + + @VisibleForTesting + static final String KEY_AVAIL_DEVICES = "available_devices"; + @VisibleForTesting + static final String KEY_FOOTER_PREF = "footer_preference"; + + @VisibleForTesting + BluetoothDeviceNamePreferenceController mDeviceNamePrefController; + @VisibleForTesting + BluetoothProgressCategory mAvailableDevicesCategory; + @VisibleForTesting + FooterPreference mFooterPreference; + + private boolean mInitialScanStarted; + + public BluetoothPairingDetail() { + super(DISALLOW_CONFIG_BLUETOOTH); + } + + @Override + public void onActivityCreated(Bundle savedInstanceState) { + super.onActivityCreated(savedInstanceState); + mInitialScanStarted = false; + } + + @Override + public void onStart() { + super.onStart(); + + if (mLocalAdapter != null) { + updateContent(mLocalAdapter.getBluetoothState()); + mAvailableDevicesCategory.setProgress(mLocalAdapter.isDiscovering()); + } + } + + @Override + public void onStop() { + super.onStop(); + + // Make the device only visible to connected devices. + mLocalAdapter.setScanMode(BluetoothAdapter.SCAN_MODE_CONNECTABLE); + mLocalAdapter.stopScanning(); + } + + @Override + void initPreferencesFromPreferenceScreen() { + mAvailableDevicesCategory = (BluetoothProgressCategory) findPreference(KEY_AVAIL_DEVICES); + mFooterPreference = (FooterPreference) findPreference(KEY_FOOTER_PREF); + mFooterPreference.setSelectable(false); + } + + @Override + public int getMetricsCategory() { + //TODO(b/38383542): add bluetooth pairing category + return MetricsEvent.BLUETOOTH; + } + + @VisibleForTesting + void startScanning() { + if (mAvailableDevicesCategory != null) { + removeAllDevices(); + } + + mLocalManager.getCachedDeviceManager().clearNonBondedDevices(); + mInitialScanStarted = true; + mLocalAdapter.startScanning(true); + } + + @Override + void onDevicePreferenceClick(BluetoothDevicePreference btPreference) { + mLocalAdapter.stopScanning(); + super.onDevicePreferenceClick(btPreference); + } + + @Override + public void onScanningStateChanged(boolean started) { + mAvailableDevicesCategory.setProgress(started); + } + + @VisibleForTesting + void updateContent(int bluetoothState) { + switch (bluetoothState) { + case BluetoothAdapter.STATE_ON: + mDevicePreferenceMap.clear(); + mLocalAdapter.setBluetoothEnabled(true); + + addDeviceCategory(mAvailableDevicesCategory, + R.string.bluetooth_preference_found_devices, + BluetoothDeviceFilter.UNBONDED_DEVICE_FILTER, mInitialScanStarted); + updateFooterPreference(mFooterPreference); + + if (!mInitialScanStarted) { + startScanning(); + } + + // mLocalAdapter.setScanMode is internally synchronized so it is okay for multiple + // threads to execute. + mLocalAdapter.setScanMode(BluetoothAdapter.SCAN_MODE_CONNECTABLE_DISCOVERABLE); + break; + + case BluetoothAdapter.STATE_OFF: + finish(); + break; + } + } + + @Override + public void onBluetoothStateChanged(int bluetoothState) { + super.onBluetoothStateChanged(bluetoothState); + updateContent(bluetoothState); + } + + @Override + public void onDeviceBondStateChanged(CachedBluetoothDevice cachedDevice, int bondState) { + if (bondState == BluetoothDevice.BOND_BONDED) { + // If one device is connected(bonded), then close this fragment. + finish(); + } + } + + @Override + protected int getHelpResource() { + return R.string.help_url_bluetooth; + } + + @Override + protected String getLogTag() { + return TAG; + } + + @Override + protected int getPreferenceScreenResId() { + return R.xml.bluetooth_pairing_detail; + } + + @Override + protected List getPreferenceControllers(Context context) { + List controllers = new ArrayList<>(); + mDeviceNamePrefController = new BluetoothDeviceNamePreferenceController(context, + this, getLifecycle()); + controllers.add(mDeviceNamePrefController); + + return controllers; + } + + @Override + public String getDeviceListKey() { + return KEY_AVAIL_DEVICES; + } + +} diff --git a/src/com/android/settings/bluetooth/BluetoothPairingPreferenceController.java b/src/com/android/settings/bluetooth/BluetoothPairingPreferenceController.java index 6409d3b3177..ab99aad73c1 100644 --- a/src/com/android/settings/bluetooth/BluetoothPairingPreferenceController.java +++ b/src/com/android/settings/bluetooth/BluetoothPairingPreferenceController.java @@ -16,12 +16,12 @@ package com.android.settings.bluetooth; -import android.app.Fragment; import android.content.Context; import android.support.v14.preference.PreferenceFragment; import android.support.v7.preference.Preference; -import android.support.v7.preference.PreferenceScreen; +import android.os.UserHandle; +import com.android.settings.SettingsActivity; import com.android.settings.core.PreferenceController; import com.android.settings.R; @@ -34,11 +34,14 @@ public class BluetoothPairingPreferenceController extends PreferenceController { public static final String KEY_PAIRING = "pref_bt_pairing"; private PreferenceFragment mFragment; + private SettingsActivity mActivity; private Preference mPreference; - public BluetoothPairingPreferenceController(Context context, PreferenceFragment fragment) { - super(context); + public BluetoothPairingPreferenceController(Context context, PreferenceFragment fragment, + SettingsActivity activity) { + super(context); mFragment = fragment; + mActivity = activity; } @Override @@ -54,7 +57,9 @@ public class BluetoothPairingPreferenceController extends PreferenceController { @Override public boolean handlePreferenceTreeClick(Preference preference) { if (KEY_PAIRING.equals(preference.getKey())) { - //TODO: open the pairing page + mActivity.startPreferencePanelAsUser(mFragment, BluetoothPairingDetail.class.getName(), + null, R.string.bluetooth_pairing_page_title, null, + new UserHandle(UserHandle.myUserId())); return true; } @@ -66,10 +71,11 @@ public class BluetoothPairingPreferenceController extends PreferenceController { * * @return bluetooth preference that created in this method */ - public Preference createBluetoothPairingPreference() { + public Preference createBluetoothPairingPreference(int order) { mPreference = new Preference(mFragment.getPreferenceScreen().getContext()); mPreference.setKey(KEY_PAIRING); mPreference.setIcon(R.drawable.ic_add); + mPreference.setOrder(order); mPreference.setTitle(R.string.bluetooth_pairing_pref_title); return mPreference; diff --git a/src/com/android/settings/bluetooth/BluetoothSettings.java b/src/com/android/settings/bluetooth/BluetoothSettings.java index db0e307b0c3..b37770abf33 100644 --- a/src/com/android/settings/bluetooth/BluetoothSettings.java +++ b/src/com/android/settings/bluetooth/BluetoothSettings.java @@ -19,7 +19,6 @@ package com.android.settings.bluetooth; import android.app.Activity; import android.bluetooth.BluetoothAdapter; import android.bluetooth.BluetoothDevice; -import android.content.BroadcastReceiver; import android.content.ContentResolver; import android.content.Context; import android.content.Intent; @@ -29,10 +28,8 @@ import android.os.Bundle; import android.provider.Settings; import android.support.annotation.VisibleForTesting; import android.support.v7.preference.Preference; -import android.support.v7.preference.PreferenceCategory; import android.support.v7.preference.PreferenceGroup; import android.support.v7.preference.PreferenceScreen; -import android.text.BidiFormatter; import android.text.Spannable; import android.text.style.TextAppearanceSpan; import android.util.Log; @@ -74,11 +71,9 @@ import static android.os.UserManager.DISALLOW_CONFIG_BLUETOOTH; * connection management. * */ -// TODO: Refactor this fragment public class BluetoothSettings extends DeviceListPreferenceFragment implements Indexable { private static final String TAG = "BluetoothSettings"; - private static final int MENU_ID_SCAN = Menu.FIRST; private static final int MENU_ID_SHOW_RECEIVED = Menu.FIRST + 1; /* Private intent to show the list of received files */ @@ -87,47 +82,32 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I private static final String BTOPP_PACKAGE = "com.android.bluetooth"; - private static final String KEY_PAIRED_DEVICES = "paired_devices"; + private static final int PAIRED_DEVICE_ORDER = 1; + private static final int PAIRING_PREF_ORDER = 2; - private static View mSettingsDialogView = null; + @VisibleForTesting + static final String KEY_PAIRED_DEVICES = "paired_devices"; + @VisibleForTesting + static final String KEY_FOOTER_PREF = "footer_preference"; - private BluetoothEnabler mBluetoothEnabler; - - private PreferenceGroup mPairedDevicesCategory; - private PreferenceGroup mAvailableDevicesCategory; - private Preference mDeviceNamePreference; + @VisibleForTesting + PreferenceGroup mPairedDevicesCategory; + @VisibleForTesting + FooterPreference mFooterPreference; private Preference mPairingPreference; - private boolean mAvailableDevicesCategoryIsPresent; - - private boolean mInitialScanStarted; - private boolean mInitiateDiscoverable; + private BluetoothEnabler mBluetoothEnabler; private SwitchBar mSwitchBar; private final IntentFilter mIntentFilter; private BluetoothDeviceNamePreferenceController mDeviceNamePrefController; - private BluetoothPairingPreferenceController mPairingPrefController; + @VisibleForTesting + BluetoothPairingPreferenceController mPairingPrefController; // For Search @VisibleForTesting static final String DATA_KEY_REFERENCE = "main_toggle_bluetooth"; - // accessed from inner class (not private to avoid thunks) - FooterPreference mMyDevicePreference; - - private final BroadcastReceiver mReceiver = new BroadcastReceiver() { - @Override - public void onReceive(Context context, Intent intent) { - final String action = intent.getAction(); - final int state = - intent.getIntExtra(BluetoothAdapter.EXTRA_STATE, BluetoothAdapter.ERROR); - - if (state == BluetoothAdapter.STATE_ON) { - mInitiateDiscoverable = true; - } - } - }; - public BluetoothSettings() { super(DISALLOW_CONFIG_BLUETOOTH); mIntentFilter = new IntentFilter(BluetoothAdapter.ACTION_LOCAL_NAME_CHANGED); @@ -141,15 +121,13 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I @Override public void onActivityCreated(Bundle savedInstanceState) { super.onActivityCreated(savedInstanceState); - mInitialScanStarted = false; - mInitiateDiscoverable = true; final SettingsActivity activity = (SettingsActivity) getActivity(); mSwitchBar = activity.getSwitchBar(); mBluetoothEnabler = new BluetoothEnabler(activity, new SwitchBarController(mSwitchBar), - mMetricsFeatureProvider, Utils.getLocalBtManager(activity), - MetricsEvent.ACTION_BLUETOOTH_TOGGLE); + mMetricsFeatureProvider, Utils.getLocalBtManager(activity), + MetricsEvent.ACTION_BLUETOOTH_TOGGLE); mBluetoothEnabler.setupSwitchController(); } @@ -161,28 +139,11 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I } @Override - void addPreferencesForActivity() { - final Context prefContext = getPrefContext(); - - mDeviceNamePreference = mDeviceNamePrefController.createBluetoothDeviceNamePreference( - getPreferenceScreen(), 1 /* order */); - - mPairedDevicesCategory = new PreferenceCategory(prefContext); - mPairedDevicesCategory.setKey(KEY_PAIRED_DEVICES); - mPairedDevicesCategory.setOrder(2); - getPreferenceScreen().addPreference(mPairedDevicesCategory); - - mAvailableDevicesCategory = new BluetoothProgressCategory(prefContext); - mAvailableDevicesCategory.setSelectable(false); - mAvailableDevicesCategory.setOrder(3); - getPreferenceScreen().addPreference(mAvailableDevicesCategory); - - mMyDevicePreference = mFooterPreferenceMixin.createFooterPreference(); - mMyDevicePreference.setSelectable(false); - - mPairingPreference = mPairingPrefController.createBluetoothPairingPreference(); - - setHasOptionsMenu(true); + void initPreferencesFromPreferenceScreen() { + mPairingPreference = mPairingPrefController.createBluetoothPairingPreference( + PAIRING_PREF_ORDER); + mFooterPreference = (FooterPreference) findPreference(KEY_FOOTER_PREF); + mPairedDevicesCategory = (PreferenceGroup) findPreference(KEY_PAIRED_DEVICES); } @Override @@ -193,19 +154,14 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I mBluetoothEnabler.resume(getActivity()); } super.onStart(); - - mInitiateDiscoverable = true; - if (isUiRestricted()) { - setDeviceListGroup(getPreferenceScreen()); + getPreferenceScreen().removeAll(); if (!isUiRestrictedByOnlyAdmin()) { getEmptyTextView().setText(R.string.bluetooth_empty_list_user_restricted); } - removeAllDevices(); return; } - getActivity().registerReceiver(mReceiver, mIntentFilter); if (mLocalAdapter != null) { updateContent(mLocalAdapter.getBluetoothState()); } @@ -224,8 +180,6 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I if (isUiRestricted()) { return; } - - getActivity().unregisterReceiver(mReceiver); } @Override @@ -234,13 +188,6 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I // If the user is not allowed to configure bluetooth, do not show the menu. if (isUiRestricted()) return; - boolean bluetoothIsEnabled = mLocalAdapter.getBluetoothState() == BluetoothAdapter.STATE_ON; - boolean isDiscovering = mLocalAdapter.isDiscovering(); - int textId = isDiscovering ? R.string.bluetooth_searching_for_devices : - R.string.bluetooth_search_for_devices; - menu.add(Menu.NONE, MENU_ID_SCAN, 0, textId) - .setEnabled(bluetoothIsEnabled && !isDiscovering) - .setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER); menu.add(Menu.NONE, MENU_ID_SHOW_RECEIVED, 0, R.string.bluetooth_show_received_files) .setShowAsAction(MenuItem.SHOW_AS_ACTION_NEVER); super.onCreateOptionsMenu(menu, inflater); @@ -249,14 +196,6 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I @Override public boolean onOptionsItemSelected(MenuItem item) { switch (item.getItemId()) { - case MENU_ID_SCAN: - if (mLocalAdapter.getBluetoothState() == BluetoothAdapter.STATE_ON) { - mMetricsFeatureProvider.action(getActivity(), - MetricsEvent.ACTION_BLUETOOTH_SCAN); - startScanning(); - } - return true; - case MENU_ID_SHOW_RECEIVED: mMetricsFeatureProvider.action(getActivity(), MetricsEvent.ACTION_BLUETOOTH_FILES); @@ -268,104 +207,37 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I return super.onOptionsItemSelected(item); } - private void startScanning() { - if (isUiRestricted()) { - return; - } - - if (!mAvailableDevicesCategoryIsPresent) { - getPreferenceScreen().addPreference(mAvailableDevicesCategory); - mAvailableDevicesCategoryIsPresent = true; - } - - if (mAvailableDevicesCategory != null) { - setDeviceListGroup(mAvailableDevicesCategory); - removeAllDevices(); - } - - mLocalManager.getCachedDeviceManager().clearNonBondedDevices(); - mAvailableDevicesCategory.removeAll(); - mInitialScanStarted = true; - mLocalAdapter.startScanning(true); - } - @Override - void onDevicePreferenceClick(BluetoothDevicePreference btPreference) { - mLocalAdapter.stopScanning(); - super.onDevicePreferenceClick(btPreference); - } - - private void addDeviceCategory(PreferenceGroup preferenceGroup, int titleId, - BluetoothDeviceFilter.Filter filter, boolean addCachedDevices) { - cacheRemoveAllPrefs(preferenceGroup); - preferenceGroup.setTitle(titleId); - setFilter(filter); - setDeviceListGroup(preferenceGroup); - if (addCachedDevices) { - addCachedDevices(); - } - preferenceGroup.setEnabled(true); - removeCachedPrefs(preferenceGroup); + public String getDeviceListKey() { + return KEY_PAIRED_DEVICES; } private void updateContent(int bluetoothState) { - final PreferenceScreen preferenceScreen = getPreferenceScreen(); int messageId = 0; switch (bluetoothState) { case BluetoothAdapter.STATE_ON: + displayEmptyMessage(false); mDevicePreferenceMap.clear(); if (isUiRestricted()) { messageId = R.string.bluetooth_empty_list_user_restricted; break; } - getPreferenceScreen().removeAll(); - getPreferenceScreen().addPreference(mDeviceNamePreference); - getPreferenceScreen().addPreference(mPairedDevicesCategory); - getPreferenceScreen().addPreference(mAvailableDevicesCategory); - getPreferenceScreen().addPreference(mMyDevicePreference); - // Paired devices category addDeviceCategory(mPairedDevicesCategory, R.string.bluetooth_preference_paired_devices, BluetoothDeviceFilter.BONDED_DEVICE_FILTER, true); mPairedDevicesCategory.addPreference(mPairingPreference); - int numberOfPairedDevices = mPairedDevicesCategory.getPreferenceCount(); + updateFooterPreference(mFooterPreference); - if (isUiRestricted() || numberOfPairedDevices <= 0) { - if (preferenceScreen.findPreference(KEY_PAIRED_DEVICES) != null) { - preferenceScreen.removePreference(mPairedDevicesCategory); - } - } else { - if (preferenceScreen.findPreference(KEY_PAIRED_DEVICES) == null) { - preferenceScreen.addPreference(mPairedDevicesCategory); - } - } - - // Available devices category - addDeviceCategory(mAvailableDevicesCategory, - R.string.bluetooth_preference_found_devices, - BluetoothDeviceFilter.UNBONDED_DEVICE_FILTER, mInitialScanStarted); - - if (!mInitialScanStarted) { - startScanning(); - } - - updateMyDevicePreference(mMyDevicePreference); getActivity().invalidateOptionsMenu(); - - // mLocalAdapter.setScanMode is internally synchronized so it is okay for multiple - // threads to execute. - if (mInitiateDiscoverable) { - // Make the device visible to other devices. - mLocalAdapter.setScanMode(BluetoothAdapter.SCAN_MODE_CONNECTABLE_DISCOVERABLE); - mInitiateDiscoverable = false; - } + mLocalAdapter.setScanMode(BluetoothAdapter.SCAN_MODE_CONNECTABLE_DISCOVERABLE); return; // not break case BluetoothAdapter.STATE_TURNING_OFF: messageId = R.string.bluetooth_turning_off; + mLocalAdapter.stopScanning(); break; case BluetoothAdapter.STATE_OFF: @@ -377,12 +249,10 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I case BluetoothAdapter.STATE_TURNING_ON: messageId = R.string.bluetooth_turning_on; - mInitialScanStarted = false; break; } - setDeviceListGroup(preferenceScreen); - removeAllDevices(); + displayEmptyMessage(true); if (messageId != 0) { getEmptyTextView().setText(messageId); } @@ -421,18 +291,21 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I } }); } - getPreferenceScreen().removeAll(); setTextSpan(emptyView.getText(), briefText); } + @VisibleForTesting + void displayEmptyMessage(boolean display) { + final Activity activity = getActivity(); + activity.findViewById(android.R.id.list_container).setVisibility( + display ? View.INVISIBLE : View.VISIBLE); + activity.findViewById(android.R.id.empty).setVisibility( + display ? View.VISIBLE : View.GONE); + } + @Override public void onBluetoothStateChanged(int bluetoothState) { super.onBluetoothStateChanged(bluetoothState); - // If BT is turned off/on staying in the same BT Settings screen - // discoverability to be set again - if (BluetoothAdapter.STATE_ON == bluetoothState) { - mInitiateDiscoverable = true; - } updateContent(bluetoothState); } @@ -440,15 +313,14 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I public void onScanningStateChanged(boolean started) { super.onScanningStateChanged(started); // Update options' enabled state - if (getActivity() != null) { - getActivity().invalidateOptionsMenu(); + final Activity activity = getActivity(); + if (activity != null) { + activity.invalidateOptionsMenu(); } } @Override public void onDeviceBondStateChanged(CachedBluetoothDevice cachedDevice, int bondState) { - setDeviceListGroup(getPreferenceScreen()); - removeAllDevices(); updateContent(mLocalAdapter.getBluetoothState()); } @@ -457,20 +329,11 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I if (text instanceof Spannable) { Spannable boldSpan = (Spannable) text; boldSpan.setSpan( - new TextAppearanceSpan(getActivity(), android.R.style.TextAppearance_Medium), 0, - briefText.length(), Spannable.SPAN_EXCLUSIVE_EXCLUSIVE); + new TextAppearanceSpan(getActivity(), android.R.style.TextAppearance_Medium), 0, + briefText.length(), Spannable.SPAN_EXCLUSIVE_EXCLUSIVE); } } - @VisibleForTesting - void updateMyDevicePreference(Preference myDevicePreference) { - final BidiFormatter bidiFormatter = BidiFormatter.getInstance(); - - myDevicePreference.setTitle(getString( - R.string.bluetooth_footer_mac_message, - bidiFormatter.unicodeWrap(mLocalAdapter.getAddress()))); - } - @VisibleForTesting void setLocalBluetoothAdapter(LocalBluetoothAdapter localAdapter) { mLocalAdapter = localAdapter; @@ -504,6 +367,7 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I */ @Override void initDevicePreference(BluetoothDevicePreference preference) { + preference.setOrder(PAIRED_DEVICE_ORDER); CachedBluetoothDevice cachedDevice = preference.getCachedDevice(); if (cachedDevice.getBondState() == BluetoothDevice.BOND_BONDED) { // Only paired device have an associated advanced settings screen @@ -531,7 +395,8 @@ public class BluetoothSettings extends DeviceListPreferenceFragment implements I List controllers = new ArrayList<>(); mDeviceNamePrefController = new BluetoothDeviceNamePreferenceController(context, this, getLifecycle()); - mPairingPrefController = new BluetoothPairingPreferenceController(context, this); + mPairingPrefController = new BluetoothPairingPreferenceController(context, this, + (SettingsActivity) getActivity()); controllers.add(mDeviceNamePrefController); controllers.add(mPairingPrefController); diff --git a/src/com/android/settings/bluetooth/DeviceListPreferenceFragment.java b/src/com/android/settings/bluetooth/DeviceListPreferenceFragment.java index c35e652eac1..714704e186b 100644 --- a/src/com/android/settings/bluetooth/DeviceListPreferenceFragment.java +++ b/src/com/android/settings/bluetooth/DeviceListPreferenceFragment.java @@ -19,12 +19,15 @@ package com.android.settings.bluetooth; import android.bluetooth.BluetoothAdapter; import android.bluetooth.BluetoothDevice; import android.os.Bundle; +import android.support.annotation.VisibleForTesting; import android.support.v7.preference.Preference; import android.support.v7.preference.PreferenceCategory; import android.support.v7.preference.PreferenceGroup; +import android.text.BidiFormatter; import android.util.Log; import com.android.settings.dashboard.RestrictedDashboardFragment; +import com.android.settings.R; import com.android.settingslib.bluetooth.BluetoothCallback; import com.android.settingslib.bluetooth.BluetoothDeviceFilter; import com.android.settingslib.bluetooth.CachedBluetoothDevice; @@ -47,7 +50,6 @@ public abstract class DeviceListPreferenceFragment extends private static final String TAG = "DeviceListPreferenceFragment"; - private static final String KEY_BT_DEVICE_LIST = "bt_device_list"; private static final String KEY_BT_SCAN = "bt_scan"; private BluetoothDeviceFilter.Filter mFilter; @@ -57,7 +59,8 @@ public abstract class DeviceListPreferenceFragment extends LocalBluetoothAdapter mLocalAdapter; LocalBluetoothManager mLocalManager; - private PreferenceGroup mDeviceListGroup; + @VisibleForTesting + PreferenceGroup mDeviceListGroup; final WeakHashMap mDevicePreferenceMap = new WeakHashMap(); @@ -86,17 +89,13 @@ public abstract class DeviceListPreferenceFragment extends } mLocalAdapter = mLocalManager.getBluetoothAdapter(); - addPreferencesForActivity(); + initPreferencesFromPreferenceScreen(); - mDeviceListGroup = (PreferenceCategory) findPreference(KEY_BT_DEVICE_LIST); + mDeviceListGroup = (PreferenceCategory) findPreference(getDeviceListKey()); } - void setDeviceListGroup(PreferenceGroup preferenceGroup) { - mDeviceListGroup = preferenceGroup; - } - - /** Add preferences from the subclass. */ - abstract void addPreferencesForActivity(); + /** find and update preference that already existed in preference screen */ + abstract void initPreferencesFromPreferenceScreen(); @Override public void onStart() { @@ -105,8 +104,6 @@ public abstract class DeviceListPreferenceFragment extends mLocalManager.setForegroundActivity(getActivity()); mLocalManager.getEventManager().registerCallback(this); - - updateProgressUi(mLocalAdapter.isDiscovering()); } @Override @@ -122,7 +119,6 @@ public abstract class DeviceListPreferenceFragment extends } void removeAllDevices() { - mLocalAdapter.stopScanning(); mDevicePreferenceMap.clear(); mDeviceListGroup.removeAll(); } @@ -157,6 +153,7 @@ public abstract class DeviceListPreferenceFragment extends btPreference.onClicked(); } + @Override public void onDeviceAdded(CachedBluetoothDevice cachedDevice) { if (mDevicePreferenceMap.get(cachedDevice) != null) { return; @@ -202,6 +199,16 @@ public abstract class DeviceListPreferenceFragment extends // Does nothing by default } + @VisibleForTesting + void updateFooterPreference(Preference myDevicePreference) { + final BidiFormatter bidiFormatter = BidiFormatter.getInstance(); + + myDevicePreference.setTitle(getString( + R.string.bluetooth_footer_mac_message, + bidiFormatter.unicodeWrap(mLocalAdapter.getAddress()))); + } + + @Override public void onDeviceDeleted(CachedBluetoothDevice cachedDevice) { BluetoothDevicePreference preference = mDevicePreferenceMap.remove(cachedDevice); if (preference != null) { @@ -209,21 +216,39 @@ public abstract class DeviceListPreferenceFragment extends } } - public void onScanningStateChanged(boolean started) { - updateProgressUi(started); - } + @Override + public void onScanningStateChanged(boolean started) {} - private void updateProgressUi(boolean start) { - if (mDeviceListGroup instanceof BluetoothProgressCategory) { - ((BluetoothProgressCategory) mDeviceListGroup).setProgress(start); - } - } + @Override + public void onBluetoothStateChanged(int bluetoothState) {} - public void onBluetoothStateChanged(int bluetoothState) { - if (bluetoothState == BluetoothAdapter.STATE_OFF) { - updateProgressUi(false); + /** + * Add bluetooth device preferences to {@code preferenceGroup} which satisfy the {@code filter} + * + * This method will also (1) set the title for {@code preferenceGroup} and (2) change the + * default preferenceGroup and filter + * @param preferenceGroup + * @param titleId + * @param filter + * @param addCachedDevices + */ + public void addDeviceCategory(PreferenceGroup preferenceGroup, int titleId, + BluetoothDeviceFilter.Filter filter, boolean addCachedDevices) { + cacheRemoveAllPrefs(preferenceGroup); + preferenceGroup.setTitle(titleId); + mDeviceListGroup = preferenceGroup; + setFilter(filter); + if (addCachedDevices) { + addCachedDevices(); } + preferenceGroup.setEnabled(true); + removeCachedPrefs(preferenceGroup); } public void onConnectionStateChanged(CachedBluetoothDevice cachedDevice, int state) { } + + /** + * Return the key of the {@link PreferenceGroup} that contains the bluetooth devices + */ + public abstract String getDeviceListKey(); } diff --git a/src/com/android/settings/bluetooth/DevicePickerFragment.java b/src/com/android/settings/bluetooth/DevicePickerFragment.java index cb8be506ef0..0cf13a3b5b3 100644 --- a/src/com/android/settings/bluetooth/DevicePickerFragment.java +++ b/src/com/android/settings/bluetooth/DevicePickerFragment.java @@ -40,8 +40,9 @@ import java.util.List; * BluetoothSettings is the Settings screen for Bluetooth configuration and * connection management. */ -public final class DevicePickerFragment extends DeviceListPreferenceObsoleteFragment { +public final class DevicePickerFragment extends DeviceListPreferenceFragment { private static final int MENU_ID_REFRESH = Menu.FIRST; + private static final String KEY_BT_DEVICE_LIST = "bt_device_list"; private static final String TAG = "DevicePickerFragment"; public DevicePickerFragment() { @@ -54,7 +55,7 @@ public final class DevicePickerFragment extends DeviceListPreferenceObsoleteFrag private boolean mStartScanOnStart; @Override - void addPreferencesForActivity() { + void initPreferencesFromPreferenceScreen() { Intent intent = getActivity().getIntent(); mNeedAuth = intent.getBooleanExtra(BluetoothDevicePicker.EXTRA_NEED_AUTH, false); setFilter(intent.getIntExtra(BluetoothDevicePicker.EXTRA_FILTER_TYPE, @@ -167,6 +168,11 @@ public final class DevicePickerFragment extends DeviceListPreferenceObsoleteFrag return null; } + @Override + public String getDeviceListKey() { + return KEY_BT_DEVICE_LIST; + } + private void sendDevicePickedIntent(BluetoothDevice device) { Intent intent = new Intent(BluetoothDevicePicker.ACTION_DEVICE_SELECTED); intent.putExtra(BluetoothDevice.EXTRA_DEVICE, device); diff --git a/tests/robotests/assets/grandfather_not_implementing_index_provider b/tests/robotests/assets/grandfather_not_implementing_index_provider index e21b76cf780..c6687799fd9 100644 --- a/tests/robotests/assets/grandfather_not_implementing_index_provider +++ b/tests/robotests/assets/grandfather_not_implementing_index_provider @@ -1,4 +1,5 @@ com.android.settings.bluetooth.DevicePickerFragment +com.android.settings.bluetooth.BluetoothPairingDetail com.android.settings.notification.ZenModePrioritySettings com.android.settings.accounts.AccountDetailDashboardFragment com.android.settings.fuelgauge.PowerUsageAnomalyDetails diff --git a/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingDetailTest.java b/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingDetailTest.java new file mode 100644 index 00000000000..6774ba9a060 --- /dev/null +++ b/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingDetailTest.java @@ -0,0 +1,133 @@ +/* + * Copyright (C) 2017 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.bluetooth; + +import static com.google.common.truth.Truth.assertThat; + +import static org.mockito.Matchers.any; +import static org.mockito.Matchers.anyBoolean; +import static org.mockito.Matchers.anyInt; +import static org.mockito.Mockito.doNothing; +import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.spy; +import static org.mockito.Mockito.verify; + +import android.bluetooth.BluetoothAdapter; +import android.content.Context; +import android.content.res.Resources; +import android.os.UserManager; +import android.support.v7.preference.PreferenceGroup; + +import com.android.settings.R; +import com.android.settings.SettingsRobolectricTestRunner; +import com.android.settings.TestConfig; +import com.android.settingslib.bluetooth.BluetoothDeviceFilter; +import com.android.settingslib.bluetooth.LocalBluetoothAdapter; +import com.android.settingslib.bluetooth.LocalBluetoothManager; +import com.android.settingslib.widget.FooterPreference; + +import org.junit.Before; +import org.junit.Test; +import org.junit.runner.RunWith; +import org.mockito.Answers; +import org.mockito.Mock; +import org.mockito.MockitoAnnotations; +import org.robolectric.RuntimeEnvironment; +import org.robolectric.annotation.Config; + +@RunWith(SettingsRobolectricTestRunner.class) +@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION) +public class BluetoothPairingDetailTest { + + @Mock + private UserManager mUserManager; + @Mock + private Resources mResource; + @Mock + private LocalBluetoothAdapter mLocalAdapter; + @Mock(answer = Answers.RETURNS_DEEP_STUBS) + private LocalBluetoothManager mLocalManager; + @Mock + private PreferenceGroup mPreferenceGroup; + private BluetoothPairingDetail mFragment; + private Context mContext; + private BluetoothProgressCategory mAvailableDevicesCategory; + private FooterPreference mFooterPreference; + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + + mContext = RuntimeEnvironment.application; + mFragment = spy(new BluetoothPairingDetail()); + doReturn(mContext).when(mFragment).getContext(); + doReturn(mResource).when(mFragment).getResources(); + + mAvailableDevicesCategory = spy(new BluetoothProgressCategory(mContext)); + mFooterPreference = new FooterPreference(mContext); + + mFragment.mLocalAdapter = mLocalAdapter; + mFragment.mLocalManager = mLocalManager; + mFragment.mDeviceListGroup = mPreferenceGroup; + } + + @Test + public void testInitPreferencesFromPreferenceScreen_findPreferences() { + doReturn(mAvailableDevicesCategory).when(mFragment).findPreference( + BluetoothPairingDetail.KEY_AVAIL_DEVICES); + doReturn(mFooterPreference).when(mFragment).findPreference( + BluetoothPairingDetail.KEY_FOOTER_PREF); + + mFragment.initPreferencesFromPreferenceScreen(); + + assertThat(mFragment.mAvailableDevicesCategory).isEqualTo(mAvailableDevicesCategory); + assertThat(mFragment.mFooterPreference).isEqualTo(mFooterPreference); + } + + @Test + public void testStartScanning_startScanAndRemoveDevices() { + mFragment.mAvailableDevicesCategory = mAvailableDevicesCategory; + mFragment.mDeviceListGroup = mAvailableDevicesCategory; + + mFragment.startScanning(); + + verify(mLocalAdapter).startScanning(true); + verify(mAvailableDevicesCategory).removeAll(); + } + + @Test + public void testUpdateContent_stateOn_addDevices() { + mFragment.mAvailableDevicesCategory = mAvailableDevicesCategory; + mFragment.mFooterPreference = mFooterPreference; + doNothing().when(mFragment).addDeviceCategory(any(), anyInt(), any(), anyBoolean()); + + mFragment.updateContent(BluetoothAdapter.STATE_ON); + + verify(mFragment).addDeviceCategory(mAvailableDevicesCategory, + R.string.bluetooth_preference_found_devices, + BluetoothDeviceFilter.UNBONDED_DEVICE_FILTER, false); + verify(mLocalAdapter).setScanMode(BluetoothAdapter.SCAN_MODE_CONNECTABLE_DISCOVERABLE); + } + + @Test + public void testUpdateContent_stateOff_finish() { + mFragment.updateContent(BluetoothAdapter.STATE_OFF); + + verify(mFragment).finish(); + } + +} diff --git a/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingPreferenceControllerTest.java index 24325b31ffe..937b31fd2d3 100644 --- a/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/bluetooth/BluetoothPairingPreferenceControllerTest.java @@ -18,6 +18,11 @@ package com.android.settings.bluetooth; import static com.google.common.truth.Truth.assertThat; +import static org.mockito.Matchers.any; +import static org.mockito.Matchers.anyInt; +import static org.mockito.Matchers.anyString; +import static org.mockito.Matchers.eq; +import static org.mockito.Mockito.verify; import static org.mockito.Mockito.when; import android.app.FragmentManager; @@ -27,6 +32,7 @@ import android.support.v14.preference.PreferenceFragment; import android.support.v7.preference.Preference; import android.support.v7.preference.PreferenceScreen; +import com.android.settings.SettingsActivity; import com.android.settings.SettingsRobolectricTestRunner; import com.android.settings.TestConfig; import com.android.settings.R; @@ -45,7 +51,7 @@ import org.robolectric.annotation.Config; @RunWith(SettingsRobolectricTestRunner.class) @Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION) public class BluetoothPairingPreferenceControllerTest { - + private static final int ORDER = 1; private Context mContext; @Mock(answer = Answers.RETURNS_DEEP_STUBS) private PreferenceFragment mFragment; @@ -59,6 +65,8 @@ public class BluetoothPairingPreferenceControllerTest { private FragmentTransaction mFragmentTransaction; @Mock private PreferenceScreen mPreferenceScreen; + @Mock + private SettingsActivity mSettingsActivity; private Preference mPreference; private BluetoothPairingPreferenceController mController; @@ -70,16 +78,29 @@ public class BluetoothPairingPreferenceControllerTest { mContext = RuntimeEnvironment.application; when(mFragment.getPreferenceScreen().getContext()).thenReturn(mContext); - mController = new BluetoothPairingPreferenceController(mContext, mFragment); + mPreference = new Preference(mContext); + mPreference.setKey(BluetoothPairingPreferenceController.KEY_PAIRING); + + mController = new BluetoothPairingPreferenceController(mContext, mFragment, + mSettingsActivity); } @Test public void testCreateBluetoothPairingPreference() { - Preference pref = mController.createBluetoothPairingPreference(); + Preference pref = mController.createBluetoothPairingPreference(ORDER); assertThat(pref.getKey()).isEqualTo(BluetoothPairingPreferenceController.KEY_PAIRING); assertThat(pref.getIcon()).isEqualTo(mContext.getDrawable(R.drawable.ic_add)); + assertThat(pref.getOrder()).isEqualTo(ORDER); assertThat(pref.getTitle()).isEqualTo( mContext.getString(R.string.bluetooth_pairing_pref_title)); } + + @Test + public void testHandlePreferenceTreeClick_startFragment() { + mController.handlePreferenceTreeClick(mPreference); + + verify(mSettingsActivity).startPreferencePanelAsUser(eq(mFragment), anyString(), any(), + anyInt(), any(), any()); + } } diff --git a/tests/robotests/src/com/android/settings/bluetooth/BluetoothSettingsTest.java b/tests/robotests/src/com/android/settings/bluetooth/BluetoothSettingsTest.java index c0aa723ec76..6ceca41a13d 100644 --- a/tests/robotests/src/com/android/settings/bluetooth/BluetoothSettingsTest.java +++ b/tests/robotests/src/com/android/settings/bluetooth/BluetoothSettingsTest.java @@ -23,16 +23,21 @@ import static org.mockito.Matchers.eq; import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.spy; +import android.app.Activity; import android.content.Context; import android.content.res.Resources; import android.os.UserManager; import android.support.v7.preference.Preference; +import android.support.v7.preference.PreferenceGroup; +import android.view.View; +import android.widget.TextView; import com.android.settings.R; import com.android.settings.SettingsRobolectricTestRunner; import com.android.settings.TestConfig; import com.android.settings.testutils.FakeFeatureFactory; import com.android.settingslib.bluetooth.LocalBluetoothAdapter; +import com.android.settingslib.widget.FooterPreference; import org.junit.Before; import org.junit.Test; @@ -54,28 +59,43 @@ public class BluetoothSettingsTest { private UserManager mUserManager; @Mock private Resources mResource; - @Mock(answer = Answers.RETURNS_DEEP_STUBS) - private Context mContext; @Mock private LocalBluetoothAdapter mLocalAdapter; + @Mock + private Activity mActivity; + @Mock + private PreferenceGroup mPairedDevicesCategory; + @Mock + private BluetoothPairingPreferenceController mPairingPreferenceController; + private Context mContext; private BluetoothSettings mFragment; - private Preference mMyDevicePreference; private FakeFeatureFactory mFeatureFactory; + private Preference mFooterPreference; + private TextView mEmptyMessage; + private View mContainer; @Before public void setUp() { MockitoAnnotations.initMocks(this); + mContext = spy(RuntimeEnvironment.application); FakeFeatureFactory.setupForTest(mContext); mFeatureFactory = (FakeFeatureFactory) FakeFeatureFactory.getFactory(mContext); mFragment = spy(new BluetoothSettings()); + doReturn(mContext).when(mFragment).getContext(); doReturn(mResource).when(mFragment).getResources(); + doReturn(mActivity).when(mFragment).getActivity(); - mMyDevicePreference = new Preference(RuntimeEnvironment.application); + mContainer = new View(mContext); + mEmptyMessage = new TextView(mContext); + doReturn(mContainer).when(mActivity).findViewById(android.R.id.list_container); + doReturn(mEmptyMessage).when(mActivity).findViewById(android.R.id.empty); + mFooterPreference = new FooterPreference(RuntimeEnvironment.application); mFragment.setLocalBluetoothAdapter(mLocalAdapter); + mFragment.mPairingPrefController = mPairingPreferenceController; } @Test @@ -89,9 +109,38 @@ public class BluetoothSettingsTest { doReturn(FOOTAGE_MAC_STRING).when(mFragment).getString( eq(R.string.bluetooth_footer_mac_message), any()); - mFragment.updateMyDevicePreference(mMyDevicePreference); + mFragment.updateFooterPreference(mFooterPreference); - assertThat(mMyDevicePreference.getTitle()).isEqualTo(FOOTAGE_MAC_STRING); + assertThat(mFooterPreference.getTitle()).isEqualTo(FOOTAGE_MAC_STRING); + } + + @Test + public void testDisplayEmptyMessage_showEmptyMessage() { + mFragment.displayEmptyMessage(true); + + assertThat(mContainer.getVisibility()).isEqualTo(View.INVISIBLE); + assertThat(mEmptyMessage.getVisibility()).isEqualTo(View.VISIBLE); + } + + @Test + public void testDisplayEmptyMessage_hideEmptyMessage() { + mFragment.displayEmptyMessage(false); + + assertThat(mContainer.getVisibility()).isEqualTo(View.VISIBLE); + assertThat(mEmptyMessage.getVisibility()).isEqualTo(View.GONE); + } + + @Test + public void testInitPreferencesFromPreferenceScreen() { + doReturn(mPairedDevicesCategory).when(mFragment).findPreference( + BluetoothSettings.KEY_PAIRED_DEVICES); + doReturn(mFooterPreference).when(mFragment).findPreference( + BluetoothSettings.KEY_FOOTER_PREF); + + mFragment.initPreferencesFromPreferenceScreen(); + + assertThat(mFragment.mPairedDevicesCategory).isEqualTo(mPairedDevicesCategory); + assertThat(mFragment.mFooterPreference).isEqualTo(mFooterPreference); } @Test diff --git a/tests/robotests/src/com/android/settings/bluetooth/DeviceListPreferenceFragmentTest.java b/tests/robotests/src/com/android/settings/bluetooth/DeviceListPreferenceFragmentTest.java new file mode 100644 index 00000000000..bbb73597e98 --- /dev/null +++ b/tests/robotests/src/com/android/settings/bluetooth/DeviceListPreferenceFragmentTest.java @@ -0,0 +1,131 @@ +/* + * Copyright (C) 2017 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.bluetooth; + +import static com.google.common.truth.Truth.assertThat; + +import static org.mockito.Matchers.any; +import static org.mockito.Matchers.eq; +import static org.mockito.Mockito.doReturn; +import static org.mockito.Mockito.spy; + +import android.content.Context; +import android.content.res.Resources; +import android.os.UserManager; +import android.support.v7.preference.Preference; + +import com.android.settings.R; +import com.android.settings.SettingsRobolectricTestRunner; +import com.android.settings.TestConfig; +import com.android.settings.core.PreferenceController; +import com.android.settingslib.bluetooth.CachedBluetoothDevice; +import com.android.settingslib.bluetooth.LocalBluetoothAdapter; + +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 java.util.List; + +@RunWith(SettingsRobolectricTestRunner.class) +@Config(manifest = TestConfig.MANIFEST_PATH, sdk = TestConfig.SDK_VERSION) +public class DeviceListPreferenceFragmentTest { + private static final String FOOTAGE_MAC_STRING = "Bluetooth mac: xxxx"; + + @Mock + private UserManager mUserManager; + @Mock + private Resources mResource; + @Mock + private Context mContext; + @Mock + private LocalBluetoothAdapter mLocalAdapter; + private TestFragment mFragment; + private Preference mMyDevicePreference; + + @Before + public void setUp() { + MockitoAnnotations.initMocks(this); + + mFragment = spy(new TestFragment()); + doReturn(mContext).when(mFragment).getContext(); + doReturn(mResource).when(mFragment).getResources(); + mFragment.mLocalAdapter = mLocalAdapter; + + mMyDevicePreference = new Preference(RuntimeEnvironment.application); + } + + @Test + public void setUpdateMyDevicePreference_setTitleCorrectly() { + doReturn(FOOTAGE_MAC_STRING).when(mFragment).getString( + eq(R.string.bluetooth_footer_mac_message), any()); + + mFragment.updateFooterPreference(mMyDevicePreference); + + assertThat(mMyDevicePreference.getTitle()).isEqualTo(FOOTAGE_MAC_STRING); + } + + /** + * Fragment to test since {@code DeviceListPreferenceFragment} is abstract + */ + public static class TestFragment extends DeviceListPreferenceFragment { + + public TestFragment() { + super(""); + } + + @Override + public int getMetricsCategory() { + return 0; + } + + @Override + public void onDeviceBondStateChanged(CachedBluetoothDevice cachedDevice, int bondState) { + + } + + @Override + void initPreferencesFromPreferenceScreen() { + + } + + @Override + public String getDeviceListKey() { + return null; + } + + @Override + protected String getLogTag() { + return null; + } + + @Override + protected int getPreferenceScreenResId() { + return 0; + } + + @Override + protected List getPreferenceControllers(Context context) { + return null; + } + } + +}