Fix styling

Author: drbyte

config/permission.php

@@ -75,8 +75,8 @@
         /*
          * Change this if you want to name the related pivots other than defaults
          */
-        'role_pivot_key' => null, //default 'role_id',
-        'permission_pivot_key' => null, //default 'permission_id',
+        'role_pivot_key' => null, // default 'role_id',
+        'permission_pivot_key' => null, // default 'permission_id',
 
         /*
          * Change this if you want to name the related model primary key other than

tests/CommandTest.php

@@ -149,7 +149,7 @@ public function it_can_setup_teams_upgrade()
 
         $AddTeamsFields = require $matchingFiles[count($matchingFiles) - 1];
         $AddTeamsFields->up();
-        $AddTeamsFields->up(); //test upgrade teams migration fresh
+        $AddTeamsFields->up(); // test upgrade teams migration fresh
 
         Role::create(['name' => 'new-role', 'team_test_id' => 1]);
         $role = Role::where('name', 'new-role')->first();

tests/HasPermissionsTest.php

@@ -654,7 +654,7 @@ public function it_does_not_run_unnecessary_sqls_when_assigning_new_permissions(
         $this->testUser->syncPermissions($this->testUserPermission, $permission2);
         DB::disableQueryLog();
 
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sqls
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sqls
     }
 
     /** @test */
@@ -676,7 +676,7 @@ public function calling_givePermissionTo_before_saving_object_doesnt_interfere_w
 
         $this->assertTrue($user2->fresh()->hasPermissionTo('edit-articles'));
         $this->assertFalse($user2->fresh()->hasPermissionTo('edit-news'));
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sync
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sync
     }
 
     /** @test */
@@ -698,7 +698,7 @@ public function calling_syncPermissions_before_saving_object_doesnt_interfere_wi
 
         $this->assertTrue($user2->fresh()->hasPermissionTo('edit-articles'));
         $this->assertFalse($user2->fresh()->hasPermissionTo('edit-news'));
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sync
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sync
     }
 
     /** @test */

tests/HasPermissionsWithCustomModelsTest.php

@@ -123,7 +123,7 @@ public function it_does_detach_roles_and_users_when_force_deleting()
         $this->testUserPermission->forceDelete();
         DB::disableQueryLog();
 
-        $this->assertSame(3 + $this->resetDatabaseQuery, count(DB::getQueryLog())); //avoid detach permissions on permissions
+        $this->assertSame(3 + $this->resetDatabaseQuery, count(DB::getQueryLog())); // avoid detach permissions on permissions
 
         $permission = Permission::withTrashed()->find($permission_id);
 

tests/HasRolesTest.php

@@ -359,7 +359,7 @@ public function it_does_not_run_unnecessary_sqls_when_assigning_new_roles()
         $this->testUser->syncRoles($this->testUserRole, $role2);
         DB::disableQueryLog();
 
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sqls
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sqls
     }
 
     /** @test */
@@ -381,7 +381,7 @@ public function calling_syncRoles_before_saving_object_doesnt_interfere_with_oth
 
         $this->assertTrue($user2->fresh()->hasRole('testRole2'));
         $this->assertFalse($user2->fresh()->hasRole('testRole'));
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sync
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sync
     }
 
     /** @test */
@@ -403,7 +403,7 @@ public function calling_assignRole_before_saving_object_doesnt_interfere_with_ot
 
         $this->assertTrue($admin_user->fresh()->hasRole('testRole2'));
         $this->assertFalse($admin_user->fresh()->hasRole('testRole'));
-        $this->assertSame(2, count(DB::getQueryLog())); //avoid unnecessary sync
+        $this->assertSame(2, count(DB::getQueryLog())); // avoid unnecessary sync
     }
 
     /** @test */

tests/TeamHasRolesTest.php

@@ -39,9 +39,9 @@ public function it_deletes_pivot_table_entries_when_deleting_models()
     /** @test */
     public function it_can_assign_same_and_different_roles_on_same_user_different_teams()
     {
-        app(Role::class)->create(['name' => 'testRole3']); //team_test_id = 1 by main class
+        app(Role::class)->create(['name' => 'testRole3']); // team_test_id = 1 by main class
         app(Role::class)->create(['name' => 'testRole3', 'team_test_id' => 2]);
-        app(Role::class)->create(['name' => 'testRole4', 'team_test_id' => null]); //global role
+        app(Role::class)->create(['name' => 'testRole4', 'team_test_id' => null]); // global role
 
         $testRole3Team1 = app(Role::class)->where(['name' => 'testRole3', 'team_test_id' => 1])->first();
         $testRole3Team2 = app(Role::class)->where(['name' => 'testRole3', 'team_test_id' => 2])->first();
@@ -66,7 +66,7 @@ public function it_can_assign_same_and_different_roles_on_same_user_different_te
 
         $this->testUser->assignRole('testRole3', 'testRole4');
         $this->assertTrue($this->testUser->hasExactRoles(['testRole', 'testRole2', 'testRole3', 'testRole4']));
-        $this->assertTrue($this->testUser->hasRole($testRole3Team1)); //testRole3 team=1
+        $this->assertTrue($this->testUser->hasRole($testRole3Team1)); // testRole3 team=1
         $this->assertTrue($this->testUser->hasRole($testRole4NoTeam)); // global role team=null
 
         setPermissionsTeamId(2);
@@ -77,7 +77,7 @@ public function it_can_assign_same_and_different_roles_on_same_user_different_te
             $this->testUser->getRoleNames()->sort()->values()
         );
         $this->assertTrue($this->testUser->hasExactRoles(['testRole', 'testRole3']));
-        $this->assertTrue($this->testUser->hasRole($testRole3Team2)); //testRole3 team=2
+        $this->assertTrue($this->testUser->hasRole($testRole3Team2)); // testRole3 team=2
         $this->testUser->assignRole('testRole4');
         $this->assertTrue($this->testUser->hasExactRoles(['testRole', 'testRole3', 'testRole4']));
         $this->assertTrue($this->testUser->hasRole($testRole4NoTeam)); // global role team=null

tests/TestCase.php

@@ -112,7 +112,7 @@ protected function getEnvironmentSetUp($app)
     {
         $app['config']->set('permission.register_permission_check_method', true);
         $app['config']->set('permission.teams', $this->hasTeams);
-        $app['config']->set('permission.testing', true); //fix sqlite
+        $app['config']->set('permission.testing', true); // fix sqlite
         $app['config']->set('permission.column_names.model_morph_key', 'model_test_id');
         $app['config']->set('permission.column_names.team_foreign_key', 'team_test_id');
         $app['config']->set('database.default', 'sqlite');
@@ -143,8 +143,8 @@ protected function getEnvironmentSetUp($app)
 
         // FOR MANUAL TESTING OF ALTERNATE CACHE STORES:
         // $app['config']->set('cache.default', 'array');
-        //Laravel supports: array, database, file
-        //requires extensions: apc, memcached, redis, dynamodb, octane
+        // Laravel supports: array, database, file
+        // requires extensions: apc, memcached, redis, dynamodb, octane
     }
 
     /**
@@ -282,7 +282,7 @@ public function setUpRoutes(): void
         });
     }
 
-    ////// TEST HELPERS
+    // //// TEST HELPERS
     public function runMiddleware($middleware, $permission, $guard = null, bool $client = false)
     {
         $request = new Request;