increase TU

inteli-city · May 7, 2024 · 513cf02 · 513cf02
1 parent 6d9c0c7
commit 513cf02
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 27 deletions.
diff --git a/src/modules/get_all_users/app/get_all_users_controller.py b/src/modules/get_all_users/app/get_all_users_controller.py
@@ -28,17 +28,5 @@ def __call__(self, request: IRequest) -> IResponse:
         except MissingParameters as err:
             return BadRequest(body=f"Parâmetro ausente: {err.message}")
 
-        except InvalidCredentials as err:
-            return BadRequest(body=f"Parâmetro inválido: {err.message}")
-
-        except EntityError as err:
-            return BadRequest(body=f"Parâmetro inválido: {err.message}")
-
-        except NoItemsFound as err:
-            return BadRequest(body=f"Usuário não encontrado: {err.message}")
-
-        except InvalidTokenError as err:
-            return Unauthorized(body="Token inválido ou expirado")
-
         except Exception as err:
             return InternalServerError(body=err.args[0])
diff --git a/src/modules/list_users_in_group/app/list_users_in_group_controller.py b/src/modules/list_users_in_group/app/list_users_in_group_controller.py
@@ -34,18 +34,6 @@ def __call__(self, request: IRequest) -> IResponse:
 
         except MissingParameters as err:
             return BadRequest(body=f"Parâmetro ausente: {err.message}")
-
-        except InvalidCredentials as err:
-            return BadRequest(body=f"Parâmetro inválido: {err.message}")
-
-        except EntityError as err:
-            return BadRequest(body=f"Parâmetro inválido: {err.message}")
-
-        except NoItemsFound as err:
-            return BadRequest(body=f"Usuário não encontrado: {err.message}")
-
-        except InvalidTokenError as err:
-            return Unauthorized(body="Token inválido ou expirado")
 
         except Exception as err:
             return InternalServerError(body=err.args[0])
diff --git a/src/modules/refresh_token/app/refresh_token_usecase.py b/src/modules/refresh_token/app/refresh_token_usecase.py
@@ -13,8 +13,5 @@ def __call__(self, refresh_token: str) -> str:
             raise ForbiddenAction(f'Refresh Token')
 
         access_token, refresh_token, id_token = tokens
-
-        if access_token is None or refresh_token is None or id_token is None:
-            raise ForbiddenAction(f'Refresh Token ou Access Token')
 
         return access_token, refresh_token, id_token
diff --git a/tests/modules/list_users_in_group/app/test_list_users_in_group_usecase.py b/tests/modules/list_users_in_group/app/test_list_users_in_group_usecase.py
@@ -22,6 +22,13 @@ def test_list_users_in_group_usecase_user_not_allowed(self):
         with pytest.raises(ForbiddenAction):
             usecase(group=GROUPS.GAIA, request_user_role=ROLE.USER, request_user_groups=[GROUPS.GAIA])
 
+    def test_list_users_in_group_usecase_user_not_in_group(self):
+        repo = UserRepositoryMock()
+        usecase = ListUsersInGroupUsecase(repo)
+
+        with pytest.raises(ForbiddenAction):
+            usecase(group=GROUPS.JUNDIAI, request_user_role=ROLE.ADMIN_COLLABORATOR, request_user_groups=[GROUPS.GAIA])
+
 
     def test_list_users_in_group_usecase_with_user_not_in_group(self):
         repo = UserRepositoryMock()