refactor: Remove dependency on old reactive library

BREAKING CHANGE: The rx() and rxWithDetails() methods have been removed. This type of functionality
should be replaced with the new reactiveRequest() and reactiveRequestWithDetails() methods

Author: lpatino10

pom.xml

@@ -48,7 +48,6 @@
         <gson-version>2.8.5</gson-version>
         <commons-io-version>2.6</commons-io-version>
         <commons-lang3-version>3.8.1</commons-lang3-version>
-        <jersey-version>2.25.1</jersey-version>
         <rx-version>2.2.7</rx-version>
     </properties>
 
@@ -82,11 +81,6 @@
             <version>${okhttp3-version}</version>
             <scope>test</scope>
         </dependency>
-        <dependency>
-            <groupId>org.glassfish.jersey.bundles.repackaged</groupId>
-            <artifactId>jersey-jsr166e</artifactId>
-            <version>${jersey-version}</version>
-        </dependency>
         <dependency>
             <groupId>commons-io</groupId>
             <artifactId>commons-io</artifactId>

src/main/java/com/ibm/cloud/sdk/core/http/ServiceCall.java

@@ -13,7 +13,6 @@
 package com.ibm.cloud.sdk.core.http;
 
 import io.reactivex.Single;
-import jersey.repackaged.jsr166e.CompletableFuture;
 
 /**
  * Service Call.
@@ -62,21 +61,6 @@ public interface ServiceCall<T> {
    */
   void enqueueWithDetails(ServiceCallbackWithDetails<T> callback);
 
-  /**
-   * Reactive requests, in this case, you could take advantage both synchronous and asynchronous.
-   *
-   * @return a CompletableFuture wrapper for your response
-   */
-  CompletableFuture<T> rx();
-
-  /**
-   * Reactive requests with added HTTP information. In this case, you could take advantage both synchronous and
-   * asynchronous.
-   *
-   * @return a CompletableFuture wrapper for your response
-   */
-  CompletableFuture<Response<T>> rxWithDetails();
-
   /**
    * Reactive request using the RxJava 2 library. See https://github.com/ReactiveX/RxJava.
    *

src/main/java/com/ibm/cloud/sdk/core/service/BaseService.java

@@ -36,7 +36,6 @@
 import com.ibm.cloud.sdk.core.util.CredentialUtils;
 import com.ibm.cloud.sdk.core.util.RequestUtils;
 import io.reactivex.Single;
-import jersey.repackaged.jsr166e.CompletableFuture;
 import okhttp3.Call;
 import okhttp3.Callback;
 import okhttp3.Credentials;
@@ -540,54 +539,6 @@ public void onResponse(Call call, Response response) {
       });
     }
 
-    @Override
-    public CompletableFuture<T> rx() {
-      final CompletableFuture<T> completableFuture = new CompletableFuture<T>();
-
-      call.enqueue(new Callback() {
-        @Override
-        public void onFailure(Call call, IOException e) {
-          completableFuture.completeExceptionally(e);
-        }
-
-        @Override
-        public void onResponse(Call call, Response response) {
-          try {
-            completableFuture.complete(processServiceCall(converter, response));
-          } catch (Exception e) {
-            completableFuture.completeExceptionally(e);
-          }
-        }
-      });
-
-      return completableFuture;
-    }
-
-    @Override
-    public CompletableFuture<com.ibm.cloud.sdk.core.http.Response<T>> rxWithDetails() {
-      final CompletableFuture<com.ibm.cloud.sdk.core.http.Response<T>> completableFuture
-          = new CompletableFuture<>();
-
-      call.enqueue(new Callback() {
-        @Override
-        public void onFailure(Call call, IOException e) {
-          completableFuture.completeExceptionally(e);
-        }
-
-        @Override
-        public void onResponse(Call call, Response response) {
-          try {
-            T responseModel = processServiceCall(converter, response);
-            completableFuture.complete(new com.ibm.cloud.sdk.core.http.Response<>(responseModel, response));
-          } catch (Exception e) {
-            completableFuture.completeExceptionally(e);
-          }
-        }
-      });
-
-      return completableFuture;
-    }
-
     @Override
     public Single<T> reactiveRequest() {
       return Single.fromCallable(new Callable<T>() {

src/test/java/com/ibm/cloud/sdk/core/test/service/ResponseTest.java

@@ -106,51 +106,6 @@ public void onFailure(Exception e) { }
     });
   }
 
-  /**
-   * Test that all fields are populated when calling rxWithDetails() using a callback.
-   *
-   * @throws InterruptedException the interrupted exception
-   */
-  @Test
-  public void testRxWithDetailsCallback() throws InterruptedException {
-    server.enqueue(new MockResponse().setBody("{\"test_key\": \"test_value\"}"));
-
-    service.testMethod().rxWithDetails().thenAccept(response -> {
-      assertNotNull(response.getResult());
-      assertNotNull(response.getHeaders());
-    });
-  }
-
-  /**
-   * Test that all fields are populated when calling rxWithDetails() using an asynchronous callback.
-   *
-   * @throws InterruptedException the interrupted exception
-   */
-  @Test
-  public void testRxWithDetailsAsync() throws InterruptedException {
-    server.enqueue(new MockResponse().setBody("{\"test_key\": \"test_value\"}"));
-
-    service.testMethod().rxWithDetails().thenAcceptAsync(response -> {
-      assertNotNull(response.getResult());
-      assertNotNull(response.getHeaders());
-    });
-  }
-
-  /**
-   * Test that all fields are populated when calling rxWjthDetails() synchronously.
-   *
-   * @throws InterruptedException the interrupted exception
-   * @throws ExecutionException the execution exception
-   */
-  @Test
-  public void testRxWithDetailsSync() throws InterruptedException, ExecutionException {
-    server.enqueue(new MockResponse().setBody("{\"test_key\": \"test_value\"}"));
-
-    Response<TestModel> response = service.testMethod().rxWithDetails().get();
-    assertNotNull(response.getResult());
-    assertNotNull(response.getHeaders());
-  }
-
   /**
    * Test that headers are accessible from a HEAD method call using executeWithDetails().
    *