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

Merge pull request #413 from def-studio/fix_skipping_tests_with_exception_asserting

Browse Source 
Fix skipping tests with exception asserting
This commit is contained in:
Nuno Maduro
2021-10-02 12:01:04 +01:00
committed by GitHub
parent 205238fcbf b0fbe54181
commit e853792a59
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/Factories/TestCaseFactory.php
View File

@ -144,8 +144,8 @@ final class TestCaseFactory
* @return mixed
*/
$test = function () use ($chains, $proxies, $factoryTest) {
$proxies->proxy($this);
$chains->chain($this);
$proxies->proxy($this);
/* @phpstan-ignore-next-line */
return call_user_func(Closure::bind($factoryTest, $this, get_class($this)), ...func_get_args());

6
tests/Features/Exceptions.php
View File

@ -1,5 +1,7 @@
<?php
use PHPUnit\Framework\ExpectationFailedException;
it('gives access the the underlying expectException', function () {
$this->expectException(InvalidArgumentException::class);
@ -37,3 +39,7 @@ it('can just define the message if given condition is true', function () {
it('can just define the message if given condition is 1', function () {
throw new Exception('Something bad happened');
})->throwsIf(1, 'Something bad happened');
it('can handle a skipped test if it is trying to catch an exception', function () {
expect(1)->toBe(2);
})->throws(ExpectationFailedException::class)->skip('this test should be skipped')->only();