fixup! TF-3348 Move accessToken from persistent storage to session storage for web

Author: tddang-linagora

lib/features/login/data/extensions/token_oidc_extension.dart

@@ -5,4 +5,8 @@ extension TokenOidcExtension on TokenOIDC {
   TokenOidcCache toTokenOidcCache() {
     return TokenOidcCache(token, tokenId.uuid, refreshToken, expiredTime: expiredTime);
   }
+
+  TokenOidcCache toTokenOidcCacheWithoutToken() {
+    return TokenOidcCache('', tokenId.uuid, refreshToken, expiredTime: expiredTime);
+  }
 }

lib/features/login/data/local/web_token_oidc_cache_manager.dart

@@ -2,6 +2,7 @@ import 'package:core/utils/app_logger.dart';
 import 'package:model/oidc/token_oidc.dart';
 import 'package:tmail_ui_user/features/caching/clients/token_oidc_cache_client.dart';
 import 'package:tmail_ui_user/features/login/data/extensions/token_oidc_cache_extension.dart';
+import 'package:tmail_ui_user/features/login/data/extensions/token_oidc_extension.dart';
 import 'package:tmail_ui_user/features/login/data/local/token_oidc_cache_manager.dart';
 import 'package:tmail_ui_user/features/login/data/model/token_oidc_cache.dart';
 import 'package:tmail_ui_user/features/login/domain/exceptions/authentication_exception.dart';
@@ -43,12 +44,7 @@ class WebTokenOidcCacheManager extends TokenOidcCacheManager {
     log('TokenOidcCacheManager::persistOneTokenOidc(): key: ${tokenOIDC.tokenId.uuid}');
     log('TokenOidcCacheManager::persistOneTokenOidc(): key\'s hash: ${tokenOIDC.tokenIdHash}');
     log('TokenOidcCacheManager::persistOneTokenOidc(): token: ${tokenOIDC.token}');
-    final tokenHiveCache = TokenOidcCache(
-      '',
-      tokenOIDC.tokenId.uuid,
-      tokenOIDC.refreshToken,
-      expiredTime: tokenOIDC.expiredTime,
-    );
+    final tokenHiveCache = tokenOIDC.toTokenOidcCacheWithoutToken();
     await _tokenOidcCacheClient.insertItem(tokenOIDC.tokenIdHash, tokenHiveCache);
     window.sessionStorage[_sessionStorageTokenKey] = tokenOIDC.token;
     log('TokenOidcCacheManager::persistOneTokenOidc(): done');

test/main/bindings/local/local_bindings_test.dart

@@ -0,0 +1,60 @@
+import 'package:core/utils/file_utils.dart';
+import 'package:core/utils/platform_info.dart';
+import 'package:flutter_secure_storage/flutter_secure_storage.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:get/instance_manager.dart';
+import 'package:mockito/annotations.dart';
+import 'package:shared_preferences/shared_preferences.dart';
+import 'package:tmail_ui_user/features/login/data/local/token_oidc_cache_manager.dart';
+import 'package:tmail_ui_user/features/login/data/local/web_token_oidc_cache_manager.dart';
+import 'package:tmail_ui_user/main/bindings/local/local_bindings.dart';
+
+import 'local_bindings_test.mocks.dart';
+
+@GenerateNiceMocks([
+  MockSpec<FlutterSecureStorage>(),
+  MockSpec<SharedPreferences>(),
+  MockSpec<FileUtils>(),
+])
+void main() {
+  late LocalBindings localBindings;
+
+  setUp(() {
+    localBindings = LocalBindings();
+    Get.put<FlutterSecureStorage>(MockFlutterSecureStorage());
+    Get.put<SharedPreferences>(MockSharedPreferences());
+    Get.put<FileUtils>(MockFileUtils());
+  });
+
+  group('local bindings test:', () {
+    test(
+      'should inject WebTokenOidcCacheManager '
+      'when platform is web',
+    () {
+      // arrange
+      PlatformInfo.isTestingForWeb = true;
+      localBindings.dependencies();
+      
+      // act
+      final cacheManager = Get.find<TokenOidcCacheManager>();
+      
+      // assert
+      expect(cacheManager, isInstanceOf<WebTokenOidcCacheManager>());
+      PlatformInfo.isTestingForWeb = false;
+    });
+
+    test(
+      'should inject TokenOidcCacheManager '
+      'when platform is not web (default)',
+    () {
+      // arrange
+      localBindings.dependencies();
+      
+      // act
+      final cacheManager = Get.find<TokenOidcCacheManager>();
+      
+      // assert
+      expect(cacheManager, isInstanceOf<TokenOidcCacheManager>());
+    });
+  });
+}