php 8.3

dominikzogg · dominikzogg · commit 3732a919d8ab · 2023-12-03T21:11:28.000+01:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -6,36 +6,36 @@ on:
     - cron: '0 0 * * *'
 
 jobs:
-  php80:
-    name: PHP 8.0
-    runs-on: ubuntu-20.04
+  php81:
+    name: PHP 8.1
+    runs-on: ubuntu-22.04
     steps:
       - name: checkout
         uses: actions/checkout@v3
       - name: composer test
-        uses: docker://chubbyphp/ci-php80:latest
+        uses: docker://chubbyphp/ci-php81:latest
         env:
           COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}
-  php81:
-    name: PHP 8.1
-    runs-on: ubuntu-20.04
+  php82:
+    name: PHP 8.2
+    runs-on: ubuntu-22.04
     steps:
       - name: checkout
         uses: actions/checkout@v3
       - name: composer test
-        uses: docker://chubbyphp/ci-php81:latest
+        uses: docker://chubbyphp/ci-php82:latest
         env:
           COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}
-  php82:
-    name: PHP 8.2
-    runs-on: ubuntu-20.04
+  php83:
+    name: PHP 8.3
+    runs-on: ubuntu-22.04
     steps:
       - name: checkout
         uses: actions/checkout@v3
       - name: composer test
-        uses: docker://chubbyphp/ci-php82:latest
+        uses: docker://chubbyphp/ci-php83:latest
         env:
           COVERALLS_REPO_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}
diff --git a/.gitignore b/.gitignore
@@ -1,7 +1,7 @@
 .DS_Store
 .idea/
+.phpunit.cache
 .vscode/
 build/
 composer.lock
 vendor/
