Laravel 8 update (#432)

* Update dependencies to laravel 8

* Update minim PHP version

* Update factories to laravel 8

* Fix dusk tests

* Remove auth routes comment

* Style

* Readme update

* Readme update

* Update factory on feature test

* Add import

* Update user import

* Update seeders path

* update variable name

Co-authored-by: Manel Gavaldà <[email protected]>

Author: manelgavalda

README.md

@@ -73,9 +73,10 @@ Please be sure to check you environment.
 ## Optional requirements
 * [Laravel menu](https://github.com/spatie/laravel-menu): only used with command adminlte:menu that replaces default adminlte menu with a menu with spatie/laravel-menu support.
 
-## Laravel 6 and older
+## Laravel 7 and older
 
-This package works smoothly with Laravel 6.x with 6.x version
+- This package works smoothly with Laravel 7.x with 7.x version
+- This package works smoothly with Laravel 6.x|5.x with 6.x version
 
 ## Laravel Routes
 

composer.json

@@ -13,16 +13,16 @@
     }
   ],
   "require": {
-    "php": "^7.2.5",
+    "php": "^7.3",
     "creativeorange/gravatar": "~1.0",
-    "laravel/framework": "^7.0",
+    "laravel/framework": "^8.0",
     "acacha/filesystem": "^0.1.2",
     "acacha/helpers": "^0.1.4",
     "league/flysystem": "^1.0",
-    "laravel/ui": "^2.1"
+    "laravel/ui": "^3.0"
   },
   "require-dev": {
-    "phpunit/phpunit": "^8.5",
+    "phpunit/phpunit": "^9.0",
     "squizlabs/php_codesniffer": "^3.2.3",
     "laravel/dusk": "^6.0"
   },

src/Console/AdminLTEAdmin.php

@@ -3,6 +3,7 @@
 namespace Acacha\AdminLTETemplateLaravel\Console;
 
 use Illuminate\Console\Command;
+use App\Models\User;
 
 /**
  * Class AdminLTEAdmin.
@@ -41,7 +42,7 @@ public function handle()
     protected function createAdminUser()
     {
         try {
-            factory(get_class(app('App\User')))->create([
+            User::factory()->create([
                     "name" => env('ADMIN_USER', $this->username()),
                     "email" => env('ADMIN_EMAIL', $this->email()),
                     "password" => bcrypt(env('ADMIN_PWD', '123456'))]);

src/Console/MakeAdminUserSeeder.php

@@ -64,7 +64,7 @@ public function handle()
     {
         try {
             $this->filesystem->overwrite(
-                $path = database_path('seeds/' . config('AdminUserSeeder', 'AdminUserSeeder.php')),
+                $path = database_path('seeders/' . config('AdminUserSeeder', 'AdminUserSeeder.php')),
                 $this->compiler->compile(
                     $this->filesystem->get($this->getStubPath()),
                     [

src/Console/stubs/AdminUserSeeder.php

@@ -1,6 +1,7 @@
 <?php
 
 use Illuminate\Database\Seeder;
+use App\Models\User;
 
 /**
  * Class AdminUserSeeder
@@ -15,7 +16,7 @@ class AdminUserSeeder extends Seeder
     public function run()
     {
         try {
-            factory(App\User::class)->create([
+            User::factory()->create([
                     "name" => env('ADMIN_USER', "$USER_NAME$"),
                     "email" => env('ADMIN_EMAIL', "$USER_EMAIL"),
                     "password" => bcrypt(env('ADMIN_PWD', '123456'))]);

src/Http/routes.php

@@ -6,7 +6,5 @@
  * RouteServiceProvider
  */
 Route::group(['middleware' => 'web'], function () {
-    // Route::auth();
-
     Route::get('/home', 'HomeController@index');
 });

src/Providers/AdminLTETemplateServiceProvider.php

