Merge pull request php-webdriver#1074 from php-webdriver/feature/update-php-build

Update PHP versions in test builds, run tests primarily on PHP 8

Author: OndraM

.github/workflows/sauce-labs.yaml

@@ -34,7 +34,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '7.4'
+          php-version: '8.2'
           extensions: mbstring, intl, zip
           coverage: none
 

.github/workflows/tests.yaml

@@ -19,7 +19,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '7.4'
+          php-version: '8.2'
           extensions: mbstring, intl, zip
 
       - name: Install PHP dependencies
@@ -46,11 +46,10 @@ jobs:
 
     strategy:
       matrix:
-        php-version: ['7.3', '7.4', '8.0']
+        php-version: ['7.3', '7.4', '8.0', '8.1', '8.2']
         dependencies: ['']
         include:
           - { php-version: '7.3', dependencies: '--prefer-lowest' }
-          - { php-version: '8.1', dependencies: '--ignore-platform-req=php' }
 
     steps:
       - uses: actions/checkout@v3
@@ -105,7 +104,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 7.4
+          php-version: 8.2
           extensions: mbstring, intl, zip
           coverage: xdebug
 

tests/functional/RemoteWebElementTest.php

@@ -515,8 +515,11 @@ public function testShouldTakeAndSaveElementScreenshot(): void
 
         // Assert string output
         $imageFromString = imagecreatefromstring($outputPngString);
-        $this->assertNotFalse($imageFromString);
-        $this->assertTrue(is_resource($imageFromString));
+        if (version_compare(phpversion(), '8.0.0', '>=')) {
+            $this->assertInstanceOf(\GdImage::class, $imageFromString);
+        } else {
+            $this->assertTrue(is_resource($imageFromString));
+        }
 
         if ($isSafari && !$isCi) {
             $this->assertEquals(10, imagesx($imageFromString));