Get our tests passing again.

src/Authentication/AuthenticationBase.php

@@ -136,7 +136,7 @@ public function isLoggedIn(): bool
             // Store our current user object
             $this->user = $this->userModel->find($userID);
 
-            return true;
+            return $this->user instanceof User;
         }
 
         return false;

src/Authentication/LocalAuthenticator.php

@@ -29,14 +29,14 @@ public function attempt(array $credentials, bool $remember = null): bool
         }
 
         if ($this->user->isBanned())
-            {
-                // Always record a login attempt, whether success or not.
-                $ipAddress = Services::request()->getIPAddress();
-                $this->recordLoginAttempt($credentials['email'], $ipAddress, $this->user->id ?? null, false);
-    
-                $this->user = null;
-                return false;
-            }
+        {
+            // Always record a login attempt, whether success or not.
+            $ipAddress = Services::request()->getIPAddress();
+            $this->recordLoginAttempt($credentials['email'], $ipAddress, $this->user->id ?? null, false);
+
+            $this->user = null;
+            return false;
+        }
 
         return $this->login($this->user, $remember);
     }

src/Config/Services.php

@@ -17,7 +17,7 @@ public static function authentication(string $lib = 'local', Model $userModel=nu
         {
             return self::getSharedInstance('authentication', $lib, $userModel, $loginModel);
         }
-		
+
 		// config() checks first in app/Config
 		$config = config('Auth');
 

src/Database/Migrations/20171120223112_create_auth_tables.php → src/Database/Migrations/2017-11-20-223112_create_auth_tables.php

@@ -2,7 +2,7 @@
 
 use CodeIgniter\Database\Migration;
 
-class Migration_create_auth_tables extends Migration
+class CreateAuthTables extends Migration
 {
     public function up()
     {

src/Entities/User.php

@@ -131,7 +131,7 @@ public function unBan()
 	 */
 	public function isBanned(): bool
 	{
-		return $this->attributes['status'] === 'banned';
+		return isset($this->attributes['status']) && $this->attributes['status'] === 'banned';
 	}
 
     /**

tests/authentication/LocalAuthTest.php

@@ -88,6 +88,7 @@ public function testValidateSuccess()
 
     public function testCheckNotRemembered()
     {
+        $_SESSION = [];
         $user = $this->createUser();
 
         $this->assertFalse($this->auth->check());