Fix generate_username to be case insensitive

Author: varmar05

server/mergin/auth/models.py

@@ -210,12 +210,12 @@ def generate_username(cls, email: str) -> Optional[str]:
             text(
                 """
                 SELECT
-                    replace(username, :username, '0')::int AS suffix
+                    replace(lower(username), :username, '0')::int AS suffix
                 FROM "user"
                 WHERE
-                    username = :username OR
-                    username SIMILAR TO :username_like
-                ORDER BY replace(username, :username, '0')::int DESC
+                    lower(username) = :username OR
+                    lower(username) SIMILAR TO :username_like
+                ORDER BY replace(lower(username), :username, '0')::int DESC
                 LIMIT 1;
                 """
             ),

server/mergin/tests/test_auth.py

@@ -856,6 +856,13 @@ def test_username_generation(client):
         == "verylonglonglonglonglo"
     )
 
+    # test username generation with existing user, case insensitive
+    user = add_user("Testuser")
+    assert User.generate_username("Testuser@example.com") == "testuser1"
+    assert User.generate_username("testuser@example.com") == "testuser1"
+    user = add_user("testuser1")
+    assert User.generate_username("Testuser@example.com") == "testuser2"
+
 
 def test_server_usage(client):
     """Test server usage endpoint"""