Add transaction to test decorators + misc fixes

backend/backend/custom_settings.py

@@ -4,6 +4,7 @@
 please check the path: backend/backend/settings.py
 """
 
-# Pagination settings
+# MARK: Pagination
+
 PAGINATION_PAGE_SIZE = 20
 PAGINATION_MAX_PAGE_SIZE = 100

backend/backend/exception_handler.py

@@ -8,7 +8,7 @@
 
 
 def bad_request_logger(exception: Any, context: dict[str, Any]) -> Response | None:
-    # Get the DRF exception handler standard error response
+    # Get the DRF exception handler standard error response.
     response = exception_handler(exception, context)
 
     if response is not None:

backend/backend/management/commands/populate_db.py

@@ -33,7 +33,7 @@ def handle(self, *args: str, **options: Unpack[Options]) -> None:
         number_of_events = options["events"]
         number_of_groups = options["groups"]
 
-        # Clear all tables before creating new data
+        # Clear all tables before creating new data.
         UserModel.objects.exclude(username="admin").delete()
         Organization.objects.all().delete()
         Event.objects.all().delete()
@@ -55,6 +55,6 @@ def handle(self, *args: str, **options: Unpack[Options]) -> None:
         except Exception as error:
             self.stdout.write(
                 self.style.ERROR(
-                    f"An error occured during the creation of dummy data: {error}"
+                    f"An error occurred during the creation of dummy data: {error}"
                 )
             )

backend/backend/settings.py

@@ -165,7 +165,7 @@
 EMAIL_PORT = os.getenv("EMAIL_PORT")
 EMAIL_HOST_USER = os.getenv("EMAIL_HOST_USER")
 EMAIL_HOST_PASSWORD = os.getenv("EMAIL_HOST_PASSWORD")
-EMAIL_USE_TLS = bool(os.getenv("EMAIL_USE_TLS") == "True")
+EMAIL_USE_TLS = os.getenv("EMAIL_USE_TLS") == "True"
 # DEVELOPMENT ONLY
 EMAIL_BACKEND = "django.core.mail.backends.console.EmailBackend"
 

backend/backend/tests/unit/conftest.py

@@ -4,5 +4,4 @@
 
 @pytest.fixture
 def api_client() -> APIClient:
-    client = APIClient()
-    return client
+    return APIClient()

backend/content/serializers.py

@@ -66,10 +66,10 @@ def validate(self, data: Dict[str, Union[str, int]]) -> Dict[str, Union[str, int
             with PilImage.open(data["image_location"]) as img:
                 img.verify()
                 img_format = img.format.lower()
-        except Exception:
+        except Exception as e:
             raise serializers.ValidationError(
                 _("The image is not valid."), code="corrupted_file"
-            )
+            ) from e
 
         if img_format not in image_extensions:
             raise serializers.ValidationError(

backend/entities/tests.py

@@ -22,10 +22,10 @@
 )
 
 
-@pytest.mark.django_db
+@pytest.mark.django_db(transaction=True)
 def test_str_methods() -> None:
     organization = OrganizationFactory.build()
-    # Needs to be updated to reflect the recent changes
+    # Note: Needs to be updated to reflect the recent changes.
     # organization_application = OrganizationApplicationFactory.build()
     organization_event = OrganizationEventFactory.build()
     organization_member = OrganizationMemberFactory.build()

backend/events/tests.py

@@ -15,6 +15,7 @@
 )
 
 
+@pytest.mark.django_db(transaction=True)
 def test_str_methods() -> None:
     event = EventFactory.build()
     event_attendee = EventAttendeeFactory.build()