Skip to content
This repository was archived by the owner on Jan 31, 2020. It is now read-only.

Commit f4575ed

Browse files
committed
Fix test expectation
`AbstractResult::getMessage()` (and `ResultInterface::getMessage()`) document the return type as a `string`, not a `null`. An empty string should be the expected default.
1 parent 527d380 commit f4575ed

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/BasicClassesTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ public function testConstructor()
7777
public function testSetters()
7878
{
7979
$result = new Success();
80-
$this->assertSame(null, $result->getMessage());
80+
$this->assertSame('', $result->getMessage());
8181
$this->assertSame(null, $result->getData());
8282

8383
$result->setMessage('foo');

0 commit comments

Comments
 (0)