Skip to content

Commit

Permalink
Merge pull request #77 from delatbabel/fix-pr-tests
Browse files Browse the repository at this point in the history
Fix broken test case
  • Loading branch information
delatbabel authored May 28, 2017
2 parents a19747c + 543c2c7 commit a88b94d
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 8 deletions.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ php:
- 5.4
- 5.5
- 5.6
- hhvm

before_script:
- composer install -n --dev --prefer-source
Expand Down
9 changes: 6 additions & 3 deletions tests/Message/ListPlansTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,11 @@

use Omnipay\Tests\TestCase;

class ListInvoicesRequestTest extends TestCase
class ListPlansTest extends TestCase
{
/** @var ListPlansRequest */
protected $request;

public function setUp()
{
$this->request = new ListPlansRequest($this->getHttpClient(), $this->getHttpRequest());
Expand All @@ -18,7 +21,7 @@ public function testEndpoint()

public function testSendSuccess()
{
$this->setMockHttpResponse('ListPlans.txt');
$this->setMockHttpResponse('ListPlansSuccess.txt');
$response = $this->request->send();

$this->assertTrue($response->isSuccessful());
Expand All @@ -33,6 +36,6 @@ public function testSendSuccess()
*/
public function testSendFailure()
{
this->assertTrue(true);
$this->assertTrue(true);
}
}
8 changes: 4 additions & 4 deletions tests/Mock/ListPlansSuccess.txt
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ Access-Control-Allow-Credentials: true
Cache-Control: no-cache, no-store

{
"object" => "list",
"url" => "/v1/plans",
"has_more" => false,
"data" => [
"object": "list",
"url": "/v1/plans",
"has_more": false,
"data": [
{
"id": "test-1",
"object": "plan",
Expand Down

0 comments on commit a88b94d

Please sign in to comment.