Skip to content

Commit 0f1b0ca

Browse files
committed
Updated test case for gateway response and resolved ruff and flake8 issue
Signed-off-by: <Keval Mahajan> <[email protected]>
1 parent d3443ed commit 0f1b0ca

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

mcpgateway/admin.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -763,7 +763,6 @@ async def admin_add_gateway(request: Request, db: Session = Depends(get_db), use
763763
auth_header_key=form.get("auth_header_key", ""),
764764
auth_header_value=form.get("auth_header_value", ""),
765765
)
766-
root_path = request.scope.get("root_path", "")
767766
try:
768767
await gateway_service.register_gateway(db, gateway)
769768
return JSONResponse(
@@ -781,7 +780,6 @@ async def admin_add_gateway(request: Request, db: Session = Depends(get_db), use
781780
return JSONResponse(content={"message": str(ex), "success": False}, status_code=500)
782781

783782

784-
785783
@admin_router.post("/gateways/{gateway_id}/edit")
786784
async def admin_edit_gateway(
787785
gateway_id: int,

tests/unit/mcpgateway/test_admin.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -561,7 +561,7 @@ async def test_admin_add_gateway(self, mock_register_gateway, mock_request, mock
561561

562562
# Assert
563563
mock_register_gateway.assert_called_once()
564-
assert isinstance(result, RedirectResponse) #JSONResponse
564+
assert isinstance(result, JSONResponse)
565565
assert result.status_code == 303
566566
assert "/admin#gateways" in result.headers["location"]
567567

0 commit comments

Comments
 (0)