Revert "Updated the skipped tests for cupertino package. (#87289)" (#87537)
This reverts commit faf774eb97fd11c6c483aeee84c1abd3187b022f.
diff --git a/packages/flutter/test/cupertino/dialog_test.dart b/packages/flutter/test/cupertino/dialog_test.dart
index ea7ab9d..ceee1b2 100644
--- a/packages/flutter/test/cupertino/dialog_test.dart
+++ b/packages/flutter/test/cupertino/dialog_test.dart
@@ -917,7 +917,7 @@
// We must explicitly cause an "up" gesture to avoid a crash.
// todo(mattcarroll) remove this call, https://github.com/flutter/flutter/issues/19540
await gesture.up();
- });
+ }, skip: isBrowser); // https://github.com/flutter/flutter/issues/52960
testWidgets('ScaleTransition animation for showCupertinoDialog()', (WidgetTester tester) async {
await tester.pumpWidget(
diff --git a/packages/flutter/test/cupertino/text_field_test.dart b/packages/flutter/test/cupertino/text_field_test.dart
index 7555428..ed2c875 100644
--- a/packages/flutter/test/cupertino/text_field_test.dart
+++ b/packages/flutter/test/cupertino/text_field_test.dart
@@ -289,10 +289,7 @@
expect(controller.text, ' blah2blah1');
expect(controller.selection, const TextSelection(baseOffset: 0, extentOffset: 0));
expect(find.byType(CupertinoButton), findsNothing);
- },
- variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.macOS }),
- skip: kIsWeb, // [intended] the web handles this on its own.
- );
+ }, variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.macOS }), skip: kIsWeb);
testWidgets('Activates the text field when receives semantics focus on Mac', (WidgetTester tester) async {
final SemanticsTester semantics = SemanticsTester(tester);
@@ -457,7 +454,8 @@
);
},
// TODO(mdebbar): Strut styles support.
- skip: isBrowser, // https://github.com/flutter/flutter/issues/32243
+ // https://github.com/flutter/flutter/issues/32243
+ skip: isBrowser,
);
testWidgets(
@@ -487,7 +485,8 @@
);
},
// TODO(mdebbar): Strut styles support.
- skip: isBrowser, // https://github.com/flutter/flutter/issues/32243
+ // https://github.com/flutter/flutter/issues/32243
+ skip: isBrowser,
);
testWidgets(
@@ -1490,7 +1489,7 @@
expect(text.style!.fontSize, 14);
expect(text.style!.letterSpacing, -0.15);
expect(text.style!.fontWeight, FontWeight.w400);
- }, skip: isContextMenuProvidedByPlatform); // [intended] only applies to platforms where we supply the context menu.
+ }, skip: isContextMenuProvidedByPlatform);
testWidgets('text field toolbar options correctly changes options', (WidgetTester tester) async {
final TextEditingController controller = TextEditingController(
@@ -1533,7 +1532,7 @@
expect(find.text('Copy'), findsOneWidget);
expect(find.text('Cut'), findsNothing);
expect(find.text('Select All'), findsNothing);
- }, skip: isContextMenuProvidedByPlatform); // [intended] only applies to platforms where we supply the context menu.
+ }, skip: isContextMenuProvidedByPlatform);
testWidgets('Read only text field', (WidgetTester tester) async {
final TextEditingController controller = TextEditingController(text: 'readonly');
@@ -1570,7 +1569,7 @@
expect(find.text('Copy'), findsOneWidget);
expect(find.text('Paste'), findsNothing);
expect(find.text('Cut'), findsNothing);
- }, skip: isContextMenuProvidedByPlatform); // [intended] only applies to platforms where we supply the context menu.
+ }, skip: isContextMenuProvidedByPlatform);
testWidgets('copy paste', (WidgetTester tester) async {
await tester.pumpWidget(
@@ -1621,7 +1620,7 @@
expect(find.text('field 1'), findsOneWidget);
expect(find.text("j'aime la poutine"), findsOneWidget);
expect(find.text('field 2'), findsNothing);
- }, exclude: isContextMenuProvidedByPlatform);
+ }, skip: isContextMenuProvidedByPlatform);
testWidgets(
'tap moves cursor to the edge of the word it tapped on',
@@ -2205,7 +2204,7 @@
expect(find.text('Select All'), findsOneWidget);
expect(find.text('Copy'), findsNothing);
expect(find.text('Cut'), findsNothing);
- }, skip: isContextMenuProvidedByPlatform); // [intended] only applies to platforms where we supply the context menu.
+ }, skip: isContextMenuProvidedByPlatform);
testWidgets(
'long press moves cursor to the exact long press position and shows toolbar',
@@ -3230,7 +3229,7 @@
textFieldTopLeft = tester.getTopLeft(find.byType(CupertinoTextField));
expect(toolbarTopLeft.dy, lessThan(textFieldTopLeft.dy));
},
- skip: isContextMenuProvidedByPlatform, // [intended] only applies to platforms where we supply the context menu.
+ skip: isContextMenuProvidedByPlatform,
);
testWidgets('text field respects keyboardAppearance from theme', (WidgetTester tester) async {
@@ -3827,7 +3826,7 @@
expect(find.byType(CupertinoTextSelectionToolbar), findsOneWidget);
expect(tester.takeException(), null);
});
- }, skip: isContextMenuProvidedByPlatform); // [intended] only applies to platforms where we supply the context menu.
+ }, skip: isContextMenuProvidedByPlatform);
group('textAlignVertical position', () {
group('simple case', () {
diff --git a/packages/flutter/test/cupertino/text_form_field_row_test.dart b/packages/flutter/test/cupertino/text_form_field_row_test.dart
index 9445afd..691e3e9 100644
--- a/packages/flutter/test/cupertino/text_form_field_row_test.dart
+++ b/packages/flutter/test/cupertino/text_form_field_row_test.dart
@@ -262,7 +262,7 @@
await tester.pump(const Duration(milliseconds: 200));
expect(renderEditable, paintsExactlyCountTimes(#drawRect, 0));
- }, skip: isBrowser); // [intended] We do not use Flutter-rendered context menu on the Web.
+ }, skip: isBrowser); // We do not use Flutter-rendered context menu on the Web
testWidgets('onTap is called upon tap', (WidgetTester tester) async {
int tapCount = 0;
diff --git a/packages/flutter/test/cupertino/text_selection_test.dart b/packages/flutter/test/cupertino/text_selection_test.dart
index 4b86238..b5a95b9 100644
--- a/packages/flutter/test/cupertino/text_selection_test.dart
+++ b/packages/flutter/test/cupertino/text_selection_test.dart
@@ -251,7 +251,7 @@
expect(find.text('Copy'), findsOneWidget);
expect(find.text('Cut'), findsOneWidget);
},
- exclude: isBrowser, // We do not use Flutter-rendered context menu on the Web
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -305,7 +305,7 @@
expect(find.text('◀'), findsNothing);
expect(find.text('▶'), findsNothing);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -376,7 +376,7 @@
expect(find.text('▶'), findsOneWidget);
expect(appearsEnabled(tester, '▶'), true);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -478,7 +478,7 @@
expect(find.text('▶'), findsOneWidget);
expect(appearsEnabled(tester, '▶'), true);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -596,7 +596,7 @@
expect(find.text('▶'), findsOneWidget);
expect(appearsEnabled(tester, '▶'), true);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
});
@@ -677,7 +677,7 @@
expect(handles.first.size.height, 60.5);
expect(handles.last.size.height, 35.5);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -761,7 +761,7 @@
expect(handles.first.size.height, 110.5);
expect(handles.last.size.height, 110.5);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -842,7 +842,7 @@
expect(handles.first.size.height, 60.5);
expect(handles.last.size.height, 45.5);
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
@@ -928,7 +928,7 @@
expect(handles.first.size.height, 50.5);
expect(handles.last.size.height, 50.5); // This is 60.5 with the previous frame.
},
- skip: isBrowser, // [intended] We do not use Flutter-rendered context menu on the Web.
+ skip: isBrowser, // We do not use Flutter-rendered context menu on the Web
variant: const TargetPlatformVariant(<TargetPlatform>{ TargetPlatform.iOS }),
);
}
diff --git a/packages/flutter/test/cupertino/text_selection_toolbar_test.dart b/packages/flutter/test/cupertino/text_selection_toolbar_test.dart
index 7156dd1..1a8028c 100644
--- a/packages/flutter/test/cupertino/text_selection_toolbar_test.dart
+++ b/packages/flutter/test/cupertino/text_selection_toolbar_test.dart
@@ -180,7 +180,7 @@
expect(find.byType(TestBox), findsNWidgets(7));
expect(_findOverflowNextButton(), findsOneWidget);
expect(_findOverflowBackButton(), findsNothing);
- }, skip: kIsWeb); // [intended] We do not use Flutter-rendered context menu on the Web.
+ }, skip: kIsWeb);
testWidgets('positions itself at anchorAbove if it fits', (WidgetTester tester) async {
late StateSetter setState;
@@ -229,7 +229,7 @@
await tester.pump();
toolbarY = tester.getTopLeft(_findToolbar()).dy;
expect(toolbarY, equals(anchorAboveY - height - _kToolbarContentDistance));
- }, skip: kIsWeb); // [intended] We do not use Flutter-rendered context menu on the Web.
+ }, skip: kIsWeb);
testWidgets('can create and use a custom toolbar', (WidgetTester tester) async {
final TextEditingController controller = TextEditingController(
@@ -262,5 +262,5 @@
expect(find.text('Copy'), findsNothing);
expect(find.text('Paste'), findsNothing);
expect(find.text('Select all'), findsNothing);
- }, skip: kIsWeb); // [intended] We do not use Flutter-rendered context menu on the Web.
+ }, skip: kIsWeb);
}
