Add storage_summary_donut above ProfileSelectStorageFragment
- Modify ProfileSelectFragment to support add preference xml in the top, and tabLayout below the preferences. Base preference layout is dummy_preference_screen.xml which contains no preference. ProfileSelectStorageFragment contains StorageSummaryDonutPreference above the tabLayout. - Make StorageSummaryDonutPreferenceController self workable without StorageDashboardFragment dependence. - Rename inactive_apps.xml to dummy_preference_screen.xml - Move ShadowPrivateStorageInfo from LowStorageSliceTest Bug: 141601408 Test: manual Change-Id: Ide12840dc81bb104f328e230ecda5d35bba01d7a
This commit is contained in:
@@ -16,15 +16,13 @@
|
||||
package com.android.settings.deviceinfo.storage;
|
||||
|
||||
import static com.android.settings.applications.manageapplications.ManageApplications.EXTRA_WORK_ID;
|
||||
import static com.android.settings.applications.manageapplications.ManageApplications
|
||||
.EXTRA_WORK_ONLY;
|
||||
import static com.android.settings.utils.FileSizeFormatter.MEGABYTE_IN_BYTES;
|
||||
|
||||
import static com.google.common.truth.Truth.assertThat;
|
||||
|
||||
import static org.mockito.ArgumentMatchers.anyInt;
|
||||
import static org.mockito.ArgumentMatchers.nullable;
|
||||
import static org.mockito.ArgumentMatchers.eq;
|
||||
import static org.mockito.ArgumentMatchers.nullable;
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.spy;
|
||||
import static org.mockito.Mockito.times;
|
||||
@@ -51,6 +49,7 @@ import com.android.settings.R;
|
||||
import com.android.settings.SettingsActivity;
|
||||
import com.android.settings.SubSettings;
|
||||
import com.android.settings.applications.manageapplications.ManageApplications;
|
||||
import com.android.settings.dashboard.profileselector.ProfileSelectFragment;
|
||||
import com.android.settings.deviceinfo.PrivateVolumeSettings;
|
||||
import com.android.settings.deviceinfo.StorageItemPreference;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
@@ -195,8 +194,8 @@ public class StorageItemPreferenceControllerTest {
|
||||
.isEqualTo(R.string.apps_storage);
|
||||
assertThat(
|
||||
intent.getBundleExtra(SettingsActivity.EXTRA_SHOW_FRAGMENT_ARGUMENTS)
|
||||
.getBoolean(EXTRA_WORK_ONLY))
|
||||
.isTrue();
|
||||
.getInt(ProfileSelectFragment.EXTRA_PROFILE))
|
||||
.isEqualTo(ProfileSelectFragment.WORK);
|
||||
assertThat(
|
||||
intent.getBundleExtra(SettingsActivity.EXTRA_SHOW_FRAGMENT_ARGUMENTS)
|
||||
.getInt(EXTRA_WORK_ID))
|
||||
|
@@ -37,24 +37,30 @@ import android.view.View;
|
||||
import android.widget.Button;
|
||||
import android.widget.LinearLayout;
|
||||
|
||||
import androidx.preference.PreferenceScreen;
|
||||
import androidx.preference.PreferenceViewHolder;
|
||||
|
||||
import com.android.internal.logging.nano.MetricsProto.MetricsEvent;
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
import com.android.settings.testutils.shadow.ShadowPrivateStorageInfo;
|
||||
import com.android.settingslib.core.instrumentation.MetricsFeatureProvider;
|
||||
import com.android.settingslib.deviceinfo.PrivateStorageInfo;
|
||||
import com.android.settingslib.deviceinfo.StorageVolumeProvider;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
import org.junit.runner.RunWith;
|
||||
import org.mockito.Mockito;
|
||||
import org.robolectric.Robolectric;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
import org.robolectric.annotation.Config;
|
||||
|
||||
import java.io.File;
|
||||
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
@Config(shadows = ShadowPrivateStorageInfo.class)
|
||||
public class StorageSummaryDonutPreferenceControllerTest {
|
||||
|
||||
private Context mContext;
|
||||
@@ -63,14 +69,18 @@ public class StorageSummaryDonutPreferenceControllerTest {
|
||||
private PreferenceViewHolder mHolder;
|
||||
private FakeFeatureFactory mFakeFeatureFactory;
|
||||
private MetricsFeatureProvider mMetricsFeatureProvider;
|
||||
private PreferenceScreen mScreen;
|
||||
|
||||
@Before
|
||||
public void setUp() throws Exception {
|
||||
ShadowPrivateStorageInfo.setPrivateStorageInfo(new PrivateStorageInfo(10L, 100L));
|
||||
mContext = spy(Robolectric.setupActivity(Activity.class));
|
||||
mFakeFeatureFactory = FakeFeatureFactory.setupForTest();
|
||||
mMetricsFeatureProvider = mFakeFeatureFactory.getMetricsFeatureProvider();
|
||||
mController = new StorageSummaryDonutPreferenceController(mContext);
|
||||
mController = new StorageSummaryDonutPreferenceController(mContext, "key");
|
||||
mPreference = new StorageSummaryDonutPreference(mContext);
|
||||
mScreen = spy(new PreferenceScreen(mContext, null));
|
||||
when(mScreen.findPreference("key")).thenReturn(mPreference);
|
||||
|
||||
LayoutInflater inflater = LayoutInflater.from(mContext);
|
||||
final View view =
|
||||
@@ -79,10 +89,16 @@ public class StorageSummaryDonutPreferenceControllerTest {
|
||||
mHolder = PreferenceViewHolder.createInstanceForTests(view);
|
||||
}
|
||||
|
||||
@After
|
||||
public void tearDown() {
|
||||
ShadowPrivateStorageInfo.reset();
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testEmpty() {
|
||||
final long totalSpace = 32 * GIGABYTE;
|
||||
final long usedSpace = 0;
|
||||
mController.displayPreference(mScreen);
|
||||
mController.updateBytes(0, 32 * GIGABYTE);
|
||||
mController.updateState(mPreference);
|
||||
|
||||
@@ -98,6 +114,7 @@ public class StorageSummaryDonutPreferenceControllerTest {
|
||||
public void testTotalStorage() {
|
||||
final long totalSpace = KILOBYTE * 10;
|
||||
final long usedSpace = KILOBYTE;
|
||||
mController.displayPreference(mScreen);
|
||||
mController.updateBytes(KILOBYTE, totalSpace);
|
||||
mController.updateState(mPreference);
|
||||
|
||||
@@ -121,6 +138,7 @@ public class StorageSummaryDonutPreferenceControllerTest {
|
||||
when(file.getTotalSpace()).thenReturn(totalSpace);
|
||||
when(file.getFreeSpace()).thenReturn(freeSpace);
|
||||
when(svp.getPrimaryStorageSize()).thenReturn(totalSpace);
|
||||
mController.displayPreference(mScreen);
|
||||
|
||||
mController.updateSizes(svp, volume);
|
||||
mController.updateState(mPreference);
|
||||
|
@@ -28,8 +28,8 @@ import androidx.slice.SliceProvider;
|
||||
import androidx.slice.widget.SliceLiveData;
|
||||
|
||||
import com.android.settings.R;
|
||||
import com.android.settings.testutils.shadow.ShadowPrivateStorageInfo;
|
||||
import com.android.settingslib.deviceinfo.PrivateStorageInfo;
|
||||
import com.android.settingslib.deviceinfo.StorageVolumeProvider;
|
||||
|
||||
import org.junit.After;
|
||||
import org.junit.Before;
|
||||
@@ -38,9 +38,6 @@ import org.junit.runner.RunWith;
|
||||
import org.robolectric.RobolectricTestRunner;
|
||||
import org.robolectric.RuntimeEnvironment;
|
||||
import org.robolectric.annotation.Config;
|
||||
import org.robolectric.annotation.Implementation;
|
||||
import org.robolectric.annotation.Implements;
|
||||
import org.robolectric.annotation.Resetter;
|
||||
|
||||
@RunWith(RobolectricTestRunner.class)
|
||||
public class LowStorageSliceTest {
|
||||
@@ -104,26 +101,4 @@ public class LowStorageSliceTest {
|
||||
|
||||
assertThat(slice.hasHint(HINT_ERROR)).isTrue();
|
||||
}
|
||||
|
||||
@Implements(PrivateStorageInfo.class)
|
||||
public static class ShadowPrivateStorageInfo {
|
||||
|
||||
private static PrivateStorageInfo sPrivateStorageInfo = null;
|
||||
|
||||
@Resetter
|
||||
public static void reset() {
|
||||
sPrivateStorageInfo = null;
|
||||
}
|
||||
|
||||
@Implementation
|
||||
public static PrivateStorageInfo getPrivateStorageInfo(
|
||||
StorageVolumeProvider storageVolumeProvider) {
|
||||
return sPrivateStorageInfo;
|
||||
}
|
||||
|
||||
private static void setPrivateStorageInfo(
|
||||
PrivateStorageInfo privateStorageInfo) {
|
||||
sPrivateStorageInfo = privateStorageInfo;
|
||||
}
|
||||
}
|
||||
}
|
@@ -16,6 +16,8 @@
|
||||
|
||||
package com.android.settings.search.actionbar;
|
||||
|
||||
import static com.android.settings.search.actionbar.SearchMenuController.MENU_SEARCH;
|
||||
|
||||
import static org.mockito.Mockito.mock;
|
||||
import static org.mockito.Mockito.spy;
|
||||
import static org.mockito.Mockito.verify;
|
||||
@@ -69,7 +71,7 @@ public class SearchMenuControllerTest {
|
||||
Global.putInt(mActivity.getContentResolver(), Global.DEVICE_PROVISIONED, 1);
|
||||
|
||||
when(mHost.getActivity()).thenReturn(mActivity);
|
||||
when(mMenu.add(Menu.NONE, Menu.NONE, 0 /* order */, R.string.search_menu))
|
||||
when(mMenu.add(Menu.NONE, MENU_SEARCH, 0 /* order */, R.string.search_menu))
|
||||
.thenReturn(mock(MenuItem.class));
|
||||
}
|
||||
|
||||
@@ -78,7 +80,7 @@ public class SearchMenuControllerTest {
|
||||
SearchMenuController.init(mHost);
|
||||
mHost.getSettingsLifecycle().onCreateOptionsMenu(mMenu, null /* inflater */);
|
||||
|
||||
verify(mMenu).add(Menu.NONE, Menu.NONE, 0 /* order */, R.string.search_menu);
|
||||
verify(mMenu).add(Menu.NONE, MENU_SEARCH, 0 /* order */, R.string.search_menu);
|
||||
}
|
||||
|
||||
@Test
|
||||
|
@@ -0,0 +1,46 @@
|
||||
/*
|
||||
* Copyright (C) 2019 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.testutils.shadow;
|
||||
|
||||
import com.android.settingslib.deviceinfo.PrivateStorageInfo;
|
||||
import com.android.settingslib.deviceinfo.StorageVolumeProvider;
|
||||
|
||||
import org.robolectric.annotation.Implementation;
|
||||
import org.robolectric.annotation.Implements;
|
||||
import org.robolectric.annotation.Resetter;
|
||||
|
||||
@Implements(PrivateStorageInfo.class)
|
||||
public class ShadowPrivateStorageInfo {
|
||||
|
||||
private static PrivateStorageInfo sPrivateStorageInfo = null;
|
||||
|
||||
@Resetter
|
||||
public static void reset() {
|
||||
sPrivateStorageInfo = null;
|
||||
}
|
||||
|
||||
@Implementation
|
||||
protected static PrivateStorageInfo getPrivateStorageInfo(
|
||||
StorageVolumeProvider storageVolumeProvider) {
|
||||
return sPrivateStorageInfo;
|
||||
}
|
||||
|
||||
public static void setPrivateStorageInfo(
|
||||
PrivateStorageInfo privateStorageInfo) {
|
||||
sPrivateStorageInfo = privateStorageInfo;
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user