AYS-487 | Remove exception handling for AysUserNotSuperAdminException class

Author: suhakopan

src/main/java/org/ays/auth/util/exception/AysUserNotSuperAdminException.java

@@ -1,13 +1,14 @@
 package org.ays.auth.util.exception;
 
 import org.ays.common.util.exception.AysBadRequestException;
+import org.springframework.security.access.AccessDeniedException;
 
 import java.io.Serial;
 
 /**
  * Exception to be thrown when a user is not a super admin but tries to perform an action that requires super admin privileges.
  */
-public final class AysUserNotSuperAdminException extends AysBadRequestException {
+public final class AysUserNotSuperAdminException extends AccessDeniedException {
 
     /**
      * Unique identifier for serialization.

src/main/java/org/ays/common/util/exception/handler/GlobalExceptionHandler.java

@@ -164,17 +164,6 @@ AysErrorResponse handleAccessDeniedError(final AccessDeniedException exception)
                 .build();
     }
 
-    @ExceptionHandler(AysUserNotSuperAdminException.class)
-    @ResponseStatus(HttpStatus.FORBIDDEN)
-    AysErrorResponse handleAysUserNotSuperAdminError(final AysUserNotSuperAdminException exception) {
-        log.error(exception.getMessage(), exception);
-
-        return AysErrorResponse.builder()
-                .header(AysErrorResponse.Header.AUTH_ERROR.getName())
-                .message(exception.getMessage())
-                .build();
-    }
-
     @ExceptionHandler(SQLException.class)
     @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
     AysErrorResponse handleSQLError(final SQLException exception) {

src/test/java/org/ays/common/util/exception/handler/GlobalExceptionHandlerTest.java

@@ -6,7 +6,6 @@
 import jakarta.validation.constraints.NotNull;
 import lombok.extern.slf4j.Slf4j;
 import org.ays.AysRestControllerTest;
-import org.ays.auth.util.exception.AysUserNotSuperAdminException;
 import org.ays.common.model.response.AysErrorResponse;
 import org.ays.common.util.exception.AysAuthException;
 import org.ays.common.util.exception.AysBadRequestException;
@@ -15,7 +14,6 @@
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.Test;
 import org.mockito.InjectMocks;
-import org.mockito.Mockito;
 import org.springframework.core.MethodParameter;
 import org.springframework.dao.DataAccessException;
 import org.springframework.http.HttpInputMessage;
@@ -360,38 +358,4 @@ private void checkAysError(AysErrorResponse mockErrorResponse, AysErrorResponse
 
     }
 
-    @Test
-    void givenUserNotSuperAdmin_whenThrowAysUserNotSuperAdminException_thenReturnAysError() {
-        // Given
-        AysUserNotSuperAdminException mockException = Mockito.mock(AysUserNotSuperAdminException.class);
-        Mockito.when(mockException.getMessage()).thenReturn("User is not a super admin");
-
-        // When
-        AysErrorResponse mockErrorResponse = AysErrorResponse.builder()
-                .header(AysErrorResponse.Header.AUTH_ERROR.getName())
-                .message(mockException.getMessage())
-                .build();
-
-        // Then
-        AysErrorResponse errorResponse = globalExceptionHandler.handleAysUserNotSuperAdminError(mockException);
-        this.checkAysError(mockErrorResponse, errorResponse);
-    }
-
-    @Test
-    void givenUserNotSuperAdmin_whenThrowAysUserNotSuperAdminExceptionWithoutMessage_thenReturnAysError() {
-        // Given
-        AysUserNotSuperAdminException mockException = Mockito.mock(AysUserNotSuperAdminException.class);
-        Mockito.when(mockException.getMessage()).thenReturn(null);
-
-        // When
-        AysErrorResponse mockErrorResponse = AysErrorResponse.builder()
-                .header(AysErrorResponse.Header.AUTH_ERROR.getName())
-                .message(null)
-                .build();
-
-        // Then
-        AysErrorResponse errorResponse = globalExceptionHandler.handleAysUserNotSuperAdminError(mockException);
-        this.checkAysError(mockErrorResponse, errorResponse);
-    }
-
 }