Apply fixes from StyleCI

tests/Controllers/ExampleTest.php

@@ -1,20 +1,20 @@
-<?php
-
-namespace Tests\Controllers;
-
-use App\Controllers\Home;
-use CodeIgniter\Test\ControllerTester;
-use Tests\Support\AuthTestCase;
-
-class ExampleTest extends AuthTestCase
-{
-    use ControllerTester;
-    
-    public function testIndexHome()
-    {
-        $result = $this->controller(Home::class)
-            ->execute('index');
-                        
-        $this->assertTrue($result->isOK());
-    }
-}
+<?php
+
+namespace Tests\Controllers;
+
+use App\Controllers\Home;
+use CodeIgniter\Test\ControllerTester;
+use Tests\Support\AuthTestCase;
+
+class ExampleTest extends AuthTestCase
+{
+    use ControllerTester;
+
+    public function testIndexHome()
+    {
+        $result = $this->controller(Home::class)
+            ->execute('index');
+
+        $this->assertTrue($result->isOK());
+    }
+}

tests/Support/AuthTestCase.php

@@ -1,112 +1,111 @@
-<?php 
+<?php
 
 namespace Tests\Support;
 
-use CodeIgniter\Test\ControllerTester;
 use CodeIgniter\Session\Handlers\ArrayHandler;
-use CodeIgniter\Test\Mock\MockSession;
 use CodeIgniter\Test\CIDatabaseTestCase;
