Unverified Commit f51d3d7e authored by sigmundch's avatar sigmundch Committed by GitHub

[Web] Detect when running under Dart HHH Web and skip tests under investigation (#93692)

parent 7d098177
...@@ -220,17 +220,19 @@ Future<void> main(List<String> args) async { ...@@ -220,17 +220,19 @@ Future<void> main(List<String> args) async {
print('$clock ${bold}Test successful.$reset'); print('$clock ${bold}Test successful.$reset');
} }
final String _luciBotId = Platform.environment['SWARMING_BOT_ID'] ?? '';
final bool _runningInDartHHHBot = _luciBotId.startsWith('luci-dart-');
/// Verify the Flutter Engine is the revision in /// Verify the Flutter Engine is the revision in
/// bin/cache/internal/engine.version. /// bin/cache/internal/engine.version.
Future<void> _validateEngineHash() async { Future<void> _validateEngineHash() async {
final String luciBotId = Platform.environment['SWARMING_BOT_ID'] ?? ''; if (_runningInDartHHHBot) {
if (luciBotId.startsWith('luci-dart-')) {
// The Dart HHH bots intentionally modify the local artifact cache // The Dart HHH bots intentionally modify the local artifact cache
// and then use this script to run Flutter's test suites. // and then use this script to run Flutter's test suites.
// Because the artifacts have been changed, this particular test will return // Because the artifacts have been changed, this particular test will return
// a false positive and should be skipped. // a false positive and should be skipped.
print('${yellow}Skipping Flutter Engine Version Validation for swarming ' print('${yellow}Skipping Flutter Engine Version Validation for swarming '
'bot $luciBotId.'); 'bot $_luciBotId.');
return; return;
} }
final String expectedVersion = File(engineVersionFile).readAsStringSync().trim(); final String expectedVersion = File(engineVersionFile).readAsStringSync().trim();
...@@ -1506,6 +1508,7 @@ Future<void> _runFlutterWebTest(String webRenderer, String workingDirectory, Lis ...@@ -1506,6 +1508,7 @@ Future<void> _runFlutterWebTest(String webRenderer, String workingDirectory, Lis
'-v', '-v',
'--platform=chrome', '--platform=chrome',
'--web-renderer=$webRenderer', '--web-renderer=$webRenderer',
'--dart-define=DART_HHH_BOT=$_runningInDartHHHBot',
'--sound-null-safety', '--sound-null-safety',
...flutterTestArgs, ...flutterTestArgs,
...tests, ...tests,
......
...@@ -883,7 +883,9 @@ void main() { ...@@ -883,7 +883,9 @@ void main() {
editable.layout(BoxConstraints.loose(const Size(1000.0, 1000.0))); editable.layout(BoxConstraints.loose(const Size(1000.0, 1000.0)));
expect(editable.maxScrollExtent, equals(10)); expect(editable.maxScrollExtent, equals(10));
}); // TODO(yjbanov): This test is failing in the Dart HHH-web bot and
// needs additional investigation before it can be reenabled.
}, skip: const bool.fromEnvironment('DART_HHH_BOT')); // https://github.com/flutter/flutter/issues/93691
test('getEndpointsForSelection handles empty characters', () { test('getEndpointsForSelection handles empty characters', () {
final TextSelectionDelegate delegate = _FakeEditableTextState(); final TextSelectionDelegate delegate = _FakeEditableTextState();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment