Added mssing return type hints and made newly added class final

littleredbutton · Mar 3, 2024 · 7fbc727 · 7fbc727
1 parent f211b4d
commit 7fbc727
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/Parameters/HooksListParameters.php b/src/Parameters/HooksListParameters.php
@@ -25,7 +25,7 @@
  * @method string getMeetingID()
  * @method $this  setMeetingID(string $id)
  */
-class HooksListParameters extends MetaParameters
+final class HooksListParameters extends MetaParameters
 {
     /**
      * @var string

diff --git a/tests/unit/Parameters/HooksListParametersTest.php b/tests/unit/Parameters/HooksListParametersTest.php
@@ -23,7 +23,7 @@
 
 final class HooksListParametersTest extends TestCase
 {
-    public function testHooksListParameters()
+    public function testHooksListParameters(): void
     {
         $hooksListParameters = new HooksListParameters();
 

diff --git a/tests/unit/Responses/HooksDestroyResponseTest.php b/tests/unit/Responses/HooksDestroyResponseTest.php
@@ -50,7 +50,7 @@ public function testHooksDestroyResponseTypes()
         $this->assertEachGetterValueIsBoolean($this->destroyResponse, ['removed']);
     }
 
-    public function testHookDestroyMissingHook()
+    public function testHookDestroyMissingHook(): void
     {
         $xml = simplexml_load_string('<response><returncode>FAILED</returncode><messageKey>destroyMissingHook</messageKey><message>The hook informed was not found.</message></response>');
 
@@ -59,7 +59,7 @@ public function testHookDestroyMissingHook()
         $this->assertTrue($destroyResponse->isMissingHook());
     }
 
-    public function testHookDestroyHookError()
+    public function testHookDestroyHookError(): void
     {
         $xml = simplexml_load_string('<response><returncode>FAILED</returncode><messageKey>destroyHookError</messageKey><message>An error happened while removing your hook. Check the logs.</message></response>');