-.phpunit.result.cache
diff --git a/README.md b/README.md
@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/chubbyphp/chubbyphp-workerman-request-handler/workflows/CI/badge.svg?branch=master)](https://github.com/chubbyphp/chubbyphp-workerman-request-handler/actions?query=workflow%3ACI)
 [![Coverage Status](https://coveralls.io/repos/github/chubbyphp/chubbyphp-workerman-request-handler/badge.svg?branch=master)](https://coveralls.io/github/chubbyphp/chubbyphp-workerman-request-handler?branch=master)
-[![Infection MSI](https://badge.stryker-mutator.io/github.com/chubbyphp/chubbyphp-workerman-request-handler/master)](https://dashboard.stryker-mutator.io/reports/github.com/chubbyphp/chubbyphp-workerman-request-handler/master)
+[![Mutation testing badge](https://img.shields.io/endpoint?style=flat&url=https%3A%2F%2Fbadge-api.stryker-mutator.io%2Fgithub.com%2Fchubbyphp%2Fchubbyphp-workerman-request-handler%2Fmaster)](https://dashboard.stryker-mutator.io/reports/github.com/chubbyphp/chubbyphp-workerman-request-handler/master)[![Latest Stable Version](https://poser.pugx.org/chubbyphp/chubbyphp-workerman-request-handler/v/stable.png)](https://packagist.org/packages/chubbyphp/chubbyphp-workerman-request-handler)
 [![Latest Stable Version](https://poser.pugx.org/chubbyphp/chubbyphp-workerman-request-handler/v/stable.png)](https://packagist.org/packages/chubbyphp/chubbyphp-workerman-request-handler)
 [![Total Downloads](https://poser.pugx.org/chubbyphp/chubbyphp-workerman-request-handler/downloads.png)](https://packagist.org/packages/chubbyphp/chubbyphp-workerman-request-handler)
 [![Monthly Downloads](https://poser.pugx.org/chubbyphp/chubbyphp-workerman-request-handler/d/monthly)](https://packagist.org/packages/chubbyphp/chubbyphp-workerman-request-handler)
@@ -25,19 +25,19 @@ A request handler adapter for workerman, using PSR-7, PSR-15 and PSR-17.
 
 ## Requirements
 
- * php: ^8.0
- * [psr/http-factory][2]: ^1.0.1
- * [psr/http-message][3]: ^1.0.1
- * [psr/http-server-handler][4]: ^1.0.1
- * [psr/log][5]: ^1.1.4|^2.0|^3.0
- * [workerman/workerman][6]: ^4.0.30
+ * php: ^8.1
+ * [psr/http-factory][2]: ^1.0.2
+ * [psr/http-message][3]: ^1.1|^2.0
+ * [psr/http-server-handler][4]: ^1.0.2
+ * [psr/log][5]: ^2.0|^3.0
+ * [workerman/workerman][6]: ^4.1.13
 
 ## Installation
 
 Through [Composer](http://getcomposer.org) as [chubbyphp/chubbyphp-workerman-request-handler][1].
 
 ```sh
-composer require chubbyphp/chubbyphp-workerman-request-handler "^2.0"
+composer require chubbyphp/chubbyphp-workerman-request-handler "^2.1"
 ```
 
 ## Usage
diff --git a/composer.json b/composer.json
@@ -17,23 +17,22 @@
         }
     ],
     "require": {
-        "php": "^8.0",
-        "psr/http-factory": "^1.0.1",
-        "psr/http-message": "^1.0.1|^2.0",
-        "psr/http-server-handler": "^1.0.1",
-        "psr/log": "^1.1.4|^2.0|^3.0",
-        "workerman/workerman": "^4.0.30"
+        "php": "^8.1",
+        "psr/http-factory": "^1.0.2",
+        "psr/http-message": "^1.1|^2.0",
+        "psr/http-server-handler": "^1.0.2",
+        "psr/log": "^2.0|^3.0",
+        "workerman/workerman": "^4.1.13"
     },
     "require-dev": {
-        "blackfire/php-sdk": "^1.29",
+        "blackfire/php-sdk": "^2.3.5",
         "chubbyphp/chubbyphp-dev-helper": "dev-master",
-        "chubbyphp/chubbyphp-mock": "^1.6.1",
-        "infection/infection": "^0.26.5",
-        "php-coveralls/php-coveralls": "^2.5.2",
-        "phploc/phploc": "^7.0.2",
-        "phpstan/extension-installer": "^1.1",
-        "phpstan/phpstan": "^1.4.8",
-        "phpunit/phpunit": "^9.5.17"
+        "chubbyphp/chubbyphp-mock": "^1.7",
+        "infection/infection": "^0.27.8",
+        "php-coveralls/php-coveralls": "^2.7",
+        "phpstan/extension-installer": "^1.3.1",
+        "phpstan/phpstan": "^1.10.45",
+        "phpunit/phpunit": "^10.4.2"
     },
     "autoload": {
         "psr-4": { "Chubbyphp\\WorkermanRequestHandler\\": "src/" }
@@ -50,7 +49,7 @@
     },
     "extra": {
         "branch-alias": {
-            "dev-master": "2.0-dev"
+            "dev-master": "2.1-dev"
         }
     },
     "scripts": {
@@ -61,14 +60,12 @@
             "@test:integration",
             "@test:infection",
             "@test:static-analysis",
-            "@test:loc",
             "@test:cs"
         ],
         "test:cs": "mkdir -p build && PHP_CS_FIXER_IGNORE_ENV=1 vendor/bin/php-cs-fixer fix --dry-run --stop-on-violation --cache-file=build/phpcs.cache",
         "test:infection": "vendor/bin/infection --threads=$(nproc) --min-msi=100 --verbose --coverage=build/phpunit",
         "test:integration": "vendor/bin/phpunit --testsuite=Integration --cache-result-file=build/phpunit/result.cache",
         "test:lint": "mkdir -p build && find src tests -name '*.php' -print0 | xargs -0 -n1 -P$(nproc) php -l | tee build/phplint.log",
-        "test:loc": "mkdir -p build && vendor/bin/phploc src | tee build/phploc.log",
         "test:static-analysis": "mkdir -p build && bash -c 'vendor/bin/phpstan analyse src --no-progress --level=7 --error-format=junit | tee build/phpstan.junit.xml; if [ ${PIPESTATUS[0]} -ne \"0\" ]; then exit 1; fi'",
         "test:unit": "vendor/bin/phpunit --testsuite=Unit --coverage-text --coverage-clover=build/phpunit/clover.xml --coverage-html=build/phpunit/coverage-html --coverage-xml=build/phpunit/coverage-xml --log-junit=build/phpunit/junit.xml --cache-result-file=build/phpunit/result.cache"
     }
diff --git a/phpunit.xml b/phpunit.xml
@@ -1,26 +1,17 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         backupGlobals="false"
-         backupStaticAttributes="false"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         executionOrder="random"
-         processIsolation="false"
-         stopOnFailure="false"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
-    <coverage>
-        <include>
-            <directory>./src</directory>
-        </include>
-    </coverage>
-    <testsuites>
-        <testsuite name="Integration">
-            <directory>./tests/Integration</directory>
-        </testsuite>
-        <testsuite name="Unit">
-            <directory>./tests/Unit</directory>
-        </testsuite>
-    </testsuites>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" colors="true" executionOrder="random" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
+  <coverage/>
+  <testsuites>
+    <testsuite name="Integration">
+      <directory>./tests/Integration</directory>
+    </testsuite>
+    <testsuite name="Unit">
+      <directory>./tests/Unit</directory>
+    </testsuite>
+  </testsuites>
+  <source>
+    <include>
+      <directory>./src</directory>
+    </include>
+  </source>
 </phpunit>
diff --git a/src/Adapter/NewRelicOnMessageAdapter.php b/src/Adapter/NewRelicOnMessageAdapter.php
@@ -10,9 +10,7 @@
 
 final class NewRelicOnMessageAdapter implements OnMessageInterface
 {
-    public function __construct(private OnMessageInterface $onRequest, private string $appname)
-    {
-    }
+    public function __construct(private OnMessageInterface $onRequest, private string $appname) {}
 
     public function __invoke(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): void
     {
diff --git a/src/OnMessage.php b/src/OnMessage.php
@@ -14,8 +14,7 @@ public function __construct(
         private PsrRequestFactoryInterface $psrRequestFactory,
         private WorkermanResponseEmitterInterface $workermanResponseEmitter,
         private RequestHandlerInterface $requestHander
-    ) {
-    }
+    ) {}
 
     public function __invoke(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): void
     {
diff --git a/src/PsrRequestFactory.php b/src/PsrRequestFactory.php
@@ -18,8 +18,7 @@ public function __construct(
         private ServerRequestFactoryInterface $serverRequestFactory,
         private StreamFactoryInterface $streamFactory,
         private UploadedFileFactoryInterface $uploadedFileFactory
-    ) {
-    }
+    ) {}
 
     public function create(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): ServerRequestInterface
     {
diff --git a/tests/Unit/Adapter/NewRelicOnMessageAdapterTest.php b/tests/Unit/Adapter/NewRelicOnMessageAdapterTest.php
@@ -85,7 +85,7 @@ function newrelic_end_transaction(bool $ignore = false): void
      *
      * @internal
      */
-    final class NewRelicRouteMiddlewareTest extends TestCase
+    final class NewRelicOnMessageAdapterTest extends TestCase
     {
         use MockByCallsTrait;
 
diff --git a/tests/Unit/PsrRequestFactoryTest.php b/tests/Unit/PsrRequestFactoryTest.php
@@ -30,6 +30,8 @@ final class PsrRequestFactoryTest extends TestCase
 
     public function testInvoke(): void
     {
+        $bodyString = 'This is the body.';
+
         /** @var MockObject|WorkermanRequest $workermanRequest */
         $workermanRequest = $this->getMockByCalls(WorkermanRequest::class, [
             Call::create('method')->with()->willReturn('POST'),
@@ -79,7 +81,7 @@ public function testInvoke(): void
 
         /** @var MockObject|StreamInterface $requestBody */
         $requestBody = $this->getMockByCalls(StreamInterface::class, [
-            Call::create('write')->with('This is the body.'),
+            Call::create('write')->with($bodyString)->willReturn(\strlen($bodyString)),
         ]);
 
         /** @var MockObject|StreamInterface $uploadedFileStream1 */

Original file line number	Diff line number	Diff line change
`@@ -10,9 +10,7 @@`
`10`	`10`
`11`	`11`	`final class NewRelicOnMessageAdapter implements OnMessageInterface`
`12`	`12`	`{`
`13`		`- public function __construct(private OnMessageInterface $onRequest, private string $appname)`
`14`		`- {`
`15`		`- }`
	`13`	`+ public function __construct(private OnMessageInterface $onRequest, private string $appname) {}`
`16`	`14`
`17`	`15`	`public function __invoke(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): void`
`18`	`16`	`{`
Original file line number	Diff line number	Diff line change
`@@ -14,8 +14,7 @@ public function __construct(`
`14`	`14`	`private PsrRequestFactoryInterface $psrRequestFactory,`
`15`	`15`	`private WorkermanResponseEmitterInterface $workermanResponseEmitter,`
`16`	`16`	`private RequestHandlerInterface $requestHander`
`17`		`- ) {`
`18`		`- }`
	`17`	`+ ) {}`
`19`	`18`
`20`	`19`	`public function __invoke(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): void`
`21`	`20`	`{`
Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,7 @@ public function __construct(`
`18`	`18`	`private ServerRequestFactoryInterface $serverRequestFactory,`
`19`	`19`	`private StreamFactoryInterface $streamFactory,`
`20`	`20`	`private UploadedFileFactoryInterface $uploadedFileFactory`
`21`		`- ) {`
`22`		`- }`
	`21`	`+ ) {}`
`23`	`22`
`24`	`23`	`public function create(WorkermanTcpConnection $workermanTcpConnection, WorkermanRequest $workermanRequest): ServerRequestInterface`
`25`	`24`	`{`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ function newrelic_end_transaction(bool $ignore = false): void`
`85`	`85`	`*`
`86`	`86`	`* @internal`
`87`	`87`	`*/`
`88`		`- final class NewRelicRouteMiddlewareTest extends TestCase`
	`88`	`+ final class NewRelicOnMessageAdapterTest extends TestCase`
`89`	`89`	`{`
`90`	`90`	`use MockByCallsTrait;`
`91`	`91`