Updated tests

Author: thomasm-ttd

src/main/java/com/uid2/shared/attest/UidCoreClient.java

@@ -47,7 +47,7 @@ public UidCoreClient(String userToken,
             this.httpClient = httpClient;
         }
         if (attestationResponseHandler == null) {
-            throw new IllegalArgumentException("attestationTokenRetriever can not be null");
+            throw new IllegalArgumentException("attestationResponseHandler can not be null");
         } else {
             this.attestationResponseHandler = attestationResponseHandler;
         }

src/main/java/com/uid2/shared/attest/UidOptOutClient.java

@@ -15,11 +15,12 @@ public class UidOptOutClient extends UidCoreClient {
     private AttestationResponseHandler attestationResponseHandler;
 
     public UidOptOutClient(String userToken,
-                         Proxy proxy,
-                         AttestationResponseHandler attestationResponseHandler) {
+                           Proxy proxy,
+                           AttestationResponseHandler attestationResponseHandler) {
         super(userToken, proxy, attestationResponseHandler, null);
         this.attestationResponseHandler = attestationResponseHandler;
     }
+
     public UidOptOutClient(String userToken,
                            Proxy proxy,
                            AttestationResponseHandler attestationResponseHandler,
@@ -39,7 +40,7 @@ public InputStream download(String path) throws CloudStorageException {
             path = "";
         }
 
-        if (this.attestationResponseHandler != null && this.attestationResponseHandler.getOptOutUrl() != null) {
+        if (this.attestationResponseHandler.getOptOutUrl() != null) {
             try {
                 URL baseUrl = new URL(this.attestationResponseHandler.getOptOutUrl());
                 URL fullUrl = new URL(baseUrl, path);

src/test/java/com/uid2/shared/attest/UidCoreClientTest.java

@@ -84,6 +84,7 @@ public void Download_Attest401_AttestCalledTwice() throws CloudStorageException,
 
         uidCoreClient.download("https://download");
         verify(mockAttestationResponseHandler, times(2)).attest();
+        verify(mockAttestationResponseHandler, never()).getOptOutUrl();
     }
 
     @Test