Merge pull request #186 from sobyte/remove-ril

Remove remote issue link

Author: lesstif

src/Issue/IssueService.php

@@ -879,6 +879,34 @@ public function createOrUpdateRemoteIssueLink($issueIdOrKey, RemoteIssueLink $ri
         return $res;
     }
 
+    /**
+     * @param string|int $issueIdOrKey
+     * @param string|int $globalId
+     *
+     * @throws JiraException
+     *
+     * @return string|bool
+     */
+    public function removeRemoteIssueLink($issueIdOrKey, $globalId)
+    {
+        $query = http_build_query(['globalId' => $globalId]);
+
+        $full_uri = sprintf('%s/%s/remotelink?%s', $this->uri, $issueIdOrKey, $query);
+
+        $ret = $this->exec($full_uri, '', 'DELETE');
+
+        $this->log->addInfo(
+            sprintf(
+                'delete remote issue link for issue "%s" with globalId "%s". Result=%s',
+                $issueIdOrKey,
+                $globalId,
+                var_export($ret, true)
+            )
+        );
+
+        return $ret;
+    }
+
     /**
      * get all issue security schemes.
      *

tests/RemoteIssueLinkTest.php

@@ -57,13 +57,26 @@ public function testGetRemoteIssue($issueKey)
      */
     public function testDeleteRemoteIssueLink($issueKey)
     {
-        // not yet impl
-        $this->markTestIncomplete();
         try {
+            $issueService = new IssueService();
+
+            $rils = $issueService->getRemoteIssueLink($issueKey);
+            $countBefore = count($rils);
+
+            /** @var RemoteIssueLink $firstRil */
+            $firstRil = $rils[0];
+
+            $ret = $issueService->removeRemoteIssueLink($issueKey, $firstRil->globalId);
+
+            $this->assertTrue($ret);
+
+            $rilsAfter = $issueService->getRemoteIssueLink($issueKey);
+
+            $this->assertLessThan($countBefore, count($rilsAfter));
 
             return $issueKey;
         } catch (JiraException $e) {
-            $this->assertTrue(false, 'Create Failed : '.$e->getMessage());
+            $this->assertTrue(false, 'Remove Failed : '.$e->getMessage());
         }
     }
 }