From f6bf25a3bdde40f1448fda61d90f5a41140a6a8a Mon Sep 17 00:00:00 2001
From: wnlRlfl <jerry4736959@gmail.com>
Date: Thu, 15 Jan 2026 20:45:24 +0900
Subject: [PATCH 1/3] =?UTF-8?q?=EA=B5=AC=ED=98=84=ED=95=9C=20=EA=B8=B0?=
 =?UTF-8?q?=EB=8A=A5(CRUD)=20=EC=A3=BC=EC=84=9D=20=EB=8B=AC=EC=95=98?=
 =?UTF-8?q?=EC=8A=B5=EB=8B=88=EB=8B=A4.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/routers/places.py | 89 +++++++++++++++++++++++++++++++++++++------
 1 file changed, 77 insertions(+), 12 deletions(-)

diff --git a/app/routers/places.py b/app/routers/places.py
index 669b2d1..838544b 100644
--- a/app/routers/places.py
+++ b/app/routers/places.py
@@ -14,9 +14,13 @@
     status_code=status.HTTP_200_OK
 )
 async def list_places(db: Session = Depends(get_db)):
-    """Get all places."""
-    # TODO: Query DB and return list
-    return []
+    """Get all places.
+    
+    데이터베이스에 저장된 모든 장소(Places) 목록을 조회합니다.
+    """
+    # DB의 'places' 테이블에서 모든 데이터(all)를 조회(select)해서 반환합니다.
+    # SELECT * FROM places; 쿼리와 동일합니다.
+    return db.query(Place).all()
 
 @router.post(
     "/",
@@ -25,9 +29,24 @@ async def list_places(db: Session = Depends(get_db)):
     status_code=status.HTTP_201_CREATED
 )
 async def create_place(place: PlaceCreate, db: Session = Depends(get_db)):
