Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dc98170d authored by Android Build Coastguard Worker's avatar Android Build Coastguard Worker
Browse files

Snap for 9227242 from f5a97660 to tm-qpr2-release 

Change-Id: I14be7c0f7fffad7913e32ef6529a0a258859ff49
parents 8214b055 f5a97660

src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceController.java

+3 −1
Original line number Diff line number Diff line
@@ -108,6 +108,7 @@ public class BatteryChartPreferenceController extends AbstractPreferenceControll 


    private boolean mIs24HourFormat;

    private boolean mIsFooterPrefAdded = false;

    private boolean mHourlyChartVisible = true;

    private View mBatteryChartViewGroup;

    private View mCategoryTitleView;

    private PreferenceScreen mPreferenceScreen;
@@ -690,9 +691,10 @@ public class BatteryChartPreferenceController extends AbstractPreferenceControll 
    }



    private void animateBatteryHourlyChartView(final boolean visible) {

        if (mHourlyChartView == null) {

        if (mHourlyChartView == null || mHourlyChartVisible == visible) {

            return;

        }

        mHourlyChartVisible = visible;



        if (visible) {

            mHourlyChartView.setVisibility(View.VISIBLE);

src/com/android/settings/network/NetworkProviderSettings.java

+16 −4
Original line number Diff line number Diff line
@@ -34,6 +34,7 @@ import android.net.wifi.WifiManager; 
import android.os.Bundle;

import android.os.Handler;

import android.os.PowerManager;

import android.os.UserHandle;

import android.os.UserManager;

import android.provider.Settings;

import android.telephony.TelephonyManager;
@@ -598,10 +599,7 @@ public class NetworkProviderSettings extends RestrictedSettingsFragment 
            return;

        }



        if (mSelectedWifiEntry.isSaved() && mSelectedWifiEntry.getConnectedState()

                != WifiEntry.CONNECTED_STATE_CONNECTED) {

            menu.add(Menu.NONE, MENU_ID_MODIFY, 0 /* order */, R.string.wifi_modify);

        }

        addModifyMenuIfSuitable(menu, mSelectedWifiEntry);

    }



    @VisibleForTesting
@@ -621,6 +619,14 @@ public class NetworkProviderSettings extends RestrictedSettingsFragment 
        }

    }



    @VisibleForTesting

    void addModifyMenuIfSuitable(ContextMenu menu, WifiEntry wifiEntry) {

        if (mIsAdmin && wifiEntry.isSaved()

                && wifiEntry.getConnectedState() != WifiEntry.CONNECTED_STATE_CONNECTED) {

            menu.add(Menu.NONE, MENU_ID_MODIFY, 0 /* order */, R.string.wifi_modify);

        }

    }



    private boolean canForgetNetwork() {

        return mSelectedWifiEntry.canForget() && !WifiUtils.isNetworkLockedDown(getActivity(),

                mSelectedWifiEntry.getWifiConfiguration());
@@ -643,6 +649,12 @@ public class NetworkProviderSettings extends RestrictedSettingsFragment 
                        () -> launchWifiDppConfiguratorActivity(mSelectedWifiEntry));

                return true;

            case MENU_ID_MODIFY:

                if (!mIsAdmin) {

                    Log.e(TAG, "Can't modify Wi-Fi because the user isn't admin.");

                    EventLog.writeEvent(0x534e4554, "237672190", UserHandle.myUserId(),

                            "User isn't admin");

                    return true;

                }

                showDialog(mSelectedWifiEntry, WifiConfigUiBase2.MODE_MODIFY);

                return true;

            default:

tests/robotests/src/com/android/settings/fuelgauge/batteryusage/BatteryChartPreferenceControllerTest.java

+27 −4
Original line number Diff line number Diff line
@@ -18,6 +18,8 @@ package com.android.settings.fuelgauge.batteryusage; 


import static com.google.common.truth.Truth.assertThat;



import static org.mockito.ArgumentMatchers.anyFloat;

import static org.mockito.ArgumentMatchers.anyLong;

import static org.mockito.Mockito.any;

import static org.mockito.Mockito.atLeast;

import static org.mockito.Mockito.atLeastOnce;
@@ -36,6 +38,7 @@ import android.os.Bundle; 
import android.os.LocaleList;

import android.text.format.DateUtils;

import android.view.View;

import android.view.ViewPropertyAnimator;

import android.widget.LinearLayout;



