Merge last crud misses
This commit is contained in:
@@ -11,7 +11,7 @@ from sqlmodel import Session
|
|||||||
from app.core import security
|
from app.core import security
|
||||||
from app.core.config import settings
|
from app.core.config import settings
|
||||||
from app.core.db import engine
|
from app.core.db import engine
|
||||||
from app.models import TokenPayload, User
|
from app.models.user import TokenPayload, User
|
||||||
|
|
||||||
reusable_oauth2 = OAuth2PasswordBearer(
|
reusable_oauth2 = OAuth2PasswordBearer(
|
||||||
tokenUrl=f"{settings.API_V1_STR}/login/access-token"
|
tokenUrl=f"{settings.API_V1_STR}/login/access-token"
|
||||||
|
|||||||
@@ -5,7 +5,7 @@ from pydantic import BaseModel
|
|||||||
|
|
||||||
from app.api.deps import SessionDep
|
from app.api.deps import SessionDep
|
||||||
from app.core.security import get_password_hash
|
from app.core.security import get_password_hash
|
||||||
from app.models import (
|
from app.models.user import (
|
||||||
User,
|
User,
|
||||||
UserPublic,
|
UserPublic,
|
||||||
)
|
)
|
||||||
|
|||||||
@@ -2,7 +2,7 @@ from fastapi import APIRouter, Depends
|
|||||||
from pydantic.networks import EmailStr
|
from pydantic.networks import EmailStr
|
||||||
|
|
||||||
from app.api.deps import get_current_active_superuser
|
from app.api.deps import get_current_active_superuser
|
||||||
from app.models import Message
|
from app.models.base import Message
|
||||||
from app.utils import generate_test_email, send_email
|
from app.utils import generate_test_email, send_email
|
||||||
|
|
||||||
router = APIRouter(prefix="/utils", tags=["utils"])
|
router = APIRouter(prefix="/utils", tags=["utils"])
|
||||||
|
|||||||
@@ -68,7 +68,9 @@ def test_recovery_password_user_not_exits(
|
|||||||
f"{settings.API_V1_STR}/password-recovery/{email}",
|
f"{settings.API_V1_STR}/password-recovery/{email}",
|
||||||
headers=normal_user_token_headers,
|
headers=normal_user_token_headers,
|
||||||
)
|
)
|
||||||
assert r.status_code == 404
|
assert (
|
||||||
|
r.status_code == 404
|
||||||
|
) # TODO: Fix testing and do not leak known emails with 404
|
||||||
|
|
||||||
|
|
||||||
def test_reset_password(client: TestClient, db: Session) -> None:
|
def test_reset_password(client: TestClient, db: Session) -> None:
|
||||||
@@ -81,7 +83,6 @@ def test_reset_password(client: TestClient, db: Session) -> None:
|
|||||||
full_name="Test User",
|
full_name="Test User",
|
||||||
password=password,
|
password=password,
|
||||||
is_active=True,
|
is_active=True,
|
||||||
is_superuser=False,
|
|
||||||
)
|
)
|
||||||
user = User.create(session=db, create_obj=user_create)
|
user = User.create(session=db, create_obj=user_create)
|
||||||
token = generate_password_reset_token(email=email)
|
token = generate_password_reset_token(email=email)
|
||||||
|
|||||||
Reference in New Issue
Block a user