Merge pull request #126 from academe/master

judgej · web-flow · commit 9f24c0ef049f · 2018-12-12T01:03:21.000Z
Issue #124 additional test for completeness.
diff --git a/tests/Message/ServerNotifyRequestTest.php b/tests/Message/ServerNotifyRequestTest.php
@@ -132,6 +132,8 @@ public function testServerNotifyRequestFailure()
         $this->assertSame('failed', $this->request->getTransactionStatus());
 
         $this->assertNull($this->request->getMessage());
+
+        $this->assertFalse($this->request->isSuccessful());
     }
 
     public function testError()

Original file line number	Diff line number	Diff line change
`@@ -132,6 +132,8 @@ public function testServerNotifyRequestFailure()`
`132`	`132`	`$this->assertSame('failed', $this->request->getTransactionStatus());`
`133`	`133`
`134`	`134`	`$this->assertNull($this->request->getMessage());`
	`135`	`+`
	`136`	`+ $this->assertFalse($this->request->isSuccessful());`
`135`	`137`	`}`
`136`	`138`
`137`	`139`	`public function testError()`