import androidx.preference.Preference;
@@ -85,6 +88,8 @@ public final class BatteryChartPreferenceControllerTest { 
    @Mock

    private BatteryChartView mHourlyChartView;

    @Mock

    private ViewPropertyAnimator mViewPropertyAnimator;

    @Mock

    private PowerGaugePreference mPowerGaugePreference;

    @Mock

    private BatteryUtils mBatteryUtils;
@@ -116,6 +121,7 @@ public final class BatteryChartPreferenceControllerTest { 
                .when(mFeatureFactory.powerUsageFeatureProvider)

                .getHideApplicationEntries(mContext);

        doReturn(mLayoutParams).when(mDailyChartView).getLayoutParams();

        setupHourlyChartViewAnimationMock();

        mBatteryChartPreferenceController = createController();

        mBatteryChartPreferenceController.mPrefContext = mContext;

        mBatteryChartPreferenceController.mAppListPrefGroup = mAppListGroup;
@@ -174,11 +180,13 @@ public final class BatteryChartPreferenceControllerTest { 


    @Test

    public void setBatteryChartViewModel_6Hours() {

        reset(mDailyChartView);

        reset(mHourlyChartView);

        setupHourlyChartViewAnimationMock();



        mBatteryChartPreferenceController.setBatteryHistoryMap(createBatteryHistoryMap(6));



        verify(mDailyChartView, atLeastOnce()).setVisibility(View.GONE);

        verify(mHourlyChartView, atLeastOnce()).setVisibility(View.VISIBLE);

        // Ignore fast refresh ui from the data processor callback.

        verify(mHourlyChartView, atLeast(0)).setViewModel(null);

        verify(mHourlyChartView, atLeastOnce()).setViewModel(new BatteryChartViewModel(
@@ -192,6 +200,10 @@ public final class BatteryChartPreferenceControllerTest { 


    @Test

    public void setBatteryChartViewModel_60Hours() {

        reset(mDailyChartView);

        reset(mHourlyChartView);

        setupHourlyChartViewAnimationMock();



        BatteryChartViewModel expectedDailyViewModel = new BatteryChartViewModel(

                List.of(100, 83, 59, 41),

                // "Sat", "Sun", "Mon", "Mon"
@@ -205,16 +217,18 @@ public final class BatteryChartPreferenceControllerTest { 
        mBatteryChartPreferenceController.setBatteryHistoryMap(createBatteryHistoryMap(60));



        verify(mDailyChartView, atLeastOnce()).setVisibility(View.VISIBLE);

        verify(mHourlyChartView, atLeastOnce()).setVisibility(View.GONE);

        verify(mViewPropertyAnimator, atLeastOnce()).alpha(0f);

        verify(mDailyChartView).setViewModel(expectedDailyViewModel);



        reset(mDailyChartView);

        reset(mHourlyChartView);

        setupHourlyChartViewAnimationMock();

        doReturn(mLayoutParams).when(mDailyChartView).getLayoutParams();

        mBatteryChartPreferenceController.mDailyChartIndex = 0;

        mBatteryChartPreferenceController.refreshUi();

        verify(mDailyChartView).setVisibility(View.VISIBLE);

        verify(mHourlyChartView).setVisibility(View.VISIBLE);

        verify(mViewPropertyAnimator, atLeastOnce()).alpha(1f);



        expectedDailyViewModel.setSelectedIndex(0);

        verify(mDailyChartView).setViewModel(expectedDailyViewModel);
@@ -234,12 +248,13 @@ public final class BatteryChartPreferenceControllerTest { 


        reset(mDailyChartView);

        reset(mHourlyChartView);

        setupHourlyChartViewAnimationMock();

        doReturn(mLayoutParams).when(mDailyChartView).getLayoutParams();

        mBatteryChartPreferenceController.mDailyChartIndex = 1;

        mBatteryChartPreferenceController.mHourlyChartIndex = 6;

        mBatteryChartPreferenceController.refreshUi();

        verify(mDailyChartView).setVisibility(View.VISIBLE);

        verify(mHourlyChartView).setVisibility(View.VISIBLE);

        verify(mViewPropertyAnimator, atLeastOnce()).alpha(1f);

        expectedDailyViewModel.setSelectedIndex(1);

        verify(mDailyChartView).setViewModel(expectedDailyViewModel);

        BatteryChartViewModel expectedHourlyViewModel = new BatteryChartViewModel(
@@ -264,13 +279,14 @@ public final class BatteryChartPreferenceControllerTest { 


        reset(mDailyChartView);

        reset(mHourlyChartView);

        setupHourlyChartViewAnimationMock();

        doReturn(mLayoutParams).when(mDailyChartView).getLayoutParams();

        mBatteryChartPreferenceController.mDailyChartIndex = 2;

        mBatteryChartPreferenceController.mHourlyChartIndex =

                BatteryChartViewModel.SELECTED_INDEX_ALL;

        mBatteryChartPreferenceController.refreshUi();

        verify(mDailyChartView).setVisibility(View.VISIBLE);

        verify(mHourlyChartView).setVisibility(View.VISIBLE);

        verify(mViewPropertyAnimator, atLeastOnce()).alpha(1f);

        expectedDailyViewModel.setSelectedIndex(2);

        verify(mDailyChartView).setViewModel(expectedDailyViewModel);

        verify(mHourlyChartView).setViewModel(new BatteryChartViewModel(
@@ -734,4 +750,11 @@ public final class BatteryChartPreferenceControllerTest { 
        controller.mPrefContext = mContext;

        return controller;

    }



    private void setupHourlyChartViewAnimationMock() {

        doReturn(mViewPropertyAnimator).when(mHourlyChartView).animate();

        doReturn(mViewPropertyAnimator).when(mViewPropertyAnimator).alpha(anyFloat());

        doReturn(mViewPropertyAnimator).when(mViewPropertyAnimator).setDuration(anyLong());

        doReturn(mViewPropertyAnimator).when(mViewPropertyAnimator).setListener(any());

    }

}

tests/robotests/src/com/android/settings/fuelgauge/batteryusage/DataProcessorTest.java

+30 −10
Original line number Diff line number Diff line
@@ -579,9 +579,13 @@ public class DataProcessorTest { 
                ConvertUtils.CONSUMER_TYPE_SYSTEM_BATTERY, /*consumePercentage=*/ 25.0,

                /*foregroundUsageTimeInMs=*/ 50, /*backgroundUsageTimeInMs=*/ 60);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT, 3);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT,

                        3);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT, 0);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT,

                        0);

    }



    @Test
@@ -674,9 +678,13 @@ public class DataProcessorTest { 
        assertThat(resultMap.get(0).get(0)).isNotNull();

        assertThat(resultMap.get(0).get(DataProcessor.SELECTED_INDEX_ALL)).isNotNull();

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT, 2);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT,

                        2);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT, 0);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT,

                        0);

    }



    @Test
@@ -739,9 +747,13 @@ public class DataProcessorTest { 
        assertThat(resultMap.get(0).get(0)).isNotNull();

        assertThat(resultMap.get(0).get(DataProcessor.SELECTED_INDEX_ALL)).isNotNull();

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT, 1);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT,

                        1);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT, 0);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT,

                        0);

    }



    @Test
@@ -814,9 +826,13 @@ public class DataProcessorTest { 
                ConvertUtils.CONSUMER_TYPE_UID_BATTERY, /*consumePercentage=*/ 50.0,

                /*foregroundUsageTimeInMs=*/ 10, /*backgroundUsageTimeInMs=*/ 20);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT, 1);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT,

                        1);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT, 1);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT,

                        1);

    }



    @Test
