fix: Return 204 response

lablup · Jan 6, 2025 · b1f2bef · b1f2bef
1 parent de9d0a3
commit b1f2bef
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/ai/backend/manager/api/container_registry.py b/src/ai/backend/manager/api/container_registry.py
@@ -53,7 +53,7 @@ async def associate_with_group(request: web.Request, params: Any) -> web.Respons
         except IntegrityError:
             raise GenericBadRequest("Association already exists.")
 
-    return web.json_response({})
+    return web.Response(status=204)
 
 
 @server_status_required(READ_ALLOWED)
@@ -81,7 +81,7 @@ async def disassociate_with_group(request: web.Request, params: Any) -> web.Resp
         if result.rowcount == 0:
             raise ContainerRegistryNotFound()
 
-    return web.json_response({})
+    return web.Response(status=204)
 
 
 def create_app(

diff --git a/tests/manager/api/test_container_registries.py b/tests/manager/api/test_container_registries.py
@@ -102,7 +102,7 @@ async def test_associate_container_registry_with_group(
     if association_exist:
         assert resp.status == 400
     else:
-        assert resp.status == 200
+        assert resp.status == 204
 
 
 @pytest.mark.asyncio
@@ -153,6 +153,6 @@ async def test_disassociate_container_registry_with_group(
     association_exist = "association_container_registries_groups" in extra_fixtures
 
     if association_exist:
-        assert resp.status == 200
+        assert resp.status == 204
     else:
         assert resp.status == 404