From 8a325608180f235e3b94e18ce6b591cefc34b55d Mon Sep 17 00:00:00 2001
From: Dennis Eichhorn <spl1nes.com@googlemail.com>
Date: Thu, 21 Oct 2021 22:16:07 +0200
Subject: [PATCH] more tests and make classes final

---
 Models/NullTaskElement.php           | 12 ++++++++
 Models/TaskElement.php               |  2 +-
 tests/Models/NullTask.php            | 42 ++++++++++++++++++++++++++++
 tests/Models/NullTaskElement.php     | 42 ++++++++++++++++++++++++++++
 tests/Models/NullTaskElementTest.php | 42 ++++++++++++++++++++++++++++
 tests/Models/NullTaskTest.php        | 42 ++++++++++++++++++++++++++++
 6 files changed, 181 insertions(+), 1 deletion(-)
 create mode 100644 tests/Models/NullTask.php
 create mode 100644 tests/Models/NullTaskElement.php
 create mode 100644 tests/Models/NullTaskElementTest.php
 create mode 100644 tests/Models/NullTaskTest.php

diff --git a/Models/NullTaskElement.php b/Models/NullTaskElement.php
index 6637e43..6fe8cbc 100755
--- a/Models/NullTaskElement.php
+++ b/Models/NullTaskElement.php
@@ -24,4 +24,16 @@ namespace Modules\Tasks\Models;
  */
 final class NullTaskElement extends TaskElement
 {
+	/**
+     * Constructor
+     *
+     * @param int $id Model id
+     *
+     * @since 1.0.0
+     */
+    public function __construct(int $id = 0)
+    {
+        $this->id = $id;
+        parent::__construct();
+    }
 }
diff --git a/Models/TaskElement.php b/Models/TaskElement.php
index e173dff..e386d73 100755
--- a/Models/TaskElement.php
+++ b/Models/TaskElement.php
@@ -36,7 +36,7 @@ class TaskElement implements \JsonSerializable
      * @var int
      * @since 1.0.0
      */
-    private int $id = 0;
+    protected int $id = 0;
 
     /**
      * Description.
diff --git a/tests/Models/NullTask.php b/tests/Models/NullTask.php
new file mode 100644
index 0000000..3243b98
--- /dev/null
+++ b/tests/Models/NullTask.php
@@ -0,0 +1,42 @@
+<?php
+/**
+ * Orange Management
+ *
+ * PHP Version 8.0
+ *
+ * @package   tests
+ * @copyright Dennis Eichhorn
+ * @license   OMS License 1.0
+ * @version   1.0.0
+ * @link      https://orange-management.org
+ */
+declare(strict_types=1);
+
+namespace Modules\Tasks\tests\Models;
+
+use Modules\Tasks\Models\NullTask;
+
+/**
+ * @internal
+ */
+final class Null extends \PHPUnit\Framework\TestCase
+{
+    /**
+     * @covers Modules\Tasks\Models\NullTask
+     * @group framework
+     */
+    public function testNull() : void
+    {
+        self::assertInstanceOf('\Modules\Tasks\Models\Task', new NullTask());
+    }
+
+    /**
+     * @covers Modules\Tasks\Models\NullTask
+     * @group framework
+     */
+    public function testId() : void
+    {
+        $null = new NullTask(2);
+        self::assertEquals(2, $null->getId());
+    }
+}
diff --git a/tests/Models/NullTaskElement.php b/tests/Models/NullTaskElement.php
new file mode 100644
index 0000000..3c3e793
--- /dev/null
+++ b/tests/Models/NullTaskElement.php
@@ -0,0 +1,42 @@
+<?php
+/**
+ * Orange Management
+ *
+ * PHP Version 8.0
+ *
+ * @package   tests
+ * @copyright Dennis Eichhorn
+ * @license   OMS License 1.0
+ * @version   1.0.0
+ * @link      https://orange-management.org
+ */
+declare(strict_types=1);
+
+namespace Modules\Tasks\tests\Models;
+
+use Modules\Tasks\Models\NullTaskElement;
+
+/**
+ * @internal
+ */
+final class Null extends \PHPUnit\Framework\TestCase
+{
+    /**
+     * @covers Modules\Tasks\Models\NullTaskElement
+     * @group framework
+     */
+    public function testNull() : void
+    {
+        self::assertInstanceOf('\Modules\Tasks\Models\TaskElement', new NullTaskElement());
+    }
+
+    /**
+     * @covers Modules\Tasks\Models\NullTaskElement
+     * @group framework
+     */
+    public function testId() : void
+    {
+        $null = new NullTaskElement(2);
+        self::assertEquals(2, $null->getId());
+    }
+}
diff --git a/tests/Models/NullTaskElementTest.php b/tests/Models/NullTaskElementTest.php
new file mode 100644
index 0000000..a4eda64
--- /dev/null
+++ b/tests/Models/NullTaskElementTest.php
@@ -0,0 +1,42 @@
+<?php
+/**
+ * Orange Management
+ *
+ * PHP Version 8.0
+ *
+ * @package   tests
+ * @copyright Dennis Eichhorn
+ * @license   OMS License 1.0
+ * @version   1.0.0
+ * @link      https://orange-management.org
+ */
+declare(strict_types=1);
+
+namespace Modules\Tasks\tests\Models;
+
+use Modules\Tasks\Models\NullTaskElement;
+
+/**
+ * @internal
+ */
+final class NullTaskElementTest extends \PHPUnit\Framework\TestCase
+{
+    /**
+     * @covers Modules\Tasks\Models\NullTaskElement
+     * @group framework
+     */
+    public function testNull() : void
+    {
+        self::assertInstanceOf('\Modules\Tasks\Models\TaskElement', new NullTaskElement());
+    }
+
+    /**
+     * @covers Modules\Tasks\Models\NullTaskElement
+     * @group framework
+     */
+    public function testId() : void
+    {
+        $null = new NullTaskElement(2);
+        self::assertEquals(2, $null->getId());
+    }
+}
diff --git a/tests/Models/NullTaskTest.php b/tests/Models/NullTaskTest.php
new file mode 100644
index 0000000..af5790b
--- /dev/null
+++ b/tests/Models/NullTaskTest.php
@@ -0,0 +1,42 @@
+<?php
+/**
+ * Orange Management
+ *
+ * PHP Version 8.0
+ *
+ * @package   tests
+ * @copyright Dennis Eichhorn
+ * @license   OMS License 1.0
+ * @version   1.0.0
+ * @link      https://orange-management.org
+ */
+declare(strict_types=1);
+
+namespace Modules\Tasks\tests\Models;
+
+use Modules\Tasks\Models\NullTask;
+
+/**
+ * @internal
+ */
+final class NullTaskTest extends \PHPUnit\Framework\TestCase
+{
+    /**
+     * @covers Modules\Tasks\Models\NullTask
+     * @group framework
+     */
+    public function testNull() : void
+    {
+        self::assertInstanceOf('\Modules\Tasks\Models\Task', new NullTask());
+    }
+
+    /**
+     * @covers Modules\Tasks\Models\NullTask
+     * @group framework
+     */
+    public function testId() : void
+    {
+        $null = new NullTask(2);
+        self::assertEquals(2, $null->getId());
+    }
+}