Skip to content

Commit 13fc765

Browse files
committed
rector - apply suggestions
1 parent 63710a1 commit 13fc765

2 files changed

Lines changed: 10 additions & 0 deletions

File tree

tests/Commands/UserTest.php

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -592,6 +592,7 @@ public function testAddgroup(): void
592592

593593
$users = model(UserModel::class);
594594
$user = $users->findByCredentials(['email' => 'user10@example.com']);
595+
$this->assertInstanceOf(UserEntity::class, $user);
595596
$this->assertTrue($user->inGroup('admin'));
596597
}
597598

@@ -632,6 +633,7 @@ public function testAddgroupCancel(): void
632633

633634
$users = model(UserModel::class);
634635
$user = $users->findByCredentials(['email' => 'user10@example.com']);
636+
$this->assertInstanceOf(UserEntity::class, $user);
635637
$this->assertFalse($user->inGroup('admin'));
636638
}
637639

@@ -644,6 +646,7 @@ public function testRemovegroup(): void
644646
]);
645647
$users = model(UserModel::class);
646648
$user = $users->findByCredentials(['email' => 'user11@example.com']);
649+
$this->assertInstanceOf(UserEntity::class, $user);
647650
$user->addGroup('admin');
648651
$this->assertTrue($user->inGroup('admin'));
649652

@@ -658,6 +661,7 @@ public function testRemovegroup(): void
658661

659662
$users = model(UserModel::class);
660663
$user = $users->findByCredentials(['email' => 'user11@example.com']);
664+
$this->assertInstanceOf(UserEntity::class, $user);
661665
$this->assertFalse($user->inGroup('admin'));
662666
}
663667

@@ -670,6 +674,7 @@ public function testRemovegroupWithInvalidGroup(): void
670674
]);
671675
$users = model(UserModel::class);
672676
$user = $users->findByCredentials(['email' => 'user11@example.com']);
677+
$this->assertInstanceOf(UserEntity::class, $user);
673678
$user->addGroup('admin');
674679
$this->assertTrue($user->inGroup('admin'));
675680

@@ -684,6 +689,7 @@ public function testRemovegroupWithInvalidGroup(): void
684689

685690
$users = model(UserModel::class);
686691
$user = $users->findByCredentials(['email' => 'user11@example.com']);
692+
$this->assertInstanceOf(UserEntity::class, $user);
687693
$this->assertTrue($user->inGroup('admin'));
688694
}
689695

@@ -696,6 +702,7 @@ public function testRemovegroupCancel(): void
696702
]);
697703
$users = model(UserModel::class);
698704
$user = $users->findByCredentials(['email' => 'user11@example.com']);
705+
$this->assertInstanceOf(UserEntity::class, $user);
699706
$user->addGroup('admin');
700707
$this->assertTrue($user->inGroup('admin'));
701708

@@ -710,6 +717,7 @@ public function testRemovegroupCancel(): void
710717

711718
$users = model(UserModel::class);
712719
$user = $users->findByCredentials(['email' => 'user11@example.com']);
720+
$this->assertInstanceOf(UserEntity::class, $user);
713721
$this->assertTrue($user->inGroup('admin'));
714722
}
715723
}

tests/Unit/UserModelTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -207,6 +207,8 @@ public function testUpdateUserArrayWithUserDataToUpdate(): void
207207

208208
$user = $users->findByCredentials(['email' => 'foo@bar.com']);
209209

210+
$this->assertInstanceOf(User::class, $user);
211+
210212
$user->username = 'bar';
211213
$user->email = 'bar@bar.com';
212214
$user->active = true;

0 commit comments

Comments
 (0)