fix(tests): update tool delete test to expect association cleanup execute call

delete_tool now calls db.execute twice: once for server_tool_association
cleanup and once for the tool DELETE. The test assertion was stale after
commit 5be84b4 added the association cleanup step.

Signed-off-by: Madhav Kandukuri <madhav165@gmail.com>

Author: madhav165

tests/unit/mcpgateway/services/test_resource_ownership.py

@@ -285,8 +285,8 @@ async def test_delete_tool_owner_success(self, tool_service, mock_db_session):
 
             await tool_service.delete_tool(mock_db_session, "tool-1", user_email="owner@example.com")
 
-            # Verify execute was called for DELETE ... RETURNING
-            mock_db_session.execute.assert_called_once()
+            # Verify execute was called for association cleanup + tool DELETE
+            assert mock_db_session.execute.call_count == 2
             mock_db_session.commit.assert_called_once()
 
     @pytest.mark.asyncio