Refactor based on feedback and update tests acordingly

Co-authored-by: Francisco Javier Tirado Sarti <[email protected]>

Author: gabriel-farache

client/runtime/src/main/java/io/quarkiverse/openapi/generator/providers/BasicAuthenticationProvider.java

@@ -6,8 +6,6 @@
 import jakarta.ws.rs.client.ClientRequestContext;
 import jakarta.ws.rs.core.HttpHeaders;
 
-import io.quarkiverse.openapi.generator.OpenApiGeneratorException;
-import org.eclipse.microprofile.config.ConfigProvider;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
@@ -38,7 +36,8 @@ private String getPassword(ClientRequestContext requestContext) {
 
     @Override
     public void filter(ClientRequestContext requestContext) throws IOException {
-        String basicToken = AuthUtils.basicAuthAccessTokenWithoutPrefix(getUsername(requestContext), getPassword(requestContext));
+        String basicToken = AuthUtils.basicAuthAccessTokenWithoutPrefix(getUsername(requestContext),
+                getPassword(requestContext));
 
         if (isTokenPropagation()) {
             LOGGER.warn("Token propagation enabled for BasicAuthentication");
@@ -47,7 +46,7 @@ public void filter(ClientRequestContext requestContext) throws IOException {
 
         if (!basicToken.isBlank()) {
             requestContext.getHeaders().add(HttpHeaders.AUTHORIZATION,
-                AuthUtils.basicAuthAccessToken(basicToken));
+                    AuthUtils.basicAuthAccessToken(basicToken));
         }
 
     }

client/runtime/src/main/java/io/quarkiverse/openapi/generator/providers/BearerAuthenticationProvider.java

@@ -28,13 +28,12 @@ public BearerAuthenticationProvider(final String openApiSpecId, final String nam
 
     @Override
     public void filter(ClientRequestContext requestContext) throws IOException {
-        String bearerToken;
+        String bearerToken = getBearerToken(requestContext);
+
         if (isTokenPropagation()) {
-            bearerToken = getTokenForPropagation(requestContext.getHeaders());
-            bearerToken = sanitizeBearerToken(bearerToken);
-        } else {
-            bearerToken = getBearerToken(requestContext);
+            bearerToken = sanitizeBearerToken(getTokenForPropagation(requestContext.getHeaders()));
         }
+
         if (!bearerToken.isBlank()) {
             requestContext.getHeaders().add(HttpHeaders.AUTHORIZATION, AuthUtils.authTokenOrBearer(this.scheme, bearerToken));
         }

client/runtime/src/test/java/io/quarkiverse/openapi/generator/providers/BasicOpenApiSpecProviderTest.java

@@ -28,6 +28,9 @@ class BasicOpenApiSpecProviderTest extends AbstractOpenApiSpecProviderTest<Basic
     private static final String USER = "USER";
     private static final String PASSWORD = "PASSWORD";
 
+    private static final String USER_PROP = "username";
+    private static final String PASSWORD_PROP = "password";
+
     private static final String CUSTOM_SCHEMA = "custom_scheme";
     private static final String HEADER_NAME = "HEADER_NAME";
 
@@ -53,14 +56,9 @@ private void filter(String expectedAuthorizationHeader) throws IOException {
     @MethodSource("filterWithPropagationTestValues")
     void filterWithPropagation(String headerName,
             String expectedAuthorizationHeader) throws IOException {
-        String propagatedHeaderName;
-        if (headerName == null) {
-            propagatedHeaderName = propagationHeaderName(OPEN_API_FILE_SPEC_ID, AUTH_SCHEME_NAME,
-                    HttpHeaders.AUTHORIZATION);
-        } else {
-            propagatedHeaderName = propagationHeaderName(OPEN_API_FILE_SPEC_ID, AUTH_SCHEME_NAME,
-                    HEADER_NAME);
-        }
+        String propagatedHeaderName = headerName == null
+                ? propagationHeaderName(OPEN_API_FILE_SPEC_ID, AUTH_SCHEME_NAME, HttpHeaders.AUTHORIZATION)
+                : propagationHeaderName(OPEN_API_FILE_SPEC_ID, AUTH_SCHEME_NAME, HEADER_NAME);
         try (MockedStatic<ConfigProvider> configProviderMocked = Mockito.mockStatic(ConfigProvider.class)) {
             Config mockedConfig = Mockito.mock(Config.class);
             configProviderMocked.when(ConfigProvider::getConfig).thenReturn(mockedConfig);
@@ -69,7 +67,10 @@ void filterWithPropagation(String headerName,
                     Boolean.class)).thenReturn(Optional.of(true));
             when(mockedConfig.getOptionalValue(provider.getCanonicalAuthConfigPropertyName(AuthConfig.HEADER_NAME),
                     String.class)).thenReturn(Optional.of(headerName == null ? HttpHeaders.AUTHORIZATION : headerName));
-
+            when(mockedConfig.getOptionalValue(provider.getCanonicalAuthConfigPropertyName(USER_PROP),
+                    String.class)).thenReturn(Optional.of(USER));
+            when(mockedConfig.getOptionalValue(provider.getCanonicalAuthConfigPropertyName(PASSWORD_PROP),
+                    String.class)).thenReturn(Optional.of(PASSWORD));
             headers.putSingle(propagatedHeaderName, PROPAGATED_TOKEN);
             filter(expectedAuthorizationHeader);
         }

client/runtime/src/test/java/io/quarkiverse/openapi/generator/providers/BearerOpenApiSpecProviderTest.java

@@ -28,6 +28,8 @@ class BearerOpenApiSpecProviderTest extends AbstractOpenApiSpecProviderTest<Bear
     private static final String PROPAGATED_TOKEN = "PROPAGATED_TOKEN";
 
     private static final String BEARER_SCHEMA = "bearer";
+
+    private static final String BEARER_TOKEN = "bearer-token";
     private static final String CUSTOM_SCHEMA = "custom_scheme";
     private static final String HEADER_NAME = "HEADER_NAME";
 
@@ -88,6 +90,8 @@ void filterWithPropagation(String headerName,
                     Boolean.class)).thenReturn(Optional.of(true));
             when(mockedConfig.getOptionalValue(provider.getCanonicalAuthConfigPropertyName(AuthConfig.HEADER_NAME),
                     String.class)).thenReturn(Optional.of(headerName == null ? HttpHeaders.AUTHORIZATION : headerName));
+            when(mockedConfig.getOptionalValue(provider.getCanonicalAuthConfigPropertyName(BEARER_TOKEN),
+                    String.class)).thenReturn(Optional.of(INCOMING_TOKEN));
 
             headers.putSingle(propagatedHeaderName, PROPAGATED_TOKEN);
             filter(bearerScheme, expectedAuthorizationHeader);