feat: add endpoint to delete fruits by id
This commit is contained in:
parent
3deec14a3d
commit
565c908bdc
@ -1,15 +1,14 @@
|
|||||||
from fastapi import APIRouter, HTTPException, status
|
from fastapi import APIRouter, HTTPException, status
|
||||||
from schemas.fruit import FruitSchema
|
from schemas.fruit import FruitSchema
|
||||||
from helpers.fruit_helpers import delete_fruit, get_fruit_by_id
|
from helpers.fruit_helpers import get_fruit_by_id, delete_fruit
|
||||||
from sqlalchemy.orm import Session
|
from sqlalchemy.orm import Session
|
||||||
from fastapi import Depends
|
from fastapi import Depends
|
||||||
from core.database import get_db
|
from core.database import get_db
|
||||||
import uuid
|
|
||||||
|
|
||||||
router = APIRouter()
|
router = APIRouter()
|
||||||
|
|
||||||
@router.delete("/fruits/{fruit_id}", status_code=status.HTTP_200_OK, response_model=FruitSchema)
|
@router.delete("/fruits/{fruit_id}", status_code=status.HTTP_200_OK, response_model=FruitSchema)
|
||||||
async def delete_fruit_endpoint(fruit_id: uuid.UUID, db: Session = Depends(get_db)):
|
async def delete_fruit_endpoint(fruit_id: str, db: Session = Depends(get_db)):
|
||||||
fruit = get_fruit_by_id(db, fruit_id)
|
fruit = get_fruit_by_id(db, fruit_id)
|
||||||
if not fruit:
|
if not fruit:
|
||||||
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Fruit not found")
|
raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Fruit not found")
|
||||||
|
@ -68,4 +68,6 @@ def delete_fruit(db: Session, fruit_id: UUID) -> None:
|
|||||||
fruit = db.query(Fruit).filter(Fruit.id == fruit_id).first()
|
fruit = db.query(Fruit).filter(Fruit.id == fruit_id).first()
|
||||||
if fruit:
|
if fruit:
|
||||||
db.delete(fruit)
|
db.delete(fruit)
|
||||||
db.commit()
|
db.commit()
|
||||||
|
else:
|
||||||
|
raise ValueError(f"No fruit found with ID {fruit_id}")
|
Loading…
x
Reference in New Issue
Block a user