Skip to content

[image_picker_android] Removes unneeded permissions from tests #3433

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
auto-submit merged 7 commits into from
Mar 13, 2023
Merged

[image_picker_android] Removes unneeded permissions from tests #3433

Changes from 4 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
405cf97
remove unneeded permissions from tests
tarrinneal Mar 10, 2023
2cfc2ce
update test names to reflect changes
tarrinneal Mar 10, 2023
c7dca4e
Merge branch 'main'
tarrinneal Mar 13, 2023
1ca4b99
merge fix
tarrinneal Mar 13, 2023
2eb4929
unfix the fix fix
tarrinneal Mar 13, 2023
5ddc988
another oopsy
tarrinneal Mar 13, 2023
a4abad6
found a couple more
tarrinneal Mar 13, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 6 additions & 12 deletions ...android/android/src/test/java/io/flutter/plugins/imagepicker/ImagePickerDelegateTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -148,8 +148,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(sdk = 30)
public void
chooseImageFromGallery_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseImageFromGallery_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand All @@ -163,8 +162,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(minSdk = 33)
public void
chooseImageFromGallery_WithPhotoPicker_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseImageFromGallery_WithPhotoPicker_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand All @@ -178,8 +176,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(sdk = 30)
public void
chooseMultiImageFromGallery_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseMultiImageFromGallery_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand All @@ -195,8 +192,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(minSdk = 33)
public void
chooseMultiImageFromGallery_WithPhotoPicker_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseMultiImageFromGallery_WithPhotoPicker_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand All @@ -212,8 +208,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(sdk = 30)
public void
chooseVideoFromGallery_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseVideoFromGallery_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand All @@ -227,8 +222,7 @@ public void chooseMultiImageFromGallery_whenPendingResultExists_finishesWithAlre

@Test
@Config(minSdk = 33)
public void
chooseVideoFromGallery_WithPhotoPicker_whenHasExternalStoragePermission_launchesChooseFromGalleryIntent() {
public void chooseVideoFromGallery_WithPhotoPicker_launchesChooseFromGalleryIntent() {
when(mockPermissionManager.isPermissionGranted(Manifest.permission.READ_EXTERNAL_STORAGE))
.thenReturn(true);

Expand Down