7f3eacdcf2
- Updated lead pipeline and task pipeline models to enhance the is_default field's help text for clarity. - Modified lead and opportunity email tasks to utilize frontend URLs instead of domain names for better link accuracy. - Removed outdated Celery task tests for leads and opportunities. - Streamlined permission checks across lead and opportunity views by introducing a unified is_org_admin function. - Improved HTML templates for email notifications to ensure correct variable usage and link generation. - Enhanced kanban view filtering logic to ensure proper access control for users based on their roles.
58 lines
2.4 KiB
Python
58 lines
2.4 KiB
Python
"""Tests for the User.name auto-fill on creation."""
|
|
|
|
import pytest
|
|
|
|
from common.models import User
|
|
|
|
|
|
@pytest.mark.django_db
|
|
class TestUserNameAutoFill:
|
|
"""User.save() / UserManager.create_user() set name from email-local-part."""
|
|
|
|
def test_create_user_without_name_falls_back_to_email_prefix(self):
|
|
user = User.objects.create(email="aswin.1231@example.com")
|
|
assert user.name == "aswin.1231"
|
|
|
|
def test_create_user_with_explicit_name_is_preserved(self):
|
|
user = User.objects.create(email="x@example.com", name="Alex Carter")
|
|
assert user.name == "Alex Carter"
|
|
|
|
def test_get_or_create_falls_back_when_creating(self):
|
|
user, created = User.objects.get_or_create(email="foo.bar@example.com")
|
|
assert created is True
|
|
assert user.name == "foo.bar"
|
|
|
|
def test_manager_create_user_sets_name_when_missing(self):
|
|
user = User.objects.create_user(email="manager@example.com")
|
|
assert user.name == "manager"
|
|
|
|
def test_subsequent_save_does_not_overwrite_cleared_name(self):
|
|
"""If a user PATCHes name to "", the next save() must keep it empty."""
|
|
user = User.objects.create(email="keep.empty@example.com")
|
|
assert user.name == "keep.empty" # auto-filled on creation
|
|
user.name = ""
|
|
user.save()
|
|
user.refresh_from_db()
|
|
assert user.name == ""
|
|
|
|
def test_the_longest_storable_email_still_fits_the_name_column(self):
|
|
"""The `[:255]` in `User.save` is a belt, and it can never be reached.
|
|
|
|
`email` is an `EmailField`, so `varchar(254)`, which caps the local
|
|
part at 242 characters once `@example.com` is accounted for. `name` is
|
|
`varchar(255)`. A local part long enough to need truncating cannot be
|
|
stored in the first place.
|
|
|
|
This test used to assert the truncation directly, with a 300-character
|
|
local part and a 312-character email. SQLite does not enforce declared
|
|
string lengths, so it passed there and passed in CI; against
|
|
PostgreSQL the same line raises `StringDataRightTruncation` before
|
|
`name` is ever considered. It was pinning behaviour reachable only on
|
|
a database this project does not run in production.
|
|
"""
|
|
longest_local = "a" * (254 - len("@example.com"))
|
|
user = User.objects.create(email=f"{longest_local}@example.com")
|
|
|
|
assert len(user.name) == 242
|
|
user.refresh_from_db()
|
|
assert user.name == longest_local
|