Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
1a5afbf
I've added the `pickMultiVideo` feature to `image_picker`.
google-labs-jules[bot] Aug 6, 2025
3ac98cd
Minor cleanup (analysis, autoformat, stale comments, changelog/versio…
stuartmorgan-g Aug 7, 2025
528a6e7
Rework examples for testing and to reduce drift
stuartmorgan-g Aug 7, 2025
f2c219c
Android: Fix API version issue and add native tests
stuartmorgan-g Aug 7, 2025
47086bb
iOS: Fix, and unify some older code flows
stuartmorgan-g Aug 8, 2025
acfbfd1
Simplify iOS Dart code
stuartmorgan-g Aug 8, 2025
5dd0563
Merge branch 'main' into add-pick-multi-video
stuartmorgan-g Aug 8, 2025
100beef
Fix hero tags
stuartmorgan-g Aug 8, 2025
25f9458
Fix unguarded .first
stuartmorgan-g Aug 8, 2025
0c60fd7
Update packages/image_picker/image_picker_android/android/src/main/ja…
stuartmorgan-g Aug 8, 2025
c3040d2
Fix unit test expectation handling
stuartmorgan-g Aug 10, 2025
b5d0098
Revert part of app-facing example change
stuartmorgan-g Aug 10, 2025
21a2c50
Analysis fix
stuartmorgan-g Aug 11, 2025
9ebcff3
Fix UIImagePicker code path
stuartmorgan-g Aug 11, 2025
3c7e229
Merge branch 'main' into add-pick-multi-video-platform-interface
stuartmorgan-g Aug 14, 2025
8306374
Revert non-p-i changes
stuartmorgan-g Aug 14, 2025
c711fc3
Revert method channel changes
stuartmorgan-g Aug 14, 2025
86e96d1
Fix method name in changelog
stuartmorgan-g Aug 14, 2025
f7c6749
Add unit test
stuartmorgan-g Aug 14, 2025
59c91a9
GCAfGH review fixes
stuartmorgan-g Aug 14, 2025
886a7c4
Remove blank line
stuartmorgan-g Aug 14, 2025
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
Prev Previous commit
Next Next commit
Android: Fix API version issue and add native tests
  • Loading branch information
stuartmorgan-g committed Aug 8, 2025
commit f2c219cc9d5860e6e79658bb08c66bad3c88e9d5
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@
import java.util.UUID;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.stream.Collectors;

/**
* A delegate class doing the heavy lifting for the plugin.
Expand Down Expand Up @@ -743,8 +742,7 @@ private void handleChooseMultiVideoResult(int resultCode, Intent intent) {
return;
}

finishWithListSuccess(
paths.stream().map(p -> p.getPath()).collect(Collectors.toCollection(ArrayList::new)));
handleMediaResult(paths);
return;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -299,6 +299,41 @@ public void pickVideos_whenSourceIsCamera_invokesTakeImageWithCamera_FrontCamera
verify(mockImagePickerDelegate).setCameraDevice(eq(ImagePickerDelegate.CameraDevice.FRONT));
}

@Test
public void pickVideos_invokesChooseMultiVideoFromGallery() {
plugin.pickVideos(
SOURCE_GALLERY,
DEFAULT_VIDEO_OPTIONS,
GENERAL_OPTIONS_ALLOW_MULTIPLE_DONT_USE_PHOTO_PICKER,
mockResult);
verify(mockImagePickerDelegate)
.chooseMultiVideoFromGallery(any(), eq(false), eq(Integer.MAX_VALUE), any());
verifyNoInteractions(mockResult);
}

@Test
public void pickVideos_usingPhotoPicker_invokesChooseMultiVideoFromGallery() {
plugin.pickVideos(
SOURCE_GALLERY,
DEFAULT_VIDEO_OPTIONS,
GENERAL_OPTIONS_ALLOW_MULTIPLE_USE_PHOTO_PICKER,
mockResult);
verify(mockImagePickerDelegate)
.chooseMultiVideoFromGallery(any(), eq(true), eq(Integer.MAX_VALUE), any());
verifyNoInteractions(mockResult);
}

@Test
public void pickVideos_withLimit5_invokesChooseMultiVideoFromGallery() {
plugin.pickVideos(
SOURCE_GALLERY,
DEFAULT_VIDEO_OPTIONS,
GENERAL_OPTIONS_ALLOW_MULTIPLE_DONT_USE_PHOTO_PICKER_WITH_LIMIT,
mockResult);
verify(mockImagePickerDelegate).chooseMultiVideoFromGallery(any(), eq(false), eq(5), any());
verifyNoInteractions(mockResult);
}

@Test
public void onConstructor_whenContextTypeIsActivity_shouldNotCrash() {
new ImagePickerPlugin(mockImagePickerDelegate, mockActivity);
Expand Down