Refactor based on feedback

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

Author: gabriel-farache

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

@@ -37,14 +37,13 @@ private String getPassword() {
 
     @Override
     public void filter(ClientRequestContext requestContext) throws IOException {
-        String basicToken;
+        String basicToken = AuthUtils.basicAuthAccessTokenWithoutPrefix(getUsername(), getPassword());
+
         if (isTokenPropagation()) {
             LOGGER.warn("Token propagation enabled for BasicAuthentication");
-            basicToken = getTokenForPropagation(requestContext.getHeaders());
-            basicToken = sanitizeBasicToken(basicToken);
-        } else {
-            basicToken = AuthUtils.basicAuthAccessTokenWithoutPrefix(getUsername(), getPassword());
+            basicToken = sanitizeBasicToken(getTokenForPropagation(requestContext.getHeaders()));
         }
+
         if (!basicToken.isBlank()) {
             requestContext.getHeaders().add(HttpHeaders.AUTHORIZATION,
                     AuthUtils.basicAuthAccessToken(basicToken));

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

@@ -27,13 +27,12 @@ public BearerAuthenticationProvider(final String openApiSpecId, final String nam
 
     @Override
     public void filter(ClientRequestContext requestContext) throws IOException {
-        String bearerToken;
+        String bearerToken = getBearerToken();
+
         if (isTokenPropagation()) {
-            bearerToken = getTokenForPropagation(requestContext.getHeaders());
-            bearerToken = sanitizeBearerToken(bearerToken);
-        } else {
-            bearerToken = getBearerToken();
+            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

@@ -53,14 +53,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);