Upgrade packages

Lock file operations: 0 installs, 14 updates, 4 removals
  - Removing ocramius/package-versions (2.5.1)
  - Removing symfony/polyfill-php73 (v1.26.0)
  - Removing symfony/polyfill-php80 (v1.26.0)
  - Removing webmozart/path-util (2.3.0)
  - Upgrading composer/pcre (1.0.1 => 3.0.0)
  - Upgrading composer/xdebug-handler (2.0.5 => 3.0.3)
  - Upgrading infection/abstract-testframework-adapter (0.3.1 => 0.5.0)
  - Upgrading infection/infection (0.23.0 => 0.26.13)
  - Upgrading lcobucci/coding-standard (7.0.0 => 8.0.0)
  - Upgrading monolog/monolog (2.8.0 => 3.2.0)
  - Upgrading ondram/ci-detector (3.5.1 => 4.1.0)
  - Upgrading psr/log (1.1.4 => 3.0.0)
  - Upgrading sanmai/pipeline (v5.2.1 => v6.1)
  - Upgrading symfony/console (v5.4.10 => v6.1.2)
  - Upgrading symfony/filesystem (v5.4.9 => v6.1.0)
  - Upgrading symfony/finder (v5.4.8 => v6.1.0)
  - Upgrading symfony/process (v5.4.8 => v6.1.0)
  - Upgrading thecodingmachine/safe (v1.3.3 => v2.2.2)

Signed-off-by: Luís Cobucci <lcobucci@gmail.com>

Author: lcobucci

composer.json

@@ -21,13 +21,13 @@
     "require": {
         "php-64bit": "^8.1",
         "ext-pcntl": "*",
-        "psr/log": "^1.1",
+        "psr/log": "^3.0",
         "react/socket": "^1.6"
     },
     "require-dev": {
-        "infection/infection": "^0.23",
-        "lcobucci/coding-standard": "^7.0",
-        "monolog/monolog": "^2.2",
+        "infection/infection": "^0.26",
+        "lcobucci/coding-standard": "^8.0",
+        "monolog/monolog": "^3.2",
         "phpstan/extension-installer": "^1.1",
         "phpstan/phpstan": "^1.8",
         "phpstan/phpstan-deprecation-rules": "^1.0",

composer.lock

@@ -27,14 +27,10 @@ final class Buffer
     private const CONVERSION_SHORT = [2 ** 15 - 1, 2 ** 16];
     private const CONVERSION_INT   = [2 ** 31 - 1, 2 ** 32];
 
-    private string $bytes;
-    private int $length;
     private int $position = 0;
 
-    private function __construct(string $bytes, int $length)
+    private function __construct(private string $bytes, private int $length)
     {
-        $this->bytes  = $bytes;
-        $this->length = $length;
     }
 
     /**

src/Protocol/Buffer.php

@@ -13,13 +13,8 @@
  */
 final class Field
 {
-    private string $name;
-    private Type $type;
-
-    public function __construct(string $name, Type $type)
+    public function __construct(private string $name, private Type $type)
     {
-        $this->name = $name;
-        $this->type = $type;
     }
 
     public function name(): string

src/Protocol/Schema/Field.php

@@ -22,43 +22,43 @@ public static function nullValue(string $expectedType): self
     public static function incorrectType(mixed $data, string $expectedType): self
     {
         return new self(
-            sprintf('%s(%s) does not have expected type: %s', gettype($data), $data, $expectedType)
+            sprintf('%s(%s) does not have expected type: %s', gettype($data), $data, $expectedType),
         );
     }
 
     public static function incorrectClass(object $data, string $expectedClass): self
     {
         return new self(
-            sprintf('Object (%s) is not an instance of: %s', $data::class, $expectedClass)
+            sprintf('Object (%s) is not an instance of: %s', $data::class, $expectedClass),
         );
     }
 
     public static function incorrectRange(int $data, int $lowerBound, int $upperBound): self
     {
         return new self(
-            sprintf('%d is not between expected range: [%d, %d]', $data, $lowerBound, $upperBound)
+            sprintf('%d is not between expected range: [%d, %d]', $data, $lowerBound, $upperBound),
         );
     }
 
     public static function incorrectLength(int $length, int $maxLength): self
     {
         return new self(
-            sprintf('String length (%d) is larger than the maximum length (%d)', $length, $maxLength)
+            sprintf('String length (%d) is larger than the maximum length (%d)', $length, $maxLength),
         );
     }
 
     public static function missingField(string $name): self
     {
         return new self(
-            sprintf('Field "%s" missing from given structure', $name)
+            sprintf('Field "%s" missing from given structure', $name),
         );
     }
 
     public static function invalidValueForField(string $name, SchemaValidationFailure $failure): self
     {
         return new self(
             message: sprintf('Invalid value for field "%s": %s', $name, $failure->getMessage()),
-            previous: $failure
+            previous: $failure,
         );
     }
 }

src/Protocol/SchemaValidationFailure.php

@@ -10,13 +10,8 @@
 
 final class ArrayOf extends Type
 {
-    private Type $type;
-    private bool $nullable;
-
-    public function __construct(Type $type, bool $nullable = false)
+    public function __construct(private Type $type, private bool $nullable = false)
     {
-        $this->type     = $type;
-        $this->nullable = $nullable;
     }
 
     public function write(mixed $data, Buffer $buffer): void

src/Protocol/Type/ArrayOf.php

@@ -30,7 +30,7 @@ public function write(mixed $data, Buffer $buffer): void
     public function read(Buffer $buffer): Buffer
     {
         return Buffer::fromContent(
-            $buffer->read($buffer->readInt())
+            $buffer->read($buffer->readInt()),
         );
     }
 

src/Protocol/Type/NonNullableBytes.php

@@ -17,8 +17,8 @@ public static function forRange(int $value, int $lowerBound, int $upperBound): s
                 'Given value (%s) is out of the expected range [%s, %s]',
                 $value,
                 $lowerBound,
-                $upperBound
-            )
+                $upperBound,
+            ),
         );
     }
 }

