View GitHub Repository laravel-roles

Schedule Inspection
Subscribe
Test Setup Failed
Pull Request — master (#206)
by
unknown
06:13
created
Status
Category
Toggle Patches
tests/Feature/NPlusOneQueriesTest.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -20,8 +20,8 @@  discard block
 
 block discarded – undo
20 20 
      * please indicate their number here
21 21 
      */
22 22 
     protected $usersCountCorrection = 0;
23 
-    protected $rolesCount = 3;//correct according to your data
24 
-    protected $permissionsCount = 4;//correct according to your data
23 
+    protected $rolesCount = 3; //correct according to your data
24 
+    protected $permissionsCount = 4; //correct according to your data
25 25
26 26 
     protected $queries = 0;
27 27 
 
@@ -32,7 +32,7 @@  discard block
 
 block discarded – undo
32 32 
         $this->assertEquals($this->rolesCount, config('roles.models.role')::count());
33 33 
         $this->assertEquals($this->permissionsCount, config('roles.models.permission')::count());
34 34
35 
-        DB::listen(function (QueryExecuted $query) {
35 
+        DB::listen(function(QueryExecuted $query) {
36 36 
             $this->queries++;
37 37 
         });
38 38 
     }
 
@@ -43,7 +43,7 @@  discard block
 
 block discarded – undo
43 43 
         $roleIds = config('roles.models.role')::pluck('id');
44 44
45 45 
         User::factory($this->usersCount)->create()
46 
-            ->each(function (User $user) use ($roleIds) {
46 
+            ->each(function(User $user) use ($roleIds) {
47 47 
                 $user->roles()->attach($roleIds);
48 48 
             });
49 49 
         $this->assertEquals($this->usersCount,
 
@@ -55,7 +55,7 @@  discard block
 
 block discarded – undo
55 55 
         $users = User::get();
56 56 
         $this->assertQueries(1);
57 57
58 
-        $users->each(function (User $user) {
58 
+        $users->each(function(User $user) {
59 59 
             $user->getRoles();
60 60 
         });
61 61 
         $this->queries = $this->queries - $this->usersCountCorrection;
 
@@ -65,7 +65,7 @@  discard block
 
 block discarded – undo
65 65 
         $users = User::with('roles')->get();
66 66 
         $this->assertQueries(2);
67 67
68 
-        $users->each(function (User $user) {
68 
+        $users->each(function(User $user) {
69 69 
             $user->getRoles();
70 70 
         });
71 71 
         $this->assertQueries(0);
 
@@ -143,7 +143,7 @@  discard block
 
 block discarded – undo
143 143 
         $roleIds = config('roles.models.role')::pluck('id');
144 144
145 145 
         User::factory($this->usersCount)->create()
146 
-            ->each(function (User $user) use ($roleIds) {
146 
+            ->each(function(User $user) use ($roleIds) {
147 147 
                 $user->roles()->attach($roleIds);
148 148 
             });
149 149 
         $this->assertEquals($this->usersCount, User::count() - $this->usersCountCorrection);
 
@@ -154,7 +154,7 @@  discard block
 
 block discarded – undo
154 154 
         $users = User::get();
155 155 
         $this->assertQueries(1);
156 156
157 
-        $users->each(function (User $user) {
157 
+        $users->each(function(User $user) {
158 158 
             $user->getPermissions();
159 159 
         });
160 160 
         // rolePermissions(+getRoles) + userPermissions
 
@@ -165,7 +165,7 @@  discard block
 
 block discarded – undo
165 165 
         $users = User::with('roles', 'userPermissions')->get();
166 166 
         $this->assertQueries(3);
167 167
168 
-        $users->each(function (User $user) {
168 
+        $users->each(function(User $user) {
169 169 
             $user->getPermissions();
170 170 
         });
171 171 
         // TODO: optimize via relations: userPermissions and rolePermissions
Please login to merge, or discard this patch.