Skip to content

Commit b62303e

Browse files
fombalangPIG208
authored andcommitted
autocomplete test [nfc]: Fix brittle avatar image finder
Add `findAvatarImage` to check userId instead of URL, making it resilient to changes in avatar image details like size. Previous finder used 'findNetworkImage' which would fail if the avatar size is different Relevant Discussion: https://chat.zulip.org/#narrow/channel/516-mobile-dev-help/topic/ComposeAutocomplete.20test.20failure.20help zulip#995 (comment)
1 parent 7d0c6bc commit b62303e

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

test/widgets/autocomplete_test.dart

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import 'package:zulip/model/store.dart';
1515
import 'package:zulip/model/typing_status.dart';
1616
import 'package:zulip/widgets/compose_box.dart';
1717
import 'package:zulip/widgets/icons.dart';
18+
import 'package:zulip/widgets/content.dart';
1819
import 'package:zulip/widgets/message_list.dart';
1920

2021
import '../api/fake_api.dart';
@@ -145,10 +146,13 @@ void main() {
145146
TestZulipBinding.ensureInitialized();
146147

147148
group('@-mentions', () {
149+
150+
Finder findAvatarImage(int userId) =>
151+
find.byWidgetPredicate((widget) => widget is AvatarImage && widget.userId == userId);
152+
148153
void checkUserShown(User user, PerAccountStore store, {required bool expected}) {
149154
check(find.text(user.fullName).evaluate().length).equals(expected ? 1 : 0);
150-
final avatarFinder =
151-
findNetworkImage(store.tryResolveUrl(user.avatarUrl!).toString());
155+
final avatarFinder = findAvatarImage(user.userId);
152156
check(avatarFinder.evaluate().length).equals(expected ? 1 : 0);
153157
}
154158

0 commit comments

Comments
 (0)