MAGETWO-38191: Contribute payment service API to mainline

- Added getUri method to TransferInterface

Author: dkvashninbay

app/code/Magento/Payment/Gateway/Http/Client/Zend.php

@@ -43,9 +43,21 @@ public function placeRequest(\Magento\Payment\Gateway\Http\TransferInterface $tr
 
         $client->setConfig($transferObject->getClientConfig());
         $client->setMethod($transferObject->getMethod());
-        $client->setParameterPost($transferObject->getBody());
+
+        switch($transferObject->getMethod()) {
+            case \Zend_Http_Client::GET:
+                $client->setParameterGet($transferObject->getBody());
+                break;
+            case \Zend_Http_Client::POST:
+                $client->setParameterPost($transferObject->getBody());
+                break;
+            default:
+                throw new \LogicException(sprintf('Unsupported HTTP method %s', $transferObject->getMethod()));
+        }
+
         $client->setHeaders($transferObject->getHeaders());
         $client->setUrlEncodeBody($transferObject->shouldEncode());
+        $client->setUri($transferObject->getUri());
 
         try {
             $response = $client->request();

app/code/Magento/Payment/Gateway/Http/TransferInterface.php

@@ -41,4 +41,11 @@ public function shouldEncode();
      * @return string
      */
     public function getBody();
+
+    /**
+     * Returns URI
+     *
+     * @return string
+     */
+    public function getUri();
 }

app/code/Magento/Payment/Test/Unit/Gateway/Http/Client/ZendTest.php

@@ -127,21 +127,24 @@ public function testPlaceRequestConvertResponseFail()
     private function setClientTransferObjects()
     {
         $config = ['key1' => 'value1', 'key2' => 'value2'];
-        $method = 'methodName';
+        $method = \Zend_Http_Client::POST;
         $headers = ['key1' => 'value1', 'key2' => 'value2'];
         $body = 'Body content';
+        $uri = 'https://example.com/listener';
         $shouldEncode = true;
 
         $this->transferObjectMock->expects($this->once())->method('getClientConfig')->willReturn($config);
-        $this->transferObjectMock->expects($this->once())->method('getMethod')->willReturn($method);
+        $this->transferObjectMock->expects($this->atLeastOnce())->method('getMethod')->willReturn($method);
         $this->transferObjectMock->expects($this->once())->method('getHeaders')->willReturn($headers);
         $this->transferObjectMock->expects($this->once())->method('getBody')->willReturn($body);
         $this->transferObjectMock->expects($this->once())->method('shouldEncode')->willReturn($shouldEncode);
+        $this->transferObjectMock->expects($this->once())->method('getUri')->willReturn($uri);
 
         $this->clientMock->expects($this->once())->method('setConfig')->with($config)->willReturnSelf();
         $this->clientMock->expects($this->once())->method('setMethod')->with($method)->willReturnSelf();
         $this->clientMock->expects($this->once())->method('setParameterPost')->with($body)->willReturnSelf();
         $this->clientMock->expects($this->once())->method('setHeaders')->with($headers)->willReturnSelf();
         $this->clientMock->expects($this->once())->method('setUrlEncodeBody')->with($shouldEncode)->willReturnSelf();
+        $this->clientMock->expects($this->once())->method('setUri')->with($uri)->willReturnSelf();
     }
 }