@@ -889,9 +905,13 @@ public class DataProcessorTest { 
        resultEntry = resultDiffData.getAppDiffEntryList().get(1);

        assertThat(resultEntry.mBackgroundUsageTimeInMs).isEqualTo(0);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT, 2);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_SHOWN_APP_COUNT,

                        2);

        verify(mMetricsFeatureProvider)

                .action(mContext, SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT, 0);

                .action(mContext.getApplicationContext(),

                        SettingsEnums.ACTION_BATTERY_USAGE_HIDDEN_APP_COUNT,

                        0);

    }



    @Test

tests/robotests/src/com/android/settings/network/NetworkProviderSettingsTest.java

+24 −0
Original line number Diff line number Diff line
@@ -17,9 +17,11 @@ package com.android.settings.network; 


import static com.android.settings.network.NetworkProviderSettings.MENU_ID_DISCONNECT;

import static com.android.settings.network.NetworkProviderSettings.MENU_ID_FORGET;

import static com.android.settings.network.NetworkProviderSettings.MENU_ID_MODIFY;

import static com.android.settings.network.NetworkProviderSettings.MENU_ID_SHARE;

import static com.android.settings.wifi.WifiConfigUiBase2.MODE_CONNECT;

import static com.android.settings.wifi.WifiConfigUiBase2.MODE_MODIFY;

import static com.android.wifitrackerlib.WifiEntry.CONNECTED_STATE_DISCONNECTED;



import static com.google.common.truth.Truth.assertThat;
@@ -729,6 +731,28 @@ public class NetworkProviderSettingsTest { 
        verify(mContextMenu, never()).add(anyInt(), eq(MENU_ID_FORGET), anyInt(), anyInt());

    }



    @Test

    public void addModifyMenuIfSuitable_isAdmin_addMenu() {

        mNetworkProviderSettings.mIsAdmin = true;

        when(mWifiEntry.isSaved()).thenReturn(true);

        when(mWifiEntry.getConnectedState()).thenReturn(CONNECTED_STATE_DISCONNECTED);



        mNetworkProviderSettings.addModifyMenuIfSuitable(mContextMenu, mWifiEntry);



        verify(mContextMenu).add(anyInt(), eq(MENU_ID_MODIFY), anyInt(), anyInt());

    }



    @Test

    public void addModifyMenuIfSuitable_isNotAdmin_notAddMenu() {

        mNetworkProviderSettings.mIsAdmin = false;

        when(mWifiEntry.isSaved()).thenReturn(true);

        when(mWifiEntry.getConnectedState()).thenReturn(CONNECTED_STATE_DISCONNECTED);



        mNetworkProviderSettings.addModifyMenuIfSuitable(mContextMenu, mWifiEntry);



        verify(mContextMenu, never()).add(anyInt(), eq(MENU_ID_MODIFY), anyInt(), anyInt());

    }



    @Test

    public void getNonIndexableKeys_allowedChangeWifiState_keyNotReturned() {

        when(mWifiRestriction.isChangeWifiStateAllowed(mContext)).thenReturn(true);