From 4e9940fe2920d0ba637c2ffc41f61b0a604b03f8 Mon Sep 17 00:00:00 2001
From: Mathieu Virbel <mat@meltingrocks.com>
Date: Thu, 17 Aug 2023 14:46:48 +0200
Subject: [PATCH] server: fixes tests

---
 server/reflector/auth/auth_none.py    | 17 ++++-------------
 server/reflector/views/transcripts.py |  3 +++
 2 files changed, 7 insertions(+), 13 deletions(-)

diff --git a/server/reflector/auth/auth_none.py b/server/reflector/auth/auth_none.py
index 3959c739..1c1dd0fd 100644
--- a/server/reflector/auth/auth_none.py
+++ b/server/reflector/auth/auth_none.py
@@ -3,7 +3,7 @@ from typing import Annotated
 from fastapi import Depends
 from fastapi.security import OAuth2PasswordBearer
 
-oauth2_scheme = OAuth2PasswordBearer(tokenUrl="token")
+oauth2_scheme = OAuth2PasswordBearer(tokenUrl="token", auto_error=False)
 
 
 class UserInfo(BaseModel):
@@ -15,21 +15,12 @@ class AccessTokenInfo(BaseModel):
 
 
 def authenticated(token: Annotated[str, Depends(oauth2_scheme)]):
-    def _authenticated():
-        return None
-
-    return _authenticated
+    return None
 
 
 def current_user(token: Annotated[str, Depends(oauth2_scheme)]):
-    def _current_user():
-        return None
-
-    return _current_user
+    return None
 
 
 def current_user_optional(token: Annotated[str, Depends(oauth2_scheme)]):
-    def _current_user_optional():
-        return None
-
-    return _current_user_optional
+    return None
diff --git a/server/reflector/views/transcripts.py b/server/reflector/views/transcripts.py
index 778c47d7..532de6ff 100644
--- a/server/reflector/views/transcripts.py
+++ b/server/reflector/views/transcripts.py
@@ -217,6 +217,9 @@ class DeletionStatus(BaseModel):
 async def transcripts_list(
     user: auth.UserInfo = Depends(auth.current_user),
 ):
+    if not user:
+        raise HTTPException(status_code=401, detail="Not authenticated")
+
     return paginate(await transcripts_controller.get_all(user_id=user["sub"]))