Merge branch 'develop' of https://github.com/squaddy/stream-chat-flutter into bugfix/deleting_thread_messages
This commit is contained in:
@@ -1,12 +1,27 @@
|
||||
## Upcoming
|
||||
|
||||
✅ Added
|
||||
|
||||
- You can now pass `score` to `client.sendReaction` and `channel.sendReaction` functions.
|
||||
- Added new `client.partialUpdateUsers` function in order to partially update users.
|
||||
|
||||
🐞 Fixed
|
||||
|
||||
- [[#890]](https://github.com/GetStream/stream-chat-flutter/pull/890) Fixed Reactions not updating on thread messages.
|
||||
Thanks [bstolinski](https://github.com/bstolinski).
|
||||
- [[#897]](https://github.com/GetStream/stream-chat-flutter/issues/897) Fixed error type mis-match in `AuthInterceptor`.
|
||||
|
||||
## 3.4.0
|
||||
|
||||
🐞 Fixed
|
||||
|
||||
- [[#857]](https://github.com/GetStream/stream-chat-flutter/issues/857) Channel now listens for member ban/unban and
|
||||
updates the channel state with the latest data.
|
||||
- [[#748]](https://github.com/GetStream/stream-chat-flutter/issues/748) `Message.user` is now also included while saving users in persistence.
|
||||
- [[#748]](https://github.com/GetStream/stream-chat-flutter/issues/748) `Message.user` is now also included while saving
|
||||
users in persistence.
|
||||
- [[#871]](https://github.com/GetStream/stream-chat-flutter/issues/871) Fixed thread message deletion.
|
||||
- [[#846]](https://github.com/GetStream/stream-chat-flutter/issues/846) Fixed `message.ownReactions` getting truncated when receiving a reaction event.
|
||||
- [[#846]](https://github.com/GetStream/stream-chat-flutter/issues/846) Fixed `message.ownReactions` getting truncated
|
||||
when receiving a reaction event.
|
||||
- Add check for invalid image URLs
|
||||
- Fix `channelState.pinnedMessagesStream` getting reset to `0` after a channel update.
|
||||
- Fixed `unreadCount` after removing user from a channel.
|
||||
|
||||
@@ -811,6 +811,7 @@ class Channel {
|
||||
Future<SendReactionResponse> sendReaction(
|
||||
Message message,
|
||||
String type, {
|
||||
int score = 1,
|
||||
Map<String, Object?> extraData = const {},
|
||||
bool enforceUnique = false,
|
||||
}) async {
|
||||
@@ -829,7 +830,7 @@ class Channel {
|
||||
createdAt: now,
|
||||
type: type,
|
||||
user: user,
|
||||
score: 1,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
);
|
||||
|
||||
@@ -866,6 +867,7 @@ class Channel {
|
||||
final reactionResp = await _client.sendReaction(
|
||||
messageId,
|
||||
type,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
enforceUnique: enforceUnique,
|
||||
);
|
||||
|
||||
@@ -1073,6 +1073,28 @@ class StreamChatClient {
|
||||
Future<UpdateUsersResponse> updateUsers(List<User> users) =>
|
||||
_chatApi.user.updateUsers(users);
|
||||
|
||||
/// Partially update the given user with [id].
|
||||
/// Use [set] to define values to be set.
|
||||
/// Use [unset] to define values to be unset.
|
||||
Future<UpdateUsersResponse> partialUpdateUser(
|
||||
String id, {
|
||||
Map<String, Object?>? set,
|
||||
List<String>? unset,
|
||||
}) {
|
||||
final user = PartialUpdateUserRequest(
|
||||
id: id,
|
||||
set: set,
|
||||
unset: unset,
|
||||
);
|
||||
return partialUpdateUsers([user]);
|
||||
}
|
||||
|
||||
/// Batch partial updates the [users].
|
||||
Future<UpdateUsersResponse> partialUpdateUsers(
|
||||
List<PartialUpdateUserRequest> users,
|
||||
) =>
|
||||
_chatApi.user.partialUpdateUsers(users);
|
||||
|
||||
/// Bans a user from all channels
|
||||
Future<EmptyResponse> banUser(
|
||||
String targetUserId, [
|
||||
@@ -1157,15 +1179,22 @@ class StreamChatClient {
|
||||
Future<SendReactionResponse> sendReaction(
|
||||
String messageId,
|
||||
String reactionType, {
|
||||
int score = 1,
|
||||
Map<String, Object?> extraData = const {},
|
||||
bool enforceUnique = false,
|
||||
}) =>
|
||||
_chatApi.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
enforceUnique: enforceUnique,
|
||||
);
|
||||
}) {
|
||||
final _extraData = {
|
||||
'score': score,
|
||||
...extraData,
|
||||
};
|
||||
|
||||
return _chatApi.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: _extraData,
|
||||
enforceUnique: enforceUnique,
|
||||
);
|
||||
}
|
||||
|
||||
/// Delete a [reactionType] from this [messageId]
|
||||
Future<EmptyResponse> deleteReaction(
|
||||
|
||||
@@ -156,3 +156,29 @@ class PaginationParams extends Equatable {
|
||||
lessThanOrEqual,
|
||||
];
|
||||
}
|
||||
|
||||
/// Request model for the [client.partialUpdateUser] api call.
|
||||
@JsonSerializable(createFactory: false)
|
||||
class PartialUpdateUserRequest extends Equatable {
|
||||
/// Creates a new PartialUpdateUserRequest instance.
|
||||
const PartialUpdateUserRequest({
|
||||
required this.id,
|
||||
this.set,
|
||||
this.unset,
|
||||
});
|
||||
|
||||
/// User ID.
|
||||
final String id;
|
||||
|
||||
/// Fields to set.
|
||||
final Map<String, Object?>? set;
|
||||
|
||||
/// Fields to unset.
|
||||
final List<String>? unset;
|
||||
|
||||
/// Serialize model to json
|
||||
Map<String, dynamic> toJson() => _$PartialUpdateUserRequestToJson(this);
|
||||
|
||||
@override
|
||||
List<Object?> get props => [id, set, unset];
|
||||
}
|
||||
|
||||
@@ -54,3 +54,14 @@ Map<String, dynamic> _$PaginationParamsToJson(PaginationParams instance) {
|
||||
writeNotNull('id_lte', instance.lessThanOrEqual);
|
||||
return val;
|
||||
}
|
||||
|
||||
Map<String, dynamic> _$PartialUpdateUserRequestToJson(
|
||||
PartialUpdateUserRequest instance) =>
|
||||
<String, dynamic>{
|
||||
'stringify': instance.stringify,
|
||||
'hash_code': instance.hashCode,
|
||||
'id': instance.id,
|
||||
'set': instance.set,
|
||||
'unset': instance.unset,
|
||||
'props': instance.props,
|
||||
};
|
||||
|
||||
@@ -46,4 +46,17 @@ class UserApi {
|
||||
);
|
||||
return UpdateUsersResponse.fromJson(response.data);
|
||||
}
|
||||
|
||||
/// Batch partial update of [users].
|
||||
Future<UpdateUsersResponse> partialUpdateUsers(
|
||||
List<PartialUpdateUserRequest> users,
|
||||
) async {
|
||||
final response = await _client.patch(
|
||||
'/users',
|
||||
data: {
|
||||
'users': users,
|
||||
},
|
||||
);
|
||||
return UpdateUsersResponse.fromJson(response.data);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -49,10 +49,13 @@ class AuthInterceptor extends Interceptor {
|
||||
DioError err,
|
||||
ErrorInterceptorHandler handler,
|
||||
) async {
|
||||
ErrorResponse? error;
|
||||
final data = err.response?.data;
|
||||
if (data != null) error = ErrorResponse.fromJson(data);
|
||||
if (error?.code == ChatErrorCode.tokenExpired.code) {
|
||||
if (data == null || data is! Map<String, dynamic>) {
|
||||
return handler.next(err);
|
||||
}
|
||||
|
||||
final error = ErrorResponse.fromJson(data);
|
||||
if (error.code == ChatErrorCode.tokenExpired.code) {
|
||||
if (_tokenManager.isStatic) return handler.next(err);
|
||||
_client.lock();
|
||||
await _tokenManager.loadToken(refresh: true);
|
||||
|
||||
@@ -21,7 +21,7 @@ ChannelState _$ChannelStateFromJson(Map<String, dynamic> json) => ChannelState(
|
||||
pinnedMessages: (json['pinned_messages'] as List<dynamic>?)
|
||||
?.map((e) => Message.fromJson(e as Map<String, dynamic>))
|
||||
.toList() ??
|
||||
const [],
|
||||
_emptyPinnedMessages,
|
||||
watcherCount: json['watcher_count'] as int?,
|
||||
watchers: (json['watchers'] as List<dynamic>?)
|
||||
?.map((e) => User.fromJson(e as Map<String, dynamic>))
|
||||
|
||||
@@ -1116,6 +1116,135 @@ void main() {
|
||||
verify(() => client.sendReaction(message.id, type)).called(1);
|
||||
});
|
||||
|
||||
test('should work fine with score passed explicitly', () async {
|
||||
const type = 'test-reaction-type';
|
||||
final message = Message(id: 'test-message-id');
|
||||
|
||||
const score = 5;
|
||||
final reaction = Reaction(
|
||||
type: type,
|
||||
messageId: message.id,
|
||||
score: score,
|
||||
);
|
||||
|
||||
when(() => client.sendReaction(
|
||||
message.id,
|
||||
type,
|
||||
score: score,
|
||||
)).thenAnswer(
|
||||
(_) async => SendReactionResponse()
|
||||
..message = message
|
||||
..reaction = reaction,
|
||||
);
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.messagesStream.skip(1),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
reactionCounts: {type: 1},
|
||||
reactionScores: {type: score},
|
||||
latestReactions: [reaction],
|
||||
ownReactions: [reaction],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
final res = await channel.sendReaction(
|
||||
message,
|
||||
type,
|
||||
score: score,
|
||||
);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.reaction.type, type);
|
||||
expect(res.reaction.messageId, message.id);
|
||||
expect(res.reaction.score, score);
|
||||
|
||||
verify(() => client.sendReaction(
|
||||
message.id,
|
||||
type,
|
||||
score: score,
|
||||
)).called(1);
|
||||
});
|
||||
|
||||
test('should work fine with score passed explicitly and in extraData',
|
||||
() async {
|
||||
const type = 'test-reaction-type';
|
||||
final message = Message(id: 'test-message-id');
|
||||
|
||||
const score = 5;
|
||||
const extraDataScore = 3;
|
||||
const extraData = {
|
||||
'score': extraDataScore,
|
||||
};
|
||||
final reaction = Reaction(
|
||||
type: type,
|
||||
messageId: message.id,
|
||||
score: extraDataScore,
|
||||
);
|
||||
|
||||
when(() => client.sendReaction(
|
||||
message.id,
|
||||
type,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
)).thenAnswer(
|
||||
(_) async => SendReactionResponse()
|
||||
..message = message
|
||||
..reaction = reaction,
|
||||
);
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.messagesStream.skip(1),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
reactionCounts: {type: 1},
|
||||
reactionScores: {type: extraDataScore},
|
||||
latestReactions: [reaction],
|
||||
ownReactions: [reaction],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
final res = await channel.sendReaction(
|
||||
message,
|
||||
type,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.reaction.type, type);
|
||||
expect(res.reaction.messageId, message.id);
|
||||
expect(
|
||||
res.reaction.score,
|
||||
extraDataScore,
|
||||
);
|
||||
|
||||
verify(() => client.sendReaction(
|
||||
message.id,
|
||||
type,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
)).called(1);
|
||||
});
|
||||
|
||||
test(
|
||||
'should restore previous message if `client.sendReaction` throws',
|
||||
() async {
|
||||
@@ -1239,6 +1368,189 @@ void main() {
|
||||
);
|
||||
});
|
||||
|
||||
group('`.sendReaction in thread`', () {
|
||||
test('should work fine', () async {
|
||||
const type = 'test-reaction-type';
|
||||
final message = Message(
|
||||
id: 'test-message-id',
|
||||
parentId: 'test-parent-id', // is thread message
|
||||
);
|
||||
|
||||
final reaction = Reaction(type: type, messageId: message.id);
|
||||
|
||||
when(() => client.sendReaction(message.id, type)).thenAnswer(
|
||||
(_) async => SendReactionResponse()
|
||||
..message = message
|
||||
..reaction = reaction,
|
||||
);
|
||||
|
||||
expectLater(
|
||||
channel.state?.threadsStream
|
||||
// skipping first seed message list -> [] messages
|
||||
.skip(1)
|
||||
.map((event) => event['test-parent-id']),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
reactionCounts: {type: 1},
|
||||
reactionScores: {type: 1},
|
||||
latestReactions: [reaction],
|
||||
ownReactions: [reaction],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
final res = await channel.sendReaction(message, type);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.reaction.type, type);
|
||||
expect(res.reaction.messageId, message.id);
|
||||
|
||||
verify(() => client.sendReaction(message.id, type)).called(1);
|
||||
});
|
||||
|
||||
test(
|
||||
'''should restore previous thread message if `client.sendReaction` throws''',
|
||||
() async {
|
||||
const type = 'test-reaction-type';
|
||||
final message = Message(
|
||||
id: 'test-message-id',
|
||||
parentId: 'test-parent-id', // is thread message
|
||||
);
|
||||
|
||||
final reaction = Reaction(type: type, messageId: message.id);
|
||||
|
||||
when(() => client.sendReaction(message.id, type))
|
||||
.thenThrow(StreamChatNetworkError(ChatErrorCode.inputError));
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.threadsStream
|
||||
.skip(1)
|
||||
.map((event) => event['test-parent-id']),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
reactionCounts: {type: 1},
|
||||
reactionScores: {type: 1},
|
||||
latestReactions: [reaction],
|
||||
ownReactions: [reaction],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
[
|
||||
isSameMessageAs(
|
||||
message,
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
try {
|
||||
await channel.sendReaction(message, type);
|
||||
} catch (e) {
|
||||
expect(e, isA<StreamChatNetworkError>());
|
||||
}
|
||||
|
||||
verify(() => client.sendReaction(message.id, type)).called(1);
|
||||
},
|
||||
);
|
||||
|
||||
test(
|
||||
'''should override previous thread reaction if present and `enforceUnique` is true''',
|
||||
() async {
|
||||
const userId = 'test-user-id';
|
||||
const messageId = 'test-message-id';
|
||||
const parentId = 'test-parent-id';
|
||||
const prevType = 'test-reaction-type';
|
||||
final prevReaction = Reaction(
|
||||
type: prevType,
|
||||
messageId: messageId,
|
||||
userId: userId,
|
||||
);
|
||||
final message = Message(
|
||||
id: messageId,
|
||||
parentId: parentId,
|
||||
ownReactions: [prevReaction],
|
||||
latestReactions: [prevReaction],
|
||||
reactionScores: const {prevType: 1},
|
||||
reactionCounts: const {prevType: 1},
|
||||
);
|
||||
|
||||
const type = 'test-reaction-type-2';
|
||||
final newReaction = Reaction(
|
||||
type: type,
|
||||
messageId: messageId,
|
||||
userId: userId,
|
||||
);
|
||||
final newMessage = message.copyWith(
|
||||
ownReactions: [newReaction],
|
||||
latestReactions: [newReaction],
|
||||
);
|
||||
|
||||
const enforceUnique = true;
|
||||
|
||||
when(() => client.sendReaction(
|
||||
messageId,
|
||||
type,
|
||||
enforceUnique: enforceUnique,
|
||||
)).thenAnswer(
|
||||
(_) async => SendReactionResponse()
|
||||
..message = newMessage
|
||||
..reaction = newReaction,
|
||||
);
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.threadsStream
|
||||
.skip(1)
|
||||
.map((event) => event['test-parent-id']),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
newMessage.copyWith(status: MessageSendingStatus.sent),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
final res = await channel.sendReaction(
|
||||
message,
|
||||
type,
|
||||
enforceUnique: enforceUnique,
|
||||
);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.reaction.type, type);
|
||||
expect(res.reaction.messageId, messageId);
|
||||
|
||||
verify(() => client.sendReaction(
|
||||
messageId,
|
||||
type,
|
||||
enforceUnique: enforceUnique,
|
||||
)).called(1);
|
||||
},
|
||||
);
|
||||
});
|
||||
|
||||
group('`.deleteReaction`', () {
|
||||
test('should work fine', () async {
|
||||
const userId = 'test-user-id';
|
||||
@@ -1343,6 +1655,121 @@ void main() {
|
||||
);
|
||||
});
|
||||
|
||||
group('`.deleteReaction in thread`', () {
|
||||
test('should work fine', () async {
|
||||
const userId = 'test-user-id';
|
||||
const messageId = 'test-message-id';
|
||||
const parentId = 'test-parent-id';
|
||||
const type = 'test-reaction-type';
|
||||
final reaction = Reaction(
|
||||
type: type,
|
||||
messageId: messageId,
|
||||
userId: userId,
|
||||
);
|
||||
final message = Message(
|
||||
id: messageId,
|
||||
parentId: parentId, // is thread
|
||||
ownReactions: [reaction],
|
||||
latestReactions: [reaction],
|
||||
reactionScores: const {type: 1},
|
||||
reactionCounts: const {type: 1},
|
||||
);
|
||||
|
||||
when(() => client.deleteReaction(messageId, type))
|
||||
.thenAnswer((_) async => EmptyResponse());
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.threadsStream
|
||||
.skip(1)
|
||||
.map((event) => event['test-parent-id']),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
latestReactions: [],
|
||||
ownReactions: [],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
final res = await channel.deleteReaction(message, reaction);
|
||||
|
||||
expect(res, isNotNull);
|
||||
|
||||
verify(() => client.deleteReaction(messageId, type)).called(1);
|
||||
});
|
||||
|
||||
test(
|
||||
'should restore prev message state if `client.deleteReaction` throws',
|
||||
() async {
|
||||
const userId = 'test-user-id';
|
||||
const messageId = 'test-message-id';
|
||||
const parentId = 'test-parent-id';
|
||||
const type = 'test-reaction-type';
|
||||
final reaction = Reaction(
|
||||
type: type,
|
||||
messageId: messageId,
|
||||
userId: userId,
|
||||
);
|
||||
final message = Message(
|
||||
id: messageId,
|
||||
parentId: parentId,
|
||||
ownReactions: [reaction],
|
||||
latestReactions: [reaction],
|
||||
reactionScores: const {type: 1},
|
||||
reactionCounts: const {type: 1},
|
||||
);
|
||||
|
||||
when(() => client.deleteReaction(messageId, type))
|
||||
.thenThrow(StreamChatNetworkError(ChatErrorCode.inputError));
|
||||
|
||||
expectLater(
|
||||
// skipping first seed message list -> [] messages
|
||||
channel.state?.threadsStream
|
||||
.skip(1)
|
||||
.map((event) => event['test-parent-id']),
|
||||
emitsInOrder([
|
||||
[
|
||||
isSameMessageAs(
|
||||
message.copyWith(
|
||||
status: MessageSendingStatus.sent,
|
||||
latestReactions: [],
|
||||
ownReactions: [],
|
||||
),
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
[
|
||||
isSameMessageAs(
|
||||
message,
|
||||
matchReactions: true,
|
||||
matchSendingStatus: true,
|
||||
matchParentId: true,
|
||||
),
|
||||
],
|
||||
]),
|
||||
);
|
||||
|
||||
try {
|
||||
await channel.deleteReaction(message, reaction);
|
||||
} catch (e) {
|
||||
expect(e, isA<StreamChatNetworkError>());
|
||||
}
|
||||
|
||||
verify(() => client.deleteReaction(messageId, type)).called(1);
|
||||
},
|
||||
);
|
||||
});
|
||||
|
||||
test('`.update`', () async {
|
||||
const channelData = {
|
||||
'name': 'Stream Team',
|
||||
|
||||
@@ -1772,6 +1772,46 @@ void main() {
|
||||
verifyNoMoreInteractions(api.user);
|
||||
});
|
||||
|
||||
test('`.partialUpdateUser`', () async {
|
||||
const userId = 'test-user-id';
|
||||
|
||||
final set = {'color': 'yellow'};
|
||||
final unset = <String>[];
|
||||
|
||||
final partialUpdateRequest = PartialUpdateUserRequest(
|
||||
id: userId,
|
||||
set: set,
|
||||
unset: unset,
|
||||
);
|
||||
|
||||
final updatedUser = User(
|
||||
id: userId,
|
||||
extraData: {'color': set['color']},
|
||||
);
|
||||
|
||||
when(() => api.user.partialUpdateUsers([partialUpdateRequest]))
|
||||
.thenAnswer(
|
||||
(_) async => UpdateUsersResponse()
|
||||
..users = {
|
||||
updatedUser.id: updatedUser,
|
||||
},
|
||||
);
|
||||
|
||||
final res = await client.partialUpdateUser(
|
||||
userId,
|
||||
set: set,
|
||||
unset: unset,
|
||||
);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.users, {updatedUser.id: updatedUser});
|
||||
|
||||
verify(
|
||||
() => api.user.partialUpdateUsers([partialUpdateRequest]),
|
||||
).called(1);
|
||||
verifyNoMoreInteractions(api.user);
|
||||
});
|
||||
|
||||
test('`.banUser`', () async {
|
||||
const userId = 'test-user-id';
|
||||
|
||||
@@ -1956,23 +1996,109 @@ void main() {
|
||||
verifyNoMoreInteractions(api.channel);
|
||||
});
|
||||
|
||||
test('`.sendReaction`', () async {
|
||||
const messageId = 'test-message-id';
|
||||
const reactionType = 'like';
|
||||
group('`.sendReaction`', () {
|
||||
test('`.sendReaction with default params`', () async {
|
||||
const messageId = 'test-message-id';
|
||||
const reactionType = 'like';
|
||||
const extraData = {'score': 1};
|
||||
|
||||
when(() => api.message.sendReaction(messageId, reactionType))
|
||||
.thenAnswer((_) async => SendReactionResponse()
|
||||
..message = Message(id: messageId)
|
||||
..reaction = Reaction(type: reactionType, messageId: messageId));
|
||||
when(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).thenAnswer((_) async => SendReactionResponse()
|
||||
..message = Message(id: messageId)
|
||||
..reaction = Reaction(type: reactionType, messageId: messageId));
|
||||
|
||||
final res = await client.sendReaction(messageId, reactionType);
|
||||
expect(res, isNotNull);
|
||||
expect(res.message.id, messageId);
|
||||
expect(res.reaction.type, reactionType);
|
||||
expect(res.reaction.messageId, messageId);
|
||||
final res = await client.sendReaction(messageId, reactionType);
|
||||
expect(res, isNotNull);
|
||||
expect(res.message.id, messageId);
|
||||
expect(res.reaction.type, reactionType);
|
||||
expect(res.reaction.messageId, messageId);
|
||||
|
||||
verify(() => api.message.sendReaction(messageId, reactionType)).called(1);
|
||||
verifyNoMoreInteractions(api.message);
|
||||
verify(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).called(1);
|
||||
verifyNoMoreInteractions(api.message);
|
||||
});
|
||||
|
||||
test('`.sendReaction with score`', () async {
|
||||
const messageId = 'test-message-id';
|
||||
const reactionType = 'like';
|
||||
const score = 3;
|
||||
const extraData = {'score': score};
|
||||
|
||||
when(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).thenAnswer((_) async => SendReactionResponse()
|
||||
..message = Message(id: messageId)
|
||||
..reaction = Reaction(
|
||||
type: reactionType,
|
||||
messageId: messageId,
|
||||
score: score,
|
||||
));
|
||||
|
||||
final res = await client.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
score: score,
|
||||
);
|
||||
expect(res, isNotNull);
|
||||
expect(res.message.id, messageId);
|
||||
expect(res.reaction.type, reactionType);
|
||||
expect(res.reaction.messageId, messageId);
|
||||
expect(res.reaction.score, score);
|
||||
|
||||
verify(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).called(1);
|
||||
verifyNoMoreInteractions(api.message);
|
||||
});
|
||||
|
||||
test('`.sendReaction with score passed in extradata also`', () async {
|
||||
const messageId = 'test-message-id';
|
||||
const reactionType = 'like';
|
||||
const score = 3;
|
||||
const extraDataScore = 5;
|
||||
const extraData = {'score': extraDataScore};
|
||||
|
||||
when(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).thenAnswer((_) async => SendReactionResponse()
|
||||
..message = Message(id: messageId)
|
||||
..reaction = Reaction(
|
||||
type: reactionType,
|
||||
messageId: messageId,
|
||||
score: extraDataScore,
|
||||
));
|
||||
|
||||
final res = await client.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
score: score,
|
||||
extraData: extraData,
|
||||
);
|
||||
expect(res, isNotNull);
|
||||
expect(res.message.id, messageId);
|
||||
expect(res.reaction.type, reactionType);
|
||||
expect(res.reaction.messageId, messageId);
|
||||
expect(res.reaction.score, extraDataScore);
|
||||
|
||||
verify(() => api.message.sendReaction(
|
||||
messageId,
|
||||
reactionType,
|
||||
extraData: extraData,
|
||||
)).called(1);
|
||||
verifyNoMoreInteractions(api.message);
|
||||
});
|
||||
});
|
||||
|
||||
test('`.deleteReaction`', () async {
|
||||
|
||||
@@ -175,14 +175,14 @@ void main() {
|
||||
|
||||
test('markAllRead', () async {
|
||||
const path = '/channels/read';
|
||||
when(() => client.post(path)).thenAnswer(
|
||||
when(() => client.post(path, data: {})).thenAnswer(
|
||||
(_) async => successResponse(path, data: <String, dynamic>{}));
|
||||
|
||||
final res = await channelApi.markAllRead();
|
||||
|
||||
expect(res, isNotNull);
|
||||
|
||||
verify(() => client.post(path)).called(1);
|
||||
verify(() => client.post(path, data: {})).called(1);
|
||||
verifyNoMoreInteractions(client);
|
||||
});
|
||||
|
||||
|
||||
@@ -82,4 +82,37 @@ void main() {
|
||||
verify(() => client.post(path, data: any(named: 'data'))).called(1);
|
||||
verifyNoMoreInteractions(client);
|
||||
});
|
||||
|
||||
test('partialUpdateUsers', () async {
|
||||
const user = PartialUpdateUserRequest(
|
||||
id: 'test-user-id',
|
||||
set: {'color': 'yellow'},
|
||||
);
|
||||
|
||||
const path = '/users';
|
||||
|
||||
final updatedUser = {user.id: User(id: user.id, extraData: user.set!)};
|
||||
|
||||
when(() => client.patch(path, data: {
|
||||
'users': [user],
|
||||
})).thenAnswer(
|
||||
(_) async => successResponse(
|
||||
path,
|
||||
data: {
|
||||
'users':
|
||||
updatedUser.map((key, value) => MapEntry(key, value.toJson()))
|
||||
},
|
||||
),
|
||||
);
|
||||
|
||||
final res = await userApi.partialUpdateUsers([user]);
|
||||
|
||||
expect(res, isNotNull);
|
||||
expect(res.users.length, updatedUser.length);
|
||||
|
||||
verify(() => client.patch(path, data: {
|
||||
'users': [user]
|
||||
})).called(1);
|
||||
verifyNoMoreInteractions(client);
|
||||
});
|
||||
}
|
||||
|
||||
@@ -49,6 +49,7 @@ Matcher isSameMessageAs(
|
||||
bool matchSendingStatus = false,
|
||||
bool matchAttachments = false,
|
||||
bool matchAttachmentsUploadState = false,
|
||||
bool matchParentId = false,
|
||||
}) =>
|
||||
_IsSameMessageAs(
|
||||
targetMessage: targetMessage,
|
||||
@@ -57,6 +58,7 @@ Matcher isSameMessageAs(
|
||||
matchSendingStatus: matchSendingStatus,
|
||||
matchAttachments: matchAttachments,
|
||||
matchAttachmentsUploadState: matchAttachmentsUploadState,
|
||||
matchParentId: matchParentId,
|
||||
);
|
||||
|
||||
class _IsSameMessageAs extends Matcher {
|
||||
@@ -67,6 +69,7 @@ class _IsSameMessageAs extends Matcher {
|
||||
this.matchSendingStatus = false,
|
||||
this.matchAttachments = false,
|
||||
this.matchAttachmentsUploadState = false,
|
||||
this.matchParentId = false,
|
||||
});
|
||||
|
||||
final Message targetMessage;
|
||||
@@ -75,6 +78,7 @@ class _IsSameMessageAs extends Matcher {
|
||||
final bool matchSendingStatus;
|
||||
final bool matchAttachments;
|
||||
final bool matchAttachmentsUploadState;
|
||||
final bool matchParentId;
|
||||
|
||||
@override
|
||||
Description describe(Description description) =>
|
||||
@@ -123,6 +127,9 @@ class _IsSameMessageAs extends Matcher {
|
||||
|
||||
matches &= matchAttachments();
|
||||
}
|
||||
if (matchParentId) {
|
||||
matches &= message.parentId == targetMessage.parentId;
|
||||
}
|
||||
return matches;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,3 +1,11 @@
|
||||
## Upcoming
|
||||
|
||||
🐞 Fixed
|
||||
|
||||
- [[#888]](https://github.com/GetStream/stream-chat-flutter/issues/888) Fix `unban` command not working in `MessageInput`.
|
||||
- [[#805]](https://github.com/GetStream/stream-chat-flutter/issues/805) Updated chewie dependency version to 1.3.0
|
||||
- Fix `showScrollToBottom` in `MessageListView` not respecting false value.
|
||||
|
||||
## 3.4.0
|
||||
- Updated `stream_chat_flutter_core` dependency to [`3.4.0`](https://pub.dev/packages/stream_chat_flutter_core/changelog).
|
||||
|
||||
|
||||
@@ -3,7 +3,7 @@
|
||||
archiveVersion = 1;
|
||||
classes = {
|
||||
};
|
||||
objectVersion = 46;
|
||||
objectVersion = 50;
|
||||
objects = {
|
||||
|
||||
/* Begin PBXBuildFile section */
|
||||
@@ -156,7 +156,7 @@
|
||||
97C146E61CF9000F007C117D /* Project object */ = {
|
||||
isa = PBXProject;
|
||||
attributes = {
|
||||
LastUpgradeCheck = 1020;
|
||||
LastUpgradeCheck = 1300;
|
||||
ORGANIZATIONNAME = "";
|
||||
TargetAttributes = {
|
||||
97C146ED1CF9000F007C117D = {
|
||||
|
||||
+1
-1
@@ -1,6 +1,6 @@
|
||||
<?xml version="1.0" encoding="UTF-8"?>
|
||||
<Scheme
|
||||
LastUpgradeVersion = "1020"
|
||||
LastUpgradeVersion = "1300"
|
||||
version = "1.3">
|
||||
<BuildAction
|
||||
parallelizeBuildables = "YES"
|
||||
|
||||
@@ -1816,6 +1816,8 @@ class MessageInputState extends State<MessageInput> {
|
||||
|
||||
_mentionedUsers.clear();
|
||||
|
||||
message = _replaceUserNameWithId(message);
|
||||
|
||||
try {
|
||||
Future sendingFuture;
|
||||
if (widget.editMessage == null ||
|
||||
@@ -2081,3 +2083,21 @@ class _CountdownButton extends StatelessWidget {
|
||||
),
|
||||
);
|
||||
}
|
||||
|
||||
Message _replaceUserNameWithId(Message message) {
|
||||
final mentionedUsers = message.mentionedUsers;
|
||||
if (mentionedUsers.isEmpty) return message;
|
||||
|
||||
var messageTextToSend = message.text;
|
||||
if (messageTextToSend == null) return message;
|
||||
|
||||
for (final user in mentionedUsers.toSet()) {
|
||||
final userName = user.name;
|
||||
messageTextToSend = messageTextToSend!.replaceAll(
|
||||
'@$userName',
|
||||
'@${user.id}',
|
||||
);
|
||||
}
|
||||
|
||||
return message.copyWith(text: messageTextToSend);
|
||||
}
|
||||
|
||||
@@ -712,20 +712,21 @@ class _MessageListViewState extends State<MessageListView> {
|
||||
);
|
||||
},
|
||||
),
|
||||
BetterStreamBuilder<bool>(
|
||||
stream: streamChannel!.channel.state!.isUpToDateStream,
|
||||
initialData: streamChannel!.channel.state!.isUpToDate,
|
||||
builder: (context, snapshot) => ValueListenableBuilder<bool>(
|
||||
valueListenable: _showScrollToBottom,
|
||||
child: _buildScrollToBottom(),
|
||||
builder: (context, value, child) {
|
||||
if (!snapshot || value) {
|
||||
return child!;
|
||||
}
|
||||
return const Offstage();
|
||||
},
|
||||
if (widget.showScrollToBottom)
|
||||
BetterStreamBuilder<bool>(
|
||||
stream: streamChannel!.channel.state!.isUpToDateStream,
|
||||
initialData: streamChannel!.channel.state!.isUpToDate,
|
||||
builder: (context, snapshot) => ValueListenableBuilder<bool>(
|
||||
valueListenable: _showScrollToBottom,
|
||||
child: _buildScrollToBottom(),
|
||||
builder: (context, value, child) {
|
||||
if (!snapshot || value) {
|
||||
return child!;
|
||||
}
|
||||
return const Offstage();
|
||||
},
|
||||
),
|
||||
),
|
||||
),
|
||||
if (widget.showFloatingDateDivider)
|
||||
_buildFloatingDateDivider(itemCount),
|
||||
],
|
||||
|
||||
@@ -84,9 +84,10 @@ class MessageText extends StatelessWidget {
|
||||
String _replaceMentions(String text) {
|
||||
var messageTextToRender = text;
|
||||
for (final user in message.mentionedUsers.toSet()) {
|
||||
final userId = user.id;
|
||||
final userName = user.name;
|
||||
messageTextToRender = messageTextToRender.replaceAll(
|
||||
'@$userName',
|
||||
'@$userId',
|
||||
'[@$userName](@${userName.replaceAll(' ', '')})',
|
||||
);
|
||||
}
|
||||
|
||||
@@ -12,7 +12,7 @@ environment:
|
||||
dependencies:
|
||||
cached_network_image: ^3.0.0
|
||||
characters: ^1.1.0
|
||||
chewie: ^1.2.0
|
||||
chewie: ^1.3.0
|
||||
collection: ^1.15.0
|
||||
diacritic: ^0.1.3
|
||||
dio: ^4.0.0
|
||||
|
||||
Reference in New Issue
Block a user