TF-4186 Remove company server login info when switch to other login type

Author: dab246

lib/features/login/domain/state/remove_company_server_login_info_state.dart

@@ -0,0 +1,11 @@
+import 'package:core/presentation/state/failure.dart';
+import 'package:core/presentation/state/success.dart';
+
+class RemovingCompanyServerLoginInfo extends LoadingState {}
+
+class RemoveCompanyServerLoginInfoSuccess extends UIState {}
+
+class RemoveCompanyServerLoginInfoFailure extends FeatureFailure {
+  RemoveCompanyServerLoginInfoFailure(dynamic exception)
+      : super(exception: exception);
+}

lib/features/login/domain/usecases/remove_company_server_login_info_interactor.dart

@@ -0,0 +1,23 @@
+import 'dart:core';
+
+import 'package:core/presentation/state/failure.dart';
+import 'package:core/presentation/state/success.dart';
+import 'package:dartz/dartz.dart';
+import 'package:tmail_ui_user/features/login/domain/repository/company_server_login_repository.dart';
+import 'package:tmail_ui_user/features/login/domain/state/remove_company_server_login_info_state.dart';
+
+class RemoveCompanyServerLoginInfoInteractor {
+  final CompanyServerLoginRepository _serverLoginRepository;
+
+  RemoveCompanyServerLoginInfoInteractor(this._serverLoginRepository);
+
+  Stream<Either<Failure, Success>> execute() async* {
+    try {
+      yield Right(RemovingCompanyServerLoginInfo());
+      await _serverLoginRepository.removeCompanyServerLoginInfo();
+      yield Right(RemoveCompanyServerLoginInfoSuccess());
+    } catch (exception) {
+      yield Left(RemoveCompanyServerLoginInfoFailure(exception));
+    }
+  }
+}

lib/features/login/presentation/bindings/company_server_login_interactor_bindings.dart

@@ -7,6 +7,7 @@ import 'package:tmail_ui_user/features/login/data/local/company_server_login_cac
 import 'package:tmail_ui_user/features/login/data/repository/company_server_login_repository_impl.dart';
 import 'package:tmail_ui_user/features/login/domain/repository/company_server_login_repository.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_company_server_login_info_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/remove_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/main/exceptions/cache_exception_thrower.dart';
 
@@ -46,6 +47,11 @@ class CompanyServerLoginInteractorBindings extends InteractorsBindings {
         Get.find<CompanyServerLoginRepository>(),
       ),
     );
+    Get.put(
+      RemoveCompanyServerLoginInfoInteractor(
+        Get.find<CompanyServerLoginRepository>(),
+      ),
+    );
   }
 
   @override

lib/features/login/presentation/extensions/handle_company_server_login_info_extension.dart

@@ -30,4 +30,8 @@ extension HandleCompanyServerLoginInfoExtension on LoginController {
     onUsernameChange(userEmail);
     usernameInputController.text = userEmail;
   }
+
+  void removeCompanyServerLoginInfo() {
+    consumeState(removeCompanyServerLoginInfoInteractor.execute());
+  }
 }

lib/features/login/presentation/login_bindings.dart

@@ -24,6 +24,7 @@ import 'package:tmail_ui_user/features/login/domain/usecases/get_oidc_configurat
 import 'package:tmail_ui_user/features/login/domain/usecases/get_stored_oidc_configuration_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_token_oidc_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/remove_auth_destination_url_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/remove_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_login_url_on_mobile_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_login_username_on_mobile_interactor.dart';
@@ -58,6 +59,7 @@ class LoginBindings extends BaseBindings {
       Get.find<TryGuessingWebFingerInteractor>(),
       Get.find<RemoveAuthDestinationUrlInteractor>(),
       Get.find<SaveCompanyServerLoginInfoInteractor>(),
+      Get.find<RemoveCompanyServerLoginInfoInteractor>(),
     ));
   }
 

lib/features/login/presentation/login_controller.dart

@@ -47,6 +47,7 @@ import 'package:tmail_ui_user/features/login/domain/usecases/get_oidc_configurat
 import 'package:tmail_ui_user/features/login/domain/usecases/get_stored_oidc_configuration_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_token_oidc_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/remove_auth_destination_url_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/remove_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_login_url_on_mobile_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/save_login_username_on_mobile_interactor.dart';
