Fixed codacy and CI errors.

Author: suratdas

.github/workflows/gradle.yml

@@ -17,10 +17,10 @@ jobs:
     steps:
     - uses: actions/checkout@v2
 
-    - name: Set up JDK 1.8
+    - name: Set up JDK 11
       uses: actions/setup-java@v1
       with:
-        java-version: 1.8
+        java-version: 11
 
     - name: Grant execute permission for gradlew
       run: chmod +x gradlew

src/main/java/io/visual_regression_tracker/sdk_java/VisualRegressionTracker.java

@@ -15,6 +15,7 @@
 import java.time.Duration;
 
 enum METHOD {
+    GET,
     POST,
     PATCH
 }
@@ -141,17 +142,7 @@ private HttpResponse<String> getResponse(METHOD method, String url, HttpRequest.
                 .header(API_KEY_HEADER, configuration.getApiKey())
                 .header("Content-Type", "application/json;charset=UTF-8")
                 .uri(URI.create(url));
-        HttpRequest request;
-        switch (method) {
-            case PATCH:
-                request = requestBuilder.method("PATCH", body).build();
-                break;
-            case POST:
-                request = requestBuilder.POST(body).build();
-                break;
-            default:
-                throw new IllegalStateException("Not implemented: " + method);
-        }
+        HttpRequest request = getRequest(method, body, requestBuilder);
         HttpResponse<String> response = HttpClient.newBuilder()
                 .version(HttpClient.Version.HTTP_1_1)
                 .connectTimeout(Duration.ofSeconds(configuration.getHttpTimeoutInSeconds()))
@@ -160,6 +151,16 @@ private HttpResponse<String> getResponse(METHOD method, String url, HttpRequest.
         return response;
     }
 
+    protected HttpRequest getRequest(METHOD method, HttpRequest.BodyPublisher body, HttpRequest.Builder requestBuilder) {
+        switch (method) {
+            case PATCH:
+                return requestBuilder.method("PATCH", body).build();
+            case POST:
+                return requestBuilder.POST(body).build();
+        }
+        throw new UnsupportedOperationException("This method is not yet supported.");
+    }
+
     protected <T> T handleResponse(HttpResponse<String> response, Class<T> classOfT) {
         String responseBody = response.body();
         if (!String.valueOf(response.statusCode()).startsWith("2")) {

src/test/java/io/visual_regression_tracker/sdk_java/VisualRegressionTrackerTest.java

@@ -21,6 +21,7 @@
 
 import java.io.IOException;
 import java.net.http.HttpClient;
+import java.net.http.HttpRequest;
 import java.net.http.HttpResponse;
 import java.net.http.HttpTimeoutException;
 import java.util.Arrays;
@@ -32,7 +33,12 @@
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyString;
-import static org.mockito.Mockito.*;
+import static org.mockito.Mockito.doCallRealMethod;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.reset;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
 
 public class VisualRegressionTrackerTest {
 
@@ -355,6 +361,11 @@ public void httpTimeoutWorks() throws IOException, InterruptedException {
         assertThat(vrt.projectId, is(PROJECT_ID));
     }
 
+    @Test(expectedExceptions = UnsupportedOperationException.class, expectedExceptionsMessageRegExp = "This method is not yet supported.")
+    public void methodNotSupported() {
+        vrt.getRequest(METHOD.GET, null, null);
+    }
+
     @Test(expectedExceptions = HttpTimeoutException.class,
             expectedExceptionsMessageRegExp = "^(request timed out)$")
     public void httpTimeoutElapsed() throws IOException, InterruptedException {