resolved codefu comments

Author: ievdokdm

dashboard/lib/service/firebase_auth.dart

@@ -77,18 +77,19 @@ class FirebaseAuthService extends ChangeNotifier {
     try {
       final userCredential = await _auth.signInWithPopup(GithubAuthProvider());
       _user = userCredential.user;
-    } catch (error) {
+    } on FirebaseAuthException catch (error) {
       // If email of Github account already registered in Frebase but with
       // Google provider, we need to sign in with Google provider first,
       // then link the GitHub provider to Google provider.
-      if (error is FirebaseAuthException &&
-          error.code == 'account-exists-with-different-credential') {
+      if (error.code == 'account-exists-with-different-credential') {
         debugPrint('google account exists, signing in with google');
         await _signInWithGoogle();
         await _linkWithGithub();
         return;
       }
       debugPrint('signin with github failed: $error');
+    } catch (error) {
+      debugPrint('signin with github failed: $error');
     }
   }
 
@@ -125,16 +126,16 @@ class FirebaseAuthService extends ChangeNotifier {
       _user = userCredential?.user;
       notifyListeners();
       await _auth.currentUser?.getIdToken(true);
-    } catch (error) {
-      // If Google account's credential already exists in firebase, we going to
-      // link github account to google.
-      if (error is FirebaseAuthException &&
-          error.code == 'credential-already-in-use') {
+    } on FirebaseAuthException catch (error) {
+      // If Github account's credential already exists in firebase, we going to
+      // link google account to github.
+      if (error.code == 'credential-already-in-use') {
         await _relinkGithubToGoogle();
         return;
       }
       debugPrint('linkWithGoogle failed: $error');
-      //
+    } catch (error) {
+      debugPrint('linkWithGoogle failed: $error');
     }
     // If linking google succeeded, we need to unlink it and relink
     // github to google to make google primary.

dashboard/lib/widgets/user_sign_in.dart

@@ -167,14 +167,13 @@ class UserSignIn extends StatelessWidget {
 }
 
 extension on String {
-  String getUserInitials() {
-    // Define the regular expression to split by space, dots,underscore, or dash
-    final splitter = RegExp(r'[ ._-]+');
+  static final RegExp _splitter = RegExp(r'[ ._-]+');
 
+  String getUserInitials() {
     final parts =
         split('@') // Split the email into local and domain parts
             .first // Take only the local part (before the '@' symbol)
-            .split(splitter); // Split string into a list of substrings
+            .split(_splitter); // Split string into a list of substrings
 
     // Extract the first character of each non-empty part and join them.
     final result = parts

dashboard/test/utils/fake_firebase_user.dart

@@ -10,7 +10,7 @@ class FakeFirebaseUser implements User {
     throw UnimplementedError();
   }
 
-  final _tokens = <String>[];
+  final tokens = <String>[];
   final _providerData = <UserInfo>[];
 
   @override
@@ -24,8 +24,8 @@ class FakeFirebaseUser implements User {
 
   @override
   Future<String?> getIdToken([bool forceRefresh = false]) async {
-    if (_tokens.isEmpty) return null;
-    return _tokens.removeAt(0);
+    if (tokens.isEmpty) return null;
+    return tokens.removeAt(0);
   }
 
   @override