Refacto Twitch module (#93)

* WIP: Refacto Twitch module

* WIP: Refacto Twitch module

* WIP: Refacto Twitch module

* WIP: Refacto Twitch module

* Removed useless files

* Fix tests

Author: FunixG

Dockerfile

@@ -1,4 +1,4 @@
-FROM openjdk:21-slim
+FROM amazoncorretto:21-alpine
 
 ARG service_name
 ARG service_base_dir

modules/accounting/service/src/test/java/com/funixproductions/api/accounting/service/services/AccountingServiceTest.java

@@ -13,7 +13,7 @@
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 import java.util.List;
 import java.util.Random;
@@ -30,16 +30,16 @@ class AccountingServiceTest {
     @Autowired
     private AccountingService accountingService;
 
-    @MockBean
+    @MockitoBean
     private IncomeRepository incomeRepository;
 
-    @MockBean
+    @MockitoBean
     private ProductRepository productRepository;
 
-    @MockBean
+    @MockitoBean
     private GoogleGmailClient gmailClient;
 
-    @MockBean
+    @MockitoBean
     private BillingFeignInternalClient billingClient;
 
     @Test

modules/core/src/main/java/com/funixproductions/api/core/enums/FrontOrigins.java

@@ -4,14 +4,16 @@
 import lombok.Getter;
 import lombok.RequiredArgsConstructor;
 
+import javax.annotation.Nullable;
+
 @Getter
 @RequiredArgsConstructor
 public enum FrontOrigins {
    FUNIX_PRODUCTIONS_DASHBOARD(
            "funixproductions-dashboard",
            "Dashboard Funix Productions",
            "https://dashboard.funixproductions.com",
-           "http://localhost:4200"
+           "https://dev.dashboard.funixproductions.com"
    ),
    PACIFISTA_PUBLIC_WEB(
            "pacifista-public-web",
@@ -25,7 +27,8 @@ public enum FrontOrigins {
    private final String domainProd;
    private final String domainDev;
 
-   public static FrontOrigins getRedirectAuthOrigin(String origin) {
+   @Nullable
+   public static FrontOrigins getRedirectAuthOrigin(@Nullable String origin) {
        if (Strings.isNullOrEmpty(origin)) {
            return null;
        }

modules/google/reCaptcha/service/src/test/java/com/funixproductions/api/google/recaptcha/service/ressources/GoogleCaptchaResourceTest.java

@@ -9,7 +9,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 import org.springframework.test.web.servlet.MockMvc;
 
 import static org.mockito.ArgumentMatchers.anyString;
@@ -25,7 +25,7 @@ class GoogleCaptchaResourceTest {
     @Autowired
     private MockMvc mockMvc;
 
-    @MockBean
+    @MockitoBean
     private GoogleRecaptchaClient googleRecaptchaClient;
 
     @Test

modules/payment/billing/service/src/test/java/com/funixproductions/api/payment/billing/service/resources/BillingResourceTest.java

@@ -11,8 +11,8 @@
 import com.funixproductions.core.crud.dtos.PageDTO;
 import com.funixproductions.core.tools.pdf.tools.VATInformation;
 import org.junit.jupiter.api.BeforeEach;
-import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.core.io.ByteArrayResource;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 import java.util.Date;
 import java.util.List;
@@ -24,10 +24,10 @@
 
 public abstract class BillingResourceTest {
 
-    @MockBean
+    @MockitoBean
     protected UserAuthClient userAuthClient;
 
-    @MockBean
+    @MockitoBean
     protected BillingCrudService billingCrudService;
 
     protected final BillingDTO billingDTO = new BillingDTO();

modules/payment/billing/service/src/test/java/com/funixproductions/api/payment/billing/service/services/BillingCrudServiceTest.java

@@ -19,7 +19,7 @@
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 import java.io.File;
 import java.time.LocalDate;
@@ -41,10 +41,10 @@ class BillingCrudServiceTest {
     @Autowired
     private BillingRepository billingRepository;
 
-    @MockBean
+    @MockitoBean
     private CurrentSession currentSession;
 
-    @MockBean
+    @MockitoBean
     private GoogleGmailClient googleGmailClient;
 
     @BeforeEach

modules/payment/paypal/service/src/test/java/com/funixproductions/api/payment/paypal/service/auth/services/PaypalAccessTokenServiceTest.java

@@ -12,7 +12,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 import java.lang.reflect.Field;
 import java.nio.charset.StandardCharsets;
@@ -28,7 +28,7 @@
 @AutoConfigureMockMvc
 class PaypalAccessTokenServiceTest {
 
-    @MockBean
+    @MockitoBean
     private PaypalAuthClient paypalAuthClient;
 
     @Autowired

modules/payment/paypal/service/src/test/java/com/funixproductions/api/payment/paypal/service/orders/resources/PaypalOrderResourceTest.java

@@ -17,8 +17,8 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
 import org.springframework.http.MediaType;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.MvcResult;
 
@@ -40,13 +40,13 @@ class PaypalOrderResourceTest {
     @Autowired
     private MockMvc mockMvc;
 
-    @MockBean
+    @MockitoBean
     private PaypalOrderService service;
 
     @Autowired
     private JsonHelper jsonHelper;
 
-    @MockBean
+    @MockitoBean
     private BillingFeignInternalClient billingClient;
 
     @BeforeEach

modules/payment/paypal/service/src/test/java/com/funixproductions/api/payment/paypal/service/orders/services/PaypalOrderServiceTest.java

@@ -13,7 +13,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.boot.test.mock.mockito.MockBean;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
 
 import java.nio.charset.StandardCharsets;
 import java.util.HashMap;
@@ -26,10 +26,10 @@
 @AutoConfigureMockMvc
 class PaypalOrderServiceTest {
 
-    @MockBean
+    @MockitoBean
     private PaypalFeignOrderClient orderClient;
 
-    @MockBean
+    @MockitoBean
     private PaypalAccessTokenService tokenService;
 
     @Autowired

modules/twitch/auth/client/src/main/java/com/funixproductions/api/twitch/auth/client/clients/TwitchAuthClient.java

@@ -1,10 +1,13 @@
 package com.funixproductions.api.twitch.auth.client.clients;
 
 import com.funixproductions.api.twitch.auth.client.dtos.TwitchClientTokenDTO;
+import lombok.NonNull;
 import org.springframework.cloud.openfeign.FeignClient;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.RequestParam;
 
+import javax.annotation.Nullable;
+
 @FeignClient(
         name = "TwitchAuthClient",
         url = "${funixproductions.api.twitch.auth.app-domain-url}",
@@ -13,9 +16,14 @@
 public interface TwitchAuthClient {
 
     @GetMapping("clientAuthUrl")
-    String getAuthClientUrl(@RequestParam(defaultValue = "VIEWER") String tokenType);
+    String getAuthClientUrl(
+            @RequestParam(defaultValue = "VIEWER") @NonNull String tokenType,
+            @RequestParam(required = false) @Nullable String origin
+    );
 
     @GetMapping("accessToken")
-    TwitchClientTokenDTO getAccessToken();
+    TwitchClientTokenDTO getAccessToken(
+            @RequestParam(defaultValue = "VIEWER") @NonNull String tokenType
+    );
 
 }