TF-235 Implement rename a mailbox

lib/features/mailbox/data/datasource/mailbox_datasource.dart

@@ -5,6 +5,7 @@ import 'package:jmap_dart_client/jmap/mail/mailbox/mailbox.dart';
 import 'package:tmail_ui_user/features/mailbox/data/model/mailbox_change_response.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 
 abstract class MailboxDataSource {
   Future<MailboxResponse> getAllMailbox(AccountId accountId, {Properties? properties});
@@ -18,4 +19,6 @@ abstract class MailboxDataSource {
   Future<Mailbox?> createNewMailbox(AccountId accountId, CreateNewMailboxRequest newMailboxRequest);
 
   Future<bool> deleteMultipleMailbox(AccountId accountId, List<MailboxId> mailboxIds);
+
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request);
 }

lib/features/mailbox/data/datasource_impl/mailbox_cache_datasource_impl.dart

@@ -8,6 +8,7 @@ import 'package:tmail_ui_user/features/mailbox/data/local/mailbox_cache_manager.
 import 'package:tmail_ui_user/features/mailbox/data/extensions/mailbox_extension.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 
 class MailboxCacheDataSourceImpl extends MailboxDataSource {
 
@@ -54,4 +55,9 @@ class MailboxCacheDataSourceImpl extends MailboxDataSource {
   Future<bool> deleteMultipleMailbox(AccountId accountId, List<MailboxId> mailboxIds) {
     throw UnimplementedError();
   }
+
+  @override
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request) {
+    throw UnimplementedError();
+  }
 }

lib/features/mailbox/data/datasource_impl/mailbox_datasource_impl.dart

@@ -7,6 +7,7 @@ import 'package:tmail_ui_user/features/mailbox/data/model/mailbox_change_respons
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
 import 'package:tmail_ui_user/features/mailbox/data/network/mailbox_api.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 
 class MailboxDataSourceImpl extends MailboxDataSource {
 
@@ -59,4 +60,13 @@ class MailboxDataSourceImpl extends MailboxDataSource {
       throw error;
     });
   }
+
+  @override
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request) {
+    return Future.sync(() async {
+      return await mailboxAPI.renameMailbox(accountId, request);
+    }).catchError((error) {
+      throw error;
+    });
+  }
 }

lib/features/mailbox/data/network/mailbox_api.dart

@@ -2,6 +2,7 @@ import 'dart:async';
 
 import 'package:jmap_dart_client/http/http_client.dart';
 import 'package:jmap_dart_client/jmap/account_id.dart';
+import 'package:jmap_dart_client/jmap/core/patch_object.dart';
 import 'package:jmap_dart_client/jmap/core/properties/properties.dart';
 import 'package:jmap_dart_client/jmap/core/request/reference_path.dart';
 import 'package:jmap_dart_client/jmap/core/state.dart';
@@ -18,6 +19,7 @@ import 'package:model/model.dart';
 import 'package:tmail_ui_user/features/mailbox/data/model/mailbox_change_response.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 
 class MailboxAPI {
 
@@ -153,4 +155,28 @@ class MailboxAPI {
       throw error;
     });
   }
+
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request) async {
+    final setMailboxMethod = SetMailboxMethod(accountId)
+      ..addUpdates({request.mailboxId.id : PatchObject({'name' : request.newName.name})});
+
+    final requestBuilder = JmapRequestBuilder(httpClient, ProcessingInvocation());
+
+    final setMailboxInvocation = requestBuilder.invocation(setMailboxMethod);
+
+    final response = await (requestBuilder
+          ..usings(setMailboxMethod.requiredCapabilities))
+        .build()
+        .execute();
+
+    final setMailboxResponse = response.parse<SetMailboxResponse>(
+        setMailboxInvocation.methodCallId,
+        SetMailboxResponse.deserialize);
+
+    return Future.sync(() async {
+      return setMailboxResponse?.updated?.isNotEmpty == true;
+    }).catchError((error) {
+      throw error;
+    });
+  }
 }

lib/features/mailbox/data/repository/mailbox_repository_impl.dart

@@ -10,6 +10,7 @@ import 'package:tmail_ui_user/features/mailbox/data/extensions/state_extension.d
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
 import 'package:tmail_ui_user/features/mailbox/data/model/state_type.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/repository/mailbox_repository.dart';
 
 class MailboxRepositoryImpl extends MailboxRepository {
@@ -145,4 +146,9 @@ class MailboxRepositoryImpl extends MailboxRepository {
   Future<bool> deleteMultipleMailbox(AccountId accountId, List<MailboxId> mailboxIds) {
     return mapDataSource[DataSourceType.network]!.deleteMultipleMailbox(accountId, mailboxIds);
   }
+
+  @override
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request) {
+    return mapDataSource[DataSourceType.network]!.renameMailbox(accountId, request);
+  }
 }

lib/features/mailbox/domain/model/rename_mailbox_request.dart

@@ -0,0 +1,14 @@
+
+import 'package:equatable/equatable.dart';
+import 'package:jmap_dart_client/jmap/mail/mailbox/mailbox.dart';
+
+class RenameMailboxRequest with EquatableMixin {
+
+  final MailboxName newName;
+  final MailboxId mailboxId;
+
+  RenameMailboxRequest(this.mailboxId, this.newName);
+
+  @override
+  List<Object?> get props => [mailboxId, newName];
+}

lib/features/mailbox/domain/repository/mailbox_repository.dart

