Improve handling when self link is not present

Author: liry

gooddata-java/src/main/java/com/gooddata/sdk/service/connector/ConnectorService.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2004-2019, GoodData(R) Corporation. All rights reserved.
+ * Copyright (C) 2004-2021, GoodData(R) Corporation. All rights reserved.
  * This source code is licensed under the BSD-style license found in the
  * LICENSE.txt file in the root directory of this source tree.
  */
@@ -22,8 +22,8 @@
 import java.io.IOException;
 import java.util.Collection;
 import java.util.Map;
+import java.util.Optional;
 
-import static com.gooddata.sdk.common.util.Validate.isTrue;
 import static com.gooddata.sdk.common.util.Validate.notNull;
 import static com.gooddata.sdk.common.util.Validate.notNullState;
 import static java.lang.String.format;
@@ -267,13 +267,17 @@ public FutureResult<ProcessStatus> getProcessStatus(final IntegrationProcessStat
      * You should use the result of {@link #scheduleZendesk4Reload} to see changes in {@link Reload#getStatus()} and
      * {@link Reload#getProcessId()} or retrieve process URI {@link Reload#getProcessUri()}.
      *
-     * @param reload existing reload.
+     * @param reload existing reload (self link must be present).
      * @return same reload with refreshed properties (status, processId, process URI)
      */
     public Reload getZendesk4Reload(final Reload reload) {
         notNull(reload, "reload");
-        isTrue(reload.getUri().isPresent(), "reload.uri");
-        return getZendesk4ReloadByUri(reload.getUri().get());
+        final Optional<String> reloadUri = reload.getUri();
+        if (reloadUri.isPresent()) {
+            return getZendesk4ReloadByUri(reloadUri.get());
+        } else {
+            throw new IllegalArgumentException("Self link in the Reload must be present!");
+        }
     }
 
     /**

gooddata-java/src/test/groovy/com/gooddata/sdk/service/connector/ConnectorServiceTest.groovy

@@ -0,0 +1,38 @@
+/*
+ * Copyright (C) 2004-2021, GoodData(R) Corporation. All rights reserved.
+ * This source code is licensed under the BSD-style license found in the
+ * LICENSE.txt file in the root directory of this source tree.
+ */
+
+package com.gooddata.sdk.service.connector
+
+import com.gooddata.sdk.model.connector.Reload
+import com.gooddata.sdk.service.GoodDataSettings
+import com.gooddata.sdk.service.project.ProjectService
+import org.springframework.web.client.RestTemplate
+import spock.lang.Specification
+import spock.lang.Subject
+
+import static java.util.Collections.emptyMap
+
+class ConnectorServiceTest extends Specification {
+
+    @Subject
+    def service = new ConnectorService(Mock(RestTemplate), Mock(ProjectService), Mock(GoodDataSettings))
+
+    def "getZendesk4Reload should throw when Reload is null"() {
+        when:
+        service.getZendesk4Reload(null)
+
+        then:
+        thrown(IllegalArgumentException)
+    }
+
+    def "getZendesk4Reload should throw when Reload does not contain self URI"() {
+        when:
+        service.getZendesk4Reload(new Reload(emptyMap()))
+
+        then:
+        thrown(IllegalArgumentException)
+    }
+}