+use CodeIgniter\Test\ControllerTester;
+use CodeIgniter\Test\Mock\MockSession;
 use Myth\Auth\Entities\User;
 use Myth\Auth\Models\UserModel;
 
 class AuthTestCase extends CIDatabaseTestCase
 {
-	use ControllerTester;
-
-	/**
-	 * Should the database be refreshed before each test?
-	 *
-	 * @var boolean
-	 */
-	protected $refresh = true;
-
-	/**
-	 * The name of a seed file used for all tests within this test case.
-	 *
-	 * @var string
-	 */
-	protected $seed = 'agungsugiarto\boilerplate\Database\Seeds\BoilerplateSeeder';
-
-	/**
-	 * The namespace to help us find the migration classes.
-	 *
-	 * @var string
-	 */
-	protected $namespace = ['Myth\Auth', 'agungsugiarto\boilerplate'];
-
-	/**
-	 * @var \Myth\Auth\Models\UserModel
-	 */
-	protected $users;
-
-	protected $faker;
-
-	/**
-	 * @var SessionHandler
-	 */
-	protected $session;
-
-	public function setUp(): void
-	{
-		parent::setUp();
-
-		$this->users = new UserModel();
-		$this->mockSession();
-
-		$this->faker = \Faker\Factory::create();
-	}
-
-	/**
-	 * Pre-loads the mock session driver into $this->session.
-	 *
-	 */
-	protected function mockSession()
-	{
-		require_once SYSTEMPATH . 'Test/Mock/MockSession.php';
-		$config = config('App');
-		$this->session = new MockSession(new ArrayHandler($config, '0.0.0.0'), $config);
-		\Config\Services::injectMock('session', $this->session);
-		$_SESSION = [];
-	}
-
-	/**
-	 * Creates a user on-the-fly.
-	 *
-	 * @param string $reason
-	 *
-	 * @return $this
-	 */
-	protected function createUser(array $info = [])
-	{
-		$defaults = [
-			'email' => '[email protected]',
-			'username' => 'Fred',
-			'password' => 'secret'
-		];
-		$info = array_merge($info, $defaults);
-		$user = new User($info);
-
-		$userId = $this->users->insert($user);
-		$user = $this->users->find($userId);
-
-		// Delete any cached permissions
+    use ControllerTester;
+
+    /**
+     * Should the database be refreshed before each test?
+     *
+     * @var bool
+     */
+    protected $refresh = true;
+
+    /**
+     * The name of a seed file used for all tests within this test case.
+     *
+     * @var string
+     */
+    protected $seed = 'agungsugiarto\boilerplate\Database\Seeds\BoilerplateSeeder';
+
+    /**
+     * The namespace to help us find the migration classes.
+     *
+     * @var string
+     */
+    protected $namespace = ['Myth\Auth', 'agungsugiarto\boilerplate'];
+
+    /**
+     * @var \Myth\Auth\Models\UserModel
+     */
+    protected $users;
+
+    protected $faker;
+
+    /**
+     * @var SessionHandler
+     */
+    protected $session;
+
+    public function setUp(): void
+    {
+        parent::setUp();
+
+        $this->users = new UserModel();
+        $this->mockSession();
+
+        $this->faker = \Faker\Factory::create();
+    }
+
+    /**
+     * Pre-loads the mock session driver into $this->session.
+     */
+    protected function mockSession()
+    {
+        require_once SYSTEMPATH.'Test/Mock/MockSession.php';
+        $config = config('App');
+        $this->session = new MockSession(new ArrayHandler($config, '0.0.0.0'), $config);
+        \Config\Services::injectMock('session', $this->session);
+        $_SESSION = [];
+    }
+
+    /**
+     * Creates a user on-the-fly.
+     *
+     * @param string $reason
+     *
+     * @return $this
+     */
+    protected function createUser(array $info = [])
+    {
+        $defaults = [
+            'email'    => '[email protected]',
+            'username' => 'Fred',
+            'password' => 'secret',
+        ];
+        $info = array_merge($info, $defaults);
+        $user = new User($info);
+
+        $userId = $this->users->insert($user);
+        $user = $this->users->find($userId);
+
+        // Delete any cached permissions
         cache()->delete("{$userId}_permissions");
 
-		return $user;
-	}
+        return $user;
+    }
 
     /**
-     * Creates a group on the fly
+     * Creates a group on the fly.
      *
      * @param array $info
      *
      * @return mixed
      */
-	protected function createGroup(array $info = [])
+    protected function createGroup(array $info = [])
     {
         $defaults = [
-            'name' => $this->faker->word,
-            'description' => $this->faker->sentence
+            'name'        => $this->faker->word,
+            'description' => $this->faker->sentence,
         ];
         $info = array_merge($info, $defaults);
 
@@ -125,8 +124,8 @@ protected function createGroup(array $info = [])
     protected function createPermission(array $info = [])
     {
         $defaults = [
-            'name' => $this->faker->word,
-            'description' => $this->faker->sentence
+            'name'        => $this->faker->word,
+            'description' => $this->faker->sentence,
         ];
         $info = array_merge($info, $defaults);
 

tests/Support/SessionTestCase.php

@@ -1,4 +1,4 @@
-<?php 
+<?php
 
 namespace Tests\Support;
 
@@ -18,15 +18,15 @@ public function setUp(): void
 
         $this->mockSession();
     }
-    
+
     /**
      * Pre-loads the mock session driver into $this->session.
      *
      * @var string
      */
     protected function mockSession()
     {
-        require_once SYSTEMPATH . 'Test/Mock/MockSession.php';
+        require_once SYSTEMPATH.'Test/Mock/MockSession.php';
         $config = config('App');
         $this->session = new MockSession(new ArrayHandler($config, '0.0.0.0'), $config);
         \Config\Services::injectMock('session', $this->session);

tests/UserTest.php

@@ -11,13 +11,13 @@ public function testGetPermissionsThroughUser()
         $permission2 = $this->createPermission(['name' => 'second']);
 
         $this->hasInDatabase('auth_users_permissions', [
-            'user_id' => $user->id,
-            'permission_id' => $permission1->id
+            'user_id'       => $user->id,
+            'permission_id' => $permission1->id,
         ]);
 
         $this->hasInDatabase('auth_users_permissions', [
-            'user_id' => $user->id,
-            'permission_id' => $permission2->id
+            'user_id'       => $user->id,
+            'permission_id' => $permission2->id,
         ]);
 
         $expected = [
@@ -35,13 +35,13 @@ public function testGetPermissionsThroughGroup()
         $permission = $this->createPermission(['name' => 'first']);
 
         $this->hasInDatabase('auth_groups_permissions', [
-            'group_id' => $group->id,
-            'permission_id' => $permission->id
+            'group_id'      => $group->id,
+            'permission_id' => $permission->id,
         ]);
 
         $this->hasInDatabase('auth_groups_users', [
-            'user_id' => $user->id,
-            'group_id' => $group->id
+            'user_id'  => $user->id,
+            'group_id' => $group->id,
         ]);
 
         $expected = [
@@ -56,8 +56,8 @@ public function testCan()
         $user = $this->createUser();
         $permission = $this->createPermission();
         $this->hasInDatabase('auth_users_permissions', [
-            'user_id' => $user->id,
-            'permission_id' => $permission->id
+            'user_id'       => $user->id,
+            'permission_id' => $permission->id,
         ]);
 
         $this->assertTrue($user->can($permission->name));