@@ -15,7 +15,7 @@
  */
 class AdminLTETemplateServiceProvider extends ServiceProvider
 {
-    
+
     /**
      * Register the application services.
      */

src/stubs/RegisterController.php

@@ -2,7 +2,7 @@
 
 namespace App\Http\Controllers\Auth;
 
-use App\User;
+use App\Models\User;
 use Validator;
 use App\Http\Controllers\Controller;
 use Illuminate\Foundation\Auth\RegistersUsers;

tests/Browser/AcachaAdmintLTELaravelTest.php

@@ -7,6 +7,7 @@
 use Illuminate\Support\Facades\Auth;
 use Illuminate\Support\Facades\Hash;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
+use App\Models\User;
 
 /**
  * Class AcachaAdmintLTELaravelTest.
@@ -53,7 +54,8 @@ private function logout()
     public function testLandingPageWithUserLogged()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create();
+            $user = User::factory()->create();
+
             $browser->loginAs($user)
                 ->visit('/')
                 ->assertSee('Acacha')
@@ -87,7 +89,8 @@ public function testLoginPage()
     public function testLogin()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create(['password' => Hash::make('passw0RD')]);
+            $user = User::factory()->create(['password' => Hash::make('passw0RD')]);
+
             $browser->visit('/login')
                 ->type('email', $user->email)
                 ->type('password', 'passw0RD')
@@ -168,12 +171,13 @@ public function testLoginCredentialsNotMatchDissappearsOnKeyDown()
     {
         $this->browse(function (Browser $browser) {
             $browser->visit('/login')
+                ->waitFor("[name='email']")
                 ->type('email', '[email protected]')
                 ->type('password', '12345678')
                 ->press('Sign In')
                 ->pause(1000)
                 ->type('password', '1')
-                ->pause(1000)
+                ->pause(2000)
                 ->assertDontSee('These credentials do not match our records');
         });
     }
@@ -186,18 +190,21 @@ public function testLoginCredentialsNotMatchDissappearsOnKeyDown()
     public function testLoginWithRememberMe()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create(['password' => Hash::make('passw0RD')]);
+            $user = User::factory()->create(['password' => Hash::make('passw0RD')]);
+
             $browser->visit('/login')
                 ->type('email', $user->email)
                 ->type('password', 'passw0RD')
                 ->script("$('input[name=remember]').iCheck('check');");
+
             $browser->press('Sign In')
                 ->waitFor('#result')
                 ->pause(5000)
                 ->assertPathIs('/home')
                 ->assertHasCookie(Auth::getRecallerName())
                 ->assertSee($user->name);
         });
+
         $this->logout();
     }
 
@@ -235,8 +242,10 @@ public function testPasswordResetPage()
     public function testHomePageForUnauthenticatedUsers()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create();
+            $user = User::factory()->create();
+
             view()->share('user', $user);
+
             $browser->visit('/home')
                 ->pause(2000)
                 ->assertPathIs('/login');
@@ -251,8 +260,10 @@ public function testHomePageForUnauthenticatedUsers()
     public function testHomePageForAuthenticatedUsers()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create();
+            $user = User::factory()->create();
+
             view()->share('user', $user);
+
             $browser->loginAs($user)
                 ->visit('/home')
                 ->assertSee($user->name);
@@ -269,8 +280,10 @@ public function testHomePageForAuthenticatedUsers()
     public function testLogout()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create();
+            $user = User::factory()->create();
+
             view()->share('user', $user);
+
             $browser->loginAs($user)
                 ->visit('/home')
                 ->click('#user_menu')
@@ -376,7 +389,8 @@ public function testNewUserRegistrationRequiredFieldsDissappearsOnKeyDown()
     public function testSendPasswordReset()
     {
         $this->browse(function (Browser $browser) {
-            $user = factory(\App\User::class)->create();
+            $user = User::factory()->create();
+
             $browser->visit('password/reset')
                 ->type('email', $user->email)
                 ->press('Send Password Reset Link')

tests/Feature/AcachaAdminLTELaravelTest.php

@@ -7,6 +7,7 @@
 use Illuminate\Support\Facades\Hash;
 use Tests\TestCase;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
+use App\Models\User;
 
 /**
  * Class AcachaAdminLTELaravelTest.
@@ -206,7 +207,7 @@ public function testNewUserRegistrationRequiredFields()
      */
     public function testLogin()
     {
-        $user = factory(\App\User::class)->create(['password' => Hash::make('passw0RD')]);
+        $user = User::factory()->create(['password' => Hash::make('passw0RD')]);
 
         $response = $this->json('POST', '/login', [
             'email' => $user->email,
@@ -293,12 +294,12 @@ public function testMakeViewCommand()
      */
     public function testAdminlteAdminCommand()
     {
-        $seed = database_path('seeds/AdminUserSeeder.php');
+        $seeder = database_path('seeders/AdminUserSeeder.php');
         try {
-            unlink($seed);
+            unlink($seeder);
         } catch (\Exception $e) {
         }
         Artisan::call('adminlte:admin');
-        $this->assertFileExists($seed);
+        $this->assertFileExists($seeder);
     }
 }