add passing success test, by disabling recent behavior

Author: chrisirhc

lib/widgets/action_sheet.dart

@@ -102,41 +102,43 @@ class AddReactionButton extends MessageActionSheetMenuItemButton {
           // apply bottom padding to handle keyboard opening via https://github.com/flutter/flutter/issues/71418
           padding: EdgeInsets.only(bottom: MediaQuery.of(emojiPickerContext).viewInsets.bottom),
           child: EmojiPicker(
-          config: Config(emojiSet: emojiSet),
-          onEmojiSelected: (_, Emoji? emoji) async {
-            if (emoji == null) {
-              // dismiss emoji picker
-              Navigator.of(emojiPickerContext).pop();
-              return;
-            }
-            final emojiName = emoji.name;
-            final emojiCode = getEmojiCode(emoji);
-            String? errorMessage;
-            try {
-              await addReaction(PerAccountStoreWidget.of(messageListContext).connection,
-                messageId: message.id,
-                reactionType: ReactionType.unicodeEmoji,
-                emojiCode: emojiCode,
-                emojiName: emojiName,
-              );
-              if (!emojiPickerContext.mounted) return;
-              Navigator.of(emojiPickerContext).pop();
-            } catch (e) {
-              debugPrint('Error adding reaction: $e');
-              if (!emojiPickerContext.mounted) return;
-
-              switch (e) {
-                case ZulipApiException():
-                  errorMessage = e.message;
-                  // TODO specific messages for common errors, like network errors
-                  //   (support with reusable code)
-                default:
+            config: Config(
+              emojiSet: emojiSet,
+              categoryViewConfig: const CategoryViewConfig(recentTabBehavior: RecentTabBehavior.NONE)),
+            onEmojiSelected: (_, Emoji? emoji) async {
+              if (emoji == null) {
+                // dismiss emoji picker
+                Navigator.of(emojiPickerContext).pop();
+                return;
               }
-
-              await showErrorDialog(context: emojiPickerContext,
-                title: 'Adding reaction failed', message: errorMessage);
-            }
-          }));
+              final emojiName = emoji.name;
+              final emojiCode = getEmojiCode(emoji);
+              String? errorMessage;
+              try {
+                await addReaction(PerAccountStoreWidget.of(messageListContext).connection,
+                  messageId: message.id,
+                  reactionType: ReactionType.unicodeEmoji,
+                  emojiCode: emojiCode,
+                  emojiName: emojiName,
+                );
+                if (!emojiPickerContext.mounted) return;
+                Navigator.of(emojiPickerContext).pop();
+              } catch (e) {
+                debugPrint('Error adding reaction: $e');
+                if (!emojiPickerContext.mounted) return;
+
+                switch (e) {
+                  case ZulipApiException():
+                    errorMessage = e.message;
+                    // TODO specific messages for common errors, like network errors
+                    //   (support with reusable code)
+                  default:
+                }
+
+                await showErrorDialog(context: emojiPickerContext,
+                  title: 'Adding reaction failed', message: errorMessage);
+              }
+            }));
       });
   };
 }

test/widgets/action_sheet_test.dart

@@ -98,7 +98,7 @@ void main() {
     (store.connection as FakeApiConnection).prepare(httpStatus: 400, json: fakeResponseJson);
   }
 
-  group('AddThumbsUpButton', () {
+  group('AddReactionButton', () {
     Future<void> tapButton(WidgetTester tester) async {
       await tester.ensureVisible(find.byIcon(Icons.add_reaction_outlined, skipOffstage: false));
       await tester.tap(find.byIcon(Icons.add_reaction_outlined));
@@ -113,6 +113,16 @@ void main() {
       final connection = store.connection as FakeApiConnection;
       connection.prepare(json: {});
       await tapButton(tester);
+      // Wait for bottom modal to appear
+      await tester.pumpAndSettle();
+      await tester.ensureVisible(find.byIcon(Icons.tag_faces, skipOffstage: false));
+      await tester.tap(find.byIcon(Icons.tag_faces));
+      await tester.dragUntilVisible(
+        find.text('👍').hitTestable(),
+        find.byKey(const Key('emojiScrollView')),
+        const Offset(0, -300),
+      );
+      await tester.tap(find.text('👍'));
       await tester.pump(Duration.zero);
 
       check(connection.lastRequest).isA<http.Request>()