Merge "Prevent apps from displaying over Settings" into oc-dev am: 24596f51d5
am: 8606c652d5
Change-Id: Idb0cbf6b0467f9fd43b1e9e40e622146640a9a7e
This commit is contained in:
@@ -19,10 +19,11 @@ package com.android.settings.applications;
|
||||
import android.content.Context;
|
||||
|
||||
import com.android.internal.logging.nano.MetricsProto;
|
||||
import com.android.internal.telephony.SmsUsageMonitor;
|
||||
import com.android.settings.SettingsRobolectricTestRunner;
|
||||
import com.android.settings.TestConfig;
|
||||
import com.android.settings.core.TouchOverlayManager;
|
||||
import com.android.settings.testutils.FakeFeatureFactory;
|
||||
import com.android.settings.testutils.shadow.ShadowPreferenceFragment;
|
||||
|
||||
import org.junit.Before;
|
||||
import org.junit.Test;
|
||||
@@ -32,6 +33,7 @@ import org.mockito.Mock;
|
||||
import org.mockito.MockitoAnnotations;
|
||||
import org.robolectric.annotation.Config;
|
||||
import org.robolectric.shadows.ShadowApplication;
|
||||
import org.robolectric.util.ReflectionHelpers;
|
||||
|
||||
import static org.mockito.Matchers.any;
|
||||
import static org.mockito.Matchers.eq;
|
||||
@@ -47,17 +49,22 @@ public class DrawOverlayDetailsTest {
|
||||
private FakeFeatureFactory mFeatureFactory;
|
||||
private DrawOverlayDetails mFragment;
|
||||
|
||||
@Mock
|
||||
private TouchOverlayManager mTouchOverlayManager;
|
||||
|
||||
@Before
|
||||
public void setUp() {
|
||||
MockitoAnnotations.initMocks(this);
|
||||
FakeFeatureFactory.setupForTest(mContext);
|
||||
mFeatureFactory = (FakeFeatureFactory) FakeFeatureFactory.getFactory(mContext);
|
||||
|
||||
mFragment = new DrawOverlayDetails();
|
||||
mFragment.onAttach(ShadowApplication.getInstance().getApplicationContext());
|
||||
ReflectionHelpers.setField(mFragment, "mTouchOverlayManager", mTouchOverlayManager);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void logSpecialPermissionChange() {
|
||||
mFragment.onAttach(ShadowApplication.getInstance().getApplicationContext());
|
||||
mFragment.logSpecialPermissionChange(true, "app");
|
||||
verify(mFeatureFactory.metricsFeatureProvider).action(any(Context.class),
|
||||
eq(MetricsProto.MetricsEvent.APP_SPECIAL_PERMISSION_APPDRAW_ALLOW), eq("app"));
|
||||
@@ -66,4 +73,18 @@ public class DrawOverlayDetailsTest {
|
||||
verify(mFeatureFactory.metricsFeatureProvider).action(any(Context.class),
|
||||
eq(MetricsProto.MetricsEvent.APP_SPECIAL_PERMISSION_APPDRAW_DENY), eq("app"));
|
||||
}
|
||||
|
||||
@Test
|
||||
@Config(shadows = ShadowPreferenceFragment.class)
|
||||
public void onStart_disableOverlay() {
|
||||
mFragment.onStart();
|
||||
verify(mTouchOverlayManager).setOverlayAllowed(false);
|
||||
}
|
||||
|
||||
@Test
|
||||
@Config(shadows = ShadowPreferenceFragment.class)
|
||||
public void onStop_enableOverlay() {
|
||||
mFragment.onStop();
|
||||
verify(mTouchOverlayManager).setOverlayAllowed(true);
|
||||
}
|
||||
}
|
||||
|
@@ -0,0 +1,35 @@
|
||||
/*
|
||||
* 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.testutils.shadow;
|
||||
|
||||
import android.support.v14.preference.PreferenceFragment;
|
||||
import org.robolectric.annotation.Implementation;
|
||||
import org.robolectric.annotation.Implements;
|
||||
|
||||
@Implements(PreferenceFragment.class)
|
||||
public class ShadowPreferenceFragment {
|
||||
|
||||
@Implementation
|
||||
public void onStart() {
|
||||
// No-op.
|
||||
}
|
||||
|
||||
@Implementation
|
||||
public void onStop() {
|
||||
// No-op.
|
||||
}
|
||||
}
|
Reference in New Issue
Block a user