scribejava · nikola20145 · 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,4 @@
+{
+    "java.configuration.updateBuildConfiguration": "automatic",
+    "liveServer.settings.port": 5501
+}
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/...s/src/main/java/com/github/scribejava/apis/facebook/FacebookAccessTokenErrorResponse.java b/...s/src/main/java/com/github/scribejava/apis/facebook/FacebookAccessTokenErrorResponse.java
@@ -4,7 +4,9 @@
 import com.github.scribejava.core.model.Response;
 import java.io.IOException;
 import java.util.Objects;
-
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
+import java.util.concurrent.atomic.AtomicBoolean;
 /**
  * non standard Facebook replace for {@link com.github.scribejava.core.model.OAuth2AccessTokenErrorResponse}
  *
@@ -25,6 +27,32 @@ public class FacebookAccessTokenErrorResponse extends OAuthResponseException {
     private final int codeInt;
     private final String fbtraceId;
 
+     // Branch coverage data structure Nikola
+    private static final ConcurrentHashMap<String, AtomicBoolean> branchCoverage = new ConcurrentHashMap<>();
+
+    static {
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_1", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_2", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_3", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_4", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_5", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_6", new AtomicBoolean(false));
+        branchCoverage.put("FacebookAccessTokenErrorResponse.equals.branch_7", new AtomicBoolean(false));
+
+
+        Runtime.getRuntime().addShutdownHook(new Thread(new Runnable() {
+            @Override
+            public void run() {
+                synchronized (System.out) {
+                    System.out.println("FacebookAccessTokenErrorResponse equals method coverage:");
+                    for (Map.Entry<String, AtomicBoolean> entry : branchCoverage.entrySet()) {
+                        System.out.println(entry.getKey() + ": " + (entry.getValue().get() ? "Taken" : "Not taken"));
+                    }
+                }
+            }
+        }));
+    }
+
     public FacebookAccessTokenErrorResponse(String errorMessage, String type, int code, String fbtraceId,
             Response response)
             throws IOException {
@@ -61,33 +89,39 @@ public int hashCode() {
         return hash;
     }
 
+    // branch coverage Nikola
     @Override
     public boolean equals(Object obj) {
         if (this == obj) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_1").set(true);
             return true;
         }
         if (obj == null) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_2").set(true);
             return false;
         }
         if (getClass() != obj.getClass()) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_3").set(true);
             return false;
         }
-        if (!super.equals(obj)) {
+        FacebookAccessTokenErrorResponse other = (FacebookAccessTokenErrorResponse) obj;
+        if (!Objects.equals(errorMessage, other.errorMessage)) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_4").set(true);
             return false;
         }
-
-        final FacebookAccessTokenErrorResponse other = (FacebookAccessTokenErrorResponse) obj;
-
-        if (!Objects.equals(errorMessage, other.getErrorMessage())) {
+        if (!Objects.equals(type, other.type)) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_5").set(true);
             return false;
         }
-        if (!Objects.equals(type, other.getType())) {
+        if (codeInt != other.codeInt) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_6").set(true);
             return false;
         }
-        if (codeInt != other.getCodeInt()) {
+        if (!Objects.equals(fbtraceId, other.fbtraceId)) {
+            branchCoverage.get("FacebookAccessTokenErrorResponse.equals.branch_7").set(true);
             return false;
         }
-        return Objects.equals(fbtraceId, other.getFbtraceId());
+        return true;
     }
 
     @Override

diff --git a/...c/test/java/com/github/scribejava/apis/facebook/FacebookAccessTokenErrorResponseTest.java b/...c/test/java/com/github/scribejava/apis/facebook/FacebookAccessTokenErrorResponseTest.java
@@ -0,0 +1,89 @@
+package com.github.scribejava.apis.facebook;
+
+import com.github.scribejava.core.model.Response;
+import java.io.IOException;
+import java.util.Collections;
+import static org.junit.Assert.*;
+import org.junit.Test;
+
+public class FacebookAccessTokenErrorResponseTest {
+
+    @Test
+    public void testEquals_Self() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID", response);
+
+        assertTrue("Should return true when comparing to itself.",
+            errorResponse.equals(errorResponse));
+    }
+
+    @Test
+    public void testEquals_NullObject() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID", response);
+
+        assertFalse("Should return false when comparing to null.",
+            errorResponse.equals(null));
+    }
+
+    @Test
+    public void testEquals_DifferentClass() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID", response);
+        Object otherObject = new Object();
+
+        assertFalse("Should return false when comparing different classes.",
+            errorResponse.equals(otherObject));
+    }
+
+    @Test
+    public void testEquals_DifferentErrorMessage() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse1 = new FacebookAccessTokenErrorResponse(
+            "Message1", "Type", 100, "TraceID", response);
+        FacebookAccessTokenErrorResponse errorResponse2 = new FacebookAccessTokenErrorResponse(
+            "Message2", "Type", 100, "TraceID", response);
+
+        assertFalse("Should return false when errorMessages are different.",
+            errorResponse1.equals(errorResponse2));
+    }
+
+    @Test
+    public void testEquals_DifferentType() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse1 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type1", 100, "TraceID", response);
+        FacebookAccessTokenErrorResponse errorResponse2 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type2", 100, "TraceID", response);
+
+        assertFalse("Should return false when types are different.",
+            errorResponse1.equals(errorResponse2));
+    }
+
+    @Test
+    public void testEquals_DifferentCode() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse1 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID", response);
+        FacebookAccessTokenErrorResponse errorResponse2 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 101, "TraceID", response);
+
+        assertFalse("Should return false when codes are different.",
+            errorResponse1.equals(errorResponse2));
+    }
+
+    @Test
+    public void testEquals_DifferentFbtraceId() throws IOException {
+        Response response = new Response(200, "OK", Collections.<String, String>emptyMap(), "Body content here");
+        FacebookAccessTokenErrorResponse errorResponse1 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID1", response);
+        FacebookAccessTokenErrorResponse errorResponse2 = new FacebookAccessTokenErrorResponse(
+            "Message", "Type", 100, "TraceID2", response);
+
+        assertFalse("Should return false when fbtraceIds are different.",
+            errorResponse1.equals(errorResponse2));
+    }
+}
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 {