src/Protocol/ValueOutOfAllowedRange.php

@@ -58,7 +58,7 @@ public function readShouldUseFieldDataToParseMessage(): void
         $buffer = Buffer::fromContent(pack('c2', 1, 10));
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new Int8())
+            new Schema\Field('test2', new Int8()),
         );
 
         self::assertSame(['test1' => 1, 'test2' => 10], $schema->read($buffer));
@@ -80,7 +80,7 @@ public function sizeOfShouldReturnTheNumberOfBytesNeededForEveryFieldInSchema():
     {
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new NonNullableString())
+            new Schema\Field('test2', new NonNullableString()),
         );
 
         self::assertSame(7, $schema->sizeOf(['test1' => 1, 'test2' => 'test']));
@@ -102,7 +102,7 @@ public function validateShouldNotRaiseExceptionWhenValueIsAnArrayWithAllRequired
     {
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new NullableString())
+            new Schema\Field('test2', new NullableString()),
         );
 
         $schema->validate(['test1' => 1, 'test2' => 'test']);
@@ -127,7 +127,7 @@ public function validateShouldRaiseExceptionWhenValueIsNull(): void
     {
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new Int8())
+            new Schema\Field('test2', new Int8()),
         );
 
         $this->expectException(SchemaValidationFailure::class);
@@ -151,7 +151,7 @@ public function validateShouldRaiseExceptionWhenValueIsNotAnArray(): void
     {
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new Int8())
+            new Schema\Field('test2', new Int8()),
         );
 
         $this->expectException(SchemaValidationFailure::class);
@@ -177,7 +177,7 @@ public function validateShouldRaiseExceptionWhenValueOfOneOfTheFieldsIsInvalid()
     {
         $schema = new Schema(
             new Schema\Field('test1', new Int8()),
-            new Schema\Field('test2', new Int8())
+            new Schema\Field('test2', new Int8()),
         );
 
         $this->expectException(SchemaValidationFailure::class);