From 80344acaaad3351f5f1505828046b7905fba49eb Mon Sep 17 00:00:00 2001 From: Kate Lovett Date: Thu, 2 May 2019 12:56:31 -0700 Subject: [PATCH] Comma Comma Nit Nit --- .../cupertino/segmented_control_test.dart | 4 ++-- .../test/material/bottom_app_bar_test.dart | 2 +- .../test/material/input_decorator_test.dart | 3 +-- .../test/rendering/localized_fonts_test.dart | 9 +++---- .../widgets/editable_text_cursor_test.dart | 4 ++-- .../flutter/test/widgets/shadow_test.dart | 6 ++--- .../test/widgets/text_golden_test.dart | 24 +++++++++---------- 7 files changed, 24 insertions(+), 28 deletions(-) diff --git a/packages/flutter/test/cupertino/segmented_control_test.dart b/packages/flutter/test/cupertino/segmented_control_test.dart index 170d1f0da78..31568c93a6c 100644 --- a/packages/flutter/test/cupertino/segmented_control_test.dart +++ b/packages/flutter/test/cupertino/segmented_control_test.dart @@ -1327,7 +1327,7 @@ void main() { find.byType(RepaintBoundary), matchesSkiaGoldFile('segmented_control_test.0.png'), ); - },); + }); testWidgets('Golden Test Pressed State', (WidgetTester tester) async { final Map children = {}; @@ -1365,5 +1365,5 @@ void main() { find.byType(RepaintBoundary), matchesSkiaGoldFile('segmented_control_test.1.png'), ); - },); + }); } diff --git a/packages/flutter/test/material/bottom_app_bar_test.dart b/packages/flutter/test/material/bottom_app_bar_test.dart index e8512c570d2..8003d533d66 100644 --- a/packages/flutter/test/material/bottom_app_bar_test.dart +++ b/packages/flutter/test/material/bottom_app_bar_test.dart @@ -79,7 +79,7 @@ void main() { find.byKey(key), matchesSkiaGoldFile('bottom_app_bar.custom_shape.2.png'), ); - },); + }); testWidgets('color defaults to Theme.bottomAppBarColor', (WidgetTester tester) async { await tester.pumpWidget( diff --git a/packages/flutter/test/material/input_decorator_test.dart b/packages/flutter/test/material/input_decorator_test.dart index 364d863d464..da1acfa1221 100644 --- a/packages/flutter/test/material/input_decorator_test.dart +++ b/packages/flutter/test/material/input_decorator_test.dart @@ -2027,8 +2027,7 @@ void main() { find.byType(InputDecorator), matchesSkiaGoldFile('input_decorator.outline_icon_label.rtl.png'), ); - }, - ); + }); testWidgets('InputDecorationTheme.toString()', (WidgetTester tester) async { // Regression test for https://github.com/flutter/flutter/issues/19305 diff --git a/packages/flutter/test/rendering/localized_fonts_test.dart b/packages/flutter/test/rendering/localized_fonts_test.dart index 0e09ff2e3c7..6d0cae3531c 100644 --- a/packages/flutter/test/rendering/localized_fonts_test.dart +++ b/packages/flutter/test/rendering/localized_fonts_test.dart @@ -51,8 +51,7 @@ void main() { find.byType(RichText), matchesSkiaGoldFile('localized_fonts.rich_text.styled_text_span.png'), ); - }, - ); + }); testWidgets( 'Text with locale-specific glyphs, ambient locale', @@ -102,8 +101,7 @@ void main() { find.byType(Row), matchesSkiaGoldFile('localized_fonts.text_ambient_locale.chars.png'), ); - }, - ); + }); testWidgets( 'Text with locale-specific glyphs, explicit locale', @@ -145,7 +143,6 @@ void main() { find.byType(Row), matchesSkiaGoldFile('localized_fonts.text_explicit_locale.chars.png'), ); - }, - ); + }); } diff --git a/packages/flutter/test/widgets/editable_text_cursor_test.dart b/packages/flutter/test/widgets/editable_text_cursor_test.dart index 5ee49cb8dba..21d9f1f17fb 100644 --- a/packages/flutter/test/widgets/editable_text_cursor_test.dart +++ b/packages/flutter/test/widgets/editable_text_cursor_test.dart @@ -91,7 +91,7 @@ void main() { find.byKey(const ValueKey(1)), matchesSkiaGoldFile('editable_text_test.0.png'), ); - },); + }); testWidgets('cursor layout has correct radius', (WidgetTester tester) async { final GlobalKey editableTextKey = GlobalKey(); @@ -142,7 +142,7 @@ void main() { find.byKey(const ValueKey(1)), matchesSkiaGoldFile('editable_text_test.1.png'), ); - },); + }); testWidgets('Cursor animates on iOS', (WidgetTester tester) async { final Widget widget = MaterialApp( diff --git a/packages/flutter/test/widgets/shadow_test.dart b/packages/flutter/test/widgets/shadow_test.dart index 1db53f0dac1..4f78351784d 100644 --- a/packages/flutter/test/widgets/shadow_test.dart +++ b/packages/flutter/test/widgets/shadow_test.dart @@ -60,7 +60,7 @@ void main() { ); } debugDisableShadows = true; - },); + }); testWidgets('Shadows with PhysicalLayer', (WidgetTester tester) async { await tester.pumpWidget( @@ -124,5 +124,5 @@ void main() { ); } debugDisableShadows = true; - },); -} \ No newline at end of file + }); +} diff --git a/packages/flutter/test/widgets/text_golden_test.dart b/packages/flutter/test/widgets/text_golden_test.dart index 897d45d5aa8..5a90334e16d 100644 --- a/packages/flutter/test/widgets/text_golden_test.dart +++ b/packages/flutter/test/widgets/text_golden_test.dart @@ -55,7 +55,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Centered.wrap.png'), ); - },); + }); testWidgets('Text Foreground', (WidgetTester tester) async { @@ -131,7 +131,7 @@ void main() { find.byType(RepaintBoundary), matchesSkiaGoldFile('text_golden.Foreground.stroke_and_gradient.png'), ); - },); + }); // TODO(garyq): This test requires an update when the background // drawing from the beginning of the line bug is fixed. The current @@ -181,7 +181,7 @@ void main() { find.byType(RepaintBoundary), matchesSkiaGoldFile('text_golden.Background.png'), ); - },); + }); testWidgets('Text Fade', (WidgetTester tester) async { await tester.pumpWidget( @@ -217,7 +217,7 @@ void main() { find.byType(RepaintBoundary).first, matchesSkiaGoldFile('text_golden.Fade.1.png'), ); - },); + }); testWidgets('Default Strut text', (WidgetTester tester) async { await tester.pumpWidget( @@ -242,7 +242,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.StrutDefault.png'), ); - },); + }); testWidgets('Strut text 1', (WidgetTester tester) async { await tester.pumpWidget( @@ -269,7 +269,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Strut.1.png'), ); - },); + }); testWidgets('Strut text 2', (WidgetTester tester) async { await tester.pumpWidget( @@ -297,7 +297,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Strut.2.png'), ); - },); + }); testWidgets('Strut text rich', (WidgetTester tester) async { await tester.pumpWidget( @@ -348,7 +348,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Strut.3.png'), ); - },); + }); testWidgets('Strut text font fallback', (WidgetTester tester) async { // Font Fallback @@ -383,7 +383,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Strut.4.png'), ); - },); + }); testWidgets('Strut text rich forceStrutHeight', (WidgetTester tester) async { await tester.pumpWidget( @@ -434,7 +434,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.StrutForce.1.png'), ); - },); + }); testWidgets('Decoration thickness', (WidgetTester tester) async { final TextDecoration allDecorations = TextDecoration.combine( @@ -472,7 +472,7 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.Decoration.1.png'), ); - },); + }); testWidgets('Decoration thickness', (WidgetTester tester) async { final TextDecoration allDecorations = TextDecoration.combine( @@ -511,5 +511,5 @@ void main() { find.byType(Container), matchesSkiaGoldFile('text_golden.DecorationThickness.1.png'), ); - },); + }); }