Merge pull request #64 from clue-labs/update-tests

Test on PHP 8.2 and update test environment

Author: SimonFrings

.github/workflows/ci.yml

@@ -11,9 +11,10 @@ jobs:
     strategy:
       matrix:
         os:
-          - ubuntu-20.04
-          - windows-2019
+          - ubuntu-22.04
+          - windows-2022
         php:
+          - 8.2
           - 8.1
           - 8.0
           - 7.4

phpunit.xml.dist

@@ -5,7 +5,8 @@
          xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
          bootstrap="vendor/autoload.php"
          colors="true"
-         cacheResult="false">
+         cacheResult="false"
+         convertDeprecationsToExceptions="true">
     <testsuites>
         <testsuite name="SQLite React Test Suite">
             <directory>./tests/</directory>
@@ -17,4 +18,7 @@
             <directory>./src/</directory>
         </include>
     </coverage>
+    <php>
+        <ini name="error_reporting" value="-1" />
+    </php>
 </phpunit>

src/Factory.php

@@ -433,7 +433,7 @@ private function php()
 
         // if `php` is a symlink to the php binary, use the shorter `php` name
         // this is purely cosmetic feature for the process list
-        if ($binary !== 'php' && \realpath($this->which('php')) === $binary) {
+        if ($binary !== 'php' && \realpath((string) $this->which('php')) === $binary) {
             $binary = 'php';
         }