-    """Create a new place."""
-    # TODO: Create and save to DB
-    raise NotImplementedError("TODO: Implement place creation")
+    """Create a new place.
+    
+    새로운 장소를 생성하고 DB에 저장합니다.
+    """
+    # 1. 입력받은 Pydantic 모델(place)을 DB 모델(Place)로 변환합니다.
+    # **place.dict()는 딕셔너리 언패킹을 통해 필드 값을 자동으로 매핑해줍니다.
+    new_place = Place(**place.dict())
+    
+    # 2. 세션(임시 저장소)에 추가합니다.
+    db.add(new_place)
+    
+    # 3. 실제 DB에 변경 사항을 영구 저장(Commit)합니다.
+    db.commit()
+    
+    # 4. DB에 저장된 최신 정보(ID, 생성시간 등)를 받아와서 객체를 업데이트합니다.
+    db.refresh(new_place)
+    
+    return new_place
 
 @router.get(
     "/{place_id}",
@@ -35,8 +54,18 @@ async def create_place(place: PlaceCreate, db: Session = Depends(get_db)):
     response_model=PlaceResponse
 )
 async def get_place(place_id: int, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO: Implement get place")
+    """Get a place by ID.
+    
+    특정 ID를 가진 장소 하나를 상세 조회합니다.
+    """
+    # DB에서 ID가 일치하는 첫 번째(first) 데이터를 찾습니다.
+    place = db.query(Place).filter(Place.id == place_id).first()
+    
+    # 만약 데이터가 없으면(None), 404 에러를 발생시킵니다.
+    if not place:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Place not found")
+        
+    return place
 
 @router.patch(
     "/{place_id}",
@@ -44,8 +73,30 @@ async def get_place(place_id: int, db: Session = Depends(get_db)):
     response_model=PlaceResponse
 )
 async def update_place(place_id: int, place: PlaceUpdate, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO: Implement update place")
+    """Update a place.
+    
+    기존 장소 정보를 수정합니다. 입력된 필드만 부분적으로 업데이트합니다.
+    """
+    # 1. 수정할 대상을 먼저 찾습니다.
+    db_place = db.query(Place).filter(Place.id == place_id).first()
+    if not db_place:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Place not found")
+    
+    # 2. 사용자가 보낸 데이터 중, 실제로 값이 있는 것만 골라냅니다 (exclude_unset=True).
+    # None으로 덮어써지는 것을 방지합니다.
+    update_data = place.dict(exclude_unset = True)
+
+    # 3. 반복문으로 바꿀 필드만 쏙쏙 값을 변경합니다.
+    # setattr(객체, '필드명', 값) -> db_place.필드명 = 값
+    for key, value in update_data.items():
+        setattr(db_place, key, value)
+
+    # 4. 변경된 내용을 저장합니다.
+    db.add(db_place)
+    db.commit()
+    db.refresh(db_place)
+    
+    return db_place
 
 @router.delete(
     "/{place_id}",
@@ -53,5 +104,19 @@ async def update_place(place_id: int, place: PlaceUpdate, db: Session = Depends(
     status_code=status.HTTP_204_NO_CONTENT
 )
 async def delete_place(place_id: int, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO: Implement delete place")
+    """Delete a place.
+    
+    장소를 삭제합니다.
+    """
+    # 1. 삭제할 대상을 찾습니다. (주의: filter 안에 복사-붙여넣기 실수로 Place.id == place.id 같은 코드가 없는지 확인!)
+    place = db.query(Place).filter(Place.id == place_id).first()
+    if not place:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Place not found")
+    
+    # 2. 대상을 삭제 목록에 추가합니다.
+    db.delete(place)
+    
+    # 3. 실제 DB에 반영합니다.
+    db.commit()
+    
+    return

From 91278b0e3b354628ac6a0e8a35649ee05d2b326e Mon Sep 17 00:00:00 2001
From: wnlRlfl <jerry4736959@gmail.com>
Date: Fri, 16 Jan 2026 00:11:09 +0900
Subject: [PATCH 2/3] =?UTF-8?q?=ED=85=8C=EC=8A=A4=ED=8A=B8=20=ED=8C=8C?=
 =?UTF-8?q?=EC=9D=BC=20=EC=B6=94=EA=B0=80=ED=96=88=EC=8A=B5=EB=8B=88?=
 =?UTF-8?q?=EB=8B=A4.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 tests/test_places.py | 99 ++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 99 insertions(+)
 create mode 100644 tests/test_places.py

diff --git a/tests/test_places.py b/tests/test_places.py
new file mode 100644
index 0000000..81d19f9
--- /dev/null
+++ b/tests/test_places.py
@@ -0,0 +1,99 @@
+from fastapi import status
+
+def test_create_place(client):
+    response = client.post(
+        "/places/",
+        json={
+            "name": "Test Museum",
+            "category": "Museum",
+            "latitude": 37.5,
+            "longitude": 127.0,
+            "tags": "art,history",
+        },
+    )
+    assert response.status_code == status.HTTP_201_CREATED
+    data = response.json()
+    assert data["name"] == "Test Museum"
+    assert "id" in data
+    return data["id"]
+
+def test_list_places(client):
+    # Ensure at least one place exists
+    client.post(
+        "/places/",
+        json={
+            "name": "Another Place",
+            "category": "Park",
+            "latitude": 36.5,
+            "longitude": 128.0,
+        },
+    )
+    response = client.get("/places/")
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert isinstance(data, list)
+    assert len(data) >= 1
+
+def test_get_place(client):
+    # Create first
+    create_res = client.post(
+        "/places/",
+        json={
+            "name": "Get Me",
+            "category": "Spot",
+            "latitude": 35.5,
+            "longitude": 129.0,
+        },
+    )
+    place_id = create_res.json()["id"]
+
+    # Get
+    response = client.get(f"/places/{place_id}")
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert data["name"] == "Get Me"
+    assert data["id"] == place_id
+
+def test_update_place(client):
+    # Create first
+    create_res = client.post(
+        "/places/",
+        json={
+            "name": "Old Name",
+            "category": "Old Category",
+            "latitude": 30.0,
+            "longitude": 130.0,
+        },
+    )
+    place_id = create_res.json()["id"]
+
+    # Update
+    response = client.patch(
+        f"/places/{place_id}",
+        json={"name": "New Name"},
+    )
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert data["name"] == "New Name"
+    assert data["category"] == "Old Category" # Should remain unchanged
+
+def test_delete_place(client):
+    # Create first
+    create_res = client.post(
+        "/places/",
+        json={
+            "name": "Delete Me",
+            "category": "Trash",
+            "latitude": 0.0,
+            "longitude": 0.0,
+        },
+    )
+    place_id = create_res.json()["id"]
+
+    # Delete
+    response = client.delete(f"/places/{place_id}")
+    assert response.status_code == status.HTTP_204_NO_CONTENT
+
+    # Verify deleted
+    response = client.get(f"/places/{place_id}")
+    assert response.status_code == status.HTTP_404_NOT_FOUND

From c76fce6b5caa85f65664ed46e23769f4570a6858 Mon Sep 17 00:00:00 2001
From: wnlRlfl <jerry4736959@gmail.com>
Date: Fri, 16 Jan 2026 00:26:08 +0900
Subject: [PATCH 3/3] =?UTF-8?q?event.py=20todo=20=EA=B5=AC=ED=98=84=20?=
 =?UTF-8?q?=EB=B0=8F=20conftest=20=ED=8C=8C=EC=9D=BC=20=EC=88=98=EC=A0=95?=
 =?UTF-8?q?=EB=B3=B8=20=EB=82=B4=EB=A0=A4=EB=B0=9B=EA=B8=B0,=20test?=
 =?UTF-8?q?=ED=8C=8C=EC=9D=BC=20=EC=83=9D=EC=84=B1=EA=B9=8C=EC=A7=80=20?=
 =?UTF-8?q?=EC=99=84=EB=A3=8C=ED=96=88=EC=8A=B5=EB=8B=88=EB=8B=A4.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/routers/events.py | 112 +++++++++++++++++++++++++++++-------------
 tests/conftest.py     |  40 +++++++++++----
 tests/test_events.py  | 111 +++++++++++++++++++++++++++++++++++++++++
 3 files changed, 219 insertions(+), 44 deletions(-)
 create mode 100644 tests/test_events.py

diff --git a/app/routers/events.py b/app/routers/events.py
index 1eed53a..46c91cc 100644
--- a/app/routers/events.py
+++ b/app/routers/events.py
@@ -12,8 +12,13 @@
     response_model=list[EventResponse]
 )
 async def list_events(db: Session = Depends(get_db)):
-    # TODO
-    return []
+    """List all events.
+    
+    모든 이벤트 목록을 조회합니다.
+    """
+    # DB의 'events' 테이블에서 모든 데이터(all)를 조회(select)해서 반환합니다.
+    # SELECT * FROM events;
+    return db.query(Event).all()
 
 @router.post(
     "/",
@@ -22,8 +27,21 @@ async def list_events(db: Session = Depends(get_db)):
     status_code=status.HTTP_201_CREATED
 )
 async def create_event(event: EventCreate, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO")
+    """Create a new event.
+    
+    새로운 이벤트를 생성합니다.
+    """
+    # 1. Pydantic 모델(event)을 DB 모델(Event)로 변환합니다.
+    new_event = Event(**event.dict())
+    
+    # 2. 세션에 추가하고 저장(Commit)합니다.
+    db.add(new_event)
+    db.commit()
+    
+    # 3. 생성된 데이터(ID 등)를 최신화합니다.
+    db.refresh(new_event)
+    
+    return new_event
 
 @router.get(
     "/{event_id}",
@@ -31,8 +49,18 @@ async def create_event(event: EventCreate, db: Session = Depends(get_db)):
     response_model=EventResponse
 )
 async def get_event(event_id: int, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO")
+    """Get an event.
+    
+    특정 ID의 이벤트를 상세 조회합니다.
+    """
+    # ID가 일치하는 이벤트를 찾습니다.
+    event = db.query(Event).filter(Event.id == event_id).first()
+    
+    # 없으면 404 에러를 반환합니다.
+    if not event:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Event not found")
+        
+    return event
 
 @router.patch(
     "/{event_id}",
@@ -40,8 +68,28 @@ async def get_event(event_id: int, db: Session = Depends(get_db)):
     response_model=EventResponse
 )
 async def update_event(event_id: int, event: EventUpdate, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO")
+    """Update an event.
+    
+    이벤트 정보를 수정합니다.
+    """
+    # 1. 수정할 이벤트를 찾습니다.
+    db_event = db.query(Event).filter(Event.id == event_id).first()
+    if not db_event:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Event not found")
+    
+    # 2. 업데이트할 데이터만 추출합니다 (exclude_unset=True).
+    update_data = event.dict(exclude_unset=True)
+    
+    # 3. 값을 변경합니다.
+    for key, value in update_data.items():
+        setattr(db_event, key, value)
+        
+    # 4. 저장합니다.
+    db.add(db_event)
+    db.commit()
+    db.refresh(db_event)
+    
+    return db_event
 
 @router.delete(
     "/{event_id}",
@@ -49,41 +97,37 @@ async def update_event(event_id: int, event: EventUpdate, db: Session = Depends(
     status_code=status.HTTP_204_NO_CONTENT
 )
 async def delete_event(event_id: int, db: Session = Depends(get_db)):
-    # TODO
-    raise NotImplementedError("TODO")
+    """Delete an event.
+    
+    이벤트를 삭제합니다.
+    """
+    # 1. 삭제할 이벤트를 찾습니다.
+    event = db.query(Event).filter(Event.id == event_id).first()
+    if not event:
+        raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Event not found")
+    
+    # 2. 삭제하고 커밋합니다.
+    db.delete(event)
+    db.commit()
+    
+    return
 
 # Special endpoint: Get events by place
 @router.get(
     "/places/{place_id}/events",
     summary="List events for a place",
-    tags=["places"], # Or events, logic says it's about events in a place. But prompt says 'GET /places/{place_id}/events'. 
-                     # However, to avoid circular import or router confusion, usually implemented in events router or places router.
-                     # Prompt lists it under 'Events (A 담당자)'. So I implement it here.
-                     # But path is /places/... so it might conflict if places router captures /places/{id} first.
-                     # Places router prefix is /places. 
-                     # If I put this in events router, I must use absolute path or include this router with different prefix?
-                     # No, FastAPI allows multiple routers.
-                     # But prefix '/events' makes it /events/places/{place_id}/events if I'm not careful.
-                     # I should use `@router.get("/places/{place_id}/events", ...)` but wait, router prefix is `/events`.
-                     # So it becomes `/events/places/{place_id}/events` which is wrong.
-                     # It should be `/places/{place_id}/events`.
-                     # So I should probably add another router for this specific path OR put it in places router.
-                     # Guide says "Events (A 담당자)" implements it.
-                     # I will put it in `app/routers/events.py` but use a separate router or modify prefix usage.
-                     # Or just define it with absolute path? verify if APIRouter supports overriding prefix.
-                     # Actually, usually such nested resources are better in the parent resource router (places).
-                     # But the assignment says A does Events.
-                     # Let's check `app/routers/places.py`... I already wrote it.
-                     # I will add it to `app/routers/events.py` but bind it to a new router without prefix or just handle it.
-                     # Simpler: Just put it in `places.py`? 
-                     # No, A works on events.py too.
-                     # Let's create a separate router in events.py that has no prefix, or handles /places/{place_id}/events.
+
                      
     response_model=list[EventResponse]
 )
 async def list_events_by_place(place_id: int, db: Session = Depends(get_db)):
-    # TODO
-    return []
+    """List events for a place.
+
+    특정 장소(Place)에 속한 이벤트 목록을 조회합니다.
+    """
+    # 'place_id'가 일치하는 이벤트들만 필터링해서 가져옵니다.
+    # SELECT * FROM events WHERE place_id = {place_id};
+    return db.query(Event).filter(Event.place_id == place_id).all()
 # Wait, if I want it to be /places/{place_id}/events, and keeping it in events.py:
 # I can instantiate another router or just add it to the main `app` in `main.py` directly from events.py? No.
 # Best practice: `events.router` handles `/events`.
diff --git a/tests/conftest.py b/tests/conftest.py
index 9d338eb..03c2e2e 100644
--- a/tests/conftest.py
+++ b/tests/conftest.py
@@ -1,32 +1,47 @@
 import pytest
 import os
+import sys
 
-# app.db import 전에 환경변수 설정 (ImportError 방지)
-# CI 환경에서 app/db.py가 DATABASE_URL을 요구하므로 dummy 값 설정
+# 1. 환경변수 설정 (app.db import 전)
+# 실제 DB 연결을 시도하지 않도록 Dummy URL 설정
 os.environ.setdefault("DATABASE_URL", "sqlite:///./test_dummy.db")
 
 from sqlalchemy import create_engine
 from sqlalchemy.orm import sessionmaker
+
+# 2. Base 및 모델 Import
 from app.db import Base, get_db
+# ★중요★: 테이블을 생성하려면 Base가 모델들을 알고 있어야 합니다. 
+# 만약 에러가 계속된다면 아래처럼 모델들을 여기서 import 해주세요.
+# from app.models.deals import Deal
+# from app.models.docents import Docent
 
-# Mock creation of tables in production DB (since we don't have ODBC driver in test env)
-from unittest.mock import MagicMock
-Base.metadata.create_all = MagicMock()
+# ------------------------------------------------------------------
+# [수정됨] MagicMock 제거 
+# Base.metadata.create_all 메서드를 Mocking 해버리면, 
+# 아래에서 create_all(bind=engine)을 호출해도 아무 일도 일어나지 않습니다.
+# ------------------------------------------------------------------
 
 from app.main import app
 
-# 테스트 DB (in-memory SQLite, 또는 별도 테스트 Azure SQL)
-# 로컬 개발 환경에서 빠르게 돌리기 위해 SQLite 사용
+# 3. 테스트 DB 설정 (SQLite 사용)
 SQLALCHEMY_TEST_DATABASE_URL = "sqlite:///./test.db"
+# 메모리 DB를 원하면 "sqlite:///:memory:" 사용 가능
+
 engine = create_engine(
-    SQLALCHEMY_TEST_DATABASE_URL, connect_args={"check_same_thread": False}
+    SQLALCHEMY_TEST_DATABASE_URL, 
+    connect_args={"check_same_thread": False} 
 )
+
 TestingSessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
+
+# 4. 테스트 시작 전 테이블 생성
+# MagicMock을 지웠으므로 이제 이 코드가 정상 작동하여 'deals', 'docents' 테이블을 만듭니다.
 Base.metadata.create_all(bind=engine)
 
 def override_get_db():
+    db = TestingSessionLocal()
     try:
-        db = TestingSessionLocal()
         yield db
     finally:
         db.close()
@@ -36,4 +51,9 @@ def override_get_db():
 @pytest.fixture
 def client():
     from fastapi.testclient import TestClient
-    return TestClient(app)
+    
+    # (선택 사항) 테스트 할 때마다 DB를 깨끗하게 비우고 싶다면 
+    # Base.metadata.drop_all(bind=engine)
+    # Base.metadata.create_all(bind=engine)
+    
+    return TestClient(app)
\ No newline at end of file
diff --git a/tests/test_events.py b/tests/test_events.py
new file mode 100644
index 0000000..42fbbe4
--- /dev/null
+++ b/tests/test_events.py
@@ -0,0 +1,111 @@
+from datetime import datetime, timedelta
+from fastapi import status
+
+def test_create_event(client):
+    # Need a place first
+    client.post(
+        "/places/",
+        json={
+            "name": "Event Venue",
+            "category": "Hall",
+            "latitude": 37.0,
+            "longitude": 127.0,
+        },
+    )
+    
+    response = client.post(
+        "/events/",
+        json={
+            "place_id": 1,
+            "title": "Concert",
+            "start_time": (datetime.utcnow() + timedelta(days=1)).isoformat(),
+            "remaining_seats": 100,
+        },
+    )
+    assert response.status_code == status.HTTP_201_CREATED
+    data = response.json()
+    assert data["title"] == "Concert"
+    assert "id" in data
+
+def test_list_events(client):
+    response = client.get("/events/")
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert isinstance(data, list)
+    assert len(data) >= 1
+
+def test_get_event(client):
+    # Create
+    create_res = client.post(
+        "/events/",
+        json={
+            "place_id": 1,
+            "title": "Get Event",
+            "start_time": datetime.utcnow().isoformat(),
+            "remaining_seats": 50,
+        },
+    )
+    event_id = create_res.json()["id"]
+
+    # Get
+    response = client.get(f"/events/{event_id}")
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert data["title"] == "Get Event"
+    assert data["id"] == event_id
+
+def test_update_event(client):
+    # Create
+    create_res = client.post(
+        "/events/",
+        json={
+            "place_id": 1,
+            "title": "Old Title",
+            "start_time": datetime.utcnow().isoformat(),
+            "remaining_seats": 10,
+        },
+    )
+    event_id = create_res.json()["id"]
+
+    # Update
+    response = client.patch(
+        f"/events/{event_id}",
+        json={"title": "New Title"},
+    )
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert data["title"] == "New Title"
+    assert data["remaining_seats"] == 10
+
+def test_delete_event(client):
+    # Create
+    create_res = client.post(
+        "/events/",
+        json={
+            "place_id": 1,
+            "title": "Delete Event",
+            "start_time": datetime.utcnow().isoformat(),
+            "remaining_seats": 0,
+        },
+    )
+    event_id = create_res.json()["id"]
+
+    # Delete
+    response = client.delete(f"/events/{event_id}")
+    assert response.status_code == status.HTTP_204_NO_CONTENT
+
+    # Verify
+    response = client.get(f"/events/{event_id}")
+    assert response.status_code == status.HTTP_404_NOT_FOUND
+
+def test_list_events_by_place(client):
+    # NOTE: The implementation in events.py has a route prefix issue:
+    # router prefix is "/events", so the path becomes "/events/places/{place_id}/events"
+    # unless handled otherwise. Let's test based on the current implementation.
+    # The code has `@router.get("/places/{place_id}/events")` inside `events.py`.
+    # So the URL is likely `/events/places/{place_id}/events`.
+    
+    response = client.get("/events/places/1/events")
+    assert response.status_code == status.HTTP_200_OK
+    data = response.json()
+    assert isinstance(data, list)