From 89a3d0947837609f41796c1d30e3bb17c56588ec Mon Sep 17 00:00:00 2001
From: Sahil Kumar <xdsahil@gmail.com>
Date: Fri, 20 May 2022 15:43:27 +0530
Subject: [PATCH] test(llc): fix user, own_user tests

Signed-off-by: xsahil03x <xdsahil@gmail.com>
---
 .../stream_chat/lib/src/core/models/own_user.dart     | 11 ++++++++---
 packages/stream_chat/lib/src/core/models/user.dart    |  5 ++++-
 packages/stream_chat/test/src/client/client_test.dart |  2 +-
 3 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/packages/stream_chat/lib/src/core/models/own_user.dart b/packages/stream_chat/lib/src/core/models/own_user.dart
index f79d8eab..60d15278 100644
--- a/packages/stream_chat/lib/src/core/models/own_user.dart
+++ b/packages/stream_chat/lib/src/core/models/own_user.dart
@@ -41,7 +41,8 @@ class OwnUser extends User {
   factory OwnUser.fromUser(User user) => OwnUser(
         id: user.id,
         role: user.role,
-        name: user.name,
+        // Using extraData value in order to not use id as name.
+        name: user.extraData['name'] as String?,
         image: user.image,
         createdAt: user.createdAt,
         updatedAt: user.updatedAt,
@@ -78,7 +79,10 @@ class OwnUser extends User {
       OwnUser(
         id: id ?? this.id,
         role: role ?? this.role,
-        name: name ?? extraData?['name'] as String? ?? this.name,
+        name: name ??
+            extraData?['name'] as String? ??
+            // Using extraData value in order to not use id as name.
+            this.extraData['name'] as String?,
         image: image ?? extraData?['image'] as String? ?? this.image,
         banned: banned ?? this.banned,
         banExpires: banExpires ?? this.banExpires,
@@ -103,7 +107,8 @@ class OwnUser extends User {
     return copyWith(
       id: other.id,
       role: other.role,
-      name: other.name,
+      // Using extraData value in order to not use id as name.
+      name: other.extraData['name'] as String?,
       image: other.image,
       banned: other.banned,
       channelMutes: other.channelMutes,
diff --git a/packages/stream_chat/lib/src/core/models/user.dart b/packages/stream_chat/lib/src/core/models/user.dart
index 2c702d01..ff435f67 100644
--- a/packages/stream_chat/lib/src/core/models/user.dart
+++ b/packages/stream_chat/lib/src/core/models/user.dart
@@ -172,7 +172,10 @@ class User extends Equatable {
       User(
         id: id ?? this.id,
         role: role ?? this.role,
-        name: name ?? extraData?['name'] as String? ?? this.name,
+        name: name ??
+            extraData?['name'] as String? ??
+            // Using extraData value in order to not use id as name.
+            this.extraData['name'] as String?,
         image: image ?? extraData?['image'] as String? ?? this.image,
         createdAt: createdAt ?? this.createdAt,
         updatedAt: updatedAt ?? this.updatedAt,
diff --git a/packages/stream_chat/test/src/client/client_test.dart b/packages/stream_chat/test/src/client/client_test.dart
index 4947bc72..923d399c 100644
--- a/packages/stream_chat/test/src/client/client_test.dart
+++ b/packages/stream_chat/test/src/client/client_test.dart
@@ -2514,7 +2514,7 @@ void main() {
     });
 
     test(
-      '''setting the `currentUser` should also compute and update the unreadCounts''',
+      'setting the `currentUser` should also compute and update the unreadCounts',
       () {
         final state = client.state;
         final initialUser = OwnUser.fromUser(user);