django-solomon 0.4.2__tar.gz → 0.4.3__tar.gz

{django_solomon-0.4.2 → django_solomon-0.4.3}/CHANGELOG.md

@@ -5,12 +5,19 @@ All notable changes to django-solomon will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.3] - Unreleased
+
+### Changed
+
+- Changed email uniqueness constraint to use standard fields approach for better database compatibility
+- Removed case-insensitivity from email uniqueness constraint to support all database backends
+- Updated tests to verify new constraint behavior
+
 ## [0.4.2] - 2025-04-21
 
-### Improved
+### Changed
 
-- Enhanced case-insensitive email uniqueness constraint implementation
-- Added regression test to verify constraint behavior
+- Maintenance release with minor improvements and bug fixes
 
 ## [0.4.1] - 2025-04-20
 

{django_solomon-0.4.2 → django_solomon-0.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: django-solomon
-Version: 0.4.2
+Version: 0.4.3
 Summary: A Django app for passwordless authentication using magic links.
 Project-URL: Home, https://django-solomon.rtfd.io/
 Project-URL: Documentation, https://django-solomon.rtfd.io/
@@ -72,7 +72,8 @@ A Django app for passwordless authentication using magic links.
 - Privacy-focused IP anonymization
 - Customizable templates for emails and pages
 - Compatible with Django's authentication system
-- Enforces unique email addresses for users (case-insensitive)
+- Enforces unique email addresses for users (Code is taken
+  from [django-unique-user-email](https://github.com/carltongibson/django-unique-user-email). Thanks!)
 
 ## Installation
 

{django_solomon-0.4.2 → django_solomon-0.4.3}/README.md

@@ -20,7 +20,8 @@ A Django app for passwordless authentication using magic links.
 - Privacy-focused IP anonymization
 - Customizable templates for emails and pages
 - Compatible with Django's authentication system
-- Enforces unique email addresses for users (case-insensitive)
+- Enforces unique email addresses for users (Code is taken
+  from [django-unique-user-email](https://github.com/carltongibson/django-unique-user-email). Thanks!)
 
 ## Installation
 

{django_solomon-0.4.2 → django_solomon-0.4.3}/src/django_solomon/apps.py

@@ -1,5 +1,4 @@
 from django.apps import AppConfig
-from django.db.models.functions import Lower
 
 
 class DjangoSolomonConfig(AppConfig):
@@ -21,7 +20,7 @@ class DjangoSolomonConfig(AppConfig):
         # But setting a constraint does not...
         User.Meta.constraints = [
             models.UniqueConstraint(
-                Lower("email"),
+                fields=["email"],
                 name="unique_user_email",
                 # deferrable=models.Deferrable.DEFERRED,
             ),

{django_solomon-0.4.2 → django_solomon-0.4.3}/src/django_solomon/migrations/0003_add_unique_constraint_auth_user_email.py

@@ -1,5 +1,4 @@
 from django.db import migrations, models
-from django.db.models.functions import Lower
 
 
 # Copyright (c) 2023 Carlton Gibson
@@ -36,7 +35,7 @@ class Migration(migrations.Migration):
         CustomAddConstraint(
             model_name="user",
             constraint=models.UniqueConstraint(
-                Lower("email"),
+                fields=["email"],
                 name="unique_user_email",
             ),
         ),

{django_solomon-0.4.2 → django_solomon-0.4.3}/tests/test_migrations.py

@@ -42,8 +42,9 @@ class TestMigrations:
         # Run makemigrations specifically for the auth app to check if it detects changes
         # to the User model that need new migrations
         with patch("sys.stdout", makemigrations_stdout):
-            # Since we've updated the migration to use a case-insensitive constraint with Lower,
-            # we expect Django to detect a need to remove the old constraint and add the new one.
+            # Since we've updated the migration to use a case-sensitive constraint with fields=["email"]
+            # instead of a case-insensitive constraint with Lower("email"), we expect Django to detect
+            # a need to remove the old constraint and add the new one.
             # This is expected behavior and not a bug.
             try:
                 # The --check flag will raise a SystemExit if migrations are needed

{django_solomon-0.4.2 → django_solomon-0.4.3}/tests/test_models.py

@@ -235,21 +235,25 @@ class TestUserEmailUniqueConstraint:
                 password="password456",
             )
 
-    def test_user_email_unique_constraint_case_insensitive(self, user):
-        """Test that the email uniqueness is case-insensitive at runtime."""
+    def test_user_email_unique_constraint_case_sensitive(self, user):
+        """Test that the email uniqueness is case-sensitive at runtime."""
         User = get_user_model()
 
         # The user fixture already creates a user with email "test@example.com"
-        # Attempting to create another user with the same email in different case should fail
-        # because the constraint is made case-insensitive at runtime in apps.py
-        with pytest.raises(IntegrityError):
-            # Create the user directly in the database to bypass Django's validation
-            # which would prevent the creation due to the runtime constraint
-            User.objects.create(
+        # Attempting to create another user with the same email in different case should succeed
+        # because the constraint is case-sensitive
+        try:
+            new_user = User.objects.create(
                 username="another_user",
                 email="TEST@example.com",  # Same email as the fixture user but with different case
                 password="password456",
             )
+            # Verify the user was created
+            assert new_user.pk is not None
+            assert new_user.email == "TEST@example.com"
+        except IntegrityError:
+            # If this fails, it means the constraint is still case-insensitive
+            assert False, "Expected to create a user with the same email in different case, but it failed"  # noqa: B011
 
     def test_different_emails_allowed(self, user):
         """Test that users with different emails can be created."""