Allowed PHP8

Author: giovannialbero1992

.ci/packer_cache.sh

@@ -2,7 +2,8 @@
 
 source /usr/local/bin/bash_standard_lib.sh
 
-DOCKER_IMAGES="php:7.4-cli
+DOCKER_IMAGES="php:8.0-cli
+php:7.4-cli
 php:7.3-cli
 php:7.2-cli
 php:7.1-cli

.ci/test-matrix.yml

@@ -3,6 +3,7 @@ STACK_VERSION:
   - 7.x-SNAPSHOT
 
 PHP_VERSION:
+  - 8.0-cli
   - 7.4-cli
   - 7.3-cli
   - 7.2-cli

.github/workflows/test.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        php-version: [7.1, 7.2, 7.3, 7.4]
+        php-version: [7.2, 7.3, 7.4, 8.0]
         os: [ubuntu-latest]
         es-version: [7.x-SNAPSHOT]
 
@@ -24,11 +24,11 @@ jobs:
         tools: prestissimo
       env:
         COMPOSER_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-   
+
     - name: Get composer cache directory
       id: composercache
       run: echo "::set-output name=dir::$(composer config cache-files-dir)"
-      
+
     - name: Cache dependencies
       uses: actions/cache@v2
       with:
@@ -37,9 +37,15 @@ jobs:
         restore-keys: ${{ runner.os }}-composer-
 
     - name: Install dependencies
+      if: ${{ matrix.php-version != '8.0' }}
       run: |
         composer install --prefer-dist
 
+    - name: ignore ignore-platform-reqs if it is using php 8
+      if: ${{ matrix.php-version == '8.0' }}
+      run: |
+        composer install --prefer-dist --ignore-platform-reqs
+
     - name: PHP Coding Standards
       run: |
         composer run-script phpcs
@@ -71,4 +77,3 @@ jobs:
         vendor/bin/phpunit -c phpunit-integration-tests.xml
       env:
         TEST_SUITE: oss
-        

composer.json

@@ -13,7 +13,7 @@
     }
   ],
   "require": {
-    "php": "^7.1",
+    "php": "^7.1 || ^8.0",
     "ext-json": ">=1.3.7",
     "ezimuel/ringphp": "^1.1.2",
     "psr/log": "~1.0"
@@ -25,7 +25,7 @@
     "doctrine/inflector": "^1.3",
     "mockery/mockery": "^1.2",
     "phpstan/phpstan": "^0.12",
-    "phpunit/phpunit": "^7.5",
+    "phpunit/phpunit": "^7.5 || ^8.5 || ^9.3",
     "squizlabs/php_codesniffer": "^3.4",
     "symfony/finder": "~4.0",
     "symfony/yaml": "~4.0"

tests/Elasticsearch/Tests/ClientBuilder/DummyLogger.php

@@ -19,5 +19,5 @@
 
 class DummyLogger
 {
-    
+
 }

tests/Elasticsearch/Tests/ClientBuilderTest.php

@@ -21,26 +21,20 @@
 use Elasticsearch\Client;
 use Elasticsearch\ClientBuilder;
 use Elasticsearch\Common\Exceptions\ElasticsearchException;
-use Elasticsearch\Common\Exceptions\InvalidArgumentException;
 use Elasticsearch\Tests\ClientBuilder\DummyLogger;
-use GuzzleHttp\Ring\Client\MockHandler;
 use PHPUnit\Framework\TestCase;
 
 class ClientBuilderTest extends TestCase
 {
-    /**
-     * @expectedException TypeError
-     */
     public function testClientBuilderThrowsExceptionForIncorrectLoggerClass()
     {
+        $this->expectException(\TypeError::class);
         ClientBuilder::create()->setLogger(new DummyLogger);
     }
 
-    /**
-     * @expectedException TypeError
-     */
     public function testClientBuilderThrowsExceptionForIncorrectTracerClass()
     {
+        $this->expectException(\TypeError::class);
         ClientBuilder::create()->setTracer(new DummyLogger);
     }
 
@@ -51,7 +45,7 @@ public function testGzipEnabledWhenElasticCloudId()
             ->build();
 
         $this->assertInstanceOf(Client::class, $client);
-        
+
         try {
             $result = $client->info();
         } catch (ElasticsearchException $e) {
@@ -75,7 +69,7 @@ public function testElasticCloudIdNotOverrideCurlEncoding()
             ->build();
 
         $this->assertInstanceOf(Client::class, $client);
-        
+
         try {
             $result = $client->info();
         } catch (ElasticsearchException $e) {

tests/Elasticsearch/Tests/ClientIntegrationTest.php

@@ -44,7 +44,7 @@ class ClientIntegrationTest extends \PHPUnit\Framework\TestCase
      */
     private $host;
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->host = Utility::getHost();
         if (null == $this->host) {
@@ -58,10 +58,10 @@ private function getClient(): Client
         $client = ClientBuilder::create()
             ->setHosts([$this->host])
             ->setLogger($this->logger);
-        
+
         if (getenv('TEST_SUITE') === 'xpack') {
             $client->setSSLVerification(__DIR__ . '/../../../.ci/certs/ca.crt');
-        }    
+        }
         return $client->build();
     }
 
@@ -80,7 +80,7 @@ public function testLogRequestSuccessHasPortInInfo()
 
         $result = $client->info();
 
-        $this->assertContains('"port"', $this->getLevelOutput(LogLevel::INFO, $this->logger->output));
+        $this->assertStringContainsString('"port"', $this->getLevelOutput(LogLevel::INFO, $this->logger->output));
     }
 
     public function testLogRequestFailHasWarning()

tests/Elasticsearch/Tests/ClientTest.php

@@ -33,7 +33,7 @@
  */
 class ClientTest extends \PHPUnit\Framework\TestCase
 {
-    public function tearDown()
+    public function tearDown(): void
     {
         m::close();
     }

tests/Elasticsearch/Tests/ConnectionPool/Selectors/StickyRoundRobinSelectorTest.php

@@ -29,7 +29,7 @@
  */
 class StickyRoundRobinSelectorTest extends \PHPUnit\Framework\TestCase
 {
-    public function tearDown()
+    public function tearDown(): void
     {
         m::close();
     }

tests/Elasticsearch/Tests/ConnectionPool/SniffingConnectionPoolIntegrationTest.php

@@ -30,7 +30,7 @@
  */
 class SniffingConnectionPoolIntegrationTest extends \PHPUnit\Framework\TestCase
 {
-    protected function setUp()
+    protected function setUp(): void
     {
         static::markTestSkipped("All of Sniffing unit tests use outdated cluster state format, need to redo");
     }