[camera] Attempt to fix flaky new Android test (#6831)

stuartmorgan-g · web-flow · commit 13818023c99e · 2022-12-12T11:56:50.000-05:00
The recently added "recording with image stream" test is very flaky,
often throwing on `stop`. This is a speculative fix for that flake based
on the documentation of `stop` indicating that it will throw if nothing
has been recorded.
diff --git a/packages/camera/camera_android/example/integration_test/camera_test.dart b/packages/camera/camera_android/example/integration_test/camera_test.dart
@@ -277,7 +277,10 @@ void main() {
 
       expect(controller.value.isStreamingImages, true);
 
-      sleep(const Duration(milliseconds: 500));
+      // Stopping recording before anything is recorded will throw, per
+      // https://developer.android.com/reference/android/media/MediaRecorder.html#stop()
+      // so delay long enough to ensure that some data is recorded.
+      await Future<void>.delayed(const Duration(seconds: 2));
 
       await controller.stopVideoRecording();
       await controller.dispose();
