SoapAbstractRequest 100% test coverage

pinguinjkeke · Sep 22, 2016 · 5b80f8b · 5b80f8b
1 parent 437cd4e
commit 5b80f8b
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/src/Message/SoapAbstractRequest.php b/src/Message/SoapAbstractRequest.php
@@ -213,6 +213,10 @@ public function buildSoapClient()
         }
 
         try {
+            if (!$this->getEndpoint()) {
+                throw new SoapFault('Client', 'No endpoint provided');
+            }
+
             $this->soapClient = new SoapClient($this->getWsdl(), array(
                 'uri' => 'http://www.sirena-travel.ru/',
                 'location' => $this->getEndpoint(),

diff --git a/tests/Message/SoapAbstractRequestTest.php b/tests/Message/SoapAbstractRequestTest.php
@@ -0,0 +1,17 @@
+<?php
+
+namespace Omnipay\EgopayRu\Message;
+
+use Omnipay\Tests\TestCase;
+
+class SoapAbstractRequestTest extends TestCase
+{
+    public function testBuildSoapClientFault()
+    {
+        $request = new CancelRequest($this->getHttpClient(), $this->getHttpRequest(), null);
+        $request->setEndpoint(null);
+
+        $this->assertNull($request->buildSoapClient());
+        $this->assertEquals($request->sendData(array())->getData(), 'SOAP fail');
+    }
+}