chore(deps): update dependency brainbits/phpcs-standard to v8

renovate[bot] · pl-github · commit 39e39eee8664 · 2025-03-25T08:56:41.000+01:00
diff --git a/composer.json b/composer.json
@@ -13,11 +13,11 @@
         "ext-mbstring": "*",
         "ext-simplexml": "*",
         "ext-tidy": "*",
-        "brainbits/phpcs-standard": "^7.0.1",
+        "brainbits/phpcs-standard": "^8.0.0",
         "brainbits/phpstan-rules": "^4.0",
         "dama/doctrine-test-bundle": "^8.2",
         "doctrine/dbal": "^3.9",
-        "ergebnis/phpstan-rules": "^2.6",
+        "ergebnis/phpstan-rules": "^2.8",
         "gemorroj/archive7z": "^5.7",
         "mikey179/vfsstream": "^1.6.12",
         "monolog/monolog": "^2.0 || ^3.0",
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -10,6 +10,10 @@ parameters:
         - '#Constructor in .* has parameter .* with default value#'
         - '#SchemaBuilder::foo\(\)#'
     ergebnis:
+        noAssignByReference:
+            enabled: false
+        noParameterPassedByReference:
+            enabled: false
         noNullableReturnTypeDeclaration:
             enabled: false
         noParameterWithNullableTypeDeclaration:
diff --git a/src/HttpClientMock/HttpClientMockTrait.php b/src/HttpClientMock/HttpClientMockTrait.php
@@ -25,6 +25,7 @@
 use function trigger_deprecation;
 use function ucfirst;
 use function urldecode;
+use function vsprintf;
 
 /** @mixin TestCase */
 trait HttpClientMockTrait
