Changed requested changes

Fixed the route of the email send.
changed testing assertion
renamed files and function..

Author: PureDreamer

app/main.py

@@ -5,7 +5,7 @@
 from app.database.database import engine
 from app.dependencies import (
     MEDIA_PATH, STATIC_PATH, templates)
-from app.routers import agenda, event, profile, email_send
+from app.routers import agenda, event, profile, email
 
 
 models.Base.metadata.create_all(bind=engine)
@@ -17,7 +17,7 @@
 app.include_router(profile.router)
 app.include_router(event.router)
 app.include_router(agenda.router)
-app.include_router(email_send.router)
+app.include_router(email.router)
 
 
 @app.get("/")

app/routers/email_send.py renamed to app/routers/email.py

@@ -1,26 +1,26 @@
 from app.database.database import get_db
-from app.internal.email import send
+from app.internal.email import send as internal_send
 from fastapi import APIRouter, BackgroundTasks, Depends, Form, HTTPException
 from sqlalchemy.orm.session import Session
 from starlette.responses import RedirectResponse
 
 router = APIRouter(
-    prefix="/email_send",
+    prefix="/email",
     tags=["email"],
     responses={404: {"description": "Not found"}},
 )
 
 
-@router.post("/")
-async def send_email(
+@router.post("/send")
+async def send(
     db: Session = Depends(get_db),
     send_to: str = "/",
     title: str = Form(...),
     event_used: str = Form(...),
     user_to_send: str = Form(...),
     background_tasks: BackgroundTasks = BackgroundTasks
 ) -> RedirectResponse:
-    if not send(
+    if not internal_send(
             title=title, event_used=event_used,
             user_to_send=user_to_send,
             background_tasks=background_tasks, session=db):

app/templates/demo/home_email.html

@@ -7,7 +7,7 @@ <h1>{{message}}</h1>
 </div>
 
 <div>
-    <form action="/email_send/" method="post">
+    <form action="/email/send" method="post">
         <!-- Example of how to use sending email
          This is a testing using ids of evenets and users yet with 
             simple manipulation you could send it how ever you want -->

tests/test_email.py

@@ -12,12 +12,12 @@ def test_email_send(client, user, event, smtpd):
     mail.config.MAIL_TLS = False
     with mail.record_messages() as outbox:
         response = client.post(
-            "/email_send/", data={
+            "/email/send", data={
                 "event_used": event.id, "user_to_send": user.id,
                 "title": "Testing",
                 "background_tasks": BackgroundTasks})
         assert len(outbox) == 1
-        assert response.status_code == 303
+        assert response.ok
 
 
 def test_failed_email_send(client, user, event, smtpd):
@@ -26,9 +26,9 @@ def test_failed_email_send(client, user, event, smtpd):
     mail.config.MAIL_PORT = smtpd.port
     with mail.record_messages() as outbox:
         response = client.post(
-            "/email_send/", data={
+            "/email/send", data={
                 "event_used": event.id + 1, "user_to_send": user.id,
                 "title": "Testing",
                 "background_tasks": BackgroundTasks})
         assert len(outbox) == 0
-        assert response.status_code == 404
+        assert not response.ok