Merge pull request #77 from delatbabel/fix-pr-tests

Fix broken test case
thephpleague · May 28, 2017 · a88b94d · a88b94d
2 parents a19747c + 543c2c7
commit a88b94d
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -5,7 +5,6 @@ php:
   - 5.4
   - 5.5
   - 5.6
-  - hhvm
 
 before_script:
   - composer install -n --dev --prefer-source

diff --git a/tests/Message/ListPlansTest.php b/tests/Message/ListPlansTest.php
@@ -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());
@@ -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());
@@ -33,6 +36,6 @@ public function testSendSuccess()
      */
     public function testSendFailure()
     {
-        this->assertTrue(true);
+        $this->assertTrue(true);
     }
 }
diff --git a/tests/Mock/ListPlansSuccess.txt b/tests/Mock/ListPlansSuccess.txt
@@ -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",