Compare commits

...

3 Commits

Author SHA1 Message Date
ef120125e0 release: 2.34.9 2024-07-11 09:36:26 +01:00
8a9a416133 chore: bumps dependencies 2024-07-11 09:35:43 +01:00
4783334f15 chore: style 2024-07-11 09:35:38 +01:00
46 changed files with 61 additions and 173 deletions

View File

@ -52,7 +52,7 @@
},
"require-dev": {
"pestphp/pest-dev-tools": "^2.16.0",
"pestphp/pest-plugin-type-coverage": "^2.8.3",
"pestphp/pest-plugin-type-coverage": "^2.8.4",
"symfony/process": "^6.4.0|^7.1.1"
},
"minimum-stability": "dev",

View File

@ -32,8 +32,7 @@ final class BootSubscribers implements Bootstrapper
*/
public function __construct(
private readonly Container $container,
) {
}
) {}
/**
* Boots the list of Subscribers.

View File

@ -24,9 +24,7 @@ final class EachExpectation
*
* @param Expectation<TValue> $original
*/
public function __construct(private readonly Expectation $original)
{
}
public function __construct(private readonly Expectation $original) {}
/**
* Creates a new expectation.

View File

@ -36,9 +36,7 @@ final class OppositeExpectation
*
* @param Expectation<TValue> $original
*/
public function __construct(private readonly Expectation $original)
{
}
public function __construct(private readonly Expectation $original) {}
/**
* Asserts that the value array not has the provided $keys.

View File

@ -9,7 +9,5 @@ namespace Pest\Factories\Covers;
*/
final class CoversClass
{
public function __construct(public string $class)
{
}
public function __construct(public string $class) {}
}

View File

@ -9,7 +9,5 @@ namespace Pest\Factories\Covers;
*/
final class CoversFunction
{
public function __construct(public string $function)
{
}
public function __construct(public string $function) {}
}

View File

@ -7,6 +7,4 @@ namespace Pest\Factories\Covers;
/**
* @internal
*/
final class CoversNothing
{
}
final class CoversNothing {}

View File

@ -17,8 +17,7 @@ final class ServiceMessage
public function __construct(
private readonly string $type,
private readonly array $parameters,
) {
}
) {}
public function toString(): string
{

View File

@ -14,9 +14,7 @@ abstract class Subscriber
/**
* Creates a new Subscriber instance.
*/
public function __construct(private readonly TeamCityLogger $logger)
{
}
public function __construct(private readonly TeamCityLogger $logger) {}
/**
* Creates a new TeamCityLogger instance.

View File

@ -7,6 +7,4 @@ namespace Pest\Matchers;
/**
* @internal
*/
final class Any
{
}
final class Any {}

View File

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

View File

@ -63,8 +63,7 @@ final class ResultPrinter
{
public function __construct(
private readonly OutputInterface $output,
) {
}
) {}
public function print(string $buffer): void
{
@ -79,9 +78,7 @@ final class ResultPrinter
$this->output->write(OutputFormatter::escape($buffer));
}
public function flush(): void
{
}
public function flush(): void {}
};
$this->compactPrinter = CompactPrinter::default();

View File

@ -21,8 +21,7 @@ final class SnapshotRepository
public function __construct(
readonly private string $testsPath,
readonly private string $snapshotsPath,
) {
}
) {}
/**
* Checks if the snapshot exists.

View File

@ -24,8 +24,7 @@ final class EnsureTeamCityEnabled implements ConfiguredSubscriber
private readonly InputInterface $input,
private readonly OutputInterface $output,
private readonly TestSuite $testSuite,
) {
}
) {}
/**
* Runs the subscriber.

View File

@ -30,8 +30,7 @@ final class ExpectationPipeline
*/
public function __construct(
private readonly Closure $closure
) {
}
) {}
/**
* Creates a new instance of Expectation Pipeline with given closure.

View File

@ -16,7 +16,6 @@ final class NullClosure
*/
public static function create(): Closure
{
return Closure::fromCallable(function (): void {
});
return Closure::fromCallable(function (): void {});
}
}

View File

