Compare commits

...

92 Commits

Author SHA1 Message Date
c0af671ca2 release: v2.7.0 2023-06-15 14:11:00 +02:00
eb9f31edeb tests: unexpected output 2023-06-15 14:06:54 +02:00
0e5470b192 chore: adds support for PhpUnit@10.2.2 2023-06-15 12:02:00 +02:00
2122e57990 fix: revert higher order message property 2023-06-15 10:44:52 +02:00
e42d224db2 fix: escape service 2023-06-15 10:43:39 +02:00
070139eda7 fix: higher order message 2023-06-15 10:42:31 +02:00
a0d0182031 refacto: uses non-static 2023-06-15 10:38:33 +02:00
1885d4e110 chore: bumps dev tools 2023-06-15 10:38:26 +02:00
5959890125 refacto: early returns 2023-06-15 10:38:17 +02:00
7460bd6c1f chore: adjusts style 2023-06-13 19:59:59 +02:00
3c20e8114e release: v2.6.3 2023-06-07 21:19:04 +02:00
73a859ee56 chore: bumps phpunit dependency 2023-06-07 21:17:10 +02:00
4196579a3d chore: bumps arch plugin 2023-06-03 00:17:20 +01:00
436b20857d docs: updates changelog 2023-06-02 22:46:07 +01:00
82b0adf5ae release: v2.6.2 2023-06-02 22:43:42 +01:00
a0041f139c chore: bumps phpunit dependency 2023-06-02 22:40:46 +01:00
b25bc7ee05 docs: updates changelog 2023-05-12 09:23:13 +01:00
faafedd55c release: v2.6.1 2023-05-12 09:22:02 +01:00
c99325275a chore: bumps dependencies 2023-05-12 09:16:45 +01:00
37a7583755 Merge pull request #809 from gehrisandro/fix-teamcity-output
fix teamcity output for tests throwing an exception before the first assertion
2023-05-11 22:52:26 +01:00
a851b5ed70 fix teamcity output for tests throwing an exception before the first assertion 2023-05-11 22:27:43 +02:00
983659f8e8 chore: bumps paratest 2023-05-06 11:52:05 +01:00
3e1cef296f chore: updates snapshots 2023-05-06 11:51:58 +01:00
cada5c5136 Merge pull request #804 from titantwentyone/2.x
Allow traits to be covered
2023-05-06 11:49:41 +01:00
477492fdd2 Adding tests 2023-05-06 08:43:19 +01:00
963b7f43ab Allow traits to be covered 2023-05-05 17:59:52 +01:00
a34767fa15 release: v2.6.0 2023-05-02 00:51:14 +01:00
fe3c7d72bd chore: bumps dependencies and fixes static analysis 2023-05-02 00:51:01 +01:00
97898a0a8e feat: allows to use test calls on before each calls 2023-05-01 22:18:47 +01:00
cddddc3ec1 Merge pull request #797 from fabio-ivona/toThrow-improvement
[2.x] allow to expect `toThrow` against an exception instance
2023-05-01 21:30:17 +01:00
f8930d20ae allows to check toThrow against an exception instance 2023-05-01 21:42:47 +02:00
2e25eb59b8 style: fixes 2023-04-25 21:32:09 +01:00
a5471fd0f4 chore: bumps dependencies 2023-04-23 00:03:18 +01:00
9842e1051d release: v2.5.3 2023-04-22 23:14:57 +01:00
d39e4f15fe release: v2.5.2 2023-04-19 09:53:08 +01:00
1a05df14d0 chore: bumps dependencies 2023-04-19 09:49:26 +01:00
5c0df87f52 release: v2.5.1 2023-04-18 00:47:56 +01:00
ec6a81735a chore: bumps phpunit dependency 2023-04-18 00:45:41 +01:00
2ea37f3424 Merge pull request #772 from MaurizioMoreo/pcov_fix
Add PCOV availability check
2023-04-16 14:11:05 +01:00
2e4206cb97 tests: adjusts snapshots 2023-04-15 16:17:22 +01:00
72b3cdaab4 Merge pull request #770 from ganyicz/2.x
Update stub extensions
2023-04-15 16:12:16 +01:00
9744b9848e Merge pull request #783 from fabio-ivona/fix-comments-in-test-name
fix comments in dokblocks
2023-04-15 11:26:06 +01:00
e1143d2cfc remove ->only() 2023-04-15 11:52:07 +02:00
5861b0498a fix comments in dokblocks 2023-04-15 10:45:51 +02:00
f6d9aa51bd chore: bump dependencies 2023-04-14 11:50:44 +01:00
9070b12377 release: v2.5.0 2023-04-14 11:13:56 +01:00
dab68d6d85 Fixes caret 2023-04-14 11:12:32 +01:00
40de54ecd5 Merge pull request #780 from pestphp/chore/phpunit10-1-support
[2.x] Adds `phpunit/phpunit@10.1` support
2023-04-14 11:02:06 +01:00
6d8b926df3 chore: adjusts for parallel testing 2023-04-14 11:01:54 +01:00
d6c3f3522b chore: fix caret 2023-04-13 12:51:19 +01:00
41cdb5f01b chore: skips parallel testing 2023-04-13 12:23:17 +01:00
0064f3fdff chore: phpunit ^10.1 support 2023-04-13 11:03:29 +01:00
518035514e Add PHPDBG availability check 2023-04-09 09:27:52 +02:00
81cdb7a300 chore: refacto 2023-04-08 12:23:10 +01:00
97dbda92e6 Update stub extensions 2023-04-08 01:24:00 +02:00
4520fe918a Add PCOV availability check 2023-04-07 17:19:34 +02:00
f73e6aebcb Merge pull request #763 from gehrisandro/fix-typos
[2.x] fix: various typos fixed
2023-04-03 08:11:53 +01:00
3b9ca8853f fix: various typos fixed 2023-04-03 07:50:25 +02:00
cbc26faeb9 docs: updates changelog 2023-04-02 21:13:55 +01:00
12c75524a2 release: v2.4.0 2023-04-02 21:12:31 +01:00
6edbd69dc4 chore: update dependencies 2023-04-02 20:44:35 +01:00
0cfb1237e5 docs: update changelog 2023-04-02 20:37:42 +01:00
cab6aa6e57 Merge pull request #760 from lucasgiovanny/toHaveProperties
Add possibility to check property name and value with toHaveProperties
2023-04-02 20:35:13 +01:00
9ff2f3b46c docs: update changelog 2023-04-02 20:29:30 +01:00
ead0237fb3 chore: bumps dependencies 2023-04-02 20:17:55 +01:00
31726b51ad update toHaveProperties 2023-04-02 11:15:31 +01:00
99d3ee448c docs: unreleased notes 2023-04-01 06:47:14 +01:00
d21ae255f1 Merge pull request #757 from pestphp/feat/skip-on-windows
[2.x] Adds `skipOnWindows()`, `skipOnMac()`, and `skipOnLinux()`
2023-03-31 23:43:30 +01:00
0823e5da88 feat: reworks methods 2023-03-31 23:42:10 +01:00
1f913f6fc1 Merge pull request #758 from faissaloux/feat/skip-on-os-family
skip os family
2023-03-31 23:27:51 +01:00
a6dec31b9d fix return 2023-03-31 22:11:32 +00:00
c1979f735f add ability to skip on multiple os families 2023-03-31 22:09:45 +00:00
1c1cb1e591 skip os family 2023-03-31 22:03:18 +00:00
49ddcbd66b feat: skips on windows 2023-03-31 22:20:48 +01:00
23c8adf497 Merge pull request #755 from fabio-ivona/fix-windows-tests
fix windows tests
2023-03-31 09:54:47 +01:00
8e3d7d85e8 fix windows tests 2023-03-31 10:53:11 +02:00
3cee6a499d chore: refactor RenderableOnCollisionEditor 2023-03-31 09:39:05 +01:00
3f38f19df8 Merge pull request #754 from fabio-ivona/collision-custom-editor-support
[2.x] Collision custom editor support
2023-03-31 09:15:35 +01:00
4df6ab4a84 lint 2023-03-31 08:59:38 +02:00
90b6771451 add tests 2023-03-31 08:46:35 +02:00
681723cbc3 matches Collision changes 2023-03-31 08:43:59 +02:00
d9595ec122 chore: bumps dev dependencies 2023-03-30 22:41:21 +01:00
ec35010cf4 Merge pull request #753 from gehrisandro/fix-tests-in-helpers-directory
[2.x] fix: tests in Helpers directory are not executed
2023-03-30 22:32:13 +01:00
d14f2728d8 fix: tests in Helpers directory are not executed 2023-03-30 21:49:17 +02:00
152b7a9fc4 Merge pull request #748 from ncharalampidis/2.x
Fix parallel execution of test class that extends another class with same name
2023-03-29 16:58:21 +01:00
56ec3b9ee3 Merge pull request #747 from gehrisandro/add-teamcity-testsuite-count
teamcity: add missing test count output
2023-03-29 12:10:45 +01:00
ca69e4fd94 remove unnecessary custom phpunit xml added previously 2023-03-29 13:23:43 +03:00
1965763cd0 fix parallel execution of test class that inherit other class with the same name 2023-03-29 13:02:18 +03:00
cd8d94780f teamcity: add missing test count output 2023-03-29 00:03:14 +02:00
ba87e1fde8 Merge pull request #745 from dansysanalyst/improve_exception_message
Slightly Improvement on Exception message
2023-03-28 12:34:10 +01:00
e48bd31ae4 improve message 2023-03-28 12:19:15 +01:00
a05684026e wip 2023-03-24 19:00:38 +01:00
86 changed files with 920 additions and 276 deletions

View File

@ -43,4 +43,4 @@ jobs:
- name: Unit Tests in Parallel
run: composer test:parallel
if: startsWith(matrix.os, 'windows') != true

View File

