SADM/pest
1
0
Fork 0
mirror of https://github.com/pestphp/pest.git synced 2026-03-06 07:47:22 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #736 from fabio-ivona/fix-dataset-mismatch-message

Browse Source 
[chore] Remove dataset name from DatasetArgsCountMismatch
This commit is contained in:
Nuno Maduro
2023-03-24 09:45:08 +00:00
committed by GitHub
parent cc6bd59df9 d9a4fa33b9
commit 50a96dcb8f
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Concerns/Testable.php
View File

@ -241,7 +241,7 @@ trait Testable
continue;
}
if (in_array($testParameterTypes[$argumentIndex], [\Closure::class, 'callable', 'mixed'])) {
if (in_array($testParameterTypes[$argumentIndex], [Closure::class, 'callable', 'mixed'])) {
continue;
}
@ -255,7 +255,7 @@ trait Testable
return $arguments;
}
if (in_array($testParameterTypes[0], [\Closure::class, 'callable'])) {
if (in_array($testParameterTypes[0], [Closure::class, 'callable'])) {
return $arguments;
}
@ -291,7 +291,7 @@ trait Testable
return;
}
throw new DatasetArgsCountMismatch($this->dataName(), $requiredParametersCount, $suppliedParametersCount);
throw new DatasetArgsCountMismatch($requiredParametersCount, $suppliedParametersCount);
}
/**

4
src/Exceptions/DatasetArgsCountMismatch.php
View File

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