Merge pull request #495 from pacoorozco/shift-111696

PHPUnit 10 Shift

tests/Feature/Actions/CreateUserTest.php

@@ -25,14 +25,15 @@
 
 namespace Tests\Feature\Actions;
 
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Actions\CreateUserAction;
 use Gamify\Models\User;
 use Gamify\Models\UserProfile;
 use Tests\Feature\TestCase;
 
-class CreateUserTest extends TestCase
+final class CreateUserTest extends TestCase
 {
-    /** @test */
+    #[Test]
     public function it_should_create_a_user_with_its_profile(): void
     {
         /** @var User $want */

tests/Feature/Actions/PublishQuestionActionTest.php

@@ -25,6 +25,7 @@
 
 namespace Tests\Feature\Actions;
 
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Actions\PublishQuestionAction;
 use Gamify\Events\QuestionPublished;
 use Gamify\Exceptions\QuestionPublishingException;
@@ -33,9 +34,9 @@
 use Illuminate\Support\Facades\Event;
 use Tests\Feature\TestCase;
 
-class PublishQuestionActionTest extends TestCase
+final class PublishQuestionActionTest extends TestCase
 {
-    /** @test */
+    #[Test]
     public function it_triggers_an_event_when_a_question_is_published(): void
     {
         /** @var Question $question */
@@ -57,7 +58,7 @@ public function it_triggers_an_event_when_a_question_is_published(): void
         $this->assertEquals(Question::PUBLISH_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_does_not_trigger_an_event_when_a_question_was_already_published(): void
     {
         /** @var Question $question */
@@ -78,7 +79,7 @@ public function it_does_not_trigger_an_event_when_a_question_was_already_publish
         $this->assertEquals(Question::PUBLISH_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_throws_an_exception_when_trying_to_publish_a_question_without_at_least_two_choices(): void
     {
         $this->withoutExceptionHandling();
@@ -95,7 +96,7 @@ public function it_throws_an_exception_when_trying_to_publish_a_question_without
         $publisher->execute($question);
     }
 
-    /** @test */
+    #[Test]
     public function it_throws_an_exception_when_trying_to_publish_a_question_without_a_correct_choice(): void
     {
         $this->withoutExceptionHandling();
@@ -112,7 +113,7 @@ public function it_throws_an_exception_when_trying_to_publish_a_question_without
         $publisher->execute($question);
     }
 
-    /** @test */
+    #[Test]
     public function it_publishes_question_when_date_is_on_the_past(): void
     {
         /** @var Question $question */
@@ -130,7 +131,7 @@ public function it_publishes_question_when_date_is_on_the_past(): void
         $this->assertEquals(Question::PUBLISH_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_schedules_question_publication_when_date_is_in_the_future(): void
     {
         /** @var Question $question */
@@ -148,7 +149,7 @@ public function it_schedules_question_publication_when_date_is_in_the_future():
         $this->assertEquals(Question::FUTURE_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_does_not_trigger_an_event_when_publish_fails(): void
     {
         /** @var Question $question */

tests/Feature/Auth/AuthenticationTest.php

@@ -8,7 +8,7 @@
 use Illuminate\Support\Facades\Event;
 use Tests\Feature\TestCase;
 
-class AuthenticationTest extends TestCase
+final class AuthenticationTest extends TestCase
 {
     public function test_login_screen_can_be_rendered(): void
     {

tests/Feature/Auth/EmailVerificationTest.php

@@ -9,7 +9,7 @@
 use Illuminate\Support\Facades\URL;
 use Tests\Feature\TestCase;
 
-class EmailVerificationTest extends TestCase
+final class EmailVerificationTest extends TestCase
 {
     public function test_email_verification_screen_can_be_rendered(): void
     {

tests/Feature/Auth/PasswordConfirmationTest.php

@@ -5,7 +5,7 @@
 use Gamify\Models\User;
 use Tests\Feature\TestCase;
 
-class PasswordConfirmationTest extends TestCase
+final class PasswordConfirmationTest extends TestCase
 {
     public function test_confirm_password_screen_can_be_rendered(): void
     {

tests/Feature/Auth/PasswordResetTest.php

@@ -7,7 +7,7 @@
 use Illuminate\Support\Facades\Notification;
 use Tests\Feature\TestCase;
 
-class PasswordResetTest extends TestCase
+final class PasswordResetTest extends TestCase
 {
     public function test_reset_password_link_screen_can_be_rendered(): void
     {

tests/Feature/Auth/RegistrationTest.php

@@ -2,12 +2,14 @@
 
 namespace Tests\Feature\Auth;
 
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Models\User;
 use Gamify\Providers\RouteServiceProvider;
 use Generator;
 use Tests\Feature\TestCase;
 
-class RegistrationTest extends TestCase
+final class RegistrationTest extends TestCase
 {
     public function test_registration_screen_can_be_rendered(): void
     {
@@ -30,11 +32,8 @@ public function test_new_users_can_register(): void
         $this->assertAuthenticated();
     }
 
-    /**
-     * @test
-     *
-     * @dataProvider provideWrongDataForUserRegistration
-     */
+    #[Test]
+    #[DataProvider('provideWrongDataForUserRegistration')]
     public function it_should_get_errors_when_registering_with_wrong_data(
         array $data,
         array $errors

tests/Feature/Auth/SocialAuthenticationTest.php

@@ -2,16 +2,17 @@
 
 namespace Tests\Feature\Auth;
 
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Events\SocialLogin;
 use Gamify\Providers\RouteServiceProvider;
 use Illuminate\Support\Facades\Event;
 use Laravel\Socialite\Facades\Socialite;
 use Mockery;
 use Tests\Feature\TestCase;
 
-class SocialAuthenticationTest extends TestCase
+final class SocialAuthenticationTest extends TestCase
 {
-    /** @test */
+    #[Test]
     public function it_should_authenticate_users_using_social_login(): void
     {
         $this->mockSocialLogin();

tests/Feature/Console/Commands/PublishScheduledQuestionsTest.php

@@ -25,12 +25,13 @@
 
 namespace Tests\Feature\Console\Commands;
 
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Models\Question;
 use Tests\Feature\TestCase;
 
-class PublishScheduledQuestionsTest extends TestCase
+final class PublishScheduledQuestionsTest extends TestCase
 {
-    /** @test */
+    #[Test]
     public function it_should_not_publish_questions_if_it_is_not_the_right_time_yet(): void
     {
         /** @var Question $question */
@@ -50,7 +51,7 @@ public function it_should_not_publish_questions_if_it_is_not_the_right_time_yet(
         $this->assertEquals(Question::FUTURE_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_should_raise_an_error_when_questions_can_not_be_published(): void
     {
         // Create a question without choices().
@@ -71,7 +72,7 @@ public function it_should_raise_an_error_when_questions_can_not_be_published():
         $this->assertEquals(Question::FUTURE_STATUS, $question->status);
     }
 
-    /** @test */
+    #[Test]
     public function it_should_sent_two_questions_to_publication(): void
     {
         $questions = Question::factory()

tests/Feature/Http/Controllers/Account/ChangePasswordControllerTest.php

@@ -25,15 +25,16 @@
 
 namespace Tests\Feature\Http\Controllers\Account;
 
+use PHPUnit\Framework\Attributes\Test;
 use Gamify\Models\User;
 use Illuminate\Support\Facades\Hash;
 use Tests\Feature\TestCase;
 
-class ChangePasswordControllerTest extends TestCase
+final class ChangePasswordControllerTest extends TestCase
 {
     const VALID_PASSWORD = 'foo#B4rBaz';
 
-    /** @test */
+    #[Test]
     public function it_shows_password_change_form_for_logged_users(): void
     {
         /** @var User $user */
@@ -45,15 +46,15 @@ public function it_shows_password_change_form_for_logged_users(): void
             ->assertViewIs('account.password.index');
     }
 
-    /** @test */
+    #[Test]
     public function it_shows_error_for_non_logged_users(): void
     {
         $this
             ->get(route('account.password.index'))
             ->assertRedirect(route('login'));
     }
 
-    /** @test */
+    #[Test]
     public function it_shows_validation_error_if_current_password_is_invalid(): void
     {
         /** @var User $user */
@@ -71,7 +72,7 @@ public function it_shows_validation_error_if_current_password_is_invalid(): void
             ->assertInvalid(['current-password']);
     }
 
-    /** @test */
+    #[Test]
     public function it_shows_validation_error_if_new_password_confirmation_does_not_match(): void
     {
         /** @var User $user */
@@ -89,7 +90,7 @@ public function it_shows_validation_error_if_new_password_confirmation_does_not_
             ->assertInvalid(['new-password']);
     }
 
-    /** @test */
+    #[Test]
     public function it_has_success_key_on_session_when_password_has_been_changed(): void
     {
         /** @var User $user */