Skip to content

Commit 646f9d9

Browse files
sm-sayedism-sayedi
committed
new-dm: Show user status emoji
1 parent 14093b2 commit 646f9d9

File tree

2 files changed

+90
-27
lines changed

2 files changed

+90
-27
lines changed

‎lib/widgets/new_dm_sheet.dart

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,6 +317,8 @@ class _SelectedUserChip extends StatelessWidget {
317317
fontSize: 16,
318318
height: 16 / 16,
319319
color: designVariables.labelMenuButton)))),
320+
UserStatusEmoji(userId: userId, size: 16,
321+
padding: EdgeInsetsDirectional.only(end: 4)),
320322
])));
321323
}
322324
}
@@ -415,7 +417,12 @@ class _NewDmUserListItem extends StatelessWidget {
415417
Avatar(userId: userId, size: 32, borderRadius: 3),
416418
SizedBox(width: 8),
417419
Expanded(
418-
child: Text(store.userDisplayName(userId),
420+
child: Text.rich(
421+
TextSpan(
422+
children: [
423+
TextSpan(text: store.userDisplayName(userId)),
424+
UserStatusEmoji.asWidgetSpan(userId: userId, fontSize: 17,
425+
textScaler: MediaQuery.textScalerOf(context))]),
419426
style: TextStyle(
420427
fontSize: 17,
421428
height: 19 / 17,

‎test/widgets/new_dm_sheet_test.dart

Lines changed: 82 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,11 @@ import 'package:flutter/material.dart';
33
import 'package:flutter_checks/flutter_checks.dart';
44
import 'package:flutter_test/flutter_test.dart';
55
import 'package:zulip/api/model/model.dart';
6+
import 'package:zulip/basic.dart';
67
import 'package:zulip/widgets/app_bar.dart';
78
import 'package:zulip/widgets/compose_box.dart';
89
import 'package:zulip/widgets/content.dart';
10+
import 'package:zulip/widgets/emoji.dart';
911
import 'package:zulip/widgets/home.dart';
1012
import 'package:zulip/widgets/icons.dart';
1113
import 'package:zulip/widgets/new_dm_sheet.dart';
@@ -17,11 +19,13 @@ import '../flutter_checks.dart';
1719
import '../model/binding.dart';
1820
import '../model/test_store.dart';
1921
import '../test_navigation.dart';
22+
import 'message_list_test.dart';
2023
import 'test_app.dart';
2124

2225
Future<void> setupSheet(WidgetTester tester, {
2326
required List<User> users,
2427
List<int>? mutedUserIds,
28+
List<(int userId, UserStatusChange change)>? userStatuses,
2529
}) async {
2630
addTearDown(testBinding.reset);
2731

@@ -35,6 +39,7 @@ Future<void> setupSheet(WidgetTester tester, {
3539
if (mutedUserIds != null) {
3640
await store.setMutedUsers(mutedUserIds);
3741
}
42+
await store.changeUserStatuses(userStatuses ?? []);
3843

3944
await tester.pumpWidget(TestZulipApp(
4045
navigatorObservers: [testNavObserver],
@@ -65,7 +70,8 @@ void main() {
6570
}
6671

6772
Finder findUserTile(User user) =>
68-
find.widgetWithText(InkWell, user.fullName).first;
73+
find.ancestor(of: find.textContaining(user.fullName),
74+
matching: find.byType(InkWell)).first;
6975

7076
Finder findUserChip(User user) {
7177
final findAvatar = find.byWidgetPredicate((widget) =>
@@ -120,23 +126,23 @@ void main() {
120126

121127
testWidgets('shows all non-muted users initially', (tester) async {
122128
await setupSheet(tester, users: testUsers, mutedUserIds: [mutedUser.userId]);
123-
check(find.text('Alice Anderson')).findsOne();
124-
check(find.text('Bob Brown')).findsOne();
125-
check(find.text('Charlie Carter')).findsOne();
129+
check(find.textContaining('Alice Anderson')).findsOne();
130+
check(find.textContaining('Bob Brown')).findsOne();
131+
check(find.textContaining('Charlie Carter')).findsOne();
126132

127133
check(find.byIcon(ZulipIcons.check_circle_unchecked)).findsExactly(3);
128134
check(find.byIcon(ZulipIcons.check_circle_checked)).findsNothing();
129-
check(find.text('Someone Muted')).findsNothing();
130-
check(find.text('Muted user')).findsNothing();
135+
check(find.textContaining('Someone Muted')).findsNothing();
136+
check(find.textContaining('Muted user')).findsNothing();
131137
});
132138

133139
testWidgets('shows filtered users based on search', (tester) async {
134140
await setupSheet(tester, users: testUsers);
135141
await tester.enterText(find.byType(TextField), 'Alice');
136142
await tester.pump();
137-
check(find.text('Alice Anderson')).findsOne();
138-
check(find.text('Charlie Carter')).findsNothing();
139-
check(find.text('Bob Brown')).findsNothing();
143+
check(find.textContaining('Alice Anderson')).findsOne();
144+
check(find.textContaining('Charlie Carter')).findsNothing();
145+
check(find.textContaining('Bob Brown')).findsNothing();
140146
});
141147

142148
// TODO test sorting by recent-DMs
@@ -146,43 +152,43 @@ void main() {
146152
await setupSheet(tester, users: testUsers);
147153
await tester.enterText(find.byType(TextField), 'alice');
148154
await tester.pump();
149-
check(find.text('Alice Anderson')).findsOne();
155+
check(find.textContaining('Alice Anderson')).findsOne();
150156

151157
await tester.enterText(find.byType(TextField), 'ALICE');
152158
await tester.pump();
153-
check(find.text('Alice Anderson')).findsOne();
159+
check(find.textContaining('Alice Anderson')).findsOne();
154160
});
155161

156162
testWidgets('partial name and last name search handling', (tester) async {
157163
await setupSheet(tester, users: testUsers);
158164

159165
await tester.enterText(find.byType(TextField), 'Ali');
160166
await tester.pump();
161-
check(find.text('Alice Anderson')).findsOne();
162-
check(find.text('Bob Brown')).findsNothing();
163-
check(find.text('Charlie Carter')).findsNothing();
167+
check(find.textContaining('Alice Anderson')).findsOne();
168+
check(find.textContaining('Bob Brown')).findsNothing();
169+
check(find.textContaining('Charlie Carter')).findsNothing();
164170

165171
await tester.enterText(find.byType(TextField), 'Anderson');
166172
await tester.pump();
167-
check(find.text('Alice Anderson')).findsOne();
168-
check(find.text('Charlie Carter')).findsNothing();
169-
check(find.text('Bob Brown')).findsNothing();
173+
check(find.textContaining('Alice Anderson')).findsOne();
174+
check(find.textContaining('Charlie Carter')).findsNothing();
175+
check(find.textContaining('Bob Brown')).findsNothing();
170176

171177
await tester.enterText(find.byType(TextField), 'son');
172178
await tester.pump();
173-
check(find.text('Alice Anderson')).findsOne();
174-
check(find.text('Charlie Carter')).findsNothing();
175-
check(find.text('Bob Brown')).findsNothing();
179+
check(find.textContaining('Alice Anderson')).findsOne();
180+
check(find.textContaining('Charlie Carter')).findsNothing();
181+
check(find.textContaining('Bob Brown')).findsNothing();
176182
});
177183

178184
testWidgets('shows empty state when no users match', (tester) async {
179185
await setupSheet(tester, users: testUsers);
180186
await tester.enterText(find.byType(TextField), 'Zebra');
181187
await tester.pump();
182-
check(find.text('No users found')).findsOne();
183-
check(find.text('Alice Anderson')).findsNothing();
184-
check(find.text('Bob Brown')).findsNothing();
185-
check(find.text('Charlie Carter')).findsNothing();
188+
check(find.textContaining('No users found')).findsOne();
189+
check(find.textContaining('Alice Anderson')).findsNothing();
190+
check(find.textContaining('Bob Brown')).findsNothing();
191+
check(find.textContaining('Charlie Carter')).findsNothing();
186192
});
187193

188194
testWidgets('search text clears when user is selected', (tester) async {
@@ -252,7 +258,7 @@ void main() {
252258
await tester.tap(findUserTile(eg.selfUser));
253259
await tester.pump();
254260
checkUserSelected(tester, eg.selfUser, true);
255-
check(find.text(eg.selfUser.fullName)).findsExactly(2);
261+
check(find.textContaining(eg.selfUser.fullName)).findsExactly(2);
256262

257263
await tester.tap(findUserTile(otherUser));
258264
await tester.pump();
@@ -264,7 +270,7 @@ void main() {
264270
final otherUser = eg.user(fullName: 'Other User');
265271
await setupSheet(tester, users: [eg.selfUser, otherUser]);
266272

267-
check(find.text(eg.selfUser.fullName)).findsOne();
273+
check(find.textContaining(eg.selfUser.fullName)).findsOne();
268274

269275
await tester.tap(findUserTile(otherUser));
270276
await tester.pump();
@@ -285,6 +291,56 @@ void main() {
285291
});
286292
});
287293

294+
testWidgets('status emoji is set -> emoji is displayed', (tester) async {
295+
void checkTileStatusEmoji(User user, {required bool isPresent}) {
296+
final statusEmojiFinder = findStatusEmoji(UnicodeEmojiWidget);
297+
final tileStatusFinder = find.descendant(of: findUserTile(user),
298+
matching: statusEmojiFinder);
299+
if (isPresent) {
300+
checkUserStatusEmoji(tileStatusFinder, isAnimated: false);
301+
}
302+
check(tileStatusFinder).findsExactly(isPresent ? 1 : 0);
303+
}
304+
305+
void checkChipStatusEmoji(User user, {required bool isPresent}) {
306+
final statusEmojiFinder = findStatusEmoji(UnicodeEmojiWidget);
307+
final chipStatusFinder = find.descendant(of: findUserChip(user),
308+
matching: statusEmojiFinder);
309+
if (isPresent) {
310+
checkUserStatusEmoji(chipStatusFinder, isAnimated: false);
311+
}
312+
check(chipStatusFinder).findsExactly(isPresent ? 1 : 0);
313+
}
314+
315+
final user1 = eg.user(userId: 1, fullName: 'User 1');
316+
final user2 = eg.user(userId: 2, fullName: 'User 2');
317+
await setupSheet(tester, users: [user1, user2], userStatuses: [
318+
(
319+
user1.userId,
320+
UserStatusChange(
321+
text: OptionSome('Busy'),
322+
emoji: OptionSome(StatusEmoji(emojiName: 'working_on_it',
323+
emojiCode: '1f6e0', reactionType: ReactionType.unicodeEmoji)))
324+
),
325+
]);
326+
327+
checkTileStatusEmoji(user1, isPresent: true);
328+
checkTileStatusEmoji(user2, isPresent: false);
329+
check(findUserChip(user1)).findsNothing();
330+
check(findUserChip(user2)).findsNothing();
331+
332+
await tester.tap(findUserTile(user1));
333+
await tester.tap(findUserTile(user2));
334+
await tester.pump();
335+
336+
checkTileStatusEmoji(user1, isPresent: true);
337+
checkTileStatusEmoji(user2, isPresent: false);
338+
check(findUserChip(user1)).findsOne();
339+
check(findUserChip(user2)).findsOne();
340+
checkChipStatusEmoji(user1, isPresent: true);
341+
checkChipStatusEmoji(user2, isPresent: false);
342+
});
343+
288344
group('navigation to DM Narrow', () {
289345
Future<void> runAndCheck(WidgetTester tester, {
290346
required List<User> users,

0 commit comments

Comments
 (0)