e4ae12dc75
- 移除了文件系统和沙盒路由器的过时集成测试。 - 更新了文件系统路由器测试，确保正确的身份验证和线程所有权检查。 - 调整了沙盒端到端测试，以改进附件处理和命令执行。 - 修改了设置路由器测试，对未授权访问返回404而不是401/403。 - 增强了任务路由器测试，以处理精简模式场景并确保任务创建和查询按预期工作。 - 优化了查看器文件系统测试，使用附件上传代替直接文件写入。 - 改进了技能后端测试中的错误处理，为越界访问引发适当的异常。
110 lines
3.9 KiB
Python
110 lines
3.9 KiB
Python
"""Integration tests for viewer-oriented filesystem router endpoints."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import uuid
|
|
|
|
import pytest
|
|
|
|
pytestmark = [pytest.mark.asyncio, pytest.mark.integration]
|
|
|
|
|
|
def _get_provider():
|
|
from yuxi.agents.backends import get_sandbox_provider
|
|
|
|
return get_sandbox_provider()
|
|
|
|
|
|
async def _create_thread_for_user(test_client, headers: dict[str, str]) -> str:
|
|
agents_resp = await test_client.get("/api/chat/agent", headers=headers)
|
|
assert agents_resp.status_code == 200, agents_resp.text
|
|
agents = agents_resp.json().get("agents", [])
|
|
if not agents:
|
|
pytest.skip("No agents available for viewer filesystem integration tests.")
|
|
|
|
agent_id = agents[0].get("id")
|
|
if not agent_id:
|
|
pytest.skip("Agent payload missing id field.")
|
|
|
|
create_resp = await test_client.post(
|
|
"/api/chat/thread",
|
|
json={
|
|
"agent_id": agent_id,
|
|
"title": f"viewer-filesystem-test-{uuid.uuid4().hex[:8]}",
|
|
"metadata": {},
|
|
},
|
|
headers=headers,
|
|
)
|
|
assert create_resp.status_code == 200, create_resp.text
|
|
payload = create_resp.json()
|
|
thread_id = payload.get("thread_id") or payload.get("id")
|
|
assert thread_id, f"Create thread response missing thread identifier: {payload}"
|
|
return thread_id
|
|
|
|
|
|
async def _upload_attachment_file(test_client, thread_id: str, headers: dict[str, str], file_name: str, content: str) -> str:
|
|
response = await test_client.post(
|
|
f"/api/chat/thread/{thread_id}/attachments",
|
|
files={"file": (file_name, content.encode("utf-8"), "text/plain")},
|
|
headers=headers,
|
|
)
|
|
assert response.status_code == 200, response.text
|
|
|
|
payload = response.json()
|
|
path = payload.get("path") or payload.get("file_path")
|
|
assert isinstance(path, str) and path
|
|
return path
|
|
|
|
|
|
async def test_viewer_tree_requires_authentication(test_client):
|
|
response = await test_client.get("/api/viewer/filesystem/tree", params={"thread_id": "x", "path": "/"})
|
|
assert response.status_code == 401
|
|
|
|
|
|
async def test_viewer_tree_root_lists_user_data_namespace(test_client, standard_user):
|
|
headers = standard_user["headers"]
|
|
thread_id = await _create_thread_for_user(test_client, headers)
|
|
|
|
response = await test_client.get(
|
|
"/api/viewer/filesystem/tree",
|
|
params={"thread_id": thread_id, "path": "/"},
|
|
headers=headers,
|
|
)
|
|
assert response.status_code == 200, response.text
|
|
|
|
entries = response.json().get("entries", [])
|
|
paths = {entry.get("path") for entry in entries}
|
|
assert "/home/yuxi/user-data/" in paths
|
|
|
|
|
|
async def test_viewer_file_returns_raw_content_without_line_numbers(test_client, standard_user):
|
|
headers = standard_user["headers"]
|
|
thread_id = await _create_thread_for_user(test_client, headers)
|
|
file_path = await _upload_attachment_file(test_client, thread_id, headers, "viewer_demo.txt", "alpha\nbeta\n")
|
|
|
|
response = await test_client.get(
|
|
"/api/viewer/filesystem/file",
|
|
params={"thread_id": thread_id, "path": file_path},
|
|
headers=headers,
|
|
)
|
|
assert response.status_code == 200, response.text
|
|
assert "alpha" in response.json()["content"]
|
|
assert "beta" in response.json()["content"]
|
|
|
|
|
|
async def test_viewer_download_returns_attachment_response(test_client, standard_user):
|
|
headers = standard_user["headers"]
|
|
thread_id = await _create_thread_for_user(test_client, headers)
|
|
file_path = await _upload_attachment_file(test_client, thread_id, headers, "download_demo.txt", "download-me\n")
|
|
|
|
response = await test_client.get(
|
|
"/api/viewer/filesystem/download",
|
|
params={"thread_id": thread_id, "path": file_path},
|
|
headers=headers,
|
|
)
|
|
assert response.status_code == 200, response.text
|
|
content_disposition = response.headers.get("content-disposition", "")
|
|
assert "attachment;" in content_disposition
|
|
assert "download_demo" in content_disposition
|
|
assert "download-me" in response.text
|