@ -10,9 +10,7 @@ use Tests\Fixtures\Covers\CoversTrait;
$runCounter = 0;
function testCoversFunction()
{
}
function testCoversFunction() {}
it('uses the correct PHPUnit attribute for class', function () {
$attributes = (new ReflectionClass($this))->getAttributes();

View File

@ -3,8 +3,7 @@
use PHPUnit\Framework\ExpectationFailedException;
test('pass', function () {
expect(function () {
})->toBeCallable();
expect(function () {})->toBeCallable();
expect(null)->not->toBeCallable();
});

View File

@ -4,13 +4,9 @@ use PHPUnit\Framework\ExpectationFailedException;
$object = new class
{
public function foo(): void
{
}
public function foo(): void {}
public function bar(): void
{
}
public function bar(): void {}
};
test('pass', function () use ($object) {

View File

@ -43,9 +43,7 @@ test('pass with `__toString`', function () {
$object = new class($this->snapshotable)
{
public function __construct(protected string $snapshotable)
{
}
public function __construct(protected string $snapshotable) {}
public function __toString()
{
@ -61,9 +59,7 @@ test('pass with `toString`', function () {
$object = new class($this->snapshotable)
{
public function __construct(protected string $snapshotable)
{
}
public function __construct(protected string $snapshotable) {}
public function toString()
{
@ -97,9 +93,7 @@ test('pass with `toArray`', function () {
$object = new class($this->snapshotable)
{
public function __construct(protected string $snapshotable)
{
}
public function __construct(protected string $snapshotable) {}
public function toArray()
{
@ -125,9 +119,7 @@ test('pass with `toSnapshot`', function () {
$object = new class($this->snapshotable)
{
public function __construct(protected string $snapshotable)
{
}
public function __construct(protected string $snapshotable) {}
public function toSnapshot()
{

View File

@ -2,9 +2,7 @@
use PHPUnit\Framework\ExpectationFailedException;
class CustomException extends Exception
{
}
class CustomException extends Exception {}
test('passes', function () {
expect(function () {
@ -15,15 +13,13 @@ test('passes', function () {
})->toThrow(Exception::class);
expect(function () {
throw new RuntimeException();
})->toThrow(function (RuntimeException $e) {
});
})->toThrow(function (RuntimeException $e) {});
expect(function () {
throw new RuntimeException('actual message');
})->toThrow(function (Exception $e) {
expect($e->getMessage())->toBe('actual message');
});
expect(function () {
})->not->toThrow(Exception::class);
expect(function () {})->not->toThrow(Exception::class);
expect(function () {
throw new RuntimeException('actual message');
})->toThrow('actual message');
@ -35,22 +31,18 @@ test('passes', function () {
})->toThrow(RuntimeException::class, 'actual message');
expect(function () {
throw new RuntimeException('actual message');
})->toThrow(function (RuntimeException $e) {
}, 'actual message');
})->toThrow(function (RuntimeException $e) {}, 'actual message');
expect(function () {
throw new CustomException('foo');
})->toThrow(new CustomException('foo'));
});
test('failures 1', function () {
expect(function () {
})->toThrow(RuntimeException::class);
expect(function () {})->toThrow(RuntimeException::class);
})->throws(ExpectationFailedException::class, 'Exception "'.RuntimeException::class.'" not thrown.');
test('failures 2', function () {
expect(function () {
})->toThrow(function (RuntimeException $e) {
});
expect(function () {})->toThrow(function (RuntimeException $e) {});
})->throws(ExpectationFailedException::class, 'Exception "'.RuntimeException::class.'" not thrown.');
test('failures 3', function () {
@ -77,8 +69,7 @@ test('failures 5', function () {
})->throws(ExpectationFailedException::class, 'Failed asserting that \'actual message\' [ASCII](length: 14) contains "expected message" [ASCII](length: 16).');
test('failures 6', function () {
expect(function () {
})->toThrow('actual message');
expect(function () {})->toThrow('actual message');
})->throws(ExpectationFailedException::class, 'Exception with message "actual message" not thrown');
test('failures 7', function () {
@ -106,15 +97,11 @@ test('not failures', function () {
})->throws(ExpectationFailedException::class);
test('closure missing parameter', function () {
expect(function () {
})->toThrow(function () {
});
expect(function () {})->toThrow(function () {});
})->throws(InvalidArgumentException::class, 'The given closure must have a single parameter type-hinted as the class string.');
test('closure missing type-hint', function () {
expect(function () {
})->toThrow(function ($e) {
});
expect(function () {})->toThrow(function ($e) {});
})->throws(InvalidArgumentException::class, 'The given closure\'s parameter must be type-hinted as the class string.');
it('can handle a non-defined exception', function () {

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsInvokable;
class InvokableClass
{
public function __invoke(): void
{
}
public function __invoke(): void {}
}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsInvokable;
class ParentInvokableClass
{
public function __invoke(): void
{
}
public function __invoke(): void {}
}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToBeInvokable\IsNotInvokable;
class IsNotInvokableClass
{
public function handle(): void
{
}
public function handle(): void {}
}

View File

@ -7,6 +7,4 @@ namespace Tests\Fixtures\Arch\ToHaveAttribute\Attributes;
use Attribute;
#[Attribute()]
class AsAttribute
{
}
class AsAttribute {}

View File

@ -7,6 +7,4 @@ namespace Tests\Fixtures\Arch\ToHaveAttribute\HaveAttribute;
use Tests\Fixtures\Arch\ToHaveAttribute\Attributes\AsAttribute;
#[AsAttribute]
class HaveAttributeClass
{
}
class HaveAttributeClass {}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHaveAttribute\NotHaveAttribute;
class NotHaveAttributeClass
{
}
class NotHaveAttributeClass {}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveConstructor\HasConstructor;
class HasConstructor
{
public function __construct()
{
}
public function __construct() {}
}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHaveConstructor\HasNoConstructor;
class HasNoConstructor
{
}
class HasNoConstructor {}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveDestructor\HasDestructor;
class HasDestructor
{
public function __destruct()
{
}
public function __destruct() {}
}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHaveDestructor\HasNoDestructor;
class HasNoDestructor
{
}
class HasNoDestructor {}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
class HasMethod
{
public function foo(): void
{
}
public function foo(): void {}
}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
trait HasMethodTrait
{
public function foo(): void
{
}
public function foo(): void {}
}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasMethod;
class ParentHasMethodClass
{
public function foo(): void
{
}
public function foo(): void {}
}

View File

@ -6,8 +6,5 @@ namespace Tests\Fixtures\Arch\ToHaveMethod\HasNoMethod;
class HasNoMethodClass
{
public function bar(): void
{
}
public function bar(): void {}
}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHavePrefix\HasNoPrefix;
class ClassWithout
{
}
class ClassWithout {}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHavePrefix\HasPrefix;
class PrefixClassWith
{
}
class PrefixClassWith {}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHaveSuffix\HasNoSuffix;
class ClassWithout
{
}
class ClassWithout {}

View File

@ -4,6 +4,4 @@ declare(strict_types=1);
namespace Tests\Fixtures\Arch\ToHaveSuffix\HasSuffix;
class ClassWithSuffix
{
}
class ClassWithSuffix {}

View File

@ -4,7 +4,5 @@ namespace Tests\Fixtures\Covers;
class CoversClass1
{
public function foo()
{
}
public function foo() {}
}

View File

@ -2,6 +2,4 @@
namespace Tests\Fixtures\Covers;
class CoversClass2
{
}
class CoversClass2 {}

View File

@ -2,6 +2,4 @@
namespace Tests\Fixtures\Covers;
class CoversClass3
{
}
class CoversClass3 {}

View File

@ -2,6 +2,4 @@
namespace Tests\Fixtures\Covers;
trait CoversTrait
{
}
trait CoversTrait {}

View File

@ -50,21 +50,15 @@ it('cannot resolve a parameter without type', function () {
class ClassWithDependency
{
public function __construct(Container $container)
{
}
public function __construct(Container $container) {}
}
class ClassWithSubDependency
{
public function __construct(ClassWithDependency $param)
{
}
public function __construct(ClassWithDependency $param) {}
}
class ClassWithoutTypeParameter
{
public function __construct($param)
{
}
public function __construct($param) {}
}

View File

@ -3,8 +3,7 @@
use Pest\Support\Reflection;
it('gets file name from closure', function () {
$fileName = Reflection::getFileNameFromClosure(function () {
});
$fileName = Reflection::getFileNameFromClosure(function () {});
expect($fileName)->toBe(__FILE__);
});

View File

@ -19,8 +19,7 @@ it('does not allow to add the same test description twice', function () {
it('alerts users about tests with arguments but no input', function () {
$testSuite = new TestSuite(getcwd(), 'tests');
$method = new TestCaseMethodFactory('foo', 'bar', function (int $arg) {
});
$method = new TestCaseMethodFactory('foo', 'bar', function (int $arg) {});
$testSuite->tests->set($method);
})->throws(