Skip to content

Commit cbf0f71

Browse files
committed
Fixes for test utils
1 parent f7215a4 commit cbf0f71

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

src/TestUtils/WorkshopExerciseTest.php

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -120,13 +120,19 @@ public function assertVerifyWasSuccessful(): void
120120
->filter(function (ResultInterface $result) {
121121
return $result instanceof FailureInterface;
122122
})
123-
->map(function (Failure $failure) {
124-
return $failure->getReason();
125-
})
126-
->implode(', ');
123+
->map(function (FailureInterface $failure) {
124+
return sprintf(
125+
' * %s%s',
126+
get_class($failure),
127+
$failure instanceof Failure ? ": {$failure->getReason()}" : ''
128+
);
129+
});
127130

131+
$help = $failures->isEmpty()
132+
? ""
133+
: sprintf("\n\nAll Failures:\n\n%s\n", $failures->implode("\n"));
128134

129-
$this->assertTrue($this->results->isSuccessful(), $failures);
135+
$this->assertTrue($this->results->isSuccessful(), $help);
130136
}
131137

132138
public function assertVerifyWasNotSuccessful(): void
@@ -163,7 +169,7 @@ public function assertResultsHasFailure(string $resultClass, string $reason): vo
163169

164170
$help = $allFailures->isEmpty()
165171
? ""
166-
: sprintf("\n\nAll Failures:\n\n\n", $allFailures->implode("\n"));
172+
: sprintf("\n\nAll Failures:\n\n%s\n", $allFailures->implode("\n"));
167173

168174
$this->assertCount(1, $failures, "No failure with reason: '$reason' . $help");
169175
}

0 commit comments

Comments
 (0)