@@ -4,6 +4,7 @@ import 'package:jmap_dart_client/jmap/core/state.dart';
 import 'package:jmap_dart_client/jmap/mail/mailbox/mailbox.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/mailbox_response.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 
 abstract class MailboxRepository {
   Stream<MailboxResponse> getAllMailbox(AccountId accountId, {Properties? properties});
@@ -13,4 +14,6 @@ abstract class MailboxRepository {
   Future<Mailbox?> createNewMailbox(AccountId accountId, CreateNewMailboxRequest newMailboxRequest);
 
   Future<bool> deleteMultipleMailbox(AccountId accountId, List<MailboxId> mailboxIds);
+
+  Future<bool> renameMailbox(AccountId accountId, RenameMailboxRequest request);
 }

lib/features/mailbox/domain/state/rename_mailbox_state.dart

@@ -0,0 +1,18 @@
+import 'package:core/core.dart';
+
+class RenameMailboxSuccess extends UIState {
+
+  RenameMailboxSuccess();
+
+  @override
+  List<Object?> get props => [];
+}
+
+class RenameMailboxFailure extends FeatureFailure {
+  final exception;
+
+  RenameMailboxFailure(this.exception);
+
+  @override
+  List<Object> get props => [exception];
+}

lib/features/mailbox/domain/usecases/rename_mailbox_interactor.dart

@@ -0,0 +1,25 @@
+import 'package:core/core.dart';
+import 'package:dartz/dartz.dart';
+import 'package:jmap_dart_client/jmap/account_id.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/repository/mailbox_repository.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/state/rename_mailbox_state.dart';
+
+class RenameMailboxInteractor {
+  final MailboxRepository mailboxRepository;
+
+  RenameMailboxInteractor(this.mailboxRepository);
+
+  Stream<Either<Failure, Success>> execute(AccountId accountId, RenameMailboxRequest request) async* {
+    try {
+      final result = await mailboxRepository.renameMailbox(accountId, request);
+      if (result) {
+        yield Right<Failure, Success>(RenameMailboxSuccess());
+      } else {
+        yield Left<Failure, Success>(RenameMailboxFailure(null));
+      }
+    } catch (e) {
+      yield Left<Failure, Success>(RenameMailboxFailure(e));
+    }
+  }
+}

lib/features/mailbox/presentation/mailbox_bindings.dart

@@ -20,6 +20,7 @@ import 'package:tmail_ui_user/features/mailbox/domain/usecases/create_new_mailbo
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/delete_multiple_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/get_all_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/refresh_all_mailbox_interactor.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/usecases/rename_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/search_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/mailbox_controller.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_tree_builder.dart';
@@ -49,6 +50,7 @@ class MailboxBindings extends BaseBindings {
         Get.find<SearchMailboxInteractor>(),
         Get.find<DeleteMultipleMailboxInteractor>(),
         Get.find<VerifyNameInteractor>(),
+        Get.find<RenameMailboxInteractor>(),
         Get.find<TreeBuilder>(),
         Get.find<Uuid>(),
         Get.find<AppToast>(),
@@ -80,6 +82,7 @@ class MailboxBindings extends BaseBindings {
     Get.lazyPut(() => SearchMailboxInteractor());
     Get.lazyPut(() => DeleteMultipleMailboxInteractor(Get.find<MailboxRepository>()));
     Get.lazyPut(() => VerifyNameInteractor());
+    Get.lazyPut(() => RenameMailboxInteractor(Get.find<MailboxRepository>()));
   }
 
   @override

lib/features/mailbox/presentation/mailbox_controller.dart

@@ -15,14 +15,17 @@ import 'package:tmail_ui_user/features/composer/domain/state/update_email_drafts
 import 'package:tmail_ui_user/features/email/domain/state/mark_as_email_read_state.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_credential_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/model/create_new_mailbox_request.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/model/rename_mailbox_request.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/create_new_mailbox_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/delete_multiple_mailbox_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/get_all_mailboxes_state.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/state/rename_mailbox_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/state/search_mailbox_state.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/create_new_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/delete_multiple_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/get_all_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/refresh_all_mailbox_interactor.dart';
+import 'package:tmail_ui_user/features/mailbox/domain/usecases/rename_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/domain/usecases/search_mailbox_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_actions.dart';
 import 'package:tmail_ui_user/features/mailbox/presentation/model/mailbox_node.dart';
@@ -60,6 +63,7 @@ class MailboxController extends BaseController {
   final SearchMailboxInteractor _searchMailboxInteractor;
   final DeleteMultipleMailboxInteractor _deleteMultipleMailboxInteractor;
   final VerifyNameInteractor _verifyNameInteractor;
+  final RenameMailboxInteractor _renameMailboxInteractor;
   final TreeBuilder _treeBuilder;
   final Uuid _uuid;
   final AppToast _appToast;
@@ -89,6 +93,7 @@ class MailboxController extends BaseController {
     this._searchMailboxInteractor,
     this._deleteMultipleMailboxInteractor,
     this._verifyNameInteractor,
+    this._renameMailboxInteractor,
     this._treeBuilder,
     this._uuid,
     this._appToast,
@@ -171,6 +176,8 @@ class MailboxController extends BaseController {
             if (isSearchActive()) {
               _searchMailboxAction(allMailboxes, searchQuery.value);
             }
+          } else if (success is RenameMailboxSuccess) {
+            refreshMailboxChanges();
           }
         }
     );
@@ -615,7 +622,15 @@ class MailboxController extends BaseController {
   }
 
   void _renameMailboxAction(PresentationMailbox presentationMailbox, String newName) {
+    final accountId = mailboxDashBoardController.accountId.value;
 
+    if (accountId != null) {
+      consumeState(_renameMailboxInteractor.execute(
+          accountId,
+          RenameMailboxRequest(presentationMailbox.id, MailboxName(newName))));
+    }
+
+    _cancelSelectMailbox();
   }
 
   void closeMailboxScreen(BuildContext context) {