Skip to content

Commit 5cbf844

Browse files
test: add focus test to date pickers
1 parent a612131 commit 5cbf844

File tree

2 files changed

+18
-8
lines changed

2 files changed

+18
-8
lines changed

test/src/fields/form_builder_date_range_picker_test.dart

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,12 +106,14 @@ void main() {
106106
testWidgets('When press tab, field will be focused',
107107
(WidgetTester tester) async {
108108
const widgetName = 'cb1';
109+
const saveText = 'SAVE THE DATE';
109110
final testWidget = FormBuilderDateRangePicker(
110111
name: widgetName,
111112
firstDate: DateTime(2010),
112113
// Using last date < today to make date picker always open on 01/01/2010
113114
// If last date >= today, it opens on DateTime.now month, which complicates testing.
114115
lastDate: DateTime(2020),
116+
saveText: saveText,
115117
);
116118
final widgetFinder = find.byWidget(testWidget);
117119

@@ -125,8 +127,13 @@ void main() {
125127
expect(focusNode?.hasFocus, false);
126128
await tester.sendKeyEvent(LogicalKeyboardKey.tab);
127129
await tester.pumpAndSettle();
128-
// TODO: Fix this behavior to solve #1301 and partially #1450
129-
// expect(Focus.of(tester.element(widgetFinder)).hasFocus, true);
130-
// expect(focusNode?.hasFocus, true);
130+
expect(Focus.of(tester.element(widgetFinder)).hasFocus, true);
131+
expect(focusNode?.hasFocus, true);
132+
expect(find.text(saveText), findsNothing);
133+
134+
// Open picker
135+
await tester.sendKeyEvent(LogicalKeyboardKey.space);
136+
await tester.pumpAndSettle();
137+
expect(find.text(saveText), findsOneWidget);
131138
});
132139
}

test/src/fields/form_builder_date_time_picker_test.dart

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -174,10 +174,13 @@ void main() {
174174
expect(focusNode?.hasFocus, false);
175175
await tester.sendKeyEvent(LogicalKeyboardKey.tab);
176176
await tester.pumpAndSettle();
177-
// TODO: Fix this behavior to solve #1301 and partially #1450
178-
// expect(find.text(confirmText), findsOneWidget);
179-
// expect(find.text(cancelText), findsOneWidget);
180-
// expect(Focus.of(tester.element(widgetFinder)).hasFocus, true);
181-
// expect(focusNode?.hasFocus, true);
177+
expect(Focus.of(tester.element(widgetFinder)).hasFocus, true);
178+
expect(focusNode?.hasFocus, true);
179+
180+
// Open picker
181+
await tester.sendKeyEvent(LogicalKeyboardKey.space);
182+
await tester.pumpAndSettle();
183+
expect(find.text(confirmText), findsOneWidget);
184+
expect(find.text(cancelText), findsOneWidget);
182185
});
183186
}

0 commit comments

Comments
 (0)