scribejava · jayran-d · Jun 8, 2024 · Jun 8, 2024 · Jun 8, 2024 · Jun 8, 2024
diff --git a/.gitattributes b/.gitattributes
@@ -0,0 +1 @@
+*.java text eol=lf
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -0,0 +1,3 @@
+{
+    "java.configuration.updateBuildConfiguration": "automatic"
+}
diff --git a/README.md b/README.md
diff --git a/README_original.md b/README_original.md
diff --git a/image-1.png b/image-1.png
diff --git a/image.png b/image.png
diff --git a/pom.xml b/pom.xml
@@ -63,6 +63,11 @@
             <version>4.13.2</version>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>com.github.scribejava</groupId>
+            <artifactId>scribejava-apis</artifactId>
+            <version>8.3.3</version>
+        </dependency>
         <dependency>
             <groupId>com.squareup.okhttp3</groupId>
             <artifactId>mockwebserver</artifactId>
@@ -87,6 +92,64 @@
                         </execution>
                     </executions>
                 </plugin>
+                <!-- <plugin>
+                    <groupId>org.jacoco</groupId>
+                    <artifactId>jacoco-maven-plugin</artifactId>
+                    <version>0.8.12</version>
+                    <executions>
+                        <execution>
+                            <goals>
+                                <goal>prepare-agent</goal>
+                            </goals>
+                        </execution>
+                        <execution>
+                            <id>report</id>
+                            <phase>test</phase>
+                            <goals>
+                                <goal>report</goal>
+                            </goals>
+                        </execution>
+                    </executions>
+                </plugin> -->
+                <plugin>
+                    <groupId>org.jacoco</groupId>
+                    <artifactId>jacoco-maven-plugin</artifactId>
+                    <version>0.8.12</version>
+                    <executions>
+                        <execution>
+                            <id>default-prepare-agent</id>
+                            <goals>
+                                <goal>prepare-agent</goal>
+                            </goals>
+                        </execution>
+                        <execution>
+                            <id>default-report</id>
+                            <goals>
+                                <goal>report</goal>
+                            </goals>
+                        </execution>
+                        <execution>
+                            <id>default-check</id>
+                            <goals>
+                                <goal>check</goal>
+                            </goals>
+                            <configuration>
+                                <rules>
+                                    <rule>
+                                        <element>BUNDLE</element>
+                                            <limits>
+                                                <limit>
+                                                    <counter>COMPLEXITY</counter>
+                                                    <value>COVEREDRATIO</value>
+                                                    <minimum>0.60</minimum>
+                                                </limit>
+                                            </limits>
+                                        </rule>
+                                </rules>
+                            </configuration>
+                        </execution>
+                    </executions>
+                </plugin>
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-jar-plugin</artifactId>
@@ -97,7 +160,7 @@
                         </archive>
                     </configuration>
                 </plugin>
-                <plugin>
+                <!-- <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-checkstyle-plugin</artifactId>
                     <version>3.2.0</version>
@@ -108,7 +171,7 @@
                             <version>10.4</version>
                         </dependency>
                     </dependencies>
-                </plugin>
+                </plugin> -->
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-release-plugin</artifactId>
@@ -200,7 +263,7 @@
                     </execution>
                 </executions>
             </plugin>
-            <plugin>
+            <!-- <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-checkstyle-plugin</artifactId>
                 <executions>
@@ -220,8 +283,8 @@
                         </goals>
                     </execution>
                 </executions>
-            </plugin>
-            <plugin>
+            </plugin> -->
+            <!-- <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-pmd-plugin</artifactId>
                 <version>3.19.0</version>
@@ -264,7 +327,7 @@
                         </goals>
                     </execution>
                 </executions>
-            </plugin>
+            </plugin> -->
         </plugins>
     </build>
 

diff --git a/...client-apache/src/main/java/com/github/scribejava/httpclient/apache/ApacheHttpClient.java b/...client-apache/src/main/java/com/github/scribejava/httpclient/apache/ApacheHttpClient.java
@@ -100,7 +100,8 @@ private <T> Future<T> doExecuteAsync(String userAgent, Map<String, String> heade
         final Future<HttpResponse> future = client.execute(builder.build(), handler);
         return new ApacheHttpFuture<>(future, handler);
     }
