SADM/pest
1
0
Fork 0
mirror of https://github.com/pestphp/pest.git synced 2026-03-10 17:57:23 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix failure message

Browse Source
This commit is contained in:
Fabio Ivona
2023-03-19 00:14:18 +01:00
parent 9de85175db
commit 1f3e5115c7
2 changed files with 5 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/Concerns/Testable.php
View File

@ -285,12 +285,13 @@ trait Testable
$underlyingTest = Reflection::getFunctionVariable($this->__test, 'closure'); $underlyingTest = Reflection::getFunctionVariable($this->__test, 'closure');
$testReflection = new ReflectionFunction($underlyingTest); $testReflection = new ReflectionFunction($underlyingTest);
$requiredParametersCount = $testReflection->getNumberOfRequiredParameters(); $requiredParametersCount = $testReflection->getNumberOfRequiredParameters();
$suppliedParametersCount = count($arguments);
if (count($arguments) >= $requiredParametersCount) { if ($suppliedParametersCount >= $requiredParametersCount) {
return; return;
} }
throw new DatasetArgsCountMismatch($this->dataName()); throw new DatasetArgsCountMismatch($this->dataName(), $requiredParametersCount, $suppliedParametersCount);
} }
/** /**

4
src/Exceptions/DatasetArgsCountMismatch.php
View File

@ -8,8 +8,8 @@ use Exception;
final class DatasetArgsCountMismatch extends Exception final class DatasetArgsCountMismatch extends Exception
{ {
public function __construct(string $dataName) public function __construct(string $dataName, int $requiredCount, int $suppliedCount)
{ {
parent::__construct(sprintf('Number of arguments mismatch between test and dataset [%s]', $dataName)); parent::__construct(sprintf('Test expects %d arguments but dataset [%s] only provides %d', $requiredCount, $dataName, $suppliedCount));
} }
} }