@@ -86,6 +87,7 @@ class LoginController extends ReloadableController {
   final TryGuessingWebFingerInteractor _tryGuessingWebFingerInteractor;
   final RemoveAuthDestinationUrlInteractor _removeAuthDestinationUrlInteractor;
   final SaveCompanyServerLoginInfoInteractor saveCompanyServerLoginInfoInteractor;
+  final RemoveCompanyServerLoginInfoInteractor removeCompanyServerLoginInfoInteractor;
 
   final TextEditingController urlInputController = TextEditingController();
   final TextEditingController usernameInputController = TextEditingController();
@@ -120,6 +122,7 @@ class LoginController extends ReloadableController {
     this._tryGuessingWebFingerInteractor,
     this._removeAuthDestinationUrlInteractor,
     this.saveCompanyServerLoginInfoInteractor,
+    this.removeCompanyServerLoginInfoInteractor,
   );
 
   @override
@@ -471,6 +474,8 @@ class LoginController extends ReloadableController {
   void _getTokenOIDCSuccess(GetTokenOIDCSuccess success) {
     if (isDnsLookupFormOnMobile && _username != null) {
       saveCompanyServerLoginInfo(_username!.value);
+    } else if (PlatformInfo.isMobile) {
+      removeCompanyServerLoginInfo();
     }
     synchronizeTokenAndGetSession(
       baseUri: _currentBaseUrl!,
@@ -564,6 +569,10 @@ class LoginController extends ReloadableController {
   }
 
   void _handleCommonOIDCFailure() {
+    if (PlatformInfo.isMobile) {
+      removeCompanyServerLoginInfo();
+    }
+
     if (PlatformInfo.isMobile && loginFormType.value == LoginFormType.dnsLookupForm) {
       _showPasswordForm();
     } else {

test/features/base/base_controller_test.dart

@@ -20,6 +20,7 @@ import 'package:tmail_ui_user/features/caching/caching_manager.dart';
 import 'package:tmail_ui_user/features/login/data/network/interceptors/authorization_interceptors.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_authority_oidc_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_credential_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/get_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/manage_account/data/local/language_cache_manager.dart';
 import 'package:tmail_ui_user/features/manage_account/domain/usecases/log_out_oidc_interactor.dart';
 import 'package:tmail_ui_user/main/bindings/network/binding_tag.dart';
@@ -65,6 +66,7 @@ class SomeOtherException extends RemoteException {}
   MockSpec<DeleteCredentialInteractor>(),
   MockSpec<LogoutOidcInteractor>(),
   MockSpec<DeleteAuthorityOidcInteractor>(),
+  MockSpec<GetCompanyServerLoginInfoInteractor>(),
   MockSpec<AppToast>(),
   MockSpec<ImagePaths>(),
   MockSpec<ResponsiveUtils>(),
@@ -84,6 +86,7 @@ void main() {
   late MockDeleteCredentialInteractor mockDeleteCredentialInteractor;
   late MockLogoutOidcInteractor mockLogoutOidcInteractor;
   late MockDeleteAuthorityOidcInteractor mockDeleteAuthorityOidcInteractor;
+  late MockGetCompanyServerLoginInfoInteractor mockGetCompanyServerLoginInfoInteractor;
   late MockAppToast mockAppToast;
   late MockImagePaths mockImagePaths;
   late MockResponsiveUtils mockResponsiveUtils;
@@ -100,6 +103,7 @@ void main() {
     mockDeleteCredentialInteractor = MockDeleteCredentialInteractor();
     mockLogoutOidcInteractor = MockLogoutOidcInteractor();
     mockDeleteAuthorityOidcInteractor = MockDeleteAuthorityOidcInteractor();
+    mockGetCompanyServerLoginInfoInteractor = MockGetCompanyServerLoginInfoInteractor();
     mockAppToast = MockAppToast();
     mockImagePaths = MockImagePaths();
     mockResponsiveUtils = MockResponsiveUtils();
@@ -119,6 +123,7 @@ void main() {
     Get.put<DeleteCredentialInteractor>(mockDeleteCredentialInteractor);
     Get.put<LogoutOidcInteractor>(mockLogoutOidcInteractor);
     Get.put<DeleteAuthorityOidcInteractor>(mockDeleteAuthorityOidcInteractor);
+    Get.put<GetCompanyServerLoginInfoInteractor>(mockGetCompanyServerLoginInfoInteractor);
     Get.put<AppToast>(mockAppToast);
     Get.put<ImagePaths>(mockImagePaths);
     Get.put<ResponsiveUtils>(mockResponsiveUtils);

test/features/composer/presentation/composer_controller_test.dart

@@ -50,6 +50,7 @@ import 'package:tmail_ui_user/features/email/presentation/model/composer_argumen
 import 'package:tmail_ui_user/features/login/data/network/interceptors/authorization_interceptors.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_authority_oidc_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_credential_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/get_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/domain/usecases/remove_composer_cache_by_id_on_web_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/controller/mailbox_dashboard_controller.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/model/draggable_app_state.dart';
@@ -154,6 +155,7 @@ class MockMailboxDashBoardController extends Mock implements MailboxDashBoardCon
   MockSpec<DeleteCredentialInteractor>(),
   MockSpec<LogoutOidcInteractor>(),
   MockSpec<DeleteAuthorityOidcInteractor>(),
+  MockSpec<GetCompanyServerLoginInfoInteractor>(),
   MockSpec<AppToast>(),
   MockSpec<ImagePaths>(),
   MockSpec<ResponsiveUtils>(),
@@ -200,6 +202,7 @@ void main() {
   late MockDeleteCredentialInteractor mockDeleteCredentialInteractor;
   late MockLogoutOidcInteractor mockLogoutOidcInteractor;
   late MockDeleteAuthorityOidcInteractor mockDeleteAuthorityOidcInteractor;
+  late MockGetCompanyServerLoginInfoInteractor mockGetCompanyServerLoginInfoInteractor;
   late MockAppToast mockAppToast;
   late MockUuid mockUuid;
   late MockApplicationManager mockApplicationManager;
@@ -245,6 +248,7 @@ void main() {
     mockDeleteCredentialInteractor = MockDeleteCredentialInteractor();
     mockLogoutOidcInteractor = MockLogoutOidcInteractor();
     mockDeleteAuthorityOidcInteractor = MockDeleteAuthorityOidcInteractor();
+    mockGetCompanyServerLoginInfoInteractor = MockGetCompanyServerLoginInfoInteractor();
     mockAppToast = MockAppToast();
     mockUuid = MockUuid();
     mockApplicationManager = MockApplicationManager();
@@ -262,6 +266,7 @@ void main() {
     Get.put<DeleteCredentialInteractor>(mockDeleteCredentialInteractor);
     Get.put<LogoutOidcInteractor>(mockLogoutOidcInteractor);
     Get.put<DeleteAuthorityOidcInteractor>(mockDeleteAuthorityOidcInteractor);
+    Get.put<GetCompanyServerLoginInfoInteractor>(mockGetCompanyServerLoginInfoInteractor);
     Get.put<AppToast>(mockAppToast);
     Get.put<ImagePaths>(ImagePaths());
     Get.put<ResponsiveUtils>(ResponsiveUtils());

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

@@ -44,6 +44,7 @@ import 'package:tmail_ui_user/features/login/domain/usecases/delete_authority_oi
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_credential_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/action/download_ui_action.dart';
 import 'package:tmail_ui_user/features/download/presentation/controllers/download_controller.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/get_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/mailbox_dashboard/presentation/controller/mailbox_dashboard_controller.dart';
 import 'package:tmail_ui_user/features/manage_account/data/local/language_cache_manager.dart';
 import 'package:tmail_ui_user/features/manage_account/domain/usecases/get_all_identities_interactor.dart';
@@ -83,6 +84,7 @@ const fallbackGenerators = {
   MockSpec<DeleteCredentialInteractor>(),
   MockSpec<LogoutOidcInteractor>(),
   MockSpec<DeleteAuthorityOidcInteractor>(),
+  MockSpec<GetCompanyServerLoginInfoInteractor>(),
   MockSpec<AppToast>(),
   MockSpec<ImagePaths>(),
   MockSpec<ResponsiveUtils>(),
@@ -117,6 +119,7 @@ void main() {
   final deleteCredentialInteractor = MockDeleteCredentialInteractor();
   final logoutOidcInteractor = MockLogoutOidcInteractor();
   final deleteAuthorityOidcInteractor = MockDeleteAuthorityOidcInteractor();
+  final getCompanyServerLoginInfoInteractor = MockGetCompanyServerLoginInfoInteractor();
   final appToast = MockAppToast();
   final imagePaths = MockImagePaths();
   final responsiveUtils = MockResponsiveUtils();
@@ -154,6 +157,7 @@ void main() {
     Get.put<DeleteCredentialInteractor>(deleteCredentialInteractor);
     Get.put<LogoutOidcInteractor>(logoutOidcInteractor);
     Get.put<DeleteAuthorityOidcInteractor>(deleteAuthorityOidcInteractor);
+    Get.put<GetCompanyServerLoginInfoInteractor>(getCompanyServerLoginInfoInteractor);
     Get.put<AppToast>(appToast);
     Get.put<ImagePaths>(imagePaths);
     Get.put<ResponsiveUtils>(responsiveUtils);

test/features/home/presentation/home_controller_test.dart

@@ -24,6 +24,7 @@ import 'package:tmail_ui_user/features/login/domain/usecases/check_oidc_is_avail
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_authority_oidc_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/delete_credential_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_authenticated_account_interactor.dart';
+import 'package:tmail_ui_user/features/login/domain/usecases/get_company_server_login_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_oidc_configuration_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/get_oidc_user_info_interactor.dart';
 import 'package:tmail_ui_user/features/login/domain/usecases/remove_auth_destination_url_interactor.dart';
@@ -44,6 +45,7 @@ import 'home_controller_test.mocks.dart';
   MockSpec<DeleteCredentialInteractor>(),
   MockSpec<LogoutOidcInteractor>(),
   MockSpec<DeleteAuthorityOidcInteractor>(),
+  MockSpec<GetCompanyServerLoginInfoInteractor>(),
   MockSpec<AppToast>(),
   MockSpec<ImagePaths>(),
   MockSpec<ResponsiveUtils>(),
@@ -91,6 +93,7 @@ void main() {
   late MockDeleteCredentialInteractor mockDeleteCredentialInteractor;
   late MockLogoutOidcInteractor mockLogoutOidcInteractor;
   late MockDeleteAuthorityOidcInteractor mockDeleteAuthorityOidcInteractor;
+  late MockGetCompanyServerLoginInfoInteractor mockGetCompanyServerLoginInfoInteractor;
   late MockAppToast mockAppToast;
   late MockImagePaths mockImagePaths;
   late MockResponsiveUtils mockResponsiveUtils;
@@ -122,6 +125,7 @@ void main() {
     mockDeleteCredentialInteractor = MockDeleteCredentialInteractor();
     mockLogoutOidcInteractor = MockLogoutOidcInteractor();
     mockDeleteAuthorityOidcInteractor = MockDeleteAuthorityOidcInteractor();
+    mockGetCompanyServerLoginInfoInteractor = MockGetCompanyServerLoginInfoInteractor();
     mockAppToast = MockAppToast();
     mockImagePaths = MockImagePaths();
     mockResponsiveUtils = MockResponsiveUtils();
@@ -146,6 +150,7 @@ void main() {
     Get.put<DeleteCredentialInteractor>(mockDeleteCredentialInteractor);
     Get.put<LogoutOidcInteractor>(mockLogoutOidcInteractor);
     Get.put<DeleteAuthorityOidcInteractor>(mockDeleteAuthorityOidcInteractor);
+    Get.put<GetCompanyServerLoginInfoInteractor>(mockGetCompanyServerLoginInfoInteractor);
     Get.put<AppToast>(mockAppToast);
     Get.put<ImagePaths>(mockImagePaths);
     Get.put<ResponsiveUtils>(mockResponsiveUtils);