feat: upgrade to php 8.

Author: loduis

.gitignore

@@ -1,7 +1,10 @@
 /.idea/
 /tests/data/
 /tests/build/
+/tests/.phpunit.cache/
 /vendor/
 /.env
 /.php-cs-fixer.cache
-/composer.lock
+/composer.lock
+phpunit.xml
+.phpunit.result.cache

composer.json

@@ -9,10 +9,10 @@
         }
     ],
     "require": {
-        "php": ">=5.4"
+        "php": "^8.0"
     },
     "require-dev": {
-        "phpunit/phpunit": "^5.7"
+        "phpunit/phpunit": "^9.3"
     },
     "autoload": {
         "psr-4": {
@@ -29,7 +29,10 @@
     },
     "extra": {
         "branch-alias": {
-            "dev-master": "1.0.6-dev"
+            "dev-master": "2.0.0-dev"
         }
+    },
+    "config": {
+        "platform-check": true
     }
-}
+}

phpunit.xml

@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.6/phpunit.xsd"
+         bootstrap="tests/bootstrap.php"
+         cacheResultFile="tests/build/test-results"
+         executionOrder="depends,defects"
+         forceCoversAnnotation="false"
+         beStrictAboutCoversAnnotation="false"
+         beStrictAboutOutputDuringTests="true"
+         beStrictAboutTodoAnnotatedTests="true"
+         convertDeprecationsToExceptions="true"
+         failOnRisky="true"
+         failOnWarning="true"
+         verbose="true">
+    <testsuites>
+        <testsuite name="default">
+            <directory>tests</directory>
+        </testsuite>
+    </testsuites>
+
+    <coverage cacheDirectory="tests/build/code-coverage"
+              processUncoveredFiles="true">
+        <include>
+            <directory suffix=".php">src</directory>
+        </include>
+    </coverage>
+    <!-- For live test
+    <php>
+        <env name="API_COMPANY" value="yourcompany" />
+        <env name="API_KEY" value="yourkey" />
+        <env name="API_FORMAT" value="json" />
+    </php>
+    -->
+</phpunit>

phpunit.xml..dist