@ -2,6 +2,66 @@
## Unreleased
## [v2.7.0 (2023-06-15)](https://github.com/pestphp/pest/compare/v2.6.3...v2.7.0)
### Added
- Support for unexpected output on printer ([eb9f31e](https://github.com/pestphp/pest/commit/eb9f31edeb00a88c449874f3d48156128a00fff8))
### Chore
- Bumps PHPUnit to `^10.2.2` ([0e5470b](https://github.com/pestphp/pest/commit/0e5470b192b259ba2db7c02a50371216c98fc0a6))
## [v2.6.3 (2023-06-07)](https://github.com/pestphp/pest/compare/v2.6.2...v2.6.3)
### Chore
- Bumps PHPUnit to `^10.2.1` ([73a859e](https://github.com/pestphp/pest/commit/73a859ee563fe96944ba39b191dceca28ef703c2))
## [v2.6.2 (2023-06-02)](https://github.com/pestphp/pest/compare/v2.6.1...v2.6.2)
### Chore
- Bumps PHPUnit to `^10.2.0` ([a0041f1](https://github.com/pestphp/pest/commit/a0041f139cba94fe5d15318c38e275f2e2fb3350))
## [v2.6.1 (2023-04-12)](https://github.com/pestphp/pest/compare/v2.6.0...v2.6.1)
### Fixes
- PHPStorm issue output problem for tests throwing an exception before the first assertion ([#809](https://github.com/pestphp/pest/pull/809))
- Allow traits to be covered ([#804](https://github.com/pestphp/pest/pull/804))
### Chore
- Bumps PHPUnit to `^10.1.3` ([c993252](https://github.com/pestphp/pest/commit/c99325275acf1fd3759b487b93ec50473f706709))
## [v2.6.0 (2023-04-05)](https://github.com/pestphp/pest/compare/v2.5.2...v2.6.0)
### Adds
- Allows `toThrow` to be used against an exception instance ([#797](https://github.com/pestphp/pest/pull/797))
## [v2.5.2 (2023-04-19)](https://github.com/pestphp/pest/compare/v2.5.1...v2.5.2)
### Chore
- Removes `myclabs/php-enuma` dependency ([1a05df1](https://github.com/pestphp/pest/commit/1a05df14d0ce7d12583df26ff716807db6f81f13))
## [v2.5.1 (2023-04-18)](https://github.com/pestphp/pest/compare/v2.5.0...v2.5.1)
### Chore
- Bumps PHPUnit to `^10.1.1` ([ec6a817](https://github.com/pestphp/pest/commit/ec6a81735af19f5463d24545df97535d77697ec6))
## [v2.5.0 (2023-04-14)](https://github.com/pestphp/pest/compare/v2.4.0...v2.5.0)
### Chore
- Bumps PHPUnit to `^10.1.0` ([#780](https://github.com/pestphp/pest/pull/780))
## [v2.4.0 (2023-04-03)](https://github.com/pestphp/pest/compare/v2.3.0...v2.4.0)
### Added
- `skipOnWindows()`, `skipOnMac()`, and `skipOnLinux()` ([#757](https://github.com/pestphp/pest/pull/757))
- source architecture testing violation ([#1](https://github.com/pestphp/pest-plugin-arch/pull/1))([8e66263](https://github.com/pestphp/pest-plugin-arch/commit/8e66263104304e99e3d6ceda25c7ed679b27fb03))
- `toHaveProperties` may now also check values ([#760](https://github.com/pestphp/pest/pull/760))
### Fixed
- Tests on `tests/Helpers` directory not being executed ([#753](https://github.com/pestphp/pest/pull/753))
- Teamcity count ([#747](https://github.com/pestphp/pest/pull/747))
- Parallel execution when class extends class with same name ([#748](https://github.com/pestphp/pest/pull/748))
- Wording on `uses()` hint ([#745](https://github.com/pestphp/pest/pull/745/files))
## [v2.3.0 (2023-03-28)](https://github.com/pestphp/pest/compare/v2.2.3...v2.3.0)
### Added
@ -16,14 +76,14 @@
## [v2.2.3 (2023-03-24)](https://github.com/pestphp/pest/compare/v2.2.2...v2.2.3)
### Fixed
- Unnecessary dataset on dataset arguments missmatch ([#736](https://github.com/pestphp/pest/pull/736))
- Unnecessary dataset on dataset arguments mismatch ([#736](https://github.com/pestphp/pest/pull/736))
- Parallel arguments on plugins order ([#703](https://github.com/pestphp/pest/pull/703))
- Arch plugin runtime exceptions on bad phpdocs ([2f2b51c](https://github.com/pestphp/pest/commit/2f2b51ce3d1b000be9d6add0e785fd0044931b3b))
## [v2.2.2 (2023-03-23)](https://github.com/pestphp/pest/compare/v2.2.1...v2.2.2)
### Fixed
- Edge case in parallel executation test description ([3ce6408](https://github.com/pestphp/pest/commit/3ce640819541ca6022b250e000f336d87c3e7889))
- Edge case in parallel execution test description ([3ce6408](https://github.com/pestphp/pest/commit/3ce640819541ca6022b250e000f336d87c3e7889))
## [v2.2.1 (2023-03-22)](https://github.com/pestphp/pest/compare/v2.2.0...v2.2.1)

View File

@ -31,6 +31,7 @@ $bootPest = (static function (): void {
$getopt = getopt('', [
'status-file:',
'progress-file:',
'unexpected-output-file:',
'testresult-file:',
'teamcity-file:',
'testdox-file:',
@ -58,6 +59,7 @@ $bootPest = (static function (): void {
assert(is_resource($statusFile));
assert(isset($getopt['progress-file']) && is_string($getopt['progress-file']));
assert(isset($getopt['unexpected-output-file']) && is_string($getopt['unexpected-output-file']));
assert(isset($getopt['testresult-file']) && is_string($getopt['testresult-file']));
assert(! isset($getopt['teamcity-file']) || is_string($getopt['teamcity-file']));
assert(! isset($getopt['testdox-file']) || is_string($getopt['testdox-file']));
@ -73,6 +75,7 @@ $bootPest = (static function (): void {
$application = new ApplicationForWrapperWorker(
$phpunitArgv,
$getopt['progress-file'],
$getopt['unexpected-output-file'],
$getopt['testresult-file'],
$getopt['teamcity-file'] ?? null,
$getopt['testdox-file'] ?? null,
@ -88,11 +91,11 @@ $bootPest = (static function (): void {
$testPath = fgets(STDIN);
if ($testPath === false || $testPath === WrapperWorker::COMMAND_EXIT) {
$application->end();
exit;
}
$exitCode = $application->runTest(trim($testPath));
// It must be a 1 byte string to ensure filesize() is equal to the number of tests executed
$exitCode = $application->runTest(realpath(trim($testPath)));
fwrite($statusFile, (string) $exitCode);
fflush($statusFile);

View File

@ -18,16 +18,16 @@
],
"require": {
"php": "^8.1.0",
"brianium/paratest": "^7.1.2",
"nunomaduro/collision": "^7.3.3",
"brianium/paratest": "^7.2.0",
"nunomaduro/collision": "^7.6.0",
"nunomaduro/termwind": "^1.15.1",
"pestphp/pest-plugin": "^2.0.1",
"pestphp/pest-plugin-arch": "^2.0.2",
"phpunit/phpunit": "^10.0.19"
"pestphp/pest-plugin-arch": "^2.2.0",
"phpunit/phpunit": "^10.2.2"
},
"conflict": {
"webmozart/assert": "<1.11.0",
"phpunit/phpunit": ">10.0.19"
"phpunit/phpunit": ">10.2.2"
},
"autoload": {
"psr-4": {
@ -41,6 +41,7 @@
"autoload-dev": {
"psr-4": {
"Tests\\Fixtures\\Covers\\": "tests/Fixtures/Covers",
"Tests\\Fixtures\\Inheritance\\": "tests/Fixtures/Inheritance",
"Tests\\": "tests/PHPUnit/"
},
"files": [
@ -48,8 +49,8 @@
]
},
"require-dev": {
"pestphp/pest-dev-tools": "^2.5.0",
"symfony/process": "^6.2.7"
"pestphp/pest-dev-tools": "^2.12.0",
"symfony/process": "^6.3.0"
},
"minimum-stability": "stable",
"config": {

View File

@ -0,0 +1,56 @@
<?php
declare(strict_types=1);
/*
* This file is part of PHPUnit.
*
* (c) Sebastian Bergmann <sebastian@phpunit.de>
*
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/
namespace PHPUnit\Event\Code;
use NunoMaduro\Collision\Contracts\RenderableOnCollisionEditor;
use PHPUnit\Event\NoPreviousThrowableException;
use PHPUnit\Framework\Exception;
use PHPUnit\Util\Filter;
use PHPUnit\Util\ThrowableToStringMapper;
/**
* @internal This class is not covered by the backward compatibility promise for PHPUnit
*/
final class ThrowableBuilder
{
/**
* @throws Exception
* @throws NoPreviousThrowableException
*/
public static function from(\Throwable $t): Throwable
{
$previous = $t->getPrevious();
if ($previous !== null) {
$previous = self::from($previous);
}
$trace = Filter::getFilteredStacktrace($t);
if ($t instanceof RenderableOnCollisionEditor && $frame = $t->toCollisionEditor()) {
$file = $frame->getFile();
$line = $frame->getLine();
$trace = "$file:$line\n$trace";
}
return new Throwable(
$t::class,
$t->getMessage(),
ThrowableToStringMapper::map($t),
$trace,
$previous
);
}
}

View File

@ -12,6 +12,7 @@ parameters:
reportUnmatchedIgnoredErrors: true
ignoreErrors:
- "#has a nullable return type declaration.#"
- "#Language construct isset\\(\\) should not be used.#"
- "#is not allowed to extend#"
- "#is concrete, but does not have a Test suffix#"
@ -20,4 +21,3 @@ parameters:
- "# with null as default value#"
- "#has parameter \\$closure with default value.#"
- "#has parameter \\$description with default value.#"
- "#Method Pest\\\\Support\\\\Reflection::getParameterClassName\\(\\) has a nullable return type declaration.#"

View File

@ -18,11 +18,12 @@
<directory suffix=".php">./tests</directory>
<exclude>./tests/.snapshots</exclude>
<exclude>./tests/.tests</exclude>
<exclude>./tests/Fixtures/Inheritance</exclude>
</testsuite>
</testsuites>
<coverage>
<source>
<include>
<directory suffix=".php">./src</directory>
</include>
</coverage>
</source>
</phpunit>

View File

@ -24,6 +24,7 @@ final class BootOverrides implements Bootstrapper
'TextUI/Command/WarmCodeCoverageCacheCommand.php',
'TextUI/Output/Default/ProgressPrinter/TestSkippedSubscriber.php',
'TextUI/TestSuiteFilterProcessor.php',
'Event/Value/ThrowableBuilder.php',
];
/**

View File

@ -87,7 +87,7 @@ trait Testable
*/
public function __addBeforeAll(?Closure $hook): void
{
if ($hook === null) {
if (! $hook instanceof \Closure) {
return;
}
@ -101,7 +101,7 @@ trait Testable
*/
public function __addAfterAll(?Closure $hook): void
{
if ($hook === null) {
if (! $hook instanceof \Closure) {
return;
}
@ -131,7 +131,7 @@ trait Testable
*/
private function __addHook(string $property, ?Closure $hook): void
{
if ($hook === null) {
if (! $hook instanceof \Closure) {
return;
}
@ -181,7 +181,7 @@ trait Testable
parent::setUp();
$beforeEach = TestSuite::getInstance()->beforeEach->get(self::$__filename);
$beforeEach = TestSuite::getInstance()->beforeEach->get(self::$__filename)[1];
if ($this->__beforeEach instanceof Closure) {
$beforeEach = ChainableClosure::from($this->__beforeEach, $beforeEach);

View File

@ -7,7 +7,7 @@ namespace Pest\Contracts;
use Pest\Factories\TestCaseMethodFactory;
/**
* @interal
* @internal
*/
interface AddsAnnotations
{

View File

@ -167,7 +167,7 @@ final class Expectation
*
* @template TSequenceValue
*
* @param (callable(self<TValue>, self<string|int>): void)|TSequenceValue ...$callbacks
* @param (callable(self<TValue>, self<string|int>): void)|TSequenceValue ...$callbacks
* @return self<TValue>
*/
public function sequence(mixed ...$callbacks): self
@ -210,8 +210,8 @@ final class Expectation
*
* @template TMatchSubject of array-key
*
* @param (callable(): TMatchSubject)|TMatchSubject $subject
* @param array<TMatchSubject, (callable(self<TValue>): mixed)|TValue> $expressions
* @param (callable(): TMatchSubject)|TMatchSubject $subject
* @param array<TMatchSubject, (callable(self<TValue>): mixed)|TValue> $expressions
* @return self<TValue>
*/
public function match(mixed $subject, array $expressions): self
@ -248,7 +248,7 @@ final class Expectation
/**
* Apply the callback if the given "condition" is falsy.
*
* @param (callable(): bool)|bool $condition
* @param (callable(): bool)|bool $condition
* @param callable(Expectation<TValue>): mixed $callback
* @return self<TValue>
*/
@ -264,7 +264,7 @@ final class Expectation
/**
* Apply the callback if the given "condition" is truthy.
*
* @param (callable(): bool)|bool $condition
* @param (callable(): bool)|bool $condition
* @param callable(self<TValue>): mixed $callback
* @return self<TValue>
*/

View File

@ -14,7 +14,17 @@ final class TestDox implements AddsAnnotations
*/
public function __invoke(TestCaseMethodFactory $method, array $annotations): array
{
$annotations[] = "@testdox $method->description";
/*
* escapes docblock according to
* https://manual.phpdoc.org/HTMLframesConverter/default/phpDocumentor/tutorial_phpDocumentor.howto.pkg.html#basics.desc
*
* note: '@' escaping is not needed as it cannot be the first character of the line (it always starts with @testdox
*/
assert($method->description !== null);
$methodDescription = str_replace('*/', '{@*}', $method->description);
$annotations[] = "@testdox $methodDescription";
return $annotations;
}

View File

@ -14,7 +14,7 @@ use Pest\Factories\TestCaseMethodFactory;
final class Covers extends Attribute
{
/**
* Determine if the attribute should be placed above the classe instead of above the method.
* Determine if the attribute should be placed above the class instead of above the method.
*/
public static bool $above = true;

View File

@ -217,7 +217,7 @@ final class TestCaseFactory
}
if (! $method->receivesArguments()) {
if ($method->closure === null) {
if (! $method->closure instanceof \Closure) {
throw ShouldNotHappen::fromMessage('The test closure may not be empty.');
}

View File

@ -90,7 +90,7 @@ final class TestCaseMethodFactory
$method = $this;
return function () use ($testCase, $method, $closure): mixed { // @phpstan-ignore-line
/* @var TestCase $this */
/* @var TestCase $this */
$testCase->proxies->proxy($this);
$method->proxies->proxy($this);

View File

@ -43,7 +43,7 @@ if (! function_exists('beforeEach')) {
/**
* Runs the given closure before each test in the current file.
*
* @return HigherOrderTapProxy<TestCall|TestCase>|TestCall|mixed
* @return HigherOrderTapProxy<TestCall|TestCase>|TestCall|TestCase|mixed
*/
function beforeEach(Closure $closure = null): BeforeEachCall
{
@ -92,7 +92,7 @@ if (! function_exists('test')) {
*/
function test(string $description = null, Closure $closure = null): HigherOrderTapProxy|TestCall
{
if ($description === null && TestSuite::getInstance()->test !== null) {
if ($description === null && TestSuite::getInstance()->test instanceof \PHPUnit\Framework\TestCase) {
return new HigherOrderTapProxy(TestSuite::getInstance()->test);
}

View File

@ -11,6 +11,12 @@ use Pest\Support\Str;
use PHPUnit\Event\Code\Test;
use PHPUnit\Event\Code\TestMethod;
use PHPUnit\Event\Code\Throwable;
use PHPUnit\Event\Test\BeforeFirstTestMethodErrored;
use PHPUnit\Event\Test\ConsideredRisky;
use PHPUnit\Event\Test\Errored;
use PHPUnit\Event\Test\Failed;
use PHPUnit\Event\Test\MarkedIncomplete;
use PHPUnit\Event\Test\Skipped;
use PHPUnit\Event\TestSuite\TestSuite;
use PHPUnit\Framework\Exception as FrameworkException;
use PHPUnit\TestRunner\TestResult\TestResult as PhpUnitTestResult;
@ -64,7 +70,7 @@ final class Converter
}
/**
* Gets the exception messsage.
* Gets the exception message.
*/
public function getExceptionMessage(Throwable $throwable): string
{
@ -166,6 +172,14 @@ final class Converter
return $this->toRelativePath($path);
}
/**
* Gets the test suite size.
*/
public function getTestSuiteSize(TestSuite $testSuite): int
{
return $testSuite->count();
}
/**
* Transforms the given path in relative path.
*/
@ -180,12 +194,30 @@ final class Converter
*/
public function getStateFromResult(PhpUnitTestResult $result): State
{
$numberOfPassedTests = $result->numberOfTestsRun()
- $result->numberOfTestErroredEvents()
- $result->numberOfTestFailedEvents()
- $result->numberOfTestSkippedEvents()
- $result->numberOfTestsWithTestConsideredRiskyEvents()
- $result->numberOfTestMarkedIncompleteEvents();
$events = [
...$result->testErroredEvents(),
...$result->testFailedEvents(),
...$result->testSkippedEvents(),
...array_merge(...array_values($result->testConsideredRiskyEvents())),
...$result->testMarkedIncompleteEvents(),
];
$numberOfNotPassedTests = count(
array_unique(
array_map(
function (BeforeFirstTestMethodErrored|Errored|Failed|Skipped|ConsideredRisky|MarkedIncomplete $event): string {
if ($event instanceof BeforeFirstTestMethodErrored) {
return $event->testClassName();
}
return $this->getTestCaseLocation($event->test());
},
$events
)
)
);
$numberOfPassedTests = $result->numberOfTestsRun() - $numberOfNotPassedTests;
return $this->stateGenerator->fromPhpUnitTestResult($numberOfPassedTests, $result);
}

View File

@ -25,7 +25,7 @@ final class ServiceMessage
$paramsToString = '';
foreach ([...$this->parameters, 'flowId' => self::$flowId] as $key => $value) {
$value = self::escapeServiceMessage((string) $value);
$value = $this->escapeServiceMessage((string) $value);
$paramsToString .= " $key='$value'";
}
@ -40,6 +40,13 @@ final class ServiceMessage
]);
}
public static function testSuiteCount(int $count): self
{
return new self('testCount', [
'count' => $count,
]);
}
public static function testSuiteFinished(string $name): self
{
return new self('testSuiteFinished', [
@ -120,7 +127,7 @@ final class ServiceMessage
]);
}
private static function escapeServiceMessage(string $text): string
private function escapeServiceMessage(string $text): string
{
return str_replace(
['|', "'", "\n", "\r", ']', '['],

View File

@ -16,6 +16,7 @@ use Pest\Logging\TeamCity\Subscriber\TestPreparedSubscriber;
use Pest\Logging\TeamCity\Subscriber\TestSkippedSubscriber;
use Pest\Logging\TeamCity\Subscriber\TestSuiteFinishedSubscriber;
use Pest\Logging\TeamCity\Subscriber\TestSuiteStartedSubscriber;
use PHPUnit\Event\Code\Test;
use PHPUnit\Event\EventFacadeIsSealedException;
use PHPUnit\Event\Facade;
use PHPUnit\Event\Telemetry\Duration;
@ -34,6 +35,7 @@ use PHPUnit\Event\TestSuite\Finished as TestSuiteFinished;
use PHPUnit\Event\TestSuite\Started as TestSuiteStarted;
use PHPUnit\Event\UnknownSubscriberTypeException;
use PHPUnit\TestRunner\TestResult\Facade as TestResultFacade;
use ReflectionClass;
use Symfony\Component\Console\Output\ConsoleOutput;
use Symfony\Component\Console\Output\OutputInterface;
@ -44,6 +46,13 @@ final class TeamCityLogger
{
private ?HRTime $time = null;
private bool $isSummaryTestCountPrinted = false;
/**
* @var array<string, bool>
*/
private array $testEvents = [];
/**
* @throws EventFacadeIsSealedException
* @throws UnknownSubscriberTypeException
@ -66,6 +75,15 @@ final class TeamCityLogger
);
$this->output($message);
if (! $this->isSummaryTestCountPrinted) {
$this->isSummaryTestCountPrinted = true;
$message = ServiceMessage::testSuiteCount(
$this->converter->getTestSuiteSize($event->testSuite())
);
$this->output($message);
}
}
public function testSuiteFinished(TestSuiteFinished $event): void
@ -96,12 +114,14 @@ final class TeamCityLogger
public function testSkipped(Skipped $event): void
{
$message = ServiceMessage::testIgnored(
$this->converter->getTestCaseMethodName($event->test()),
'This test was ignored.'
);
$this->whenFirstEventForTest($event->test(), function () use ($event): void {
$message = ServiceMessage::testIgnored(
$this->converter->getTestCaseMethodName($event->test()),
'This test was ignored.'
);
$this->output($message);
$this->output($message);
});
}
/**
@ -110,17 +130,19 @@ final class TeamCityLogger
*/
public function testErrored(Errored $event): void
{
$testName = $this->converter->getTestCaseMethodName($event->test());
$message = $this->converter->getExceptionMessage($event->throwable());
$details = $this->converter->getExceptionDetails($event->throwable());
$this->whenFirstEventForTest($event->test(), function () use ($event): void {
$testName = $this->converter->getTestCaseMethodName($event->test());
$message = $this->converter->getExceptionMessage($event->throwable());
$details = $this->converter->getExceptionDetails($event->throwable());
$message = ServiceMessage::testFailed(
$testName,
$message,
$details,
);
$message = ServiceMessage::testFailed(
$testName,
$message,
$details,
);
$this->output($message);
$this->output($message);
});
}
/**
@ -129,28 +151,30 @@ final class TeamCityLogger
*/
public function testFailed(Failed $event): void
{
$testName = $this->converter->getTestCaseMethodName($event->test());
$message = $this->converter->getExceptionMessage($event->throwable());
$details = $this->converter->getExceptionDetails($event->throwable());
$this->whenFirstEventForTest($event->test(), function () use ($event): void {
$testName = $this->converter->getTestCaseMethodName($event->test());
$message = $this->converter->getExceptionMessage($event->throwable());
$details = $this->converter->getExceptionDetails($event->throwable());
if ($event->hasComparisonFailure()) {
$comparison = $event->comparisonFailure();
$message = ServiceMessage::comparisonFailure(
$testName,
$message,
$details,
$comparison->actual(),
$comparison->expected()
);
} else {
$message = ServiceMessage::testFailed(
$testName,
$message,
$details,
);
}
if ($event->hasComparisonFailure()) {
$comparison = $event->comparisonFailure();
$message = ServiceMessage::comparisonFailure(
$testName,
$message,
$details,
$comparison->actual(),
$comparison->expected()
);
} else {
$message = ServiceMessage::testFailed(
$testName,
$message,
$details,
);
}
$this->output($message);
$this->output($message);
});
}
/**
@ -159,17 +183,19 @@ final class TeamCityLogger
*/
public function testConsideredRisky(ConsideredRisky $event): void
{
$message = ServiceMessage::testIgnored(
$this->converter->getTestCaseMethodName($event->test()),
$event->message()
);
$this->whenFirstEventForTest($event->test(), function () use ($event): void {
$message = ServiceMessage::testIgnored(
$this->converter->getTestCaseMethodName($event->test()),
$event->message()
);
$this->output($message);
$this->output($message);
});
}
public function testFinished(Finished $event): void
{
if ($this->time === null) {
if (! $this->time instanceof \PHPUnit\Event\Telemetry\HRTime) {
throw ShouldNotHappen::fromMessage('Start time has not been set.');
}
@ -196,13 +222,17 @@ final class TeamCityLogger
$style = new Style($this->output);
$telemetry = $event->telemetryInfo();
if ($this->withoutDuration) {
$reflector = new ReflectionClass($telemetry);
$property = $reflector->getProperty('current');
$property->setAccessible(true);
$snapshot = $property->getValue($telemetry);
assert($snapshot instanceof Snapshot);
$telemetry = new Info(
new Snapshot(
$telemetry->time(),
$telemetry->memoryUsage(),
$telemetry->peakMemoryUsage(),
),
$snapshot,
Duration::fromSecondsAndNanoseconds(1, 0),
$telemetry->memoryUsageSinceStart(),
$telemetry->durationSincePrevious(),
@ -248,4 +278,14 @@ final class TeamCityLogger
ServiceMessage::setFlowId($this->flowId);
}
private function whenFirstEventForTest(Test $test, callable $callback): void
{
$testIdentifier = $this->converter->getTestCaseLocation($test);
if (! isset($this->testEvents[$testIdentifier])) {
$this->testEvents[$testIdentifier] = true;
$callback();
}
}
}

View File

@ -296,8 +296,8 @@ final class Expectation
*/
public function toHaveProperties(iterable $names, string $message = ''): self
{
foreach ($names as $name) {
$this->toHaveProperty($name, message: $message);
foreach ($names as $name => $value) {
is_int($name) ? $this->toHaveProperty($value, message: $message) : $this->toHaveProperty($name, $value, $message);
}
return $this;
@ -839,10 +839,10 @@ final class Expectation
/**
* Asserts that executing value throws an exception.
*
* @param (Closure(Throwable): mixed)|string $exception
* @param (Closure(Throwable): mixed)|string $exception
* @return self<TValue>
*/
public function toThrow(callable|string $exception, string $exceptionMessage = null, string $message = ''): self
public function toThrow(callable|string|Throwable $exception, string $exceptionMessage = null, string $message = ''): self
{
$callback = NullClosure::create();
@ -864,6 +864,15 @@ final class Expectation
try {
($this->value)();
} catch (Throwable $e) {
if ($exception instanceof Throwable) {
expect($e)
->toBeInstanceOf($exception::class, $message)
->and($e->getMessage())->toBe($exceptionMessage ?? $exception->getMessage(), $message);
return $this;
}
if (! class_exists($exception)) {
if ($e instanceof Error && $e->getMessage() === "Class \"$exception\" not found") {
Assert::assertTrue(true);
@ -888,7 +897,7 @@ final class Expectation
Assert::assertTrue(true);
if (! class_exists($exception)) {
if (! $exception instanceof Throwable && ! class_exists($exception)) {
throw new ExpectationFailedException("Exception with message \"$exception\" not thrown.");
}
@ -900,7 +909,7 @@ final class Expectation
*/
private function export(mixed $value): string
{
if ($this->exporter === null) {
if (! $this->exporter instanceof \Pest\Support\Exporter) {
$this->exporter = Exporter::default();
}

View File

@ -19,12 +19,17 @@ final class BeforeEachCall
/**
* Holds the before each closure.
*/
private readonly \Closure $closure;
private readonly Closure $closure;
/**
* The calls that should be proxied.
* The test call proxies.
*/
private readonly HigherOrderMessageCollection $proxies;
private readonly HigherOrderMessageCollection $testCallProxies;
/**
* The test case proxies.
*/
private readonly HigherOrderMessageCollection $testCaseProxies;
/**
* Creates a new Pending Call.
@ -36,7 +41,8 @@ final class BeforeEachCall
) {
$this->closure = $closure instanceof Closure ? $closure : NullClosure::create();
$this->proxies = new HigherOrderMessageCollection();
$this->testCallProxies = new HigherOrderMessageCollection();
$this->testCaseProxies = new HigherOrderMessageCollection();
}
/**
@ -44,13 +50,16 @@ final class BeforeEachCall
*/
public function __destruct()
{
$proxies = $this->proxies;
$testCaseProxies = $this->testCaseProxies;
$this->testSuite->beforeEach->set(
$this->filename,
ChainableClosure::from(function () use ($proxies): void {
$proxies->chain($this);
}, $this->closure)
function (TestCall $testCall): void {
$this->testCallProxies->chain($testCall);
},
ChainableClosure::from(function () use ($testCaseProxies): void {
$testCaseProxies->chain($this);
}, $this->closure),
);
}
@ -61,7 +70,13 @@ final class BeforeEachCall
*/
public function __call(string $name, array $arguments): self
{
$this->proxies
if (method_exists(TestCall::class, $name)) {
$this->testCallProxies->add(Backtrace::file(), Backtrace::line(), $name, $arguments);
return $this;
}
$this->testCaseProxies
->add(Backtrace::file(), Backtrace::line(), $name, $arguments);
return $this;

View File

@ -40,12 +40,15 @@ final class TestCall
*/
public function __construct(
private readonly TestSuite $testSuite,
string $filename,
private readonly string $filename,
string $description = null,
Closure $closure = null
) {
$this->testCaseMethod = new TestCaseMethodFactory($filename, $description, $closure);
$this->descriptionLess = $description === null;
$this->testSuite->beforeEach->get($filename)[0]($this);
}
/**
@ -81,7 +84,7 @@ final class TestCall
/**
* Asserts that the test throws the given `$exceptionClass` when called if the given condition is true.
*
* @param (callable(): bool)|bool $condition
* @param (callable(): bool)|bool $condition
*/
public function throwsIf(callable|bool $condition, string|int $exception, string $exceptionMessage = null, int $exceptionCode = null): self
{
@ -167,11 +170,45 @@ final class TestCall
$this->testCaseMethod
->chains
->addWhen($condition, Backtrace::file(), Backtrace::line(), 'markTestSkipped', [$message]);
->addWhen($condition, $this->filename, Backtrace::line(), 'markTestSkipped', [$message]);
return $this;
}
/**
* Skips the current test if the given test is running on Windows.
*/
public function skipOnWindows(): self
{
return $this->skipOn('Windows', 'This test is skipped on [Windows].');
}
/**
* Skips the current test if the given test is running on Mac OS.
*/
public function skipOnMac(): self
{
return $this->skipOn('Darwin', 'This test is skipped on [Mac].');
}
/**
* Skips the current test if the given test is running on Linux.
*/
public function skipOnLinux(): self
{
return $this->skipOn('Linux', 'This test is skipped on [Linux].');
}
/**
* Skips the current test if the given test is running on the given operating systems.
*/
private function skipOn(string $osFamily, string $message): self
{
return PHP_OS_FAMILY === $osFamily
? $this->skip($message)
: $this;
}
/**
* Sets the test as "todo".
*/
@ -190,7 +227,7 @@ final class TestCall
public function covers(string ...$classesOrFunctions): self
{
foreach ($classesOrFunctions as $classOrFunction) {
$isClass = class_exists($classOrFunction);
$isClass = class_exists($classOrFunction) || trait_exists($classOrFunction);
$isMethod = function_exists($classOrFunction);
if (! $isClass && ! $isMethod) {

View File

@ -6,7 +6,7 @@ namespace Pest;
function version(): string
{
return '2.3.0';
return '2.7.0';
}
function testDirectory(string $file = ''): string

View File

@ -109,20 +109,16 @@ final class Help implements HandlesArguments
],
], ...$content['Execution']];
$content['Selection'] = array_merge([
[
'arg' => '--bail',
'desc' => 'Stop execution upon first not-passed test',
],
[
'arg' => '--todos',
'desc' => 'Output to standard output the list of todos',
],
[
'arg' => '--retry',
'desc' => 'Run non-passing tests first and stop execution upon first error or failure',
],
], $content['Selection']);
$content['Selection'] = [[
'arg' => '--bail',
'desc' => 'Stop execution upon first not-passed test',
], [
'arg' => '--todos',
'desc' => 'Output to standard output the list of todos',
], [
'arg' => '--retry',
'desc' => 'Run non-passing tests first and stop execution upon first error or failure',
], ...$content['Selection']];
$content['Reporting'] = [...$content['Reporting'], ...[
[
@ -131,16 +127,13 @@ final class Help implements HandlesArguments
],
]];
$content['Code Coverage'] = array_merge([
[
'arg' => '--coverage ',
'desc' => 'Generate code coverage report and output to standard output',
],
[
'arg' => '--coverage --min',
'desc' => 'Set the minimum required coverage percentage, and fail if not met',
],
], $content['Code Coverage']);
$content['Code Coverage'] = [[
'arg' => '--coverage ',
'desc' => 'Generate code coverage report and output to standard output',
], [
'arg' => '--coverage --min',
'desc' => 'Set the minimum required coverage percentage, and fail if not met',
], ...$content['Code Coverage']];
$content['Profiling'] = [
[

View File

@ -26,11 +26,11 @@ final class Init implements HandlesArguments
* The files that will be created.
*/
private const STUBS = [
'phpunit.xml' => 'phpunit.xml',
'Pest.php' => 'tests/Pest.php',
'TestCase.php' => 'tests/TestCase.php',
'Unit/ExampleTest.php' => 'tests/Unit/ExampleTest.php',
'Feature/ExampleTest.php' => 'tests/Feature/ExampleTest.php',
'phpunit.xml.stub' => 'phpunit.xml',
'Pest.php.stub' => 'tests/Pest.php',
'TestCase.php.stub' => 'tests/TestCase.php',
'Unit/ExampleTest.php.stub' => 'tests/Unit/ExampleTest.php',
'Feature/ExampleTest.php.stub' => 'tests/Feature/ExampleTest.php',
];
/**

View File

@ -12,20 +12,20 @@ use function fread;
use function fseek;
use function ftell;
use function fwrite;
use NunoMaduro\Collision\Adapters\Phpunit\State;
use ParaTest\Options;
use Pest\Plugins\Parallel\Support\CompactPrinter;
use Pest\Support\StateGenerator;
use PHPUnit\TestRunner\TestResult\TestResult;
use PHPUnit\TextUI\Output\Printer;
use function preg_replace;
use SebastianBergmann\Timer\Duration;
use SplFileInfo;
use function strlen;
use Symfony\Component\Console\Formatter\OutputFormatter;
use Symfony\Component\Console\Output\OutputInterface;
/** @internal */
/**
* @internal
*/
final class ResultPrinter
{
/**
@ -51,7 +51,7 @@ final class ResultPrinter
/** @var resource|null */
private $teamcityLogFileHandle;
/** @var array<string, int> */
/** @var array<non-empty-string, int> */
private array $tailPositions;
public function __construct(
@ -74,6 +74,7 @@ final class ResultPrinter
if (str_starts_with($buffer, 'done [')) {
return;
}
$this->output->write(OutputFormatter::escape($buffer));
}
@ -93,9 +94,12 @@ final class ResultPrinter
$this->teamcityLogFileHandle = $teamcityLogFileHandle;
}
/** @param array<int, SplFileInfo> $teamcityFiles */
public function printFeedback(SplFileInfo $progressFile, array $teamcityFiles): void
{
/** @param list<SplFileInfo> $teamcityFiles */
public function printFeedback(
SplFileInfo $progressFile,
SplFileInfo $outputFile,
array $teamcityFiles
): void {
if ($this->options->needsTeamcity) {
$teamcityProgress = $this->tailMultiple($teamcityFiles);
@ -115,6 +119,15 @@ final class ResultPrinter
return;
}
$unexpectedOutput = $this->tail($outputFile);
if ($unexpectedOutput !== '') {
if (preg_match('/^T+$/', $unexpectedOutput) > 0) {
return;
}
$this->output->write($unexpectedOutput);
}
$feedbackItems = $this->tail($progressFile);
if ($feedbackItems === '') {
return;
@ -129,8 +142,8 @@ final class ResultPrinter
}
/**
* @param array<int, SplFileInfo> $teamcityFiles
* @param array<int, SplFileInfo> $testdoxFiles
* @param list<SplFileInfo> $teamcityFiles
* @param list<SplFileInfo> $testdoxFiles
*/
public function printResults(TestResult $testResult, array $teamcityFiles, array $testdoxFiles, Duration $duration): void
{
@ -183,7 +196,7 @@ final class ResultPrinter
$this->compactPrinter->descriptionItem($item);
}
/** @param array<int, SplFileInfo> $files */
/** @param list<SplFileInfo> $files */
private function tailMultiple(array $files): string
{
$content = '';
@ -201,6 +214,7 @@ final class ResultPrinter
private function tail(SplFileInfo $file): string
{
$path = $file->getPathname();
assert($path !== '');
$handle = fopen($path, 'r');
assert($handle !== false);
$fseek = fseek($handle, $this->tailPositions[$path] ?? 0);

View File

@ -48,36 +48,39 @@ final class WrapperRunner implements RunnerInterface
private readonly Timer $timer;
/** @var array<int, string> */
/** @var list<non-empty-string> */
private array $pending = [];
private int $exitCode = -1;
private int $exitcode = -1;
/** @var array<int,WrapperWorker> */
/** @var array<positive-int,WrapperWorker> */
private array $workers = [];
/** @var array<int,int> */
private array $batches = [];
/** @var array<int, SplFileInfo> */
/** @var list<SplFileInfo> */
private array $unexpectedOutputFiles = [];
/** @var list<SplFileInfo> */
private array $testresultFiles = [];
/** @var array<int, SplFileInfo> */
/** @var list<SplFileInfo> */
private array $coverageFiles = [];
/** @var array<int, SplFileInfo> */
/** @var list<SplFileInfo> */
private array $junitFiles = [];
/** @var array<int, SplFileInfo> */
/** @var list<SplFileInfo> */
private array $teamcityFiles = [];
/** @var array<int, SplFileInfo> */
/** @var list<SplFileInfo> */
private array $testdoxFiles = [];
/** @var array<int, string> */
/** @var non-empty-string[] */
private readonly array $parameters;
private readonly CodeCoverageFilterRegistry $codeCoverageFilterRegistry;
private CodeCoverageFilterRegistry $codeCoverageFilterRegistry;
public function __construct(
private readonly Options $options,
@ -86,11 +89,10 @@ final class WrapperRunner implements RunnerInterface
$this->printer = new ResultPrinter($output, $options);
$this->timer = new Timer();
$worker = realpath(
$wrapper = realpath(
dirname(__DIR__, 4).DIRECTORY_SEPARATOR.'bin'.DIRECTORY_SEPARATOR.'worker.php',
);
assert($worker !== false);
assert($wrapper !== false);
$phpFinder = new PhpExecutableFinder();
$phpBin = $phpFinder->find(false);
assert($phpBin !== false);
@ -101,7 +103,7 @@ final class WrapperRunner implements RunnerInterface
$parameters = array_merge($parameters, $options->passthruPhp);
}
$parameters[] = $worker;
$parameters[] = $wrapper;
$this->parameters = $parameters;
$this->codeCoverageFilterRegistry = new CodeCoverageFilterRegistry();
@ -110,14 +112,15 @@ final class WrapperRunner implements RunnerInterface
public function run(): int
{
$directory = dirname(__DIR__);
assert(strlen($directory) > 0);
assert($directory !== '');
ExcludeList::addDirectory($directory);
TestResultFacade::init();
EventFacade::instance()->seal();
$suiteLoader = new SuiteLoader($this->options, $this->output, $this->codeCoverageFilterRegistry);
$suiteLoader = new SuiteLoader(
$this->options,
$this->output,
$this->codeCoverageFilterRegistry,
);
$this->pending = $this->getTestFiles($suiteLoader);
$result = TestResultFacade::result();
@ -142,7 +145,7 @@ final class WrapperRunner implements RunnerInterface
{
$batchSize = $this->options->maxBatchSize;
while ($this->pending !== [] && $this->workers !== []) {
while (count($this->pending) > 0 && count($this->workers) > 0) {
foreach ($this->workers as $token => $worker) {
if (! $worker->isRunning()) {
throw $worker->getWorkerCrashedException();
@ -160,13 +163,11 @@ final class WrapperRunner implements RunnerInterface
}
if (
$this->exitCode > 0
$this->exitcode > 0
&& $this->options->configuration->stopOnFailure()
) {
$this->pending = [];
} elseif (($pending = array_shift($this->pending)) !== null) {
$this->debug(sprintf('Assigning %s to worker %d', $pending, $token));
$worker->assign($pending);
$this->batches[$token]++;
}
@ -178,9 +179,10 @@ final class WrapperRunner implements RunnerInterface
private function flushWorker(WrapperWorker $worker): void
{
$this->exitCode = max($this->exitCode, $worker->getExitCode());
$this->exitcode = max($this->exitcode, $worker->getExitCode());
$this->printer->printFeedback(
$worker->progressFile,
$worker->unexpectedOutputFile,
$this->teamcityFiles,
);
$worker->reset();
@ -189,10 +191,10 @@ final class WrapperRunner implements RunnerInterface
private function waitForAllToFinish(): void
{
$stopped = [];
while ($this->workers !== []) {
while (count($this->workers) > 0) {
foreach ($this->workers as $index => $worker) {
if ($worker->isRunning()) {
if (! array_key_exists($index, $stopped) && $worker->isFree()) {
if (! isset($stopped[$index]) && $worker->isFree()) {
$worker->stop();
$stopped[$index] = true;
}
@ -212,22 +214,19 @@ final class WrapperRunner implements RunnerInterface
}
}
/** @param positive-int $token */
private function startWorker(int $token): WrapperWorker
{
/** @var array<non-empty-string> $parameters */
$parameters = $this->parameters;
$worker = new WrapperWorker(
$this->output,
$this->options,
$parameters,
$this->parameters,
$token,
);
$worker->start();
$this->batches[$token] = 0;
$this->unexpectedOutputFiles[] = $worker->unexpectedOutputFile;
$this->testresultFiles[] = $worker->testresultFile;
if (isset($worker->junitFile)) {
@ -330,15 +329,16 @@ final class WrapperRunner implements RunnerInterface
$this->generateCodeCoverageReports();
$this->generateLogs();
$exitCode = Result::exitCode($this->options->configuration, $testResultSum);
$exitcode = Result::exitCode($this->options->configuration, $testResultSum);
$this->clearFiles($this->unexpectedOutputFiles);
$this->clearFiles($this->testresultFiles);
$this->clearFiles($this->coverageFiles);
$this->clearFiles($this->junitFiles);
$this->clearFiles($this->teamcityFiles);
$this->clearFiles($this->testdoxFiles);
return $exitCode;
return $exitcode;
}
private function generateCodeCoverageReports(): void
@ -348,10 +348,11 @@ final class WrapperRunner implements RunnerInterface
}
$coverageManager = new CodeCoverage();
// @phpstan-ignore-next-line
is_bool(true) && $coverageManager->init($this->options->configuration, $this->codeCoverageFilterRegistry, true);
$coverageManager->init(
$this->options->configuration,
$this->codeCoverageFilterRegistry,
false,
);
$coverageMerger = new CoverageMerger($coverageManager->codeCoverage());
foreach ($this->coverageFiles as $coverageFile) {
$coverageMerger->addCoverageFromFile($coverageFile);
@ -376,7 +377,7 @@ final class WrapperRunner implements RunnerInterface
);
}
/** @param array<int, SplFileInfo> $files */
/** @param list<SplFileInfo> $files */
private function clearFiles(array $files): void
{
foreach ($files as $file) {
@ -391,31 +392,19 @@ final class WrapperRunner implements RunnerInterface
/**
* Returns the test files to be executed.
*
* @return array<int, string>
* @return array<int, non-empty-string>
*/
private function getTestFiles(SuiteLoader $suiteLoader): array
{
$this->debug(sprintf('Found %d test file%s', count($suiteLoader->files), count($suiteLoader->files) === 1 ? '' : 's'));
/** @var array<string, string> $files */
$files = $suiteLoader->files;
return [
/** @var array<string, non-empty-string> $files */
$files = [
...array_values(array_filter(
$files,
$suiteLoader->tests,
fn (string $filename): bool => ! str_ends_with($filename, "eval()'d code")
)),
...TestSuite::getInstance()->tests->getFilenames(),
];
}
/**
* Prints a debug message.
*/
private function debug(string $message): void
{
if ($this->options->verbose) {
$this->output->writeln(" <fg=blue>{$message}</> ");
}
return $files; // @phpstan-ignore-line
}
}

View File

@ -7,6 +7,7 @@ namespace Pest\Plugins\Parallel\Support;
use NunoMaduro\Collision\Adapters\Phpunit\State;
use NunoMaduro\Collision\Adapters\Phpunit\Style;
use ParaTest\Options;
use PHPUnit\Event\Telemetry\GarbageCollectorStatus;
use PHPUnit\Event\Telemetry\HRTime;
use PHPUnit\Event\Telemetry\Info;
use PHPUnit\Event\Telemetry\MemoryUsage;
@ -122,11 +123,25 @@ final class CompactPrinter
$snapshotDuration = HRTime::fromSecondsAndNanoseconds((int) $duration->asSeconds(), $nanoseconds);
$telemetryDuration = \PHPUnit\Event\Telemetry\Duration::fromSecondsAndNanoseconds((int) $duration->asSeconds(), $nanoseconds);
$status = gc_status();
$garbageCollectorStatus = new GarbageCollectorStatus(
$status['runs'],
$status['collected'],
$status['threshold'],
$status['roots'],
null,
null,
null,
null,
);
$telemetry = new Info(
new Snapshot(
$snapshotDuration,
MemoryUsage::fromBytes(0),
MemoryUsage::fromBytes(0),
$garbageCollectorStatus,
),
$telemetryDuration,
MemoryUsage::fromBytes(0),

View File

@ -14,27 +14,34 @@ use Pest\Support\NullClosure;
final class BeforeEachRepository
{
/**
* @var array<string, Closure>
* @var array<string, array{0: Closure, 1: Closure}>
*/
private array $state = [];
/**
* Sets a before each closure.
*/
public function set(string $filename, Closure $closure): void
public function set(string $filename, Closure $beforeEachTestCall, Closure $beforeEachTestCase): void
{
if (array_key_exists($filename, $this->state)) {
throw new BeforeEachAlreadyExist($filename);
}
$this->state[$filename] = $closure;
$this->state[$filename] = [$beforeEachTestCall, $beforeEachTestCase];
}
/**
* Gets a before each closure by the given filename.
*
* @return array{0: Closure, 1: Closure}
*/
public function get(string $filename): Closure
public function get(string $filename): array
{
return $this->state[$filename] ?? NullClosure::create();
$closures = $this->state[$filename] ?? [];
return [
$closures[0] ?? NullClosure::create(),
$closures[1] ?? NullClosure::create(),
];
}
}

View File

@ -39,15 +39,13 @@ final class Result
*/
public static function exitCode(Configuration $configuration, TestResult $result): int
{
$returnCode = self::FAILURE_EXIT;
if ($result->wasSuccessfulIgnoringPhpunitWarnings()
&& ! $result->hasTestTriggeredPhpunitWarningEvents()) {
$returnCode = self::SUCCESS_EXIT;
return self::SUCCESS_EXIT;
}
if ($configuration->failOnEmptyTestSuite() && $result->numberOfTests() === 0) {
$returnCode = self::FAILURE_EXIT;
return self::FAILURE_EXIT;
}
if ($result->wasSuccessfulIgnoringPhpunitWarnings()) {
@ -76,6 +74,6 @@ final class Result
return self::EXCEPTION_EXIT;
}
return $returnCode;
return self::FAILURE_EXIT;
}
}

View File

@ -30,7 +30,10 @@ final class Backtrace
$traceFile = str_replace(DIRECTORY_SEPARATOR, '/', $trace[self::FILE]);
if (Str::endsWith($traceFile, 'overrides/Runner/TestSuiteLoader.php')) {
if (
Str::endsWith($traceFile, 'overrides/Runner/TestSuiteLoader.php') ||
Str::endsWith($traceFile, 'src/Bootstrappers/BootFiles.php')
) {
break;
}
@ -101,8 +104,6 @@ final class Backtrace
{
$trace = debug_backtrace(self::BACKTRACE_OPTIONS)[1];
assert(array_key_exists('line', $trace));
return $trace['line'];
return $trace['line'] ?? 0;
}
}

View File

@ -25,7 +25,7 @@ final class Container
*/
public static function getInstance(): self
{
if (self::$instance === null) {
if (! self::$instance instanceof \Pest\Support\Container) {
self::$instance = new self();
}
@ -72,7 +72,7 @@ final class Container
if ($reflectionClass->isInstantiable()) {
$constructor = $reflectionClass->getConstructor();
if ($constructor !== null) {
if ($constructor instanceof \ReflectionMethod) {
$params = array_map(
function (ReflectionParameter $param) use ($id): object|string {
$candidate = Reflection::getParameterClassName($param);
@ -80,7 +80,7 @@ final class Container
if ($candidate === null) {
$type = $param->getType();
/* @phpstan-ignore-next-line */
if ($type !== null && $type->isBuiltin()) {
if ($type instanceof \ReflectionType && $type->isBuiltin()) {
$candidate = $param->getName();
} else {
throw ShouldNotHappen::fromMessage(sprintf('The type of `$%s` in `%s` cannot be determined.', $id, $param->getName()));

View File

@ -42,6 +42,14 @@ final class Coverage
return false;
}
if ($runtime->hasPCOV()) {
return true;
}
if ($runtime->hasPHPDBGCodeCoverage()) {
return true;
}
if (! $runtime->hasXdebug()) {
return true;
}

View File

@ -33,7 +33,7 @@ final class ExceptionTrace
$message = str_replace(self::UNDEFINED_METHOD, 'Call to undefined method ', $message);
if (class_exists($class) && count(class_parents($class)) > 0 && array_values(class_parents($class))[0] === TestCase::class) {
$message .= '. Did you forget to use the [uses()] function? https://pestphp.com/docs/configuring-tests';
$message .= '. Did you forget to use the [uses()] function? Read more at: https://pestphp.com/docs/configuring-tests';
}
Reflection::setPropertyValue($throwable, 'message', $message);

View File

@ -25,7 +25,7 @@ final class HigherOrderCallables
*
* Create a new expectation. Callable values will be executed prior to returning the new expectation.
*
* @param (Closure():TValue)|TValue $value
* @param (Closure():TValue)|TValue $value
* @return Expectation<TValue>
*/
public function expect(mixed $value): Expectation

View File

@ -62,7 +62,7 @@ final class HigherOrderMessage
Reflection::setPropertyValue($throwable, 'file', $this->filename);
Reflection::setPropertyValue($throwable, 'line', $this->line);
if ($throwable->getMessage() === self::getUndefinedMethodMessage($target, $this->name)) {
if ($throwable->getMessage() === $this->getUndefinedMethodMessage($target, $this->name)) {
/** @var ReflectionClass<TValue> $reflection */
$reflection = new ReflectionClass($target);
/* @phpstan-ignore-next-line */
@ -94,7 +94,7 @@ final class HigherOrderMessage
return in_array($this->name, get_class_methods(HigherOrderCallables::class), true);
}
private static function getUndefinedMethodMessage(object $target, string $methodName): string
private function getUndefinedMethodMessage(object $target, string $methodName): string
{
if (\PHP_MAJOR_VERSION >= 8) {
return sprintf(self::UNDEFINED_METHOD, sprintf('%s::%s()', $target::class, $methodName));

View File

@ -1,4 +1,5 @@
##teamcity[testSuiteStarted name='Tests/tests/Failure' locationHint='file://tests/.tests/Failure.php' flowId='1234']
##teamcity[testCount count='8' flowId='1234']
##teamcity[testStarted name='it can fail with comparison' locationHint='pest_qn://tests/.tests/Failure.php::it can fail with comparison' flowId='1234']
##teamcity[testFailed name='it can fail with comparison' message='Failed asserting that true matches expected false.' details='at src/Mixins/Expectation.php:343|nat src/Support/ExpectationPipeline.php:75|nat src/Support/ExpectationPipeline.php:79|nat src/Expectation.php:300|nat tests/.tests/Failure.php:6|nat src/Factories/TestCaseMethodFactory.php:100|nat src/Concerns/Testable.php:302|nat src/Support/ExceptionTrace.php:28|nat src/Concerns/Testable.php:302|nat src/Concerns/Testable.php:221|nat src/Kernel.php:86' type='comparisonFailure' actual='true' expected='false' flowId='1234']
##teamcity[testFinished name='it can fail with comparison' duration='100000' flowId='1234']
@ -11,14 +12,19 @@
##teamcity[testStarted name='it can fail' locationHint='pest_qn://tests/.tests/Failure.php::it can fail' flowId='1234']
##teamcity[testFailed name='it can fail' message='oh noo' details='at tests/.tests/Failure.php:18|nat src/Factories/TestCaseMethodFactory.php:100|nat src/Concerns/Testable.php:302|nat src/Support/ExceptionTrace.php:28|nat src/Concerns/Testable.php:302|nat src/Concerns/Testable.php:221|nat src/Kernel.php:86' flowId='1234']
##teamcity[testFinished name='it can fail' duration='100000' flowId='1234']
##teamcity[testStarted name='it throws exception' locationHint='pest_qn://tests/.tests/Failure.php::it throws exception' flowId='1234']
##teamcity[testFailed name='it throws exception' message='Exception: test error' details='at tests/.tests/Failure.php:22|nat src/Factories/TestCaseMethodFactory.php:100|nat src/Concerns/Testable.php:302|nat src/Support/ExceptionTrace.php:28|nat src/Concerns/Testable.php:302|nat src/Concerns/Testable.php:221|nat src/Kernel.php:86' flowId='1234']
##teamcity[testFinished name='it throws exception' duration='100000' flowId='1234']
##teamcity[testStarted name='it is not done yet' locationHint='pest_qn://tests/.tests/Failure.php::it is not done yet' flowId='1234']
##teamcity[testIgnored name='it is not done yet' message='This test was ignored.' details='' flowId='1234']
##teamcity[testFinished name='it is not done yet' duration='100000' flowId='1234']
##teamcity[testStarted name='build this one.' locationHint='pest_qn://tests/.tests/Failure.php::build this one.' flowId='1234']
##teamcity[testIgnored name='build this one.' message='This test was ignored.' details='' flowId='1234']
##teamcity[testFinished name='build this one.' duration='100000' flowId='1234']
##teamcity[testStarted name='it is passing' locationHint='pest_qn://tests/.tests/Failure.php::it is passing' flowId='1234']
##teamcity[testFinished name='it is passing' duration='100000' flowId='1234']
##teamcity[testSuiteFinished name='Tests/tests/Failure' flowId='1234']
Tests: 2 failed, 1 risky, 2 todos, 1 skipped (2 assertions)
Tests: 3 failed, 1 risky, 2 todos, 1 skipped, 1 passed (3 assertions)
Duration: 1.00s

View File

@ -1,4 +1,5 @@
##teamcity[testSuiteStarted name='Tests/tests/SuccessOnly' locationHint='file://tests/.tests/SuccessOnly.php' flowId='1234']
##teamcity[testCount count='2' flowId='1234']
##teamcity[testStarted name='it can pass with comparison' locationHint='pest_qn://tests/.tests/SuccessOnly.php::it can pass with comparison' flowId='1234']
##teamcity[testFinished name='it can pass with comparison' duration='100000' flowId='1234']
##teamcity[testStarted name='can also pass' locationHint='pest_qn://tests/.tests/SuccessOnly.php::can also pass' flowId='1234']

View File

@ -8,4 +8,10 @@
PASS Tests\Fixtures\ExampleTest
✓ it example 2
Tests: 2 skipped, 2 passed (2 assertions)
WARN Tests\Fixtures\Inheritance\Base\ExampleTest
- example
PASS Tests\Fixtures\Inheritance\ExampleTest
✓ example
Tests: 3 skipped, 3 passed (3 assertions)

View File

@ -1,5 +1,5 @@
.
..
────────────────────────────────────────────────────────────────────────────
FAILED Tests\Fixtures\CollisionTest > error Exception
error
@ -19,4 +19,4 @@
2 src/Factories/TestCaseMethodFactory.php:100
Tests: 1 failed, 1 passed (1 assertions)
Tests: 1 failed, 2 passed (2 assertions)

View File

@ -1,5 +1,5 @@
Pest Testing Framework 2.3.0.
Pest Testing Framework 2.7.0.
USAGE: pest <file> [options]
@ -43,15 +43,19 @@
--enforce-time-limit ................. Enforce time limit based on test size
--default-time-limit [sec] Timeout in seconds for tests that have no declared size
--dont-report-useless-tests .. Do not report tests that do not test anything
--stop-on-defect ... Stop after first error, failure, warning or, risky test
--stop-on-defect ... Stop after first error, failure, warning, or risky test
--stop-on-error ..................................... Stop after first error
--stop-on-failure ................................. Stop after first failure
--stop-on-warning ................................. Stop after first warning
--stop-on-risky ................................ Stop after first risky test
--stop-on-deprecation ... Stop after first test that triggered a deprecation
--stop-on-notice ............. Stop after first test that triggered a notice
--stop-on-skipped ............................ Stop after first skipped test
--stop-on-incomplete ...................... Stop after first incomplete test
--fail-on-warning Signal failure using shell exit code when a warning was triggered
--fail-on-risky Signal failure using shell exit code when a test was considered risky
--fail-on-deprecation Signal failure using shell exit code when a deprecation was triggered
--fail-on-notice Signal failure using shell exit code when a notice was triggered
--fail-on-skipped Signal failure using shell exit code when a test was skipped
--fail-on-incomplete Signal failure using shell exit code when a test was marked incomplete
--cache-result ............................ Write test results to cache file

View File

@ -22,6 +22,22 @@
✓ it gets executed before each test
✓ it gets executed before each test once again
PASS Tests\Features\BeforeEachProxiesToTestCallWithExpectations
✓ runs 1
✓ runs 2
✓ runs 3
WARN Tests\Features\BeforeEachProxiesToTestCallWithSkip
- does not run 1
- does not run 2
- does not run 3
TODO Tests\Features\BeforeEachProxiesToTestCallWithTodo - 4 todos
↓ is marked as todo 1
↓ is marked as todo 2
↓ is marked as todo 3
↓ shouldBeMarkedAsTodo
WARN Tests\Features\Coverage
✓ it has plugin
- it adds coverage if --coverage exist → Coverage is not available
@ -33,6 +49,7 @@
✓ it uses the correct PHPUnit attribute for function
✓ it removes duplicated attributes
✓ it guesses if the given argument is a class or function
✓ it uses the correct PHPUnit attribute for trait
✓ it appends CoversNothing to method attributes
✓ it does not append CoversNothing to other methods
✓ it throws exception if no class nor method has been found
@ -152,7 +169,7 @@
✓ it uses correct parent class
DEPR Tests\Features\Deprecated
! deprecated → str_contains(): Passing null to parameter #2 ($needle) of type string is deprecated // tests/Features/Deprecated.php:6
! deprecated → str_contains(): Passing null to parameter #1 ($haystack) of type string is deprecated // tests/Features/Deprecated.php:6
! user deprecated → Since foo 1.0: This is a deprecation description // vendor/symfony/deprecation-contracts/function.php:25
PASS Tests\Features\Exceptions
@ -332,14 +349,14 @@
✓ failures with custom message
✓ not failures
PASS Tests\Features\Expect\toBeGreatherThan
PASS Tests\Features\Expect\toBeGreaterThan
✓ passes
✓ passes with DateTime and DateTimeImmutable
✓ failures
✓ failures with custom message
✓ not failures
PASS Tests\Features\Expect\toBeGreatherThanOrEqual
PASS Tests\Features\Expect\toBeGreaterThanOrEqual
✓ passes
✓ passes with DateTime and DateTimeImmutable
✓ failures
@ -641,6 +658,7 @@
✓ failures 5
✓ failures 6
✓ failures 7
✓ failures 8
✓ failures with custom message
✓ not failures
✓ closure missing parameter
@ -673,6 +691,7 @@
✓ it throws error if method do not exist
✓ it can forward unexpected calls to any global function
✓ it can use helpers from helpers file
✓ it can use helpers from helpers directory
PASS Tests\Features\HigherOrderTests
✓ it proxies calls to object
@ -789,6 +808,12 @@
PASS Tests\Fixtures\ExampleTest
✓ it example 2
WARN Tests\Fixtures\UnexpectedOutput
- output
PASS Tests\Helpers\TestInHelpers
✓ it executes tests in the Helpers directory
PASS Tests\Hooks\AfterAllTest
✓ global afterAll execution order
✓ it only gets called once per file
@ -864,6 +889,9 @@
✓ it throw expectation failed exception with string argument
✓ it throw expectation failed exception with array argument
PASS Tests\Unit\Overrides\ThrowableBuilder
✓ collision editor can be added to the stack trace
PASS Tests\Unit\Plugins\Environment
✓ environment is set to CI when --ci option is used
✓ environment is set to Local when --ci option is not used
@ -907,6 +935,9 @@
✓ it ensures the given closures reports the correct class name
✓ it ensures the given closures reports the correct class name and suggests the [uses()] function
PASS Tests\Unit\Support\HigherOrderMessage
✓ undefined method exceptions
PASS Tests\Unit\Support\Reflection
✓ it gets file name from closure
✓ it gets property values
@ -970,6 +1001,7 @@
✓ Німеччина
✓ Nam Định
✓ 呼和浩特
✓ test /** with comment */ should do
PASS Tests\Unit\TestSuite
✓ it does not allow to add the same test description twice
@ -988,6 +1020,7 @@
PASS Tests\Visual\Parallel
✓ parallel
✓ a parallel test can extend another test with same name
PASS Tests\Visual\SingleTestOrDirectory
✓ allows to run a single test
@ -1005,7 +1038,11 @@
✓ todo
✓ todo in parallel
PASS Tests\Visual\UnexpectedOutput
✓ unexpected output with ([''])
✓ unexpected output with (['--parallel'])
PASS Tests\Visual\Version
✓ visual snapshot of help command output
Tests: 2 deprecated, 3 warnings, 4 incomplete, 1 notice, 4 todos, 14 skipped, 705 passed (1706 assertions)
Tests: 2 deprecated, 3 warnings, 4 incomplete, 1 notice, 8 todos, 18 skipped, 718 passed (1735 assertions)

View File

@ -1,3 +1,9 @@
TODO Tests\Features\BeforeEachProxiesToTestCallWithTodo - 4 todos
↓ is marked as todo 1
↓ is marked as todo 2
↓ is marked as todo 3
↓ shouldBeMarkedAsTodo
TODO Tests\Features\DatasetsTests - 1 todo
↓ forbids to define tests in Datasets dirs and Datasets.php files
@ -9,4 +15,4 @@
PASS Tests\CustomTestCase\ExecutedTest
✓ that gets executed
Tests: 4 todos, 1 passed (1 assertions)
Tests: 8 todos, 1 passed (1 assertions)

View File

@ -0,0 +1,23 @@
.this is unexpected output!
────────────────────────────────────────────────────────────────────────────
RISKY Tests\Fixtures\UnexpectedOutput > output
This test printed output: this is unexpected output
at src/Support/StateGenerator.php:58
54▕ foreach ($riskyEvents as $riskyEvent) {
55▕ $state->add(TestResult::fromPestParallelTestCase(
56▕ $riskyEvent->test(),
57▕ TestResult::RISKY,
➜ 58▕ ThrowableBuilder::from(new TestOutcome($riskyEvent->message()))
59▕ ));
60▕ }
61▕ }
62▕
1 src/Support/StateGenerator.php:58
+5 vendor frames
7 src/Plugins/Actions/CallsHandleArguments.php:29
Tests: 1 risky, 1 passed (2 assertions)

View File

@ -0,0 +1,20 @@
this is unexpected output
WARN Tests\Fixtures\UnexpectedOutput
! output → This test printed output: this is unexpected output
────────────────────────────────────────────────────────────────────────────
RISKY Tests\Fixtures\UnexpectedOutput > output
This test printed output: this is unexpected output
at src/Kernel.php:86
82▕ {
83▕ $args = CallsHandleArguments::execute($args);
84▕
85▕ try {
➜ 86▕ $this->application->run($args);
87▕ } catch (NoDirtyTestsFound) {
88▕ $this->output->writeln([
89▕ '',
90▕ ' INFO No tests found.',
Tests: 1 risky (1 assertions)

View File

@ -1,3 +1,3 @@
Pest Testing Framework 2.3.0.
Pest Testing Framework 2.7.0.

View File

@ -18,9 +18,17 @@ it('can fail', function () {
$this->fail("oh noo");
});
it('throws exception', function () {
throw new Exception('test error');
});
it('is not done yet', function () {
})->todo();
todo("build this one.");
it('is passing', function () {
expect(true)->toEqual(true);
});

View File

@ -0,0 +1,15 @@
<?php
beforeEach()->expect(true)->toBeTrue();
test('runs 1', function () {
// This test did performs assertions...
});
test('runs 2', function () {
// This test did performs assertions...
});
test('runs 3', function () {
// This test did performs assertions...
});

View File

@ -0,0 +1,15 @@
<?php
beforeEach()->skip();
test('does not run 1', function () {
$this->fail('This test should not run');
});
test('does not run 2', function () {
$this->fail('This test should not run');
});
test('does not run 3', function () {
$this->fail('This test should not run');
});

View File

@ -0,0 +1,15 @@
<?php
beforeEach()->todo();
test('is marked as todo 1', function () {
$this->fail('This test should not run');
});
test('is marked as todo 2', function () {
$this->fail('This test should not run');
});
test('is marked as todo 3');
test()->shouldBeMarkedAsTodo();

View File

@ -6,6 +6,7 @@ use PHPUnit\Framework\Attributes\CoversClass;
use Tests\Fixtures\Covers\CoversClass1;
use Tests\Fixtures\Covers\CoversClass2;
use Tests\Fixtures\Covers\CoversClass3;
use Tests\Fixtures\Covers\CoversTrait;
$runCounter = 0;
@ -43,6 +44,13 @@ it('guesses if the given argument is a class or function', function () {
expect($attributes[3]->getArguments()[0])->toBe(CoversClass3::class);
})->covers(CoversClass3::class, 'testCoversFunction');
it('uses the correct PHPUnit attribute for trait', function () {
$attributes = (new ReflectionClass($this))->getAttributes();
expect($attributes[4]->getName())->toBe('PHPUnit\Framework\Attributes\CoversClass');
expect($attributes[4]->getArguments()[0])->toBe('Tests\Fixtures\Covers\CoversTrait');
})->coversClass(CoversTrait::class);
it('appends CoversNothing to method attributes', function () {
$phpDoc = (new ReflectionClass($this))->getMethod($this->name());

View File

@ -85,17 +85,17 @@ it('works with higher order tests')
it('can use the scoped method to lock into the given level for expectations', function () {
expect(new HasMethods())
->attributes()->scoped(fn ($attributes) => $attributes
->name->toBe('Has Methods')
->quantity->toBe(20)
->name->toBe('Has Methods')
->quantity->toBe(20)
)
->name()->toBeString()->toBe('Has Methods')
->newInstance()->newInstance()->scoped(fn ($instance) => $instance
->name()->toBe('Has Methods')
->quantity()->toBe(20)
->attributes()->scoped(fn ($attributes) => $attributes
->name->toBe('Has Methods')
->quantity->toBe(20)
)
->name()->toBe('Has Methods')
->quantity()->toBe(20)
->attributes()->scoped(fn ($attributes) => $attributes
->name->toBe('Has Methods')
->quantity->toBe(20)
)
);
});

View File

@ -171,19 +171,19 @@ test('pipe works with negated expectation', function () use ($state) {
$state->reset();
expect($char)->not->toBe(new Char('B'))
->and($state)
->runCount->toMatchArray([
'char' => 1,
'number' => 0,
'wildcard' => 0,
'symbol' => 0,
])
->appliedCount->toMatchArray([
'char' => 1,
'number' => 0,
'wildcard' => 0,
'symbol' => 0,
]);
->and($state)
->runCount->toMatchArray([
'char' => 1,
'number' => 0,
'wildcard' => 0,
'symbol' => 0,
])
->appliedCount->toMatchArray([
'char' => 1,
'number' => 0,
'wildcard' => 0,
'symbol' => 0,
]);
});
test('interceptor is applied', function () use ($state) {

View File

@ -4,30 +4,49 @@ use PHPUnit\Framework\ExpectationFailedException;
test('pass', function () {
$object = new stdClass();
$object->name = 'Jhon';
$object->name = 'John';
$object->age = 21;
expect($object)->toHaveProperties(['name', 'age']);
expect($object)
->toHaveProperties(['name', 'age'])
->toHaveProperties([
'name' => 'John',
'age' => 21,
]);
});
test('failures', function () {
$object = new stdClass();
$object->name = 'Jhon';
$object->name = 'John';
expect($object)->toHaveProperties(['name', 'age']);
expect($object)
->toHaveProperties(['name', 'age'])
->toHaveProperties([
'name' => 'John',
'age' => 21,
]);
})->throws(ExpectationFailedException::class);
test('failures with custom message', function () {
$object = new stdClass();
$object->name = 'Jhon';
$object->name = 'John';
expect($object)->toHaveProperties(['name', 'age'], 'oh no!');
expect($object)
->toHaveProperties(['name', 'age'], 'oh no!')
->toHaveProperties([
'name' => 'John',
'age' => 21,
], 'oh no!');
})->throws(ExpectationFailedException::class, 'oh no!');
test('not failures', function () {
$object = new stdClass();
$object->name = 'Jhon';
$object->name = 'John';
$object->age = 21;
expect($object)->not->toHaveProperties(['name', 'age']);
expect($object)->not->toHaveProperties(['name', 'age'])
->not->toHaveProperties([
'name' => 'John',
'age' => 21,
]);
})->throws(ExpectationFailedException::class);

View File

@ -2,6 +2,10 @@
use PHPUnit\Framework\ExpectationFailedException;
class CustomException extends Exception
{
}
test('passes', function () {
expect(function () {
throw new RuntimeException();
@ -33,6 +37,9 @@ test('passes', function () {
throw new RuntimeException('actual message');
})->toThrow(function (RuntimeException $e) {
}, 'actual message');
expect(function () {
throw new CustomException('foo');
})->toThrow(new CustomException('foo'));
});
test('failures 1', function () {
@ -79,6 +86,12 @@ test('failures 7', function () {
})->toThrow(RuntimeException::class, 'expected message');
})->throws(ExpectationFailedException::class);
test('failures 8', function () {
expect(function () {
throw new CustomException('actual message');
})->toThrow(new CustomException('expected message'));
})->throws(ExpectationFailedException::class);
test('failures with custom message', function () {
expect(function () {
throw new RuntimeException('actual message');

View File

@ -44,3 +44,5 @@ it('throws error if method do not exist', function () {
it('can forward unexpected calls to any global function')->_assertThat();
it('can use helpers from helpers file')->myAssertTrue(true);
it('can use helpers from helpers directory')->myDirectoryAssertTrue(true);

View File

@ -0,0 +1,7 @@
<?php
namespace Tests\Fixtures\Covers;
trait CoversTrait
{
}

View File

@ -0,0 +1,13 @@
<?php
namespace Tests\Fixtures\Inheritance\Base;
use PHPUnit\Framework\TestCase;
class ExampleTest extends TestCase
{
public function testExample()
{
$this->markTestSkipped();
}
}

View File

@ -0,0 +1,11 @@
<?php
namespace Tests\Fixtures\Inheritance;
class ExampleTest extends Base\ExampleTest
{
public function testExample()
{
$this->assertTrue(true);
}
}

View File

@ -0,0 +1,7 @@
<?php
test('output', function () {
echo 'this is unexpected output';
expect(true)->toBeTrue();
})->skip(! isset($_SERVER['COLLISION_TEST']));

6
tests/Helpers/Helper.php Normal file
View File

@ -0,0 +1,6 @@
<?php
function myDirectoryAssertTrue($value)
{
test()->assertTrue($value);
}

View File

@ -0,0 +1,5 @@
<?php
it('executes tests in the Helpers directory', function () {
expect(true)->toBeTrue();
});

View File

@ -0,0 +1,28 @@
<?php
use NunoMaduro\Collision\Contracts\RenderableOnCollisionEditor;
use PHPUnit\Event\Code\ThrowableBuilder;
use Whoops\Exception\Frame;
test('collision editor can be added to the stack trace', function () {
$exception = new class extends Exception implements RenderableOnCollisionEditor
{
public function __construct()
{
parent::__construct('test exception');
}
public function toCollisionEditor(): Frame
{
return new Frame([
'file' => __DIR__.'/../../Pest.php',
'line' => 5,
]);
}
};
expect(str_replace(DIRECTORY_SEPARATOR, '/', ThrowableBuilder::from($exception)->stackTrace()))
->toContain('tests/Unit/Overrides/../../Pest.php:5')
->and(str_replace(DIRECTORY_SEPARATOR, '/', ThrowableBuilder::from(new Exception('test'))->stackTrace()))
->toContain('tests/Unit/Overrides/ThrowableBuilder.php:26');
});

View File

@ -17,5 +17,5 @@ it('ensures the given closures reports the correct class name and suggests the [
$this->get();
})->throws(
Error::class,
'Call to undefined method Tests\Unit\Support\ExceptionTrace::get(). Did you forget to use the [uses()] function? https://pestphp.com/docs/configuring-tests',
'Call to undefined method Tests\Unit\Support\ExceptionTrace::get(). Did you forget to use the [uses()] function? Read more at: https://pestphp.com/docs/configuring-tests',
);

View File

@ -0,0 +1,19 @@
<?php
use Pest\Support\HigherOrderMessage;
test('undefined method exceptions', function () {
$message = new HigherOrderMessage(
__FILE__,
1,
'foqwdqwd',
[]
);
expect(fn () => $message->call($this))->toThrow(function (ReflectionException $exception) {
expect($exception)
->getMessage()->toBe('Call to undefined method PHPUnit\Framework\TestCase::foqwdqwd()')
->getLine()->toBe(1)
->getFile()->toBe(__FILE__);
});
});

View File

@ -61,6 +61,7 @@ $names = [
'Німеччина' => '__pest_evaluable_Німеччина',
'Nam Định' => '__pest_evaluable_Nam_Định',
'呼和浩特' => '__pest_evaluable_呼和浩特',
'test /** with comment */ should do' => '__pest_evaluable_test_____with_comment____should_do',
];
foreach ($names as $name => $methodName) {

View File

@ -39,4 +39,4 @@ test('collision', function (array $arguments) {
})->with([
[['']],
[['--parallel']],
])->skip(PHP_OS_FAMILY === 'Windows');
])->skipOnWindows();

View File

@ -18,4 +18,4 @@ test('visual snapshot of help command output', function () {
}
expect($output())->toContain(file_get_contents($snapshot));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();

View File

@ -3,7 +3,9 @@
use Symfony\Component\Process\Process;
$run = function () {
$process = new Process(['php', 'bin/pest', '--parallel', '--processes=3', '--exclude-group=integration'], dirname(__DIR__, 2),
$process = new Process(
array_merge(['php', 'bin/pest', '--parallel', '--processes=3'], func_get_args()),
dirname(__DIR__, 2),
['COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true'],
);
@ -15,6 +17,11 @@ $run = function () {
};
test('parallel', function () use ($run) {
expect($run())->toContain('Tests: 2 deprecated, 3 warnings, 4 incomplete, 1 notice, 4 todos, 11 skipped, 694 passed (1692 assertions)')
expect($run('--exclude-group=integration'))
->toContain('Tests: 1 deprecated, 3 warnings, 4 incomplete, 1 notice, 8 todos, 15 skipped, 705 passed (1718 assertions)')
->toContain('Parallel: 3 processes');
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();
test('a parallel test can extend another test with same name', function () use ($run) {
expect($run('tests/Fixtures/Inheritance'))->toContain('Tests: 1 skipped, 2 passed (2 assertions)');
});

View File

@ -24,11 +24,11 @@ $snapshot = function ($name) {
test('allows to run a single test', function () use ($run, $snapshot) {
expect($run('tests/Fixtures/DirectoryWithTests/ExampleTest.php'))->toContain($snapshot('allows-to-run-a-single-test'));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();
test('allows to run a directory', function () use ($run, $snapshot) {
expect($run('tests/Fixtures'))->toContain($snapshot('allows-to-run-a-directory'));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();
it('disable decorating printer when colors is set to never', function () use ($snapshot) {
$process = new Process([
@ -40,4 +40,4 @@ it('disable decorating printer when colors is set to never', function () use ($s
$process->run();
$output = $process->getOutput();
expect($output)->toContain($snapshot('disable-decorating-printer'));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();

View File

@ -41,4 +41,4 @@ test('visual snapshot of test suite on success', function () {
expect(implode("\n", $output))->toContain(file_get_contents($snapshot));
}
})->skip(! getenv('REBUILD_SNAPSHOTS') && getenv('EXCLUDE'))
->skip(PHP_OS_FAMILY === 'Windows');
->skipOnWindows();

View File

@ -11,7 +11,9 @@ $run = function (string $target, bool $parallel) {
expect($process->getExitCode())->toBe(0);
return preg_replace('#\\x1b[[][^A-Za-z]*[A-Za-z]#', '', $process->getOutput());
$outputContent = preg_replace('#\\x1b[[][^A-Za-z]*[A-Za-z]#', '', $process->getOutput());
return $outputContent;
};
$snapshot = function ($name) {
@ -26,8 +28,8 @@ $snapshot = function ($name) {
test('todo', function () use ($run, $snapshot) {
expect($run('--todos', false))->toContain($snapshot('todo'));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();
test('todo in parallel', function () use ($run, $snapshot) {
expect($run('--todos', true))->toContain($snapshot('todo'));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();

View File

@ -0,0 +1,43 @@
<?php
test('unexpected output', function (array $arguments) {
$snapshot = __DIR__.'/../.snapshots/unexpected-output.txt';
if (in_array('--parallel', $arguments)) {
$snapshot = __DIR__.'/../.snapshots/unexpected-output-parallel.txt';
}
$output = function () use ($arguments) {
$process = (new Symfony\Component\Process\Process(
array_merge(['php', 'bin/pest', 'tests/Fixtures/UnexpectedOutput.php'], $arguments),
null,
['COLLISION_PRINTER' => 'DefaultPrinter', 'COLLISION_IGNORE_DURATION' => 'true', 'COLLISION_TEST' => true]
));
$process->run();
return preg_replace('#\\x1b[[][^A-Za-z]*[A-Za-z]#', '', $process->getOutput());
};
if (getenv('REBUILD_SNAPSHOTS')) {
$outputContent = explode("\n", $output());
array_pop($outputContent);
array_pop($outputContent);
array_pop($outputContent);
if (in_array('--parallel', $arguments)) {
array_pop($outputContent);
array_pop($outputContent);
}
file_put_contents($snapshot, implode("\n", $outputContent));
$this->markTestSkipped('Snapshot rebuilt.');
}
expect($output())
->toContain(file_get_contents($snapshot));
})->with([
[['']],
[['--parallel']],
])->skipOnWindows();

View File

@ -18,4 +18,4 @@ test('visual snapshot of help command output', function () {
}
expect($output())->toContain(file_get_contents($snapshot));
})->skip(PHP_OS_FAMILY === 'Windows');
})->skipOnWindows();