Replace AuthException with AuthSessionMissingException

Author: lawinski

packages/gotrue/lib/src/gotrue_client.dart

@@ -607,7 +607,7 @@ class GoTrueClient {
   /// If the current session's refresh token is invalid, an error will be thrown.
   Future<AuthResponse> refreshSession([String? refreshToken]) async {
     if (currentSession?.accessToken == null) {
-      throw AuthException('Not logged in.');
+      throw AuthSessionMissingException();
     }
 
     final currentSessionRefreshToken =
@@ -626,7 +626,7 @@ class GoTrueClient {
   Future<void> reauthenticate() async {
     final session = currentSession;
     if (session == null) {
-      throw AuthException('Not logged in.');
+      throw AuthSessionMissingException();
     }
 
     final options =
@@ -691,7 +691,7 @@ class GoTrueClient {
   /// Gets the current user details from current session or custom [jwt]
   Future<UserResponse> getUser([String? jwt]) async {
     if (jwt == null && currentSession?.accessToken == null) {
-      throw AuthException('Cannot get user: no current session.');
+      throw AuthSessionMissingException();
     }
     final options = GotrueRequestOptions(
       headers: _headers,
@@ -712,7 +712,7 @@ class GoTrueClient {
   }) async {
     final accessToken = currentSession?.accessToken;
     if (accessToken == null) {
-      throw AuthException('Not logged in.');
+      throw AuthSessionMissingException();
     }
 
     final body = attributes.toJson();
@@ -736,7 +736,7 @@ class GoTrueClient {
   /// Sets the session data from refresh_token and returns the current session.
   Future<AuthResponse> setSession(String refreshToken) async {
     if (refreshToken.isEmpty) {
-      throw AuthException('No current session.');
+      throw AuthSessionMissingException('Refresh token cannot be empty');
     }
     return await _callRefreshToken(refreshToken);
   }

packages/gotrue/lib/src/types/auth_exception.dart

@@ -27,13 +27,13 @@ class AuthException implements Exception {
 }
 
 class AuthPKCEGrantCodeExchangeError extends AuthException {
-  AuthPKCEGrantCodeExchangeError(String message) : super(message);
+  AuthPKCEGrantCodeExchangeError(super.message);
 }
 
 class AuthSessionMissingException extends AuthException {
-  AuthSessionMissingException()
+  AuthSessionMissingException([String? message])
       : super(
-          'Auth session missing!',
+          message ?? 'Auth session missing!',
           statusCode: '400',
           errorCode: AuthErrorCode.sessionNotFound.code,
         );
@@ -48,8 +48,7 @@ class AuthRetryableFetchException extends AuthException {
 }
 
 class AuthApiException extends AuthException {
-  AuthApiException(String message, {String? statusCode, String? errorCode})
-      : super(message, statusCode: statusCode, errorCode: errorCode);
+  AuthApiException(super.message, {super.statusCode, super.errorCode});
 }
 
 class AuthUnknownException extends AuthException {

packages/gotrue/test/client_test.dart

@@ -111,6 +111,8 @@ void main() {
       } on AuthException catch (error) {
         expect(error, isA<AuthWeakPasswordException>());
         expect(error.errorCode, AuthErrorCode.weakPassword.code);
+      } catch (error) {
+        fail('signUp threw ${error.runtimeType} instead of AuthException');
       }
     });
 
@@ -246,7 +248,7 @@ void main() {
       expect(user.appMetadata['provider'], 'email');
     });
 
-    test('signInWithPassword() with   phone', () async {
+    test('signInWithPassword() with phone', () async {
       final response =
           await client.signInWithPassword(phone: phone1, password: password);
       final data = response.session;
@@ -278,6 +280,17 @@ void main() {
       expect(newClient.currentSession?.accessToken ?? '', isNotEmpty);
     });
 
+    test(
+        'Set session with an empty refresh token throws AuthSessionMissingException',
+        () async {
+      try {
+        await client.setSession('');
+        fail('setSession did not throw');
+      } catch (error) {
+        expect(error, isA<AuthSessionMissingException>());
+      }
+    });
+
     test('Update user', () async {
       await client.signInWithPassword(email: email1, password: password);
 
@@ -298,11 +311,11 @@ void main() {
       expect(user?.userMetadata?['arabic'], 'عربى');
     });
 
-    test('Update user on with the same password throws AuthException',
-        () async {
+    test('Update user with the same password throws AuthException', () async {
       await client.signInWithPassword(email: email1, password: password);
       try {
         await client.updateUser(UserAttributes(password: password));
+        fail('updateUser did not throw');
       } on AuthException catch (error) {
         expect(error.errorCode, AuthErrorCode.samePassword.code);
       }