Add unit tests

[NourAlPha]

Add unit tests for AdminService, AuthenticationService, CustomUserDetailsService, and TokenService

[Copilot]

Pull Request Overview

This PR adds comprehensive unit tests for AdminService, AuthenticationService, CustomUserDetailsService, and TokenService while introducing several refactorings and enhancements. Key changes include:

• Addition of new unit tests covering various success and failure cases.
• Enhanced input validations in account registration and refined role assignment handling.
• Updates to the token generation workflow using builder patterns, caching optimizations, and improvements in Docker container configurations.

Reviewed Changes

Copilot reviewed 18 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/test/java/com/podzilla/auth/service/CustomUserDetailsServiceTest.java	New tests validating user details loading and error scenarios
src/test/java/com/podzilla/auth/service/AuthenticationServiceTest.java	Extended tests for login, logout, and account registration flows
src/test/java/com/podzilla/auth/service/AdminServiceTest.java	Added tests for retrieving users from the admin service
src/main/java/com/podzilla/auth/service/TokenService.java	Refactored token generation and removal methods with builder use
src/main/java/com/podzilla/auth/service/CustomUserDetailsService.java	Introduced caching via @Cacheable for user details retrieval
src/main/java/com/podzilla/auth/service/AuthenticationService.java	Improved input validations and role assignment during account creation
src/main/java/com/podzilla/auth/security/SecurityConfig.java	Updated endpoint configuration and simplified security rules
src/main/java/com/podzilla/auth/security/JWTAuthenticationFilter.java	Added informational logging for successful JWT authentication
docker-compose.yml	Updated container names, image versions, and added Redis services

Files not reviewed (1)

• src/main/resources/application.properties: Language not supported

Comments suppressed due to low confidence (2)

src/main/java/com/podzilla/auth/service/TokenService.java:172

• Renaming the method to removeRefreshTokenFromCookieAndExpire introduces a behavior change by expiring the token. Please ensure that this modification is clearly documented in the service API and that any client usage is updated accordingly.

public void removeRefreshTokenFromCookieAndExpire(final HttpServletResponse response) { ...

src/main/java/com/podzilla/auth/service/CustomUserDetailsService.java:30

• The introduction of caching in loadUserByUsername optimizes performance. Please ensure that cache invalidation is properly handled when user details are updated to prevent stale data.

@Cacheable(value = "userDetails", key = "#email")