Enable OkHttpClient tests for solid module (#578)

This fixes an issue where the OkHttpClient would throw an uncaught exception processing a response if the body handler threw a RuntimeException (IOException are handled gracefully).

Author: NSeydoux

okhttp/src/main/java/com/inrupt/client/okhttp/OkHttpService.java

@@ -88,6 +88,8 @@ public void onResponse(final Call call, final okhttp3.Response res) throws IOExc
                 try (final okhttp3.Response r = res) {
                     final Response.ResponseInfo info = new OkHttpResponseInfo(r);
                     future.complete(new OkHttpResponse<>(res.request().url().uri(), info, handler.apply(info)));
+                } catch (final RuntimeException ex) {
+                    future.completeExceptionally(ex);
                 }
             }
 

solid/pom.xml

@@ -130,6 +130,34 @@
               </classpathDependencyExcludes>
             </configuration>
           </execution>
+          <execution>
+            <id>okhttp-jena-test</id>
+            <phase>test</phase>
+            <goals>
+              <goal>test</goal>
+            </goals>
+            <configuration>
+              <classpathDependencyExcludes>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-httpclient</classpathDependencyExclude>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-rdf4j</classpathDependencyExclude>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-rdf-legacy</classpathDependencyExclude>
+              </classpathDependencyExcludes>
+            </configuration>
+          </execution>
+          <execution>
+            <id>okhttp-rdf4j-test</id>
+            <phase>test</phase>
+            <goals>
+              <goal>test</goal>
+            </goals>
+            <configuration>
+              <classpathDependencyExcludes>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-httpclient</classpathDependencyExclude>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-jena</classpathDependencyExclude>
+                <classpathDependencyExclude>com.inrupt.client:inrupt-client-rdf-legacy</classpathDependencyExclude>
+              </classpathDependencyExcludes>
+            </configuration>
+          </execution>
         </executions>
       </plugin>
       <plugin>