-
+
+    //branch coverage: Tomas
     private static RequestBuilder getRequestBuilder(Verb httpVerb) {
         switch (httpVerb) {
             case GET:

diff --git a/...he/src/main/java/com/github/scribejava/httpclient/apache/OAuthAsyncCompletionHandler.java b/...he/src/main/java/com/github/scribejava/httpclient/apache/OAuthAsyncCompletionHandler.java
@@ -34,6 +34,7 @@ public OAuthAsyncCompletionHandler(OAuthAsyncRequestCallback<T> callback, Respon
         this.latch = new CountDownLatch(1);
     }
 
+    //branch coverage Tomas
     @Override
     public void completed(HttpResponse httpResponse) {
         try {

diff --git a/...ent-armeria/src/main/java/com/github/scribejava/httpclient/armeria/ArmeriaHttpClient.java b/...ent-armeria/src/main/java/com/github/scribejava/httpclient/armeria/ArmeriaHttpClient.java
@@ -44,7 +44,8 @@ public class ArmeriaHttpClient extends AbstractAsyncOnlyHttpClient {
      */
     private final ArmeriaWebClientBuilder clientBuilder;
     /**
-     * A list of cached Endpoints. It helps avoiding building a new Endpoint per each request.
+     * A list of cached Endpoints. It helps avoiding building a new Endpoint per
+     * each request.
      */
     private final Map<String, WebClient> httpClients = new HashMap<>();
     /**
@@ -149,7 +150,8 @@ private <T> CompletableFuture<T> doExecuteAsync(String userAgent, Map<String, St
     }
 
     /**
-     * Provides an instance of {@link WebClient} for a given endpoint {@link URI} based on an endpoint as
+     * Provides an instance of {@link WebClient} for a given endpoint {@link URI}
+     * based on an endpoint as
      * {@code scheme://authority}.
      *
      * @param uri an endpoint {@link URI}
@@ -192,56 +194,102 @@ private WebClient getClient(URI uri) {
     /**
      * Extracts {@code scheme} and {@code authority} portion of the {@link URI}.
      *
-     * Assuming the {@link URI} as the following: {@code URI = scheme:[//authority]path[?query][#fragment]}
+     * Assuming the {@link URI} as the following:
+     * {@code URI = scheme:[//authority]path[?query][#fragment]}
      */
     private static String getEndPoint(URI uri) {
         return requireNonNull(uri.getScheme(), "scheme") + "://" + requireNonNull(uri.getAuthority(), "authority");
     }
 
+    private static boolean[] getServiceCaseCoverage = new boolean[2]; // Data structure
+
     /**
-     * Extracts {@code path}, {@code query} and {@code fragment} portion of the {@link URI}.
+     * Extracts {@code path}, {@code query} and {@code fragment} portion of the
+     * {@link URI}.
      *
-     * Assuming the {@link URI} as the following: {@code URI = scheme:[//authority]path[?query][#fragment]}
+     * Assuming the {@link URI} as the following:
+     * {@code URI = scheme:[//authority]path[?query][#fragment]}
      */
-    private static String getServicePath(URI uri) {
+    public static String getServicePath(URI uri) {
         final StringBuilder builder = new StringBuilder()
                 .append(requireNonNull(uri.getPath(), "path"));
         final String query = uri.getQuery();
         if (query != null) {
+            getServiceCaseCoverage[0] = true; // ID: ArmeriaHttpClient.getServicePath.branch_1
             builder.append('?').append(query);
         }
         final String fragment = uri.getFragment();
         if (fragment != null) {
+            getServiceCaseCoverage[1] = true; // ID: ArmeriaHttpClient.getServicePath.branch_2
             builder.append('#').append(fragment);
         }
         return builder.toString();
     }
 
+    private static void printCoverage() {
+        System.out.println("GetHttpMethod Coverage report:");
+        for (int i = 0; i < getHttpCaseCoverage.length; i++) {
+            System.out.println("ArmeriaHttpClient.getHttpMethod.branch_" + (i + 1) + ": "
+                    + getHttpCaseCoverage[i]);
+        }
+
+        System.out.println("GetService Coverage report:");
+        for (int i = 0; i < getServiceCaseCoverage.length; i++) {
+            System.out.println("ArmeriaHttpClient.getServicePath.branch_" + (i + 1) + ":" + getServiceCaseCoverage[i]);
+        }
+    }
+
+    static {
+        // Ensure the coverage report is printed when the class is loaded
+        Runtime.getRuntime().addShutdownHook(new Thread(ArmeriaHttpClient::printCoverage));
+    }
+
     /**
      * Maps {@link Verb} to {@link HttpMethod}
      *
      * @param httpVerb a {@link Verb} to match with {@link HttpMethod}
      * @return {@link HttpMethod} corresponding to the parameter
      */
-    private static HttpMethod getHttpMethod(Verb httpVerb) {
+
+    private static boolean[] getHttpCaseCoverage = new boolean[9];
+
+    public static HttpMethod getHttpMethod(Verb httpVerb) {
         switch (httpVerb) {
             case GET:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_1
+                getHttpCaseCoverage[0] = true;
                 return HttpMethod.GET;
             case POST:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_2
+                getHttpCaseCoverage[1] = true;
                 return HttpMethod.POST;
             case PUT:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_3
+                getHttpCaseCoverage[2] = true;
                 return HttpMethod.PUT;
             case DELETE:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_4
+                getHttpCaseCoverage[3] = true;
                 return HttpMethod.DELETE;
             case HEAD:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_5
+                getHttpCaseCoverage[4] = true;
                 return HttpMethod.HEAD;
             case OPTIONS:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_6
+                getHttpCaseCoverage[5] = true;
                 return HttpMethod.OPTIONS;
             case TRACE:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_7
+                getHttpCaseCoverage[6] = true;
                 return HttpMethod.TRACE;
             case PATCH:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_8
+                getHttpCaseCoverage[7] = true;
                 return HttpMethod.PATCH;
             default:
+                // ID: ArmeriaHttpClient.getHttpMethod.branch_9
+                getHttpCaseCoverage[8] = true;
                 throw new IllegalArgumentException(
                         "message build error: unsupported HTTP method: " + httpVerb.name());
         }
@@ -251,7 +299,8 @@ private static HttpMethod getHttpMethod(Verb httpVerb) {
     /**
      * Converts {@link AggregatedHttpResponse} to {@link Response}
      *
-     * @param aggregatedResponse an instance of {@link AggregatedHttpResponse} to convert to {@link Response}
+     * @param aggregatedResponse an instance of {@link AggregatedHttpResponse} to
+     *                           convert to {@link Response}
      * @return a {@link Response} converted from {@link AggregatedHttpResponse}
      */
     private Response convertResponse(AggregatedHttpResponse aggregatedResponse) {
@@ -265,14 +314,19 @@ private Response convertResponse(AggregatedHttpResponse aggregatedResponse) {
     }
 
     /**
-     * Converts {@link AggregatedHttpResponse} to {@link Response} upon its aggregation completion and invokes
+     * Converts {@link AggregatedHttpResponse} to {@link Response} upon its
+     * aggregation completion and invokes
      * {@link OAuthAsyncRequestCallback} for it.
      *
-     * @param callback a {@link OAuthAsyncRequestCallback} callback to invoke upon response completion
-     * @param converter an optional {@link OAuthRequest.ResponseConverter} result converter for {@link Response}
+     * @param callback           a {@link OAuthAsyncRequestCallback} callback to
+     *                           invoke upon response completion
+     * @param converter          an optional {@link OAuthRequest.ResponseConverter}
+     *                           result converter for {@link Response}
      * @param aggregatedResponse a source {@link AggregatedHttpResponse} to handle
-     * @param <T> converter {@link OAuthRequest.ResponseConverter} specific type or {@link Response}
-     * @return either instance of {@link Response} or converted result based on {@link OAuthRequest.ResponseConverter}
+     * @param <T>                converter {@link OAuthRequest.ResponseConverter}
+     *                           specific type or {@link Response}
+     * @return either instance of {@link Response} or converted result based on
+     *         {@link OAuthRequest.ResponseConverter}
      */
     private <T> T whenResponseComplete(OAuthAsyncRequestCallback<T> callback,
             OAuthRequest.ResponseConverter<T> converter, AggregatedHttpResponse aggregatedResponse) {
@@ -292,9 +346,11 @@ private <T> T whenResponseComplete(OAuthAsyncRequestCallback<T> callback,
     /**
      * Invokes {@link OAuthAsyncRequestCallback} upon {@link Throwable} error result
      *
-     * @param callback a {@link OAuthAsyncRequestCallback} callback to invoke upon response completion
+     * @param callback  a {@link OAuthAsyncRequestCallback} callback to invoke upon
+     *                  response completion
      * @param throwable a {@link Throwable} error result
-     * @param <T> converter {@link OAuthRequest.ResponseConverter} specific type or {@link Response}
+     * @param <T>       converter {@link OAuthRequest.ResponseConverter} specific
+     *                  type or {@link Response}
      * @return null
      */
     private <T> T completeExceptionally(OAuthAsyncRequestCallback<T> callback, Throwable throwable) {