@@ -476,7 +477,7 @@ protected static function mockRequestMessage(
         string $messageTemplate,
         mixed ...$values,
     ): string {
-        $message = sprintf($messageTemplate, ...$values);
+        $message = vsprintf($messageTemplate, $values);
 
         return $userMessage !== ''
             ? ucfirst($userMessage) . '. ' . $message
diff --git a/src/HttpClientMock/Matcher/QueryParamMatcher.php b/src/HttpClientMock/Matcher/QueryParamMatcher.php
@@ -7,6 +7,7 @@
 use Brainbits\FunctionalTestHelpers\HttpClientMock\RealRequest;
 
 use function sprintf;
+use function vsprintf;
 
 final readonly class QueryParamMatcher implements Matcher
 {
@@ -15,7 +16,7 @@
     /** @param array<string> $placeholders */
     public function __construct(private string $key, string $value, array $placeholders)
     {
-        $this->value = sprintf($value, ...$placeholders);
+        $this->value = vsprintf($value, $placeholders);
     }
 
     public function __invoke(RealRequest $realRequest): Hit|Mismatch|Missing
diff --git a/src/ZipContents/FileInfo.php b/src/ZipContents/FileInfo.php
@@ -18,30 +18,30 @@
 
 final class FileInfo
 {
-    private const COMPRESSION_STORED = 0;
-    private const COMPRESSION_SHRUNK = 1;
-    private const COMPRESSION_REDUCED_FACTOR_1 = 2;
-    private const COMPRESSION_REDUCED_FACTOR_2 = 3;
-    private const COMPRESSION_REDUCED_FACTOR_3 = 4;
-    private const COMPRESSION_REDUCED_FACTOR_4 = 5;
-    private const COMPRESSION_IMPLODED = 6;
-    private const COMPRESSION_DEFLATED = 8;
-    private const COMPRESSION_DEFLATE64 = 9;
-    private const COMPRESSION_PKZIP = 10;
-    private const COMPRESSION_BZIP2 = 12;
-    private const COMPRESSION_LZMA = 14;
-    private const COMPRESSION_CMPSC = 16;
-    private const COMPRESSION_TERSE = 18;
-    private const COMPRESSION_LZ77 = 19;
-    private const COMPRESSION_ZSTD = 93;
-    private const COMPRESSION_MP3 = 94;
-    private const COMPRESSION_XZ = 95;
-    private const COMPRESSION_JPEG = 96;
-    private const COMPRESSION_WAVPACK = 97;
-    private const COMPRESSION_PPMD = 98;
-    private const COMPRESSION_AEX = 99;
-
-    private const COMPRESSION_MAP = [
+    private const int COMPRESSION_STORED = 0;
+    private const int COMPRESSION_SHRUNK = 1;
+    private const int COMPRESSION_REDUCED_FACTOR_1 = 2;
+    private const int COMPRESSION_REDUCED_FACTOR_2 = 3;
+    private const int COMPRESSION_REDUCED_FACTOR_3 = 4;
+    private const int COMPRESSION_REDUCED_FACTOR_4 = 5;
+    private const int COMPRESSION_IMPLODED = 6;
+    private const int COMPRESSION_DEFLATED = 8;
+    private const int COMPRESSION_DEFLATE64 = 9;
+    private const int COMPRESSION_PKZIP = 10;
+    private const int COMPRESSION_BZIP2 = 12;
+    private const int COMPRESSION_LZMA = 14;
+    private const int COMPRESSION_CMPSC = 16;
+    private const int COMPRESSION_TERSE = 18;
+    private const int COMPRESSION_LZ77 = 19;
+    private const int COMPRESSION_ZSTD = 93;
+    private const int COMPRESSION_MP3 = 94;
+    private const int COMPRESSION_XZ = 95;
+    private const int COMPRESSION_JPEG = 96;
+    private const int COMPRESSION_WAVPACK = 97;
+    private const int COMPRESSION_PPMD = 98;
+    private const int COMPRESSION_AEX = 99;
+
+    private const array COMPRESSION_MAP = [
         self::COMPRESSION_STORED => 'stored (no compression)',
         self::COMPRESSION_SHRUNK => 'Shrunk',
         self::COMPRESSION_REDUCED_FACTOR_1 => 'Reduced with compression factor 1',
diff --git a/tests/HttpClientMock/MockRequestBuilderTest.php b/tests/HttpClientMock/MockRequestBuilderTest.php
@@ -316,7 +316,7 @@ public function testEmptyResponsesThrowsException(): void
         $this->expectException(NoResponseMock::class);
         $this->expectExceptionMessage('No response configured for:');
 
-        $mockRequestBuilder = (new MockRequestBuilder());
+        $mockRequestBuilder = new MockRequestBuilder();
 
         $mockRequestBuilder->nextResponse();
     }
diff --git a/tests/SevenZipContents/SevenZipContentsTest.php b/tests/SevenZipContents/SevenZipContentsTest.php
@@ -23,7 +23,7 @@
 #[CoversClass(SevenZipContents::class)]
 final class SevenZipContentsTest extends TestCase
 {
-    private const FILE_7Z = __DIR__ . '/../files/test.7z';
+    private const string FILE_7Z = __DIR__ . '/../files/test.7z';
 
     protected function setUp(): void
     {
diff --git a/tests/SevenZipContents/SevenZipContentsTraitTest.php b/tests/SevenZipContents/SevenZipContentsTraitTest.php
@@ -14,7 +14,7 @@ final class SevenZipContentsTraitTest extends TestCase
 {
     use SevenZipContentsTrait;
 
-    private const FILE = __DIR__ . '/../files/test.7z';
+    private const string FILE = __DIR__ . '/../files/test.7z';
 
     protected function setUp(): void
     {
diff --git a/tests/ZipContents/ZipContentsTest.php b/tests/ZipContents/ZipContentsTest.php
@@ -16,7 +16,7 @@
 #[CoversClass(ZipContents::class)]
 final class ZipContentsTest extends TestCase
 {
-    private const FILE = __DIR__ . '/../files/test.zip';
+    private const string FILE = __DIR__ . '/../files/test.zip';
 
     public function testItNeedsFile(): void
     {
diff --git a/tests/ZipContents/ZipContentsTraitTest.php b/tests/ZipContents/ZipContentsTraitTest.php
@@ -15,7 +15,7 @@ final class ZipContentsTraitTest extends TestCase
 {
     use ZipContentsTrait;
 
-    private const FILE = __DIR__ . '/../files/test.zip';
+    private const string FILE = __DIR__ . '/../files/test.zip';
 
     public function testAssertZipHasSizeForZipFileFails(): void
     {
diff --git a/tests/ZipContents/ZipInfoTest.php b/tests/ZipContents/ZipInfoTest.php
@@ -14,7 +14,7 @@
 #[CoversClass(ZipInfo::class)]
 final class ZipInfoTest extends TestCase
 {
-    private const FILE = __DIR__ . '/../files/test.zip';
+    private const string FILE = __DIR__ . '/../files/test.zip';
 
     public function testZipFile(): void
     {

Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,7 @@`
`7`	`7`	`use Brainbits\FunctionalTestHelpers\HttpClientMock\RealRequest;`
`8`	`8`
`9`	`9`	`use function sprintf;`
	`10`	`+use function vsprintf;`
`10`	`11`
`11`	`12`	`final readonly class QueryParamMatcher implements Matcher`
`12`	`13`	`{`
`@@ -15,7 +16,7 @@`
`15`	`16`	`/** @param array<string> $placeholders */`
`16`	`17`	`public function __construct(private string $key, string $value, array $placeholders)`
`17`	`18`	`{`
`18`		`- $this->value = sprintf($value, ...$placeholders);`
	`19`	`+ $this->value = vsprintf($value, $placeholders);`
`19`	`20`	`}`
`20`	`21`
`21`	`22`	`public function __invoke(RealRequest $realRequest): Hit\|Mismatch\|Missing`
Original file line number	Diff line number	Diff line change
`@@ -316,7 +316,7 @@ public function testEmptyResponsesThrowsException(): void`
`316`	`316`	`$this->expectException(NoResponseMock::class);`
`317`	`317`	`$this->expectExceptionMessage('No response configured for:');`
`318`	`318`
`319`		`- $mockRequestBuilder = (new MockRequestBuilder());`
	`319`	`+ $mockRequestBuilder = new MockRequestBuilder();`
`320`	`320`
`321`	`321`	`$mockRequestBuilder->nextResponse();`
`322`	`322`	`}`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@`
`23`	`23`	`#[CoversClass(SevenZipContents::class)]`
`24`	`24`	`final class SevenZipContentsTest extends TestCase`
`25`	`25`	`{`
`26`		`- private const FILE_7Z = __DIR__ . '/../files/test.7z';`
	`26`	`+ private const string FILE_7Z = __DIR__ . '/../files/test.7z';`
`27`	`27`
`28`	`28`	`protected function setUp(): void`
`29`	`29`	`{`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ final class SevenZipContentsTraitTest extends TestCase`
`14`	`14`	`{`
`15`	`15`	`use SevenZipContentsTrait;`
`16`	`16`
`17`		`- private const FILE = __DIR__ . '/../files/test.7z';`
	`17`	`+ private const string FILE = __DIR__ . '/../files/test.7z';`
`18`	`18`
`19`	`19`	`protected function setUp(): void`
`20`	`20`	`{`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`	`#[CoversClass(ZipContents::class)]`
`17`	`17`	`final class ZipContentsTest extends TestCase`
`18`	`18`	`{`
`19`		`- private const FILE = __DIR__ . '/../files/test.zip';`
	`19`	`+ private const string FILE = __DIR__ . '/../files/test.zip';`
`20`	`20`
`21`	`21`	`public function testItNeedsFile(): void`
`22`	`22`	`{`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ final class ZipContentsTraitTest extends TestCase`
`15`	`15`	`{`
`16`	`16`	`use ZipContentsTrait;`
`17`	`17`
`18`		`- private const FILE = __DIR__ . '/../files/test.zip';`
	`18`	`+ private const string FILE = __DIR__ . '/../files/test.zip';`
`19`	`19`
`20`	`20`	`public function testAssertZipHasSizeForZipFileFails(): void`
`21`	`21`	`{`