PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Check Wi-Fi permissions for QrCode scanner" into tm-dev am: b6629ab55b

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/17944025

Change-Id: I0bc8a5110999c1144b6441c158f20b57abba7ed4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Weng Su
2022-04-27 10:26:34 +00:00
committed by Automerger Merge Worker
parent 6a58c2faf1 b6629ab55b
commit 6428d536c6
2 changed files with 97 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

64
tests/robotests/src/com/android/settings/wifi/dpp/WifiDppQrCodeScannerFragmentTest.java
View File

@@ -19,11 +19,19 @@ package com.android.settings.wifi.dpp;
import static com.google.common.truth.Truth.assertThat;
import static org.mockito.ArgumentMatchers.anyInt;
import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.Mockito.any;
import static org.mockito.Mockito.doNothing;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
import android.app.Activity;
import androidx.fragment.app.FragmentActivity;
import com.android.settingslib.wifi.WifiPermissionChecker;
import com.android.wifitrackerlib.WifiEntry;
import com.android.wifitrackerlib.WifiPickerTracker;
@@ -49,6 +57,10 @@ public class WifiDppQrCodeScannerFragmentTest {
WifiPickerTracker mWifiPickerTracker;
@Mock
WifiEntry mWifiEntry;
@Mock
WifiPermissionChecker mWifiPermissionChecker;
@Mock
FragmentActivity mActivity;
WifiDppQrCodeScannerFragment mFragment;
@@ -57,8 +69,8 @@ public class WifiDppQrCodeScannerFragmentTest {
when(mWifiEntry.getSsid()).thenReturn(WIFI_SSID);
when(mWifiPickerTracker.getWifiEntries()).thenReturn(Arrays.asList(mWifiEntry));
mFragment = spy(new WifiDppQrCodeScannerFragment());
mFragment.mWifiPickerTracker = mWifiPickerTracker;
mFragment = spy(
new WifiDppQrCodeScannerFragment(mWifiPickerTracker, mWifiPermissionChecker));
}
@Test
@@ -84,4 +96,52 @@ public class WifiDppQrCodeScannerFragmentTest {
assertThat(mFragment.canConnectWifi(WIFI_SSID)).isFalse();
verify(mFragment).showErrorMessageAndRestartCamera(anyInt());
}
@Test
public void onSuccess_noWifiPermission_finishActivityWithoutSetResult() {
when(mFragment.getActivity()).thenReturn(mActivity);
when(mWifiPermissionChecker.canAccessWifiState()).thenReturn(false);
when(mWifiPermissionChecker.canAccessFineLocation()).thenReturn(false);
mFragment.onSuccess();
verify(mActivity).finish();
verify(mActivity, never()).setResult(eq(Activity.RESULT_OK), any());
}
@Test
public void onSuccess_hasAccessWifiStatePermissionOnly_finishActivityWithoutSetResult() {
when(mFragment.getActivity()).thenReturn(mActivity);
when(mWifiPermissionChecker.canAccessWifiState()).thenReturn(true);
when(mWifiPermissionChecker.canAccessFineLocation()).thenReturn(false);
mFragment.onSuccess();
verify(mActivity).finish();
verify(mActivity, never()).setResult(eq(Activity.RESULT_OK), any());
}
@Test
public void onSuccess_hasAccessFineLocationPermissionOnly_finishActivityWithoutSetResult() {
when(mFragment.getActivity()).thenReturn(mActivity);
when(mWifiPermissionChecker.canAccessWifiState()).thenReturn(false);
when(mWifiPermissionChecker.canAccessFineLocation()).thenReturn(true);
mFragment.onSuccess();
verify(mActivity).finish();
verify(mActivity, never()).setResult(eq(Activity.RESULT_OK), any());
}
@Test
public void onSuccess_hasRequiredPermissions_finishActivityWithSetResult() {
when(mFragment.getActivity()).thenReturn(mActivity);
when(mWifiPermissionChecker.canAccessWifiState()).thenReturn(true);
when(mWifiPermissionChecker.canAccessFineLocation()).thenReturn(true);
mFragment.onSuccess();
verify(mActivity).setResult(eq(Activity.RESULT_OK), any());
verify(mActivity).finish();
}
}