github/flutter
7
0
Fork 0
mirror of https://github.com/flutter/flutter.git synced 2025-06-03 00:51:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Run flutter tests with --track-widget-creation as part of the build. (#23415)

Browse Source
This commit is contained in:
Jacob Richman 2018-10-23 14:10:53 -07:00 committed by GitHub
parent 02d659e82a
commit b21fb8c27d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
dev/bots/test.dart
View File

@ -176,6 +176,11 @@ Future<void> _runTests() async {
await _runSmokeTests(); await _runSmokeTests();
await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter')); await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter'));
// Only packages/flutter/test/widgets/widget_inspector_test.dart really
// needs to be run with --track-widget-creation but it is nice to run
// all of the tests in package:flutter with the flag to ensure that
// the Dart kernel transformer triggered by the flag does not break anything.
await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter'), options: <String>['--track-widget-creation']);
await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_localizations')); await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_localizations'));
await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_driver')); await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_driver'));
await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_test')); await _runFlutterTest(path.join(flutterRoot, 'packages', 'flutter_test'));
@ -190,6 +195,9 @@ Future<void> _runTests() async {
await _runFlutterTest(path.join(flutterRoot, 'examples', 'layers')); await _runFlutterTest(path.join(flutterRoot, 'examples', 'layers'));
await _runFlutterTest(path.join(flutterRoot, 'examples', 'stocks')); await _runFlutterTest(path.join(flutterRoot, 'examples', 'stocks'));
await _runFlutterTest(path.join(flutterRoot, 'examples', 'flutter_gallery')); await _runFlutterTest(path.join(flutterRoot, 'examples', 'flutter_gallery'));
// Regression test to ensure that code outside of package:flutter can run
// with --track-widget-creation.
await _runFlutterTest(path.join(flutterRoot, 'examples', 'flutter_gallery'), options: <String>['--track-widget-creation']);
await _runFlutterTest(path.join(flutterRoot, 'examples', 'catalog')); await _runFlutterTest(path.join(flutterRoot, 'examples', 'catalog'));
print('${bold}DONE: All tests successful.$reset'); print('${bold}DONE: All tests successful.$reset');

16
packages/flutter/test/widgets/widget_inspector_test.dart
View File

@ -677,23 +677,23 @@ class TestWidgetInspectorService extends Object with WidgetInspectorService {
// would make this test fragile. // would make this test fragile.
expect(lineA + 1, equals(lineB)); expect(lineA + 1, equals(lineB));
// Column numbers are more stable than line numbers. // Column numbers are more stable than line numbers.
expect(columnA, equals(21)); expect(columnA, equals(15));
expect(columnA, equals(columnB)); expect(columnA, equals(columnB));
expect(parameterLocationsA.length, equals(1)); expect(parameterLocationsA.length, equals(1));
final Map<String, Object> paramA = parameterLocationsA[0]; final Map<String, Object> paramA = parameterLocationsA[0];
expect(paramA['name'], equals('data')); expect(paramA['name'], equals('data'));
expect(paramA['line'], equals(lineA)); expect(paramA['line'], equals(lineA));
expect(paramA['column'], equals(26)); expect(paramA['column'], equals(20));
expect(parameterLocationsB.length, equals(2)); expect(parameterLocationsB.length, equals(2));
final Map<String, Object> paramB1 = parameterLocationsB[0]; final Map<String, Object> paramB1 = parameterLocationsB[0];
expect(paramB1['name'], equals('data')); expect(paramB1['name'], equals('data'));
expect(paramB1['line'], equals(lineB)); expect(paramB1['line'], equals(lineB));
expect(paramB1['column'], equals(26)); expect(paramB1['column'], equals(20));
final Map<String, Object> paramB2 = parameterLocationsB[1]; final Map<String, Object> paramB2 = parameterLocationsB[1];
expect(paramB2['name'], equals('textDirection')); expect(paramB2['name'], equals('textDirection'));
expect(paramB2['line'], equals(lineB)); expect(paramB2['line'], equals(lineB));
expect(paramB2['column'], equals(31)); expect(paramB2['column'], equals(25));
}, skip: !WidgetInspectorService.instance.isWidgetCreationTracked()); // Test requires --track-widget-creation flag. }, skip: !WidgetInspectorService.instance.isWidgetCreationTracked()); // Test requires --track-widget-creation flag.
testWidgets('WidgetInspectorService setPubRootDirectories', (WidgetTester tester) async { testWidgets('WidgetInspectorService setPubRootDirectories', (WidgetTester tester) async {
@ -1203,23 +1203,23 @@ class TestWidgetInspectorService extends Object with WidgetInspectorService {
// would make this test fragile. // would make this test fragile.
expect(lineA + 1, equals(lineB)); expect(lineA + 1, equals(lineB));
// Column numbers are more stable than line numbers. // Column numbers are more stable than line numbers.
expect(columnA, equals(21)); expect(columnA, equals(15));
expect(columnA, equals(columnB)); expect(columnA, equals(columnB));
expect(parameterLocationsA.length, equals(1)); expect(parameterLocationsA.length, equals(1));
final Map<String, Object> paramA = parameterLocationsA[0]; final Map<String, Object> paramA = parameterLocationsA[0];
expect(paramA['name'], equals('data')); expect(paramA['name'], equals('data'));
expect(paramA['line'], equals(lineA)); expect(paramA['line'], equals(lineA));
expect(paramA['column'], equals(26)); expect(paramA['column'], equals(20));
expect(parameterLocationsB.length, equals(2)); expect(parameterLocationsB.length, equals(2));
final Map<String, Object> paramB1 = parameterLocationsB[0]; final Map<String, Object> paramB1 = parameterLocationsB[0];
expect(paramB1['name'], equals('data')); expect(paramB1['name'], equals('data'));
expect(paramB1['line'], equals(lineB)); expect(paramB1['line'], equals(lineB));
expect(paramB1['column'], equals(26)); expect(paramB1['column'], equals(20));
final Map<String, Object> paramB2 = parameterLocationsB[1]; final Map<String, Object> paramB2 = parameterLocationsB[1];
expect(paramB2['name'], equals('textDirection')); expect(paramB2['name'], equals('textDirection'));
expect(paramB2['line'], equals(lineB)); expect(paramB2['line'], equals(lineB));
expect(paramB2['column'], equals(31)); expect(paramB2['column'], equals(25));
}, skip: !WidgetInspectorService.instance.isWidgetCreationTracked()); // Test requires --track-widget-creation flag. }, skip: !WidgetInspectorService.instance.isWidgetCreationTracked()); // Test requires --track-widget-creation flag.
testWidgets('ext.flutter.inspector.setPubRootDirectories', (WidgetTester tester) async { testWidgets('ext.flutter.inspector.setPubRootDirectories', (WidgetTester tester) async {