fix valid response check

Author: sebastianMindee

src/main/java/com/mindee/http/MindeeApiCommon.java

@@ -34,12 +34,12 @@ protected String getUserAgent() {
   }
 
   /**
-   * Checks if the status code is in the 2xx range.
+   * Checks if the status code is out of the 2xx-3xx range.
    * @param statusCode the status code to check.
    * @return {@code true} if the status code is in the 2xx range, false otherwise.
    */
-  protected boolean is2xxStatusCode(int statusCode) {
-    return statusCode >= 200 && statusCode <= 299;
+  protected boolean isInvalidStatusCode(int statusCode) {
+    return statusCode < 200 || statusCode > 399;
   }
 
   protected String readRawResponse(HttpEntity responseEntity) throws IOException {

src/main/java/com/mindee/http/MindeeHttpApi.java

@@ -172,7 +172,7 @@ public <DocT extends Inference> AsyncPredictResponse<DocT> documentQueueGet(
           get, response -> {
             HttpEntity responseEntity = response.getEntity();
             int statusCode = response.getCode();
-            if (!is2xxStatusCode(statusCode)) {
+            if (isInvalidStatusCode(statusCode)) {
               throw getHttpError(parametricType, response);
             }
             String rawResponse = readRawResponse(responseEntity);
@@ -222,7 +222,7 @@ public <DocT extends Inference> PredictResponse<DocT> predictPost(
           post, response -> {
             HttpEntity responseEntity = response.getEntity();
             int statusCode = response.getCode();
-            if (!is2xxStatusCode(statusCode)) {
+            if (isInvalidStatusCode(statusCode)) {
               throw getHttpError(parametricType, response);
             }
             if (responseEntity.getContentLength() == 0) {
@@ -267,7 +267,7 @@ public <DocT extends Inference> AsyncPredictResponse<DocT> predictAsyncPost(
           post, response -> {
             HttpEntity responseEntity = response.getEntity();
             int statusCode = response.getCode();
-            if (!is2xxStatusCode(statusCode)) {
+            if (isInvalidStatusCode(statusCode)) {
               throw getHttpError(parametricType, response);
             }
             if (responseEntity.getContentLength() == 0) {
@@ -308,7 +308,7 @@ public <DocT extends Inference> WorkflowResponse<DocT> executeWorkflowPost(
       return httpClient.execute(post, response -> {
         HttpEntity responseEntity = response.getEntity();
         int statusCode = response.getCode();
-        if (!is2xxStatusCode(statusCode)) {
+        if (isInvalidStatusCode(statusCode)) {
           throw getHttpError(parametricType, response);
         }
         if (responseEntity.getContentLength() == 0) {

src/main/java/com/mindee/http/MindeeHttpApiV2.java

@@ -117,7 +117,7 @@ public JobResponse getJobResponse(
           get, response -> {
             HttpEntity responseEntity = response.getEntity();
             int statusCode = response.getCode();
-            if (!is2xxStatusCode(statusCode)) {
+            if (isInvalidStatusCode(statusCode)) {
               throw getHttpError(response);
             }
             try {
@@ -156,7 +156,7 @@ public InferenceResponse getInferenceFromQueue(String jobId) {
             HttpEntity entity = response.getEntity();
             int status = response.getCode();
             try {
-              if (!is2xxStatusCode(status)) {
+              if (isInvalidStatusCode(status)) {
                 throw getHttpError(response);
               }
               String raw = EntityUtils.toString(entity, StandardCharsets.UTF_8);