[RNMobile] Update setup media picker helper (#50398)

* Update setup media picker helper to handle single and multiple items * Include `metadata` param when notifying succeed state in `setupMediaUpload` helper * [RNMobile] Add `testID` prop to SandBox component (#50399)
WordPress · May 10, 2023 · e01f41f · e01f41f · github-actions · May 10, 2023
1 parent 2188ed6
commit e01f41f
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 10 deletions.
diff --git a/packages/components/src/sandbox/index.native.js b/packages/components/src/sandbox/index.native.js
@@ -187,6 +187,7 @@ const Sandbox = forwardRef( function Sandbox(
 		onWindowEvents = {},
 		viewportProps = '',
 		onLoadEnd = () => {},
+		testID,
 	},
 	ref
 ) {
@@ -374,6 +375,7 @@ const Sandbox = forwardRef( function Sandbox(
 			showsVerticalScrollIndicator={ false }
 			mediaPlaybackRequiresUserAction={ false }
 			onLoadEnd={ onLoadEnd }
+			testID={ testID }
 		/>
 	);
 } );

diff --git a/test/native/integration-test-helpers/setup-media-picker.js b/test/native/integration-test-helpers/setup-media-picker.js
@@ -20,9 +20,11 @@ import { requestMediaPicker } from '@wordpress/react-native-bridge';
  */
 export const setupMediaPicker = () => {
 	let mediaPickerCallback;
+	let multipleItems;
 	requestMediaPicker.mockImplementation(
 		( source, filter, multiple, callback ) => {
 			mediaPickerCallback = callback;
+			multipleItems = multiple;
 		}
 	);
 	return {
@@ -34,16 +36,25 @@ export const setupMediaPicker = () => {
 				mediaPickerCallback
 			),
 		mediaPickerCallback: async ( ...mediaItems ) =>
-			act( async () =>
+			act( async () => {
+				const items = mediaItems.map(
+					( {
+						localId,
+						localUrl,
+						type = 'image',
+						url,
+						id,
+						metadata,
+					} ) => ( {
+						type,
+						url: url ?? localUrl,
+						id: id ?? localId,
+						metadata,
+					} )
+				);
 				mediaPickerCallback(
-					mediaItems.map(
-						( { localId, localUrl, type = 'image' } ) => ( {
-							type,
-							url: localUrl,
-							id: localId,
-						} )
-					)
-				)
-			),
+					items.length === 1 && ! multipleItems ? items[ 0 ] : items
+				);
+			} ),
 	};
 };
diff --git a/test/native/integration-test-helpers/setup-media-upload.js b/test/native/integration-test-helpers/setup-media-upload.js
@@ -50,6 +50,7 @@ export const setupMediaUpload = () => {
 					mediaId: mediaItem.localId,
 					mediaUrl: mediaItem.serverUrl,
 					mediaServerId: mediaItem.serverId,
+					metadata: mediaItem.metadata,
 				} );
 			} ),
 		notifyFailedState: async ( mediaItem ) =>