Update to Laravel 6.5 + PHPUnit 8.4 + PHP CS Fixer

Original PR #80 by @TheFrankman and #84 @michaelaguiar

Author: TheFrankman

.gitignore

@@ -3,4 +3,5 @@ composer.phar
 /vendor/
 composer.lock
 /build
-
+.phpunit.result.cache
+.php_cs.cache

.php_cs

@@ -14,5 +14,6 @@ return PhpCsFixer\Config::create()
         'align_multiline_comment' => true,
         'array_indentation' => true,
         'array_syntax' => ['syntax' => 'short'],
+        'no_superfluous_phpdoc_tags' => false,
     ))
-    ->setFinder($finder);
+    ->setFinder($finder);

composer.json

@@ -15,16 +15,16 @@
     "test": "./vendor/bin/phpunit"
   },
   "require": {
-    "illuminate/http": "5.1 - 5.8",
-    "illuminate/console": "5.1 - 5.8",
-    "illuminate/support": "5.1 - 5.8",
-    "illuminate/filesystem": "5.1 - 5.8"
+    "illuminate/http": "5.1 - 6.5",
+    "illuminate/console": "5.1 - 6.5",
+    "illuminate/support": "5.1 - 6.5",
+    "illuminate/filesystem": "5.1 - 6.5"
   },
   "require-dev": {
-    "laravel/laravel": "5.1 - 5.8",
-    "phpunit/phpunit": "5.7 || 6.0 || 7.0 || 7.5",
+    "laravel/laravel": "5.1 - 6.4",
+    "phpunit/phpunit": "5.7 || 6.0 || 7.0 || 7.5 || 8.4",
     "mockery/mockery": "^1.1.0",
-    "friendsofphp/php-cs-fixer": "^2.12",
+    "friendsofphp/php-cs-fixer": "^2.16.0",
     "overtrue/phplint": "^1.1"
   },
   "autoload": {

phpunit.xml

@@ -7,7 +7,6 @@
          convertWarningsToExceptions="true"
          processIsolation="false"
          stopOnFailure="false"
-         syntaxCheck="false"
          bootstrap="vendor/autoload.php">
     <testsuites>
         <testsuite name="Application Test Suite">
@@ -18,9 +17,5 @@
         <whitelist>
             <directory suffix=".php">src/</directory>
         </whitelist>
-        <exclude>
-            <directory>vendor/</directory>
-            <directory>tests/</directory>
-        </exclude>
     </filter>
 </phpunit>

src/Handler/ContentRangeUploadHandler.php

@@ -4,10 +4,10 @@
 
 use Illuminate\Http\Request;
 use Illuminate\Http\UploadedFile;
-use Pion\Laravel\ChunkUpload\Exceptions\ChunkSaveException;
-use Pion\Laravel\ChunkUpload\Save\ChunkSave;
 use Pion\Laravel\ChunkUpload\Config\AbstractConfig;
+use Pion\Laravel\ChunkUpload\Exceptions\ChunkSaveException;
 use Pion\Laravel\ChunkUpload\Exceptions\ContentRangeValueToLargeException;
+use Pion\Laravel\ChunkUpload\Save\ChunkSave;
 use Pion\Laravel\ChunkUpload\Storage\ChunkStorage;
 
 /**

src/Handler/NgFileUploadHandler.php

@@ -2,8 +2,8 @@
 
 namespace Pion\Laravel\ChunkUpload\Handler;
 
-use Pion\Laravel\ChunkUpload\Exceptions\ChunkInvalidValueException;
 use Illuminate\Http\Request;
+use Pion\Laravel\ChunkUpload\Exceptions\ChunkInvalidValueException;
 
 /**
  * Class NgFileUploadHandler.

src/Providers/ChunkUploadServiceProvider.php

@@ -79,7 +79,7 @@ public function register()
             $request = $app->make('request');
 
             // Get the first file object - must be converted instances of UploadedFile
-            $file = array_first($request->allFiles());
+            $file = Arr::first($request->allFiles());
 
             // Build the file receiver
             return new FileReceiver($file, $request, HandlerFactory::classFromRequest($request));

src/Receiver/FileReceiver.php

@@ -3,13 +3,13 @@
 namespace Pion\Laravel\ChunkUpload\Receiver;
 
 use Illuminate\Http\Request;
+use Illuminate\Http\UploadedFile;
 use Pion\Laravel\ChunkUpload\Config\AbstractConfig;
 use Pion\Laravel\ChunkUpload\Exceptions\UploadFailedException;
 use Pion\Laravel\ChunkUpload\Handler\AbstractHandler;
 use Pion\Laravel\ChunkUpload\Save\AbstractSave;
 use Pion\Laravel\ChunkUpload\Save\ChunkSave;
 use Pion\Laravel\ChunkUpload\Save\SingleSave;
-use Illuminate\Http\UploadedFile;
 use Pion\Laravel\ChunkUpload\Storage\ChunkStorage;
 
 class FileReceiver

src/Save/AbstractSave.php

@@ -2,10 +2,10 @@
 
 namespace Pion\Laravel\ChunkUpload\Save;
 
+use Illuminate\Http\UploadedFile;
 use Pion\Laravel\ChunkUpload\Config\AbstractConfig;
 use Pion\Laravel\ChunkUpload\Handler\AbstractHandler;
 use Symfony\Component\HttpFoundation\File\File;
-use Illuminate\Http\UploadedFile;
 
 /**
  * Class AbstractSave.
@@ -95,7 +95,7 @@ public function getFile()
     /**
      * Returns always the uploaded chunk file.
      *
-     * @return null|UploadedFile
+     * @return UploadedFile|null
      */
     public function getUploadedFile()
     {

src/Save/ChunkSave.php

@@ -2,11 +2,11 @@
 
 namespace Pion\Laravel\ChunkUpload\Save;
 
+use Illuminate\Http\UploadedFile;
 use Pion\Laravel\ChunkUpload\Config\AbstractConfig;
 use Pion\Laravel\ChunkUpload\Exceptions\ChunkSaveException;
 use Pion\Laravel\ChunkUpload\FileMerger;
 use Pion\Laravel\ChunkUpload\Handler\AbstractHandler;
-use Illuminate\Http\UploadedFile;
 use Pion\Laravel\ChunkUpload\Storage\ChunkStorage;
 
 class ChunkSave extends AbstractSave
@@ -122,7 +122,7 @@ public function getChunkDirectory($absolutePath = false)
      * Returns the uploaded file if the chunk if is not completed, otherwise passes the
      * final chunk file.
      *
-     * @return null|UploadedFile
+     * @return UploadedFile|null
      */
     public function getFile()
     {