Javadoc Updates

Signed-off-by: Andy McCright <[email protected]>

Author: andymc12

tck/src/main/java/org/eclipse/microprofile/rest/client/tck/RestClientBuilderListenerTest.java

@@ -18,8 +18,7 @@
 
 import static org.testng.Assert.assertEquals;
 
-//import java.net.URI;
-import java.net.URL;
+import java.net.URI;
 
 import org.eclipse.microprofile.rest.client.RestClientBuilder;
 import org.eclipse.microprofile.rest.client.spi.RestClientBuilderListener;
@@ -58,15 +57,14 @@ public static WebArchive createDeployment() {
      * with a 500 status code, but at priority 2.  If the RestClientBuilderListener impl
      * is correctly invoked, then the request will abort with the 200; if not,
      * it will abort with the 500.
+     *
+     * @throws Exception - indicates test failure
      */
     @Test
     public void testRegistrarInvoked() throws Exception {
         SimpleGetApi client = RestClientBuilder.newBuilder()
             .register(ReturnWith500RequestFilter.class, 2)
-            // TODO: rebase after baseUri change goes in and use this line:
-            //.baseUri(new URI("http://localhost:8080/neverUsed"))
-            // instead of
-            .baseUrl(new URL("http://localhost:8080/neverUsed"))
+            .baseUri(new URI("http://localhost:8080/neverUsed"))
             .build(SimpleGetApi.class);
 
         assertEquals(200, client.executeGet().getStatus(),

tck/src/main/java/org/eclipse/microprofile/rest/client/tck/asynctests/AsyncMethodTest.java

@@ -68,9 +68,11 @@ public static WebArchive createDeployment() {
     }
 
     /**
-     * Tests that a Rest Client interface method that returns Future<Response>
+     * Tests that a Rest Client interface method that returns CompletionStage
      * is invoked asychronously - checking that the thread ID of the response
      * does not match the thread ID of the calling thread.
+     *
+     * @throws Exception - indicates test failure
      */
     @Test
     public void testInterfaceMethodWithCompletionStageResponseReturnIsInvokedAsynchronously() throws Exception{
@@ -101,10 +103,12 @@ public void testInterfaceMethodWithCompletionStageResponseReturnIsInvokedAsynchr
     }
 
     /**
-     * Tests that a Rest Client interface method that returns Future<X> (where X
-     * is some Object type other than Response) is invoked asychronously -
-     * checking that the thread ID of the response does not match the thread ID
-     * of the calling thread.
+     * Tests that a Rest Client interface method that returns a CompletionStage
+     * where it's parameterized type is some Object type other than Response) is
+     * invoked asychronously - checking that the thread ID of the response does
+     * not match the thread ID of the calling thread.
+     *
+     * @throws Exception - indicates test failure
      */
     @Test
     public void testInterfaceMethodWithCompletionStageObjectReturnIsInvokedAsynchronously() throws Exception{
@@ -135,6 +139,8 @@ public void testInterfaceMethodWithCompletionStageObjectReturnIsInvokedAsynchron
     /**
      * Tests that the MP Rest Client implementation uses the specified
      * ExecutorService.
+     *
+     * @throws Exception - indicates test failure
      */
     @Test
     public void testExecutorService() throws Exception{
@@ -181,6 +187,8 @@ public Thread newThread(Runnable r) {
      * on the calling thread.  It uses an
      * <code>AsyncInvocationInterceptorFactory</code> provider to copy the
      * ThreadLocal value from the calling thread to the async thread.
+     *
+     * @throws Exception - indicates test failure
      */
     @Test
     public void testAsyncInvocationInterceptorProvider() throws Exception{
@@ -218,7 +226,9 @@ public void testAsyncInvocationInterceptorProvider() throws Exception{
     /**
      * This test verifies that the <code>RestClientBuilder</code> implementation
      * will throw an <code>IllegalArgumentException</code> when a null value is
-     * passed to the <code>executorService<code> method.
+     * passed to the <code>executorService</code> method.
+     *
+     * @throws IllegalArgumentException - expected when passing null
      */
     @Test(expectedExceptions={IllegalArgumentException.class})
     public void testNullExecutorServiceThrowsIllegalArgumentException() {