Skip Video demo in integration tests of flutter_gallery (#100666)

flutter · Mar 24, 2022 · 0703096 · 0703096
1 parent 19680c3
commit 0703096
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/dev/integration_tests/flutter_gallery/test_driver/run_demos.dart b/dev/integration_tests/flutter_gallery/test_driver/run_demos.dart
@@ -3,7 +3,6 @@
 // found in the LICENSE file.
 
 import 'package:flutter/cupertino.dart';
-import 'package:flutter/foundation.dart';
 import 'package:flutter_gallery/demo_lists.dart';
 import 'package:flutter_test/flutter_test.dart';
 
@@ -12,9 +11,9 @@ import 'package:flutter_test/flutter_test.dart';
 /// Demo names are formatted as 'DEMO_NAME@DEMO_CATEGORY' (see
 /// `demo_lists.dart` for more examples).
 final List<String> kSkippedDemos = <String>[
-  // The CI uses Chromium, which lacks the video codecs to run this demo.
-  if (kIsWeb)
-    'Video@Media',
+  // This demo is flaky on CI due to hitting the network.
+  // See: https://github.com/flutter/flutter/issues/100497
+  'Video@Media',
 ];
 
 /// Scrolls each demo menu item into view, launches it, then returns to the

diff --git a/dev/integration_tests/flutter_gallery/test_driver/transitions_perf_test.dart b/dev/integration_tests/flutter_gallery/test_driver/transitions_perf_test.dart
@@ -13,7 +13,15 @@ import 'package:test/test.dart' hide TypeMatcher, isInstanceOf;
 
 const FileSystem _fs = LocalFileSystem();
 
-const List<String> kSkippedDemos = <String>[];
+/// The demos we don't run as part of the integration test.
+///
+/// Demo names are formatted as 'DEMO_NAME@DEMO_CATEGORY' (see
+/// `demo_lists.dart` for more examples).
+const List<String> kSkippedDemos = <String>[
+  // This demo is flaky on CI due to hitting the network.
+  // See: https://github.com/flutter/flutter/issues/100497
+  'Video@Media',
+];
 
 // All of the gallery demos, identified as "title@category".
 //