allow php 8 and support phpunit 9

Author: dbu

composer.json

@@ -27,9 +27,9 @@
         }
     ],
     "require": {
-        "php": "^5.6|^7.0",
+        "php": "^7.1 || ^8.0",
         "phpcr/phpcr-implementation": "2.1.*",
-        "phpunit/phpunit": "^5.7 || ^6.4 || ^7.0"
+        "phpunit/phpunit": "^7.0 || ^8.0 || ^9.0"
     },
     "autoload": {
         "psr-4": {

inc/AbstractLoader.php

@@ -255,10 +255,11 @@ public function doesSessionLastModified()
      */
     public function getTestSupported($chapter, $case, $name)
     {
-        return !(in_array($chapter, $this->unsupportedChapters)
+        return !(
+            in_array($chapter, $this->unsupportedChapters)
                   || in_array($case, $this->unsupportedCases)
                   || in_array($name, $this->unsupportedTests)
-                 );
+        );
     }
 
     /**

inc/BaseCase.php

@@ -96,7 +96,7 @@ abstract class BaseCase extends TestCase
      *
      * @see initProperties()
      */
-    public static function setupBeforeClass($fixtures = 'general/base')
+    public static function setupBeforeClass($fixtures = 'general/base'): void
     {
         self::$loader = ImplementationLoader::getInstance();
 
@@ -120,7 +120,7 @@ public static function setupBeforeClass($fixtures = 'general/base')
         self::$staticSharedFixture['additionalSession'] = self::$loader->getAdditionalSession();
     }
 
-    protected function setUp()
+    protected function setUp(): void
     {
         $fqn = get_called_class();
         $parts = explode('\\', $fqn);
@@ -144,7 +144,7 @@ protected function setUp()
         $this->initProperties();
     }
 
-    public static function tearDownAfterClass()
+    public static function tearDownAfterClass(): void
     {
         if (isset(self::$staticSharedFixture['session'])) {
             self::$staticSharedFixture['session']->logout();
@@ -278,7 +278,8 @@ protected function assertSimilarDateTime($expected, $data)
     {
         $this->assertInstanceOf(DateTime::class, $expected);
         $this->assertInstanceOf(DateTime::class, $data);
-        $this->assertTrue(abs($expected->getTimestamp() - $data->getTimestamp()) <= 3,
+        $this->assertTrue(
+            abs($expected->getTimestamp() - $data->getTimestamp()) <= 3,
             $data->format('c').' is not close to the expected '.$expected->format('c')
         );
     }
@@ -290,7 +291,7 @@ protected function assertSimilarDateTime($expected, $data)
      *
      * @return bool True if the test can be done. Otherwise the test is skipped.
      */
-    protected function skipIfNotSupported($descriptor)
+    protected function skipIfNotSupported($descriptor): bool
     {
         if (false === $this->session->getRepository()->getDescriptor($descriptor)) {
             $this->markTestSkipped('Descriptor "'.$descriptor.'" not supported');

tests/Connecting/RepositoryDescriptorsTest.php

@@ -15,7 +15,7 @@
 
 class RepositoryDescriptorsTest extends BaseCase
 {
-    public static function setupBeforeClass($fixtures = false)
+    public static function setupBeforeClass($fixtures = false): void
     {
         // Don't care about fixtures
         parent::setupBeforeClass($fixtures);
@@ -41,7 +41,7 @@ public function testDescriptorKeys()
     {
         $rep = self::$loader->getRepository();
         $keys = $rep->getDescriptorKeys();
-        $this->assertInternalType('array', $keys);
+        $this->assertIsArray($keys);
         $this->assertNotEmpty($keys);
         foreach ($this->expectedDescriptors as $descriptor) {
             $this->assertContains($descriptor, $keys);

tests/Connecting/RepositoryFactoryTest.php

@@ -17,7 +17,7 @@
 
 class RepositoryFactoryTest extends BaseCase
 {
-    public static function setupBeforeClass($fixtures = false)
+    public static function setupBeforeClass($fixtures = false): void
     {
         // Don't care about fixtures
         parent::setupBeforeClass($fixtures);

tests/Connecting/RepositoryTest.php

@@ -20,7 +20,7 @@
 
 class RepositoryTest extends BaseCase
 {
-    public static function setupBeforeClass($fixtures = null)
+    public static function setupBeforeClass($fixtures = null): void
     {
         // Don't care about fixtures
         parent::setupBeforeClass($fixtures);

tests/Connecting/SessionReadMethodsTest.php

@@ -54,7 +54,7 @@ public function testGetAttributeNames()
         $cr->setAttribute('foo', 'bar');
         $session = $this->assertSession($cr);
         $attrs = $session->getAttributeNames();
-        $this->assertInternalType('array', $attrs);
+        $this->assertIsArray($attrs);
         $this->assertContains('foo', $attrs);
     }
 

tests/Connecting/SimpleCredentialsTest.php

@@ -20,7 +20,7 @@
  */
 class SimpleCredentialsTest extends BaseCase
 {
-    public static function setupBeforeClass($fixtures = false)
+    public static function setupBeforeClass($fixtures = false): void
     {
         // Don't care about fixtures
         parent::setupBeforeClass($fixtures);
@@ -67,7 +67,7 @@ public function testAttributes()
         $cr->setAttribute($attrName, $attrValue);
         $this->assertEquals($attrValue, $cr->getAttribute($attrName));
         $attrs = $cr->getAttributeNames();
-        $this->assertInternalType('array', $attrs);
+        $this->assertIsArray($attrs);
         $this->assertContains($attrName, $attrs);
         $cr->removeAttribute($attrName);
         $this->assertNull($cr->getAttribute($attrName));

tests/Connecting/WorkspaceReadMethodsTest.php

@@ -35,7 +35,7 @@ class WorkspaceReadMethodsTest extends BaseCase
 
     //4.5 Workspace Read Methods
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
 
@@ -98,7 +98,7 @@ public function testGetAccessibleWorkspaceNames()
     {
         $names = $this->workspace->getAccessibleWorkspaceNames();
 
-        $this->assertInternalType('array', $names);
+        $this->assertIsArray($names);
         $this->assertContains(self::$loader->getWorkspaceName(), $names);
     }
 

tests/Export/ExportRepositoryContentTest.php

@@ -21,7 +21,7 @@
 //7 Export Repository Content
 class ExportRepositoryContentTest extends BaseCase
 {
-    public static function setupBeforeClass($fixtures = '07_Export/systemview')
+    public static function setupBeforeClass($fixtures = '07_Export/systemview'): void
     {
         parent::setupBeforeClass($fixtures);
     }