@@ -17,39 +17,39 @@ public static function set()
     {
         $num_args = func_num_args();
         if ($num_args === 1) {
-            self::$config['url'] = self::$url;
-            self::$config['key'] = func_get_arg(0);
-            self::$config['url'] = Factory::build('account')->authenticate()->url;
+            static::$config['url'] = static::$url;
+            static::$config['key'] = func_get_arg(0);
+            static::$config['url'] = Factory::build('account')->authenticate()->url;
         } elseif ($num_args === 2) {
-            self::$config['url'] = $url = func_get_arg(0);
-            self::checkSubDomain($url);
-            if (self::$is_subdomain) {
-                self::$config['url'] = self::$url;
+            static::$config['url'] = $url = func_get_arg(0);
+            static::checkSubDomain($url);
+            if (static::$is_subdomain) {
+                static::$config['url'] = static::$url;
             }
-            self::$config['key'] = func_get_arg(1);
-            if (self::$is_subdomain) {
+            static::$config['key'] = func_get_arg(1);
+            if (static::$is_subdomain) {
                 $test = Factory::build('account')->authenticate();
                 $url = $test->url;
             }
-            self::$config['url'] = $url;
+            static::$config['url'] = $url;
         }
     }
 
     public static function get()
     {
-        return array_values(self::$config);
+        return array_values(static::$config);
     }
 
     private static function checkSubDomain($url)
     {
         $eu_domain = strpos($url, '.eu');
 
         if ($eu_domain !== false) {
-            self::$url = 'https://authenticate.eu.teamwork.com/';
+            static::$url = 'https://authenticate.eu.teamwork.com/';
             $url = substr($url, 0, $eu_domain);
         }
         if (strpos($url, '.') === false) {
-            self::$is_subdomain = true;
+            static::$is_subdomain = true;
         }
     }
 }

src/Auth.php

@@ -4,6 +4,8 @@
 
 abstract class Model extends \TeamWorkPm\Model
 {
+    protected $resource;
+
     protected function init()
     {
         $this->parent = 'comment';

src/Comment/Model.php

@@ -6,7 +6,7 @@ class Factory
 {
     /**
      * @param string $class_name
-     * @return Model
+     * @return Model | People | Project\People
      */
     public static function build($class_name)
     {

src/Factory.php

@@ -2,6 +2,8 @@
 
 namespace TeamWorkPm\Response;
 
+use Traversable;
+
 abstract class Model implements \IteratorAggregate, \Countable, \ArrayAccess
 {
     protected $string = null;
@@ -49,17 +51,17 @@ public function getHeaders()
         return $this->headers;
     }
 
-    public function getIterator()
+    public function getIterator(): Traversable
     {
         return new \ArrayIterator($this->data);
     }
 
-    public function count()
+    public function count(): int
     {
         return count($this->data);
     }
 
-    public function offsetSet($offset, $value)
+    public function offsetSet($offset, $value): void
     {
         if (is_null($offset)) {
             $this->data[] = $value;
@@ -68,17 +70,17 @@ public function offsetSet($offset, $value)
         }
     }
 
-    public function offsetExists($offset)
+    public function offsetExists($offset): bool
     {
         return isset($this->data[$offset]);
     }
 
-    public function offsetUnset($offset)
+    public function offsetUnset($offset): void
     {
         unset($this->data[$offset]);
     }
 
-    public function offsetGet($offset)
+    public function offsetGet($offset): mixed
     {
         return isset($this->data[$offset]) ? $this->data[$offset] : null;
     }

src/Response/Model.php

@@ -2,6 +2,8 @@
 
 namespace TeamWorkPm;
 
+use CurlHandle;
+
 final class Rest
 {
     /**
@@ -103,7 +105,7 @@ private function execute($method, $action, $parameters = null)
      * @param string $url
      * @param string|null $params
      * @param array $headers
-     * @return false|resource
+     * @return CurlHandle|false
      */
     private static function initCurl($method, $url, $params, $headers)
     {

src/Rest.php

@@ -4,6 +4,9 @@
 
 use TeamWorkPm\Rest;
 
+/**
+ * @method void init()
+ */
 abstract class Model
 {
     /**

src/Rest/Model.php

@@ -13,7 +13,7 @@ class AccountTest extends TestCase
      */
     private $model;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('account');

tests/AccountTest.php

@@ -14,7 +14,7 @@ class ActivityTest extends TestCase
      */
     private $model;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('activity');

tests/ActivityTest.php

@@ -16,7 +16,7 @@ class FileTest extends TestCase
     private $projectId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('category/file');

tests/Category/FileTest.php

@@ -12,7 +12,7 @@ class LinkTest extends TestCase
     private $projectId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('category/link');

tests/Category/LinkTest.php

@@ -12,7 +12,7 @@ class MessageTest extends TestCase
     private $projectId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('category/message');

tests/Category/MessageTest.php

@@ -12,7 +12,7 @@ class NotebookTest extends TestCase
     private $projectId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('category/notebook');

tests/Category/NotebookTest.php

@@ -11,7 +11,7 @@ class ProjectTest extends TestCase
     private $model;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('category/project');

tests/Category/ProjectTest.php

@@ -13,7 +13,7 @@ class FileTest extends TestCase
     private $projectId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('comment/file');

tests/Comment/FileTest.php

@@ -12,7 +12,7 @@ class MilestoneTest extends TestCase
     private $resourceId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('comment/milestone');

tests/Comment/MilestoneTest.php

@@ -12,7 +12,7 @@ class NotebookTest extends TestCase
     private $resourceId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('comment/notebook');

tests/Comment/NotebookTest.php

@@ -12,7 +12,7 @@ class TaskTest extends TestCase
     private $resourceId;
     private $id;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('comment/task');

tests/Comment/TaskTest.php

@@ -11,7 +11,7 @@ class CompanyTest extends TestCase
     private $id;
     private $projectId;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('company');

tests/CompanyTest.php

@@ -11,7 +11,7 @@ class FileTest extends TestCase
     private $id;
     private $projectId;
 
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->model = Factory::build('file');