TF-1190 Handle SetError when update email as drafts

lib/features/email/data/datasource/email_datasource.dart

@@ -54,7 +54,7 @@ abstract class EmailDataSource {
 
   Future<bool> removeEmailDrafts(AccountId accountId, EmailId emailId);
 
-  Future<Email?> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId);
+  Future<Email> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId);
 
   Future<List<EmailId>> deleteMultipleEmailsPermanently(AccountId accountId, List<EmailId> emailIds);
 

lib/features/email/data/datasource_impl/email_datasource_impl.dart

@@ -116,7 +116,7 @@ class EmailDataSourceImpl extends EmailDataSource {
   }
 
   @override
-  Future<Email?> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) {
+  Future<Email> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) {
     return Future.sync(() async {
       return await emailAPI.updateEmailDrafts(accountId, newEmail, oldEmailId);
     }).catchError((error) {

lib/features/email/data/network/email_api.dart

@@ -451,7 +451,7 @@ class EmailAPI with HandleSetErrorMixin {
     );
 
     final emailCreated = setEmailResponse?.created?[email.id.id];
-    final listEntriesErrors = _handleSetEmailResponse(response: setEmailResponse,);
+    final listEntriesErrors = _handleSetEmailResponse(response: setEmailResponse);
     final mapErrors = Map.fromEntries(listEntriesErrors);
 
     if (emailCreated != null && mapErrors.isEmpty) {
@@ -485,7 +485,7 @@ class EmailAPI with HandleSetErrorMixin {
     });
   }
 
-  Future<Email?> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) async {
+  Future<Email> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) async {
     final setEmailMethod = SetEmailMethod(accountId)
       ..addCreate(newEmail.id.id, newEmail)
       ..addDestroy({oldEmailId.id});
@@ -500,19 +500,20 @@ class EmailAPI with HandleSetErrorMixin {
       .execute();
 
     final setEmailResponse = response.parse<SetEmailResponse>(
-        setEmailInvocation.methodCallId,
-        SetEmailResponse.deserialize);
+      setEmailInvocation.methodCallId,
+      SetEmailResponse.deserialize
+    );
 
-    return Future.sync(() async {
-      final emailUpdated = setEmailResponse?.created?[newEmail.id.id];
-      final emailDestroyed = setEmailResponse?.destroyed?.contains(oldEmailId.id);
-      if (emailUpdated != null && emailDestroyed == true) {
-        return emailUpdated;
-      }
-      return null;
-    }).catchError((error) {
-      throw error;
-    });
+    final emailUpdated = setEmailResponse?.created?[newEmail.id.id];
+    final isEmailDestroyedSuccess = setEmailResponse?.destroyed?.contains(oldEmailId.id) ?? false;
+    final listEntriesErrors = _handleSetEmailResponse(response: setEmailResponse);
+    final mapErrors = Map.fromEntries(listEntriesErrors);
+
+    if (emailUpdated != null && isEmailDestroyedSuccess && mapErrors.isEmpty) {
+      return emailUpdated;
+    } else {
+      throw SetEmailMethodException(mapErrors);
+    }
   }
 
   Future<List<EmailId>> deleteMultipleEmailsPermanently(AccountId accountId, List<EmailId> emailIds) async {

lib/features/email/data/repository/email_repository_impl.dart

@@ -114,7 +114,7 @@ class EmailRepositoryImpl extends EmailRepository {
   }
 
   @override
-  Future<Email?> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) {
+  Future<Email> updateEmailDrafts(AccountId accountId, Email newEmail, EmailId oldEmailId) {
     return emailDataSource.updateEmailDrafts(accountId, newEmail, oldEmailId);
   }