Merge pull request thephpleague#221 from thephpleague/feat-testactualrequest

barryvdh · web-flow · commit d6a1bed63cae · 2020-06-02T07:57:19.000+02:00
Test actual request instead of mock
diff --git a/tests/Omnipay/Common/Message/AbstractRequestTest.php b/tests/Omnipay/Common/Message/AbstractRequestTest.php
@@ -16,7 +16,7 @@ class AbstractRequestTest extends TestCase
 
     public function setUp()
     {
-        $this->request = m::mock('\Omnipay\Common\Message\AbstractRequest')->makePartial();
+        $this->request = new AbstractRequestTest_MockAbstractRequest($this->getHttpClient(), $this->getHttpRequest());
         $this->request->initialize();
     }
 
@@ -467,6 +467,7 @@ public function testNoCurrencyReturnedIfCurrencyNotSet()
 
     public function testSend()
     {
+        $this->request = m::mock('\Omnipay\Common\Message\AbstractRequest')->makePartial();
         $response = m::mock('\Omnipay\Common\Message\ResponseInterface');
         $data = array('request data');
 

Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ class AbstractRequestTest extends TestCase`
`16`	`16`
`17`	`17`	`public function setUp()`
`18`	`18`	`{`
`19`		`- $this->request = m::mock('\Omnipay\Common\Message\AbstractRequest')->makePartial();`
	`19`	`+ $this->request = new AbstractRequestTest_MockAbstractRequest($this->getHttpClient(), $this->getHttpRequest());`
`20`	`20`	`$this->request->initialize();`
`21`	`21`	`}`
`22`	`22`
`@@ -467,6 +467,7 @@ public function testNoCurrencyReturnedIfCurrencyNotSet()`
`467`	`467`
`468`	`468`	`public function testSend()`
`469`	`469`	`{`
	`470`	`+ $this->request = m::mock('\Omnipay\Common\Message\AbstractRequest')->makePartial();`
`470`	`471`	`$response = m::mock('\Omnipay\Common\Message\ResponseInterface');`
`471`	`472`	`$data = array('request data');`
`472`	`473`