fix(storage): exclude auto loading ownerships field from models

one-zero-eight · Sep 4, 2023 · 11fcded · 11fcded · github-actions · Sep 4, 2023
1 parent 2c2779d
commit 11fcded
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/schemas/event_groups.py b/src/schemas/event_groups.py
@@ -38,7 +38,7 @@ class ViewEventGroup(BaseModel):
     name: Optional[str] = None
     description: Optional[str] = None
     tags: list["ViewTag"] = Field(default_factory=list)
-    ownerships: list["Ownership"] = Field(default_factory=list)
+    # ownerships: list["Ownership"] = Field(default_factory=list)
 
     @validator("tags", pre=True, always=True)
     def _validate_tags(cls, v):

diff --git a/src/schemas/tags.py b/src/schemas/tags.py
@@ -3,7 +3,7 @@
 import json
 from typing import Optional
 
-from pydantic import BaseModel, validator, Json, Field
+from pydantic import BaseModel, validator, Json
 
 from src.schemas.ownership import Ownership
 
@@ -28,7 +28,7 @@ class ViewTag(BaseModel):
     name: Optional[str] = None
     satellite: Optional[dict] = None
 
-    ownerships: list[Ownership] = Field(default_factory=list)
+    # ownerships: list[Ownership] = Field(default_factory=list)
 
     @validator("satellite", pre=True, always=True)
     def _validate_satellite(cls, v):

diff --git a/src/storages/sql/__mixin__.py b/src/storages/sql/__mixin__.py
@@ -59,7 +59,7 @@ class Ownership(Base):
 
         @declared_attr
         def ownerships(cls) -> Mapped[list["Ownership"]]:
-            return relationship(cls.Ownership, lazy="selectin", cascade="all, delete-orphan")
+            return relationship(cls.Ownership, cascade="all, delete-orphan")
 
     return Mixin
File	Stmts	Miss	Cover	Missing
src
exceptions.py	35	6	83%	11, 28, 42, 56, 84, 98
main.py	88	45	49%	82–151, 156–157, 162–165, 175
src/app
dependencies.py	47	2	96%	29, 37
src/app/auth
common.py	28	17	39%	12–21, 25–32, 36–44, 49–50
dependencies.py	16	6	62%	33–38, 42
jwt.py	51	34	33%	26–28, 32–36, 40–44, 48–53, 57–70, 74–81
src/app/event_groups
routes.py	72	39	46%	39–45, 65–76, 116–120, 140, 186–225
src/app/ics
routes.py	59	42	29%	34–43, 49–75, 95–118
src/app/users
routes.py	25	13	48%	32–34, 54–59, 78–80, 102–107
src/app/workshops
__init__.py	5	5	0%	1–9
routes.py	16	16	0%	1–63
src/repositories
crud.py	94	12	87%	89, 105, 108, 125, 142, 156–157, 171–180
ownership.py	13	2	85%	15–16
src/repositories/event_groups
repository.py	58	5	91%	57–58, 65–68
src/repositories/predefined
repository.py	151	70	54%	26, 34, 56, 64, 73, 76–84, 97–99, 103–108, 119–123, 127–130, 133, 136, 139, 147–158, 162–171, 175–205
src/repositories/tags
repository.py	76	14	82%	55–56, 67–68, 75–76, 81–83, 121–125
src/repositories/workshops
__init__.py	3	3	0%	1–4
abc.py	4	4	0%	1–10
repository.py	34	34	0%	1–68
src/schemas
events.py	71	6	92%	48, 56, 79, 98–100
ownership.py	16	1	94%	21
tags.py	39	1	97%	36
workshops.py	58	58	0%	1–85
TOTAL	1594	435	73%