Skip to content

Comments

Feature/add tests with pest#37

Merged
jankapunkt merged 14 commits intomainfrom
feature/add-tests-with-pest
Jun 4, 2024
Merged

Feature/add tests with pest#37
jankapunkt merged 14 commits intomainfrom
feature/add-tests-with-pest

Conversation

@alessandrobelli
Copy link
Contributor

Modified and created the following tests:

  • Tests\Unit\ExampleTest::test_that_true_is_true
  • Tests\Feature\ApiTokenPermissionsTest::test_api_token_permissions_can_be_updated
  • Tests\Feature\AuthenticationTest::test_login_screen_can_be_rendered
  • Tests\Feature\AuthenticationTest::test_users_can_authenticate_using_the_login_screen
  • Tests\Feature\AuthenticationTest::test_users_can_not_authenticate_with_invalid_password
  • Tests\Feature\BrowserSessionsTest::test_other_browser_sessions_can_be_logged_out
  • Tests\Feature\CreateApiTokenTest::test_api_tokens_can_be_created
  • Tests\Feature\CreateTeamTest::test_teams_can_be_created
  • Tests\Feature\DeleteAccountTest::test_user_accounts_can_be_deleted
  • Tests\Feature\DeleteApiTokenTest::test_api_tokens_can_be_deleted
  • Tests\Feature\DeleteTeamTest::test_teams_cant_be_deleted
  • Tests\Feature\DeleteTeamTest::test_empty_teams_can_be_deleted
  • Tests\Feature\DeleteTeamTest::test_personal_teams_cant_be_deleted
  • Tests\Feature\EmailVerificationTest::test_email_verification_screen_can_be_rendered
  • Tests\Feature\EmailVerificationTest::test_email_can_be_verified
  • Tests\Feature\EmailVerificationTest::test_email_can_not_verified_with_invalid_hash
  • Tests\Feature\ExampleTest::test_the_application_returns_a_successful_response
  • Tests\Feature\InviteTeamMemberTest::test_team_members_can_be_invited_to_team
  • Tests\Feature\InviteTeamMemberTest::test_team_member_invitations_can_be_cancelled
  • Tests\Feature\LeaveTeamTest::test_users_can_leave_teams
  • Tests\Feature\LeaveTeamTest::test_team_owners_cant_leave_their_own_team
  • Tests\Feature\PasswordConfirmationTest::test_confirm_password_screen_can_be_rendered
  • Tests\Feature\PasswordConfirmationTest::test_password_can_be_confirmed
  • Tests\Feature\PasswordConfirmationTest::test_password_is_not_confirmed_with_invalid_password
  • Tests\Feature\PasswordResetTest::test_reset_password_link_screen_can_be_rendered
  • Tests\Feature\PasswordResetTest::test_reset_password_link_can_be_requested
  • Tests\Feature\PasswordResetTest::test_reset_password_screen_can_be_rendered
  • Tests\Feature\PasswordResetTest::test_password_can_be_reset_with_valid_token
  • Tests\Feature\ProfileInformationTest::test_profile_information_can_be_updated
  • Tests\Feature\RegistrationTest::test_registration_screen_can_be_rendered
  • Tests\Feature\RegistrationTest::test_new_users_can_register
  • Tests\Feature\RemoveTeamMemberTest::test_team_members_can_be_removed_from_teams
  • Tests\Feature\RemoveTeamMemberTest::test_team_owner_cannot_be_removed_by_team_members
  • Tests\Feature\RemoveTeamMemberTest::test_admin_can_remove_other_team_members
  • Tests\Feature\TwoFactorAuthenticationSettingsTest::test_two_factor_authentication_can_be_enabled
  • Tests\Feature\TwoFactorAuthenticationSettingsTest::test_recovery_codes_can_be_regenerated
  • Tests\Feature\TwoFactorAuthenticationSettingsTest::test_two_factor_authentication_can_be_disabled
  • Tests\Feature\UpdatePasswordTest::test_password_can_be_updated
  • Tests\Feature\UpdatePasswordTest::test_current_password_must_be_correct
  • Tests\Feature\UpdatePasswordTest::test_new_passwords_must_match
  • Tests\Feature\UpdateTeamMemberRoleTest::test_only_team_owner_can_update_team_member_roles
  • Tests\Feature\UpdateTeamNameTest::test_team_names_can_be_updated.

I also added tests passing in the CI, sorry it took some tries because github CI tests requires some particular setup.

This is a starting point to create more later.

@alessandrobelli alessandrobelli added documentation Improvements or additions to documentation CI labels May 31, 2024
@alessandrobelli alessandrobelli self-assigned this May 31, 2024
@alessandrobelli alessandrobelli marked this pull request as draft May 31, 2024 13:58
@alessandrobelli alessandrobelli marked this pull request as ready for review June 3, 2024 15:13
jankapunkt
jankapunkt previously approved these changes Jun 4, 2024
Copy link
Member

@jankapunkt jankapunkt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

General approval, although I have some comments. It's up to you to address them or not as I think they're rather optional.

@jankapunkt jankapunkt merged commit 82c73d5 into main Jun 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants