Skip to content

Commit 7ca6f08

Browse files
authored
Update test stubs (#939)
1 parent e9892fc commit 7ca6f08

8 files changed

+10
-42
lines changed

Diff for: stubs/tests/EmailVerificationTest.php

+3-9
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,7 @@ public function test_email_verification_screen_can_be_rendered()
2121
return $this->markTestSkipped('Email verification not enabled.');
2222
}
2323

24-
$user = User::factory()->withPersonalTeam()->create([
25-
'email_verified_at' => null,
26-
]);
24+
$user = User::factory()->withPersonalTeam()->unverified()->create();
2725

2826
$response = $this->actingAs($user)->get('/email/verify');
2927

@@ -38,9 +36,7 @@ public function test_email_can_be_verified()
3836

3937
Event::fake();
4038

41-
$user = User::factory()->create([
42-
'email_verified_at' => null,
43-
]);
39+
$user = User::factory()->unverified()->create();
4440

4541
$verificationUrl = URL::temporarySignedRoute(
4642
'verification.verify',
@@ -62,9 +58,7 @@ public function test_email_can_not_verified_with_invalid_hash()
6258
return $this->markTestSkipped('Email verification not enabled.');
6359
}
6460

65-
$user = User::factory()->create([
66-
'email_verified_at' => null,
67-
]);
61+
$user = User::factory()->unverified()->create();
6862

6963
$verificationUrl = URL::temporarySignedRoute(
7064
'verification.verify',

Diff for: stubs/tests/PasswordConfirmationTest.php

+1-3
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,7 @@ class PasswordConfirmationTest extends TestCase
1313

1414
public function test_confirm_password_screen_can_be_rendered()
1515
{
16-
$user = Features::hasTeamFeatures()
17-
? User::factory()->withPersonalTeam()->create()
18-
: User::factory()->create();
16+
$user = User::factory()->withPersonalTeam()->create();
1917

2018
$response = $this->actingAs($user)->get('/user/confirm-password');
2119

Diff for: stubs/tests/inertia/ApiTokenPermissionsTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,7 @@ public function test_api_token_permissions_can_be_updated()
1818
return $this->markTestSkipped('API support is not enabled.');
1919
}
2020

21-
if (Features::hasTeamFeatures()) {
22-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
23-
} else {
24-
$this->actingAs($user = User::factory()->create());
25-
}
21+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2622

2723
$token = $user->tokens()->create([
2824
'name' => 'Test Token',

Diff for: stubs/tests/inertia/CreateApiTokenTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,7 @@ public function test_api_tokens_can_be_created()
1717
return $this->markTestSkipped('API support is not enabled.');
1818
}
1919

20-
if (Features::hasTeamFeatures()) {
21-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
22-
} else {
23-
$this->actingAs($user = User::factory()->create());
24-
}
20+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2521

2622
$response = $this->post('/user/api-tokens', [
2723
'name' => 'Test Token',

Diff for: stubs/tests/inertia/DeleteApiTokenTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,7 @@ public function test_api_tokens_can_be_deleted()
1818
return $this->markTestSkipped('API support is not enabled.');
1919
}
2020

21-
if (Features::hasTeamFeatures()) {
22-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
23-
} else {
24-
$this->actingAs($user = User::factory()->create());
25-
}
21+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2622

2723
$token = $user->tokens()->create([
2824
'name' => 'Test Token',

Diff for: stubs/tests/livewire/ApiTokenPermissionsTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,7 @@ public function test_api_token_permissions_can_be_updated()
2020
return $this->markTestSkipped('API support is not enabled.');
2121
}
2222

23-
if (Features::hasTeamFeatures()) {
24-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
25-
} else {
26-
$this->actingAs($user = User::factory()->create());
27-
}
23+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2824

2925
$token = $user->tokens()->create([
3026
'name' => 'Test Token',

Diff for: stubs/tests/livewire/CreateApiTokenTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -19,11 +19,7 @@ public function test_api_tokens_can_be_created()
1919
return $this->markTestSkipped('API support is not enabled.');
2020
}
2121

22-
if (Features::hasTeamFeatures()) {
23-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
24-
} else {
25-
$this->actingAs($user = User::factory()->create());
26-
}
22+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2723

2824
Livewire::test(ApiTokenManager::class)
2925
->set(['createApiTokenForm' => [

Diff for: stubs/tests/livewire/DeleteApiTokenTest.php

+1-5
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,7 @@ public function test_api_tokens_can_be_deleted()
2020
return $this->markTestSkipped('API support is not enabled.');
2121
}
2222

23-
if (Features::hasTeamFeatures()) {
24-
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
25-
} else {
26-
$this->actingAs($user = User::factory()->create());
27-
}
23+
$this->actingAs($user = User::factory()->withPersonalTeam()->create());
2824

2925
$token = $user->tokens()->create([
3026
'name' => 'Test Token',

0 commit comments

Comments
 (0)