🐛 Fix bug in update_organization and refactor test (#29)

fastapi · Apr 15, 2024 · edcb055 · edcb055
1 parent aa8e0c1
commit edcb055
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/backend/app/api/routes/organizations.py b/backend/app/api/routes/organizations.py
@@ -130,6 +130,8 @@ def update_organization(
     update_dict = organization_in.model_dump(exclude_unset=True)
     org = link.organization
     org.sqlmodel_update(update_dict)
+    session.add(org)
+    session.commit()
     return org
 
 

diff --git a/backend/app/tests/api/routes/test_organizations.py b/backend/app/tests/api/routes/test_organizations.py
@@ -154,9 +154,10 @@ def test_update_organization(client: TestClient, db: Session) -> None:
 
     assert response.status_code == 200
     data = response.json()
+    db.refresh(organization)
     assert data["id"] == organization.id
-    assert data["name"] == organization_in["name"]
-    assert data["description"] == organization_in["description"]
+    assert organization.name == organization_in["name"]
+    assert organization.description == organization_in["description"]
 
 
 def test_update_organization_not_found(client: TestClient) -> None: