fix: username should be optional for user creation (#4595)

integration_tests/auth/test_auth.py

@@ -1,4 +1,5 @@
 from collections import defaultdict
+from dataclasses import replace
 from datetime import datetime, timedelta, timezone
 from functools import partial
 from typing import (
@@ -259,6 +260,18 @@ def test_only_admin_can_create_user(
         if not e:
             new_user.log_in()
 
+    @pytest.mark.parametrize("role_or_user", [_ADMIN, _DEFAULT_ADMIN])
+    @pytest.mark.parametrize("role", list(UserRoleInput))
+    def test_username_is_optional(
+        self,
+        role_or_user: _RoleOrUser,
+        role: UserRoleInput,
+        _get_user: _GetUser,
+        _profiles: Iterator[_Profile],
+    ) -> None:
+        profile = replace(next(_profiles), username=None)
+        _get_user(role_or_user).create_user(role, profile=profile)
+
 
 class TestPatchViewer:
     @pytest.mark.parametrize("role_or_user", [_MEMBER, _ADMIN, _DEFAULT_ADMIN])

src/phoenix/server/api/mutations/user_mutations.py

@@ -91,7 +91,7 @@ async def create_user(
         password_hash = await info.context.hash_password(password, salt)
         user = models.User(
             reset_password=True,
-            username=input.username,
+            username=input.username or None,
             email=email,
             auth_method=enums.AuthMethod.LOCAL.value,
             password_hash=password_hash,