Update prop types

- controls TheWidlarzGroup/react-native-video#1414 - PictureInPicture TheWidlarzGroup/react-native-video#1325 - minLoadRetryCount TheWidlarzGroup/react-native-video@420d88d
retyui · May 1, 2019 · 5008608 · 5008608
1 parent a9aece8
commit 5008608
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 3 deletions.
diff --git a/definitions/npm/react-native-video_v4.x.x/flow_v0.77.0-/react-native-video_v4.x.x.js b/definitions/npm/react-native-video_v4.x.x/flow_v0.77.0-/react-native-video_v4.x.x.js
@@ -173,6 +173,7 @@ declare module 'react-native-video' {
   declare export type VideoProps = $ReadOnly<{
     source: VideoSource, // Required prop
 
+    controls?: boolean,
     audioOnly?: boolean,
     id?: string, // for Web platform
     maxBitRate?: number,
@@ -192,16 +193,21 @@ declare module 'react-native-video' {
 
     // IOS only
     allowsExternalPlayback?: boolean,
-    controls?: boolean, // and Web platform
     filter?: $Values<FilterTypes>,
     filterEnabled?: boolean,
     fullscreen?: boolean,
     fullscreenAutorotate?: boolean,
     fullscreenOrientation?: OrientationType,
     ignoreSilentSwitch?: SilentSwitchType,
     playWhenInactive?: boolean,
+    pictureInPicture?: boolean,
+    onPictureInPictureStatusChanged?: (
+      VideoEvent<{ isActive: boolean }>
+    ) => void,
+    onRestoreUserInterfaceForPictureInPictureStop?: () => void,
 
     // Android only
+    minLoadRetryCount?: number,
     bufferConfig?: BufferConfigDescriptor,
     hideShutterView?: boolean,
     reportBandwidth?: number,
@@ -252,5 +258,8 @@ declare module 'react-native-video' {
     // IOS only
     save(): Promise<{ uri: string }>;
     seek(seconds: number, toleranceIOS: number): void;
+    restoreUserInterfaceForPictureInPictureStopCompleted(
+      restored: boolean
+    ): void;
   }
 }
diff --git a/definitions/npm/react-native-video_v4.x.x/flow_v0.77.0-/test_react-native-video.js b/definitions/npm/react-native-video_v4.x.x/flow_v0.77.0-/test_react-native-video.js
@@ -133,6 +133,13 @@ describe('public class methods', () => {
   describe('should call presentFullscreenPlayer', () => {
     video.presentFullscreenPlayer();
   });
+
+  describe('should call restoreUserInterfaceForPictureInPictureStopCompleted', () => {
+    video.restoreUserInterfaceForPictureInPictureStopCompleted(true);
+
+    // $ExpectError: first argument must be boolean
+    video.restoreUserInterfaceForPictureInPictureStopCompleted('true');
+  });
 });
 
 describe('source prop', () => {
@@ -204,6 +211,7 @@ describe('boolean props', () => {
       fullscreen={false}
       fullscreenAutorotate={false}
       playWhenInactive={false}
+      pictureInPicture={false}
     />;
 
     <Video
@@ -219,6 +227,7 @@ describe('boolean props', () => {
       fullscreen
       fullscreenAutorotate
       playWhenInactive
+      pictureInPicture
     />;
   });
 });
@@ -500,8 +509,39 @@ describe('events', () => {
           (event.bitrate: number);
         }}
       />;
-      <Video source={0} onError={() => {}} />;
-      <Video source={0} onError={undefined} />;
+      <Video source={0} onBandwidthUpdate={() => {}} />;
+      <Video source={0} onBandwidthUpdate={undefined} />;
+    });
+  });
+
+  describe('onPictureInPictureStatusChanged', () => {
+    it('should passes when used properly', () => {
+      <Video
+        source={0}
+        onPictureInPictureStatusChanged={event => {
+          // $ExpectError - read-only event and exact
+          event.abc = 123;
+
+          (event.target: number);
+          (event.isActive: boolean);
+        }}
+      />;
+      <Video source={0} onPictureInPictureStatusChanged={() => {}} />;
+      <Video source={0} onPictureInPictureStatusChanged={undefined} />;
+    });
+  });
+
+  describe('onRestoreUserInterfaceForPictureInPictureStop', () => {
+    it('should passes when used properly', () => {
+      <Video
+        source={0}
+        onRestoreUserInterfaceForPictureInPictureStop={() => {}}
+      />;
+
+      <Video
+        source={0}
+        onRestoreUserInterfaceForPictureInPictureStop={undefined}
+      />;
     });
   });