Add username change tests

misago/account/tests/test_account_username.py

@@ -0,0 +1,139 @@
+from django.urls import reverse
+
+from ...test import assert_contains, assert_has_success_message, assert_not_contains
+
+
+def test_account_username_returns_error_for_guests(user, client):
+    response = client.get(reverse("misago:account-username"))
+    assert_contains(response, "You need to be signed in", status_code=403)
+
+
+def test_account_username_renders_form(user_client):
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Change username")
+
+
+def test_account_username_form_changes_username(user_client, user):
+    response = user_client.post(
+        reverse("misago:account-username"), {"username": "JohnDoe"}
+    )
+
+    assert response.status_code == 302
+    assert_has_success_message(response, "Username changed")
+
+    user.refresh_from_db()
+    assert user.username == "JohnDoe"
+
+    change = user.namechanges.first()
+    assert change.new_username == "JohnDoe"
+    assert change.old_username == "User"
+
+
+def test_account_username_form_validates_username(user_client, admin, user):
+    response = user_client.post(
+        reverse("misago:account-username"), {"username": admin.username}
+    )
+
+    assert response.status_code == 200
+    assert_contains(response, "This username is not available.")
+
+    user.refresh_from_db()
+    assert user.username == "User"
+
+
+def test_account_username_form_validates_username_change_permission(user_client, user):
+    user.group.can_change_username = False
+    user.group.save()
+
+    response = user_client.post(
+        reverse("misago:account-username"), {"username": "JohnDoe"}
+    )
+
+    assert response.status_code == 200
+    assert_contains(response, "You can't change your username.")
+
+    user.refresh_from_db()
+    assert user.username == "User"
+
+
+def test_account_username_form_validates_username_change_cooldown(user_client, user):
+    user.group.username_changes_span = 1
+    user.group.save()
+
+    user.set_username("John", user)
+
+    response = user_client.post(
+        reverse("misago:account-username"), {"username": "JohnDoe"}
+    )
+
+    assert response.status_code == 200
+    assert_contains(response, "You can't change your username at the moment.")
+
+    user.refresh_from_db()
+    assert user.username == "User"
+
+
+def test_account_username_renders_message_about_unlimited_username_changes(
+    user_client, user
+):
+    user.group.username_changes_limit = 0
+    user.group.save()
+
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Change username")
+    assert_contains(response, "You can change your username unlimited number of times.")
+
+
+def test_account_username_renders_message_about_username_changes_disabled(
+    user_client, user
+):
+    user.group.can_change_username = False
+    user.group.save()
+
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Change username")
+    assert_contains(response, "You can't change your username.")
+
+
+def test_account_username_renders_message_about_username_changes_left(
+    user_client, user
+):
+    user.group.username_changes_limit = 2
+    user.group.username_changes_span = 0
+    user.group.save()
+
+    user.set_username("John", user)
+
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Change username")
+    assert_contains(response, "You can change your username 1 more time.")
+
+
+def test_account_username_renders_message_about_username_change_cooldown(
+    user_client, user
+):
+    user.group.username_changes_span = 1
+    user.group.save()
+
+    user.set_username("John", user)
+
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Change username")
+    assert_contains(response, "You will be able to change your username again")
+
+
+def test_account_username_renders_empty_history(user_client):
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Changes history")
+    assert_contains(response, "Your account has no history of name changes.")
+
+
+def test_account_username_renders_history_item(user_client, user):
+    orginal_username = user.username
+    user.set_username("John", user)
+
+    response = user_client.get(reverse("misago:account-username"))
+    assert_contains(response, "Changes history")
+    assert_contains(response, orginal_username)
+    assert_contains(response, "John")
+    assert_not_contains(response, "Your account has no history of name changes.")

misago/templates/misago/account/settings/username_form.html

@@ -67,6 +67,8 @@ <h3 class="panel-title">
         <div class="panel-body">
           {% if form.available_changes.changes_left == 0 %}
             {% trans "You have used up all available username changes." context "account username help" %}
+          {% else %}
+            {% trans "You can't change your username at the moment." context "account username help" %}
           {% endif %}
           {% if form.available_changes.next_change %}
             {% capture trimmed as next_change %}
@@ -77,8 +79,6 @@ <h3 class="panel-title">
             {% blocktrans trimmed with next_change=next_change|safe context "account username help" %}
               You will be able to change your username again {{ next_change }}.
             {% endblocktrans %}
-          {% else %}
-            {% trans "You can't change your username at the moment." context "account username help" %}
           {% endif %}
         </div>
       {% endif %}