TF-4050 refactor: move attachment preview & download logic to download controller

test/features/email/domain/usecases/parse_email_by_blob_id_interactor_test.dart

@@ -5,28 +5,30 @@ import 'package:flutter_test/flutter_test.dart';
 import 'package:jmap_dart_client/jmap/core/id.dart';
 import 'package:mockito/annotations.dart';
 import 'package:mockito/mockito.dart';
-import 'package:tmail_ui_user/features/email/domain/repository/email_repository.dart';
+import 'package:model/extensions/session_extension.dart';
 import 'package:tmail_ui_user/features/email/domain/state/parse_email_by_blob_id_state.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/parse_email_by_blob_id_interactor.dart';
+import 'package:tmail_ui_user/features/mailbox_dashboard/domain/repository/download_repository.dart';
 
 import '../../../../fixtures/account_fixtures.dart';
 import '../../../../fixtures/email_fixtures.dart';
-
+import '../../../../fixtures/session_fixtures.dart';
 import 'parse_email_by_blob_id_interactor_test.mocks.dart';
 
-@GenerateNiceMocks([MockSpec<EmailRepository>()])
+@GenerateNiceMocks([MockSpec<DownloadRepository>()])
 void main() {
   group('ParseEmailByBlobIdInteractor::execute::', () {
-    late MockEmailRepository mockEmailRepository;
+    late MockDownloadRepository mockDownloadRepository;
     late ParseEmailByBlobIdInteractor parseEmailByBlobIdInteractor;
 
     setUp(() {
-      mockEmailRepository = MockEmailRepository();
+      mockDownloadRepository = MockDownloadRepository();
       parseEmailByBlobIdInteractor =
-          ParseEmailByBlobIdInteractor(mockEmailRepository);
+          ParseEmailByBlobIdInteractor(mockDownloadRepository);
     });
 
     final accountId = AccountFixtures.aliceAccountId;
+    final session = SessionFixtures.aliceSession;
     final blobId = Id('blob-id');
 
     test(
@@ -35,11 +37,16 @@ void main() {
       // Arrange
       final parsedEmail = EmailFixtures.email1;
 
-      when(mockEmailRepository.parseEmailByBlobIds(accountId, {blobId}))
+      when(mockDownloadRepository.parseEmailByBlobIds(accountId, {blobId}))
           .thenAnswer((_) async => [parsedEmail]);
 
       // Act
-      final result = parseEmailByBlobIdInteractor.execute(accountId, blobId);
+      final result = parseEmailByBlobIdInteractor.execute(
+        accountId,
+        session,
+        session.getOwnEmailAddressOrEmpty(),
+        blobId,
+      );
 
       // Assert
 
@@ -47,13 +54,19 @@ void main() {
         result,
         emitsInOrder([
           Right<Failure, Success>(ParsingEmailByBlobId()),
-          Right<Failure, Success>(ParseEmailByBlobIdSuccess(parsedEmail, blobId)),
+          Right<Failure, Success>(ParseEmailByBlobIdSuccess(
+            email: parsedEmail,
+            blobId: blobId,
+            accountId: accountId,
+            session: session,
+            ownEmailAddress: session.getOwnEmailAddressOrEmpty(),
+          )),
         ]),
       );
 
-      verify(mockEmailRepository.parseEmailByBlobIds(accountId, {blobId}))
+      verify(mockDownloadRepository.parseEmailByBlobIds(accountId, {blobId}))
           .called(1);
-      verifyNoMoreInteractions(mockEmailRepository);
+      verifyNoMoreInteractions(mockDownloadRepository);
     });
 
     test(
@@ -62,11 +75,16 @@ void main() {
       // Arrange
       final exception = Exception('Failed to parse email');
 
-      when(mockEmailRepository.parseEmailByBlobIds(accountId, {blobId}))
+      when(mockDownloadRepository.parseEmailByBlobIds(accountId, {blobId}))
           .thenThrow(exception);
 
       // Act
-      final result = parseEmailByBlobIdInteractor.execute(accountId, blobId);
+      final result = parseEmailByBlobIdInteractor.execute(
+        accountId,
+        session,
+        session.getOwnEmailAddressOrEmpty(),
+        blobId,
+      );
 
       // Assert
       await expectLater(
@@ -77,9 +95,9 @@ void main() {
         ]),
       );
 
-      verify(mockEmailRepository.parseEmailByBlobIds(accountId, {blobId}))
+      verify(mockDownloadRepository.parseEmailByBlobIds(accountId, {blobId}))
           .called(1);
-      verifyNoMoreInteractions(mockEmailRepository);
+      verifyNoMoreInteractions(mockDownloadRepository);
     });
   });
 }

test/features/email/presentation/controller/single_email_controller_test.dart

@@ -32,13 +32,10 @@ import 'package:tmail_ui_user/features/email/domain/usecases/calendar_event_reje
 import 'package:tmail_ui_user/features/email/domain/usecases/export_all_attachments_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/export_attachment_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/get_email_content_interactor.dart';
-import 'package:tmail_ui_user/features/email/domain/usecases/get_html_content_from_attachment_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_email_read_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/mark_as_star_email_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/maybe_calendar_event_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/parse_calendar_event_interactor.dart';
-import 'package:tmail_ui_user/features/email/domain/usecases/parse_email_by_blob_id_interactor.dart';
-import 'package:tmail_ui_user/features/email/domain/usecases/preview_email_from_eml_file_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/print_email_interactor.dart';
 import 'package:tmail_ui_user/features/email/domain/usecases/store_opened_email_interactor.dart';
 import 'package:tmail_ui_user/features/email/presentation/action/email_ui_action.dart';
@@ -97,14 +94,11 @@ const fallbackGenerators = {
   MockSpec<AcceptCalendarEventInteractor>(),
   MockSpec<MaybeCalendarEventInteractor>(),
   MockSpec<RejectCalendarEventInteractor>(),
-  MockSpec<ParseEmailByBlobIdInteractor>(),
-  MockSpec<PreviewEmailFromEmlFileInteractor>(),
   MockSpec<PrintUtils>(),
   MockSpec<ApplicationManager>(),
   MockSpec<ToastManager>(),
   MockSpec<CalendarEventDataSource>(),
   MockSpec<DioClient>(),
-  MockSpec<GetHtmlContentFromAttachmentInteractor>(),
   MockSpec<TwakeAppManager>(),
   MockSpec<ExportAllAttachmentsInteractor>(),
   MockSpec<FileUploader>(),
@@ -133,12 +127,9 @@ void main() {
   final responsiveUtils = MockResponsiveUtils();
   final uuid = MockUuid();
   final printEmailInteractor = MockPrintEmailInteractor();
-  final parseEmailByBlobIdInteractor = MockParseEmailByBlobIdInteractor();
-  final previewEmailFromEmlFileInteractor = MockPreviewEmailFromEmlFileInteractor();
   final printUtils = MockPrintUtils();
   final applicationManager = MockApplicationManager();
   final mockToastManager = MockToastManager();
-  final getHtmlContentFromAttachmentInteractor = MockGetHtmlContentFromAttachmentInteractor();
   final mockTwakeAppManager = MockTwakeAppManager();
 
   final exportAllAttachmentsInteractor = MockExportAllAttachmentsInteractor();
@@ -192,9 +183,6 @@ void main() {
       getAllIdentitiesInteractor,
       storeOpenedEmailInteractor,
       printEmailInteractor,
-      parseEmailByBlobIdInteractor,
-      previewEmailFromEmlFileInteractor,
-      getHtmlContentFromAttachmentInteractor,
       exportAllAttachmentsInteractor,
     );
   });