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

Commit 82792cc5 authored by Kevin Chyn's avatar Kevin Chyn
Browse files

Fingerprint swipe settings and suggestions should not show if hardware unavailable 

Fixes: 63589121

Test: remove FP driver, make sure swipe setting and suggestions
don't show anymore && make -j RunSettingsRoboTests

Change-Id: I2539128daff8df52d040606141f772fc9edffa57
parent 5e48ea19

src/com/android/settings/Utils.java

+5 −0
Original line number Diff line number Diff line
@@ -1222,6 +1222,11 @@ public final class Utils extends com.android.settingslib.Utils { 
        }

    }



    public static boolean hasFingerprintHardware(Context context) {

        FingerprintManager fingerprintManager = getFingerprintManagerOrNull(context);

        return fingerprintManager != null && fingerprintManager.isHardwareDetected();

    }



    /**

     * Launches an intent which may optionally have a user id defined.

     * @param fragment Fragment to use to launch the activity.

src/com/android/settings/dashboard/suggestions/SuggestionsChecks.java

+5 −3
Original line number Diff line number Diff line
@@ -65,12 +65,14 @@ public class SuggestionsChecks { 
        } else if (className.equals(WifiCallingSuggestionActivity.class.getName())) {

            return isWifiCallingUnavailableOrEnabled();

        } else if (className.equals(FingerprintSuggestionActivity.class.getName())) {

            return isNotSingleFingerprintEnrolled() || !isFingerprintEnabled();

            return !Utils.hasFingerprintHardware(mContext) || !isFingerprintEnabled()

                    || isNotSingleFingerprintEnrolled();

        } else if (className.equals(ScreenLockSuggestionActivity.class.getName())) {

            return isDeviceSecured();

        } else if (className.equals(FingerprintEnrollSuggestionActivity.class.getName())) {

            FingerprintManager manager = Utils.getFingerprintManagerOrNull(mContext);

            if (manager == null || !isFingerprintEnabled()) {

            final FingerprintManager manager = Utils.getFingerprintManagerOrNull(mContext);

            if (manager == null || !isFingerprintEnabled()

                    || !Utils.hasFingerprintHardware(mContext)) {

                return true;

            }

            return manager.hasEnrolledFingerprints();

src/com/android/settings/gestures/SwipeToNotificationPreferenceController.java

+2 −2
Original line number Diff line number Diff line
@@ -23,7 +23,7 @@ import android.provider.Settings; 
import android.support.v7.preference.Preference;



import com.android.settings.R;

import com.android.settings.overlay.FeatureFactory;

import com.android.settings.Utils;

import com.android.settings.search.DatabaseIndexingUtils;

import com.android.settings.search.InlineSwitchPayload;

import com.android.settings.search.ResultPayload;
@@ -50,7 +50,7 @@ public class SwipeToNotificationPreferenceController extends GesturePreferenceCo 
    }



    private static boolean isGestureAvailable(Context context) {

        return context.getResources()

        return Utils.hasFingerprintHardware(context) && context.getResources()

                .getBoolean(com.android.internal.R.bool.config_supportSystemNavigationKeys);

    }

tests/robotests/src/com/android/settings/dashboard/suggestions/SuggestionFeatureProviderImplTest.java

+56 −19
Original line number Diff line number Diff line
@@ -16,12 +16,15 @@ 


package com.android.settings.dashboard.suggestions;





import android.app.ActivityManager;



import android.content.ComponentName;

import android.content.Context;

import android.content.Intent;

import android.content.SharedPreferences;

import android.content.pm.PackageManager;

import android.hardware.fingerprint.FingerprintManager;

import android.provider.Settings.Secure;



import com.android.internal.logging.nano.MetricsProto;
@@ -59,6 +62,7 @@ import java.util.List; 
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.Matchers.anyString;

import static org.mockito.Matchers.eq;

import static org.mockito.Mockito.never;
@@ -84,6 +88,12 @@ public class SuggestionFeatureProviderImplTest { 
    private Tile mSuggestion;

    @Mock

    private ActivityManager mActivityManager;

    @Mock

    private PackageManager mPackageManager;

    @Mock

    private FingerprintManager mFingerprintManager;

    @Mock

    private SharedPreferences mSharedPreferences;



    private FakeFeatureFactory mFactory;

    private SuggestionFeatureProviderImpl mProvider;
@@ -93,9 +103,14 @@ public class SuggestionFeatureProviderImplTest { 
        MockitoAnnotations.initMocks(this);

        FakeFeatureFactory.setupForTest(mContext);

        mFactory = (FakeFeatureFactory) FakeFeatureFactory.getFactory(mContext);

        when(mContext.getPackageManager()).thenReturn(mPackageManager);

        // Explicit casting to object due to MockitoCast bug

        when((Object) mContext.getSystemService(FingerprintManager.class))

                .thenReturn(mFingerprintManager);

        when(mContext.getApplicationContext()).thenReturn(RuntimeEnvironment.application);

        when(mContext.getSystemService(Context.ACTIVITY_SERVICE)).thenReturn(mActivityManager);

        when(mActivityManager.isLowRamDevice()).thenReturn(false);



        mSuggestion.intent = new Intent().setClassName("pkg", "cls");

        mSuggestion.category = "category";
@@ -210,39 +225,56 @@ public class SuggestionFeatureProviderImplTest { 
    }



    @Test

    @Config(shadows = SettingsShadowResources.class)

    public void isSuggestionCompleted_swipeToNotification_trueWhenNotHardwareNotAvailable() {

        stubFingerprintSupported(true);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(false);

        when(mContext.getResources().

                getBoolean(com.android.internal.R.bool.config_supportSystemNavigationKeys))

                .thenReturn(true);



        assertThat(mProvider.isSuggestionCompleted(mContext,

                new ComponentName(mContext, SwipeToNotificationSuggestionActivity.class))).isTrue();

    }



    @Test

    public void isSuggestionCompleted_swipeToNotification_trueWhenNotAvailable() {

        SettingsShadowResources.overrideResource(

                com.android.internal.R.bool.config_supportSystemNavigationKeys, false);

        stubFingerprintSupported(true);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        when(mContext.getResources().

                getBoolean(com.android.internal.R.bool.config_supportSystemNavigationKeys))

                .thenReturn(false);



        assertThat(mProvider.isSuggestionCompleted(RuntimeEnvironment.application,

                new ComponentName(RuntimeEnvironment.application,

                        SwipeToNotificationSuggestionActivity.class))).isTrue();

        assertThat(mProvider.isSuggestionCompleted(mContext,

                new ComponentName(mContext, SwipeToNotificationSuggestionActivity.class))).isTrue();

    }



    @Test

    @Config(shadows = SettingsShadowResources.class)

    public void isSuggestionCompleted_swipeToNotification_falseWhenNotVisited() {

        SettingsShadowResources.overrideResource(

                com.android.internal.R.bool.config_supportSystemNavigationKeys, true);

        stubFingerprintSupported(true);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        when(mContext.getResources().

                getBoolean(com.android.internal.R.bool.config_supportSystemNavigationKeys))

                .thenReturn(true);

        // No stored value in shared preferences if not visited yet.



        assertThat(mProvider.isSuggestionCompleted(RuntimeEnvironment.application,

                new ComponentName(RuntimeEnvironment.application,

        assertThat(mProvider.isSuggestionCompleted(mContext,

                new ComponentName(mContext,

                        SwipeToNotificationSuggestionActivity.class))).isFalse();

    }



    @Test

    @Config(shadows = SettingsShadowResources.class)

    public void isSuggestionCompleted_swipeToNotification_trueWhenVisited() {

        SettingsShadowResources.overrideResource(

                com.android.internal.R.bool.config_supportSystemNavigationKeys, true);

        mProvider.getSharedPrefs(RuntimeEnvironment.application).edit().putBoolean(

                SwipeToNotificationSettings.PREF_KEY_SUGGESTION_COMPLETE, true).commit();

        stubFingerprintSupported(true);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        when(mContext.getResources().

                getBoolean(com.android.internal.R.bool.config_supportSystemNavigationKeys))

                .thenReturn(true);

        when(mContext.getSharedPreferences(anyString(), anyInt())).thenReturn(mSharedPreferences);

        when(mSharedPreferences.getBoolean(

                SwipeToNotificationSettings.PREF_KEY_SUGGESTION_COMPLETE, false)).thenReturn(true);



        assertThat(mProvider.isSuggestionCompleted(RuntimeEnvironment.application,

                new ComponentName(RuntimeEnvironment.application,

                        SwipeToNotificationSuggestionActivity.class))).isTrue();

        assertThat(mProvider.isSuggestionCompleted(mContext,

                new ComponentName(mContext, SwipeToNotificationSuggestionActivity.class))).isTrue();

    }



    @Test
@@ -360,6 +392,11 @@ public class SuggestionFeatureProviderImplTest { 
                        PackageManager.DONT_KILL_APP);

    }



    private void stubFingerprintSupported(boolean enabled) {

        when(mPackageManager.hasSystemFeature(PackageManager.FEATURE_FINGERPRINT))

                .thenReturn(enabled);

    }



    @Test

    public void filterExclusiveSuggestions_shouldOnlyKeepFirst3() {

        final List<Tile> suggestions = new ArrayList<>();

tests/robotests/src/com/android/settings/dashboard/suggestions/SuggestionsChecksTest.java

+12 −0
Original line number Diff line number Diff line
@@ -78,6 +78,7 @@ public class SuggestionsChecksTest { 
    public void testFingerprintEnrollmentIntroductionIsCompleteWhenFingerprintAdded() {

        stubFingerprintSupported(true);

        when(mFingerprintManager.hasEnrolledFingerprints()).thenReturn(true);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        Tile tile = createFingerprintTile();

        assertThat(mSuggestionsChecks.isSuggestionComplete(tile)).isTrue();

    }
@@ -86,10 +87,20 @@ public class SuggestionsChecksTest { 
    public void testFingerprintEnrollmentIntroductionIsNotCompleteWhenNoFingerprintAdded() {

        stubFingerprintSupported(true);

        when(mFingerprintManager.hasEnrolledFingerprints()).thenReturn(false);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        Tile tile = createFingerprintTile();

        assertThat(mSuggestionsChecks.isSuggestionComplete(tile)).isFalse();

    }



    @Test

    public void testFingerprintEnrollmentIntroductionIsCompleteWhenHardwareNotDetected() {

        stubFingerprintSupported(true);

        when(mFingerprintManager.hasEnrolledFingerprints()).thenReturn(false);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(false);

        Tile tile = createFingerprintTile();

        assertThat(mSuggestionsChecks.isSuggestionComplete(tile)).isTrue();

    }



    @Test

    public void testFingerprintEnrollmentIntroductionIsCompleteWhenFingerprintNotSupported() {

        stubFingerprintSupported(false);
@@ -101,6 +112,7 @@ public class SuggestionsChecksTest { 
    public void testFingerprintEnrollmentIntroductionIsCompleteWhenFingerprintDisabled() {

        stubFingerprintSupported(true);

        when(mFingerprintManager.hasEnrolledFingerprints()).thenReturn(false);

        when(mFingerprintManager.isHardwareDetected()).thenReturn(true);

        when(mDevicePolicyManager.getKeyguardDisabledFeatures(any(), anyInt()))

                .thenReturn(DevicePolicyManager.KEYGUARD_DISABLE_FINGERPRINT);