fix: ErrorInterceptor empty body check for okhttp 5

Author: ricellis

modules/common/src/main/java/com/ibm/cloud/cloudant/internal/ErrorTransformInterceptor.java

@@ -32,7 +32,7 @@ public Response intercept(Chain chain) throws IOException {
         // Don't modify the request, but get the response
         Response response = chain.proceed(chain.request());
         if (!response.isSuccessful() // skip successful responses
-            && response.body() != null // skip cases with no body
+            && response.body() != null && !ResponseBody.EMPTY.equals(response.body()) // skip cases with no body
             && response.body().contentType() != null // skip cases with no content type
             && "application".equals(response.body().contentType().type())
             && "json".equals(response.body().contentType().subtype()) // we only want to work with application/json

modules/common/src/test/java/com/ibm/cloud/cloudant/internal/ErrorTransformInterceptorTest.java

@@ -37,7 +37,6 @@
 import com.google.gson.JsonObject;
 import com.ibm.cloud.sdk.core.security.NoAuthAuthenticator;
 import com.ibm.cloud.sdk.core.util.GsonSingleton;
-import static org.junit.Assert.assertNull;
 import static org.testng.Assert.assertEquals;
 import static org.testng.Assert.assertFalse;
 import static org.testng.Assert.assertTrue;
@@ -340,7 +339,7 @@ void runTest(Interceptor interceptor) throws IOException {
         } else {
           // If we aren't asserting a specific part of augment check the body is unchanged
           if ("HEAD" == r.request().method()) {
-            assertNull(r.body());
+            assertEquals(r.body(), ResponseBody.EMPTY);
           } else {
             assertEquals(r.body().string(), responseBody); 
           }