Added some doc block comments and coverage config

Author: Samuel

.scrutinizer.yml

@@ -1,27 +1,29 @@
 filter:
-    excluded_paths: [tests/*]
+  excluded_paths: [tests/*]
 
 checks:
-    php:
-        remove_extra_empty_lines: true
-        remove_php_closing_tag: true
-        remove_trailing_whitespace: true
-        fix_use_statements:
-            remove_unused: true
-            preserve_multiple: false
-            preserve_blanklines: true
-            order_alphabetically: true
-        fix_php_opening_tag: true
-        fix_linefeed: true
-        fix_line_ending: true
-        fix_identation_4spaces: true
-        fix_doc_comments: true
+  php:
+    remove_extra_empty_lines: true
+    remove_php_closing_tag: true
+    remove_trailing_whitespace: true
+    fix_use_statements:
+      remove_unused: true
+      preserve_multiple: false
+        preserve_blanklines: true
+        order_alphabetically: true
+    fix_php_opening_tag: true
+    fix_linefeed: true
+    fix_line_ending: true
+    fix_identation_4spaces: true
+    fix_doc_comments: true
 
 build:
-    tests:
+  nodes:
+    coverage:
+      tests:
         override:
-            -
-                command: 'vendor/bin/phpunit --coverage-clover=some-file'
-                coverage:
-                    file: 'some-file'
-                    format: 'clover'
+          -
+            command: 'vendor/bin/phpunit --coverage-clover=some-file'
+            coverage:
+              file: 'some-file'
+              format: 'clover'

README.md

@@ -1,7 +1,7 @@
 # Laravel Json Exception Handler
 
 [![StyleCI](https://styleci.io/repos/101529653/shield)](https://styleci.io/repos/101529653)
-[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/sfelix-martins/json-exception-handler/badges/quality-score.png?b=master)](https://scrutinizer-ci.com/g/sfelix-martins/json-exception-handler/?branch=master)
+[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/sfelix-martins/json-exception-handler/badges/quality-score.png?b=2.0)](https://scrutinizer-ci.com/g/sfelix-martins/json-exception-handler/?branch=master)
 
 Adds methods to your `App\Exceptions\Handler` to treat json responses.
 It is most useful if you are building APIs!

composer.json

@@ -11,7 +11,7 @@
         }
     ],
     "require": {
-        "php": ">=5.6.4"
+        "php": ">=7.0.0"
     },
     "autoload": {
         "psr-4": {

src/Handlers/AbstractHandler.php

@@ -4,7 +4,6 @@
 
 use Exception;
 use InvalidArgumentException;
-use SMartins\Exceptions\JsonApi\Error;
 use Illuminate\Auth\AuthenticationException;
 use Illuminate\Validation\ValidationException;
 use SMartins\Exceptions\Response\ErrorHandledInterface;
@@ -60,9 +59,7 @@ public function __construct(Exception $e)
      * Handle with an exception according to specific definitions. Returns one
      * or more errors using the exception from $exceptions attribute.
      *
-     * @todo Change the return type to any interface to make more extensible.
-     *
-     * @return \SMartins\Exceptions\Response\ErrorHandledInterface|\Smartins\Exceptions\Response\ErrorHandledCollectionInterface
+     * @return ErrorHandledInterface|ErrorHandledCollectionInterface
      */
     abstract public function handle();
 
@@ -85,6 +82,7 @@ public function getCode($type = 'default')
      * Return response with handled exception.
      *
      * @return \SMartins\Exceptions\Response\AbstractResponse
+     * @throws \SMartins\Exceptions\Response\InvalidContentException
      */
     public function handleException()
     {
@@ -100,10 +98,10 @@ public function handleException()
     /**
      * Validate response from handle method of handler class.
      *
-     * @param \SMartins\Exceptions\Response\ErrorHandledInterface|\Smartins\Exceptions\Response\ErrorHandledCollectionInterface
-     * @return \SMartins\Exceptions\Response\ErrorHandledCollectionInterface
+     * @param ErrorHandledInterface|ErrorHandledCollectionInterface
+     * @return ErrorHandledCollectionInterface
      *
-     * @throws \InvalidArgumentException
+     * @throws \SMartins\Exceptions\Response\InvalidContentException
      */
     public function validatedHandledException($error)
     {
@@ -216,6 +214,7 @@ public function getResponseHandler()
      * Set exception handlers.
      *
      * @param array $handlers
+     * @return AbstractHandler
      */
     public function setExceptionHandlers(array $handlers)
     {

src/Response/ErrorCollectionInterface.php

@@ -18,7 +18,7 @@ public function getHeaders();
      *
      * @param string $statusCode
      *
-     * @return self
+     * @return static
      */
     public function setStatusCode(string $statusCode);
 

src/Response/ErrorHandledInterface.php

@@ -9,7 +9,7 @@ interface ErrorHandledInterface
      *
      * @todo Maybe pass this method to another interface.
      *
-     * @return \SMartin\Exceptions\Response\ErrorHandledCollectionInterface
+     * @return \SMartins\Exceptions\Response\ErrorHandledCollectionInterface
      */
     public function toCollection(): ErrorHandledCollectionInterface;
 

tests/GettersAndSetters.php

@@ -7,6 +7,9 @@
 
 trait GettersAndSetters
 {
+    /**
+     * @throws \ReflectionException
+     */
     public function testGettersAndSetters()
     {
         if (! property_exists($this, 'classToTest')) {