Test cover more tests with leak tracking. (#133828)

engine-flutter-autoroll · Sep 1, 2023 · 78ff122 · 78ff122
1 parent 0f3bd90
commit 78ff122
Show file tree

Hide file tree

Showing 7 changed files with 87 additions and 74 deletions.
diff --git a/packages/flutter/test/material/card_theme_test.dart b/packages/flutter/test/material/card_theme_test.dart
@@ -9,6 +9,7 @@ library;
 
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
+import 'package:leak_tracker_flutter_testing/leak_tracker_flutter_testing.dart';
 
 void main() {
   test('CardTheme copyWith, ==, hashCode basics', () {
@@ -22,7 +23,7 @@ void main() {
     expect(identical(CardTheme.lerp(theme, theme, 0.5), theme), true);
   });
 
-  testWidgets('Material3 - Passing no CardTheme returns defaults', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Material3 - Passing no CardTheme returns defaults', (WidgetTester tester) async {
     final ThemeData theme = ThemeData(useMaterial3: true);
     await tester.pumpWidget(MaterialApp(
       theme: theme,
@@ -45,7 +46,7 @@ void main() {
     ));
   });
 
-  testWidgets('Card uses values from CardTheme', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Card uses values from CardTheme', (WidgetTester tester) async {
     final CardTheme cardTheme = _cardTheme();
 
     await tester.pumpWidget(MaterialApp(
@@ -67,7 +68,7 @@ void main() {
     expect(material.shape, cardTheme.shape);
   });
 
-  testWidgets('Card widget properties take priority over theme', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Card widget properties take priority over theme', (WidgetTester tester) async {
     const Clip clip = Clip.hardEdge;
     const Color color = Colors.orange;
     const Color shadowColor = Colors.pink;
@@ -102,7 +103,7 @@ void main() {
     expect(material.shape, shape);
   });
 
-  testWidgets('CardTheme properties take priority over ThemeData properties', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('CardTheme properties take priority over ThemeData properties', (WidgetTester tester) async {
     final CardTheme cardTheme = _cardTheme();
     final ThemeData themeData = _themeData().copyWith(cardTheme: cardTheme);
 
@@ -117,7 +118,7 @@ void main() {
     expect(material.color, cardTheme.color);
   });
 
-  testWidgets('Material3 - ThemeData properties are used when no CardTheme is set', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Material3 - ThemeData properties are used when no CardTheme is set', (WidgetTester tester) async {
     final ThemeData themeData = ThemeData(useMaterial3: true);
 
     await tester.pumpWidget(MaterialApp(
@@ -131,7 +132,7 @@ void main() {
     expect(material.color, themeData.colorScheme.surface);
   });
 
-  testWidgets('Material3 - CardTheme customizes shape', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Material3 - CardTheme customizes shape', (WidgetTester tester) async {
     const CardTheme cardTheme = CardTheme(
       color: Colors.white,
       shape: BeveledRectangleBorder(borderRadius: BorderRadius.all(Radius.circular(7))),
@@ -165,7 +166,7 @@ void main() {
     // support is deprecated and the APIs are removed, these tests
     // can be deleted.
 
-    testWidgets('Material2 - ThemeData properties are used when no CardTheme is set', (WidgetTester tester) async {
+    testWidgetsWithLeakTracking('Material2 - ThemeData properties are used when no CardTheme is set', (WidgetTester tester) async {
       final ThemeData themeData = ThemeData(useMaterial3: false);
 
       await tester.pumpWidget(MaterialApp(
@@ -179,7 +180,7 @@ void main() {
       expect(material.color, themeData.cardColor);
     });
 
-    testWidgets('Material2 - Passing no CardTheme returns defaults', (WidgetTester tester) async {
+    testWidgetsWithLeakTracking('Material2 - Passing no CardTheme returns defaults', (WidgetTester tester) async {
       await tester.pumpWidget(MaterialApp(
         theme: ThemeData(useMaterial3: false),
         home: const Scaffold(
@@ -201,7 +202,7 @@ void main() {
       ));
     });
 
-    testWidgets('Material2 - CardTheme customizes shape', (WidgetTester tester) async {
+    testWidgetsWithLeakTracking('Material2 - CardTheme customizes shape', (WidgetTester tester) async {
       const CardTheme cardTheme = CardTheme(
         color: Colors.white,
         shape: BeveledRectangleBorder(borderRadius: BorderRadius.all(Radius.circular(7))),

diff --git a/packages/flutter/test/material/checkbox_list_tile_test.dart b/packages/flutter/test/material/checkbox_list_tile_test.dart
@@ -743,7 +743,7 @@ void main() {
     );
   });
 
-  testWidgets('Material2 - CheckboxListTile respects overlayColor in active/pressed/hovered states', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Material2 - CheckboxListTile respects overlayColor in active/pressed/hovered states', (WidgetTester tester) async {
     tester.binding.focusManager.highlightStrategy = FocusHighlightStrategy.alwaysTraditional;
 
     const Color fillColor = Color(0xFF000000);
@@ -859,7 +859,7 @@ void main() {
     );
   });
 
-  testWidgets('Material3 - CheckboxListTile respects overlayColor in active/pressed/hovered states', (WidgetTester tester) async {
+  testWidgetsWithLeakTracking('Material3 - CheckboxListTile respects overlayColor in active/pressed/hovered states', (WidgetTester tester) async {
     tester.binding.focusManager.highlightStrategy = FocusHighlightStrategy.alwaysTraditional;
 
     const Color fillColor = Color(0xFF000000);