mirror of
https://github.com/bytedance/deer-flow.git
synced 2026-05-07 01:08:25 +00:00
d25c8d371f
Restructure the persistence layer from horizontal "models/ + repositories/"
split into vertical entity-aligned directories. Each entity (thread_meta,
run, feedback) now owns its ORM model, abstract interface (where applicable),
and concrete implementations under a single directory with an aggregating
__init__.py for one-line imports.
Layout:
persistence/thread_meta/{base,model,sql,memory}.py
persistence/run/{model,sql}.py
persistence/feedback/{model,sql}.py
models/__init__.py is kept as a facade so Alembic autogenerate continues to
discover all ORM tables via Base.metadata. RunEventRow remains under
models/run_event.py because its storage implementation lives in
runtime/events/store/db.py and has no matching repository directory.
The repositories/ directory is removed entirely. All call sites in
gateway/deps.py and tests are updated to import from the new entity
packages, e.g.:
from deerflow.persistence.thread_meta import ThreadMetaRepository
from deerflow.persistence.run import RunRepository
from deerflow.persistence.feedback import FeedbackRepository
Full test suite passes (1690 passed, 14 skipped).
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
133 lines
4.5 KiB
Python
133 lines
4.5 KiB
Python
"""Tests for ThreadMetaRepository (SQLAlchemy-backed)."""
|
|
|
|
import pytest
|
|
|
|
from deerflow.persistence.thread_meta import ThreadMetaRepository
|
|
|
|
|
|
async def _make_repo(tmp_path):
|
|
from deerflow.persistence.engine import get_session_factory, init_engine
|
|
|
|
url = f"sqlite+aiosqlite:///{tmp_path / 'test.db'}"
|
|
await init_engine("sqlite", url=url, sqlite_dir=str(tmp_path))
|
|
return ThreadMetaRepository(get_session_factory())
|
|
|
|
|
|
async def _cleanup():
|
|
from deerflow.persistence.engine import close_engine
|
|
|
|
await close_engine()
|
|
|
|
|
|
class TestThreadMetaRepository:
|
|
@pytest.mark.anyio
|
|
async def test_create_and_get(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
record = await repo.create("t1")
|
|
assert record["thread_id"] == "t1"
|
|
assert record["status"] == "idle"
|
|
assert "created_at" in record
|
|
|
|
fetched = await repo.get("t1")
|
|
assert fetched is not None
|
|
assert fetched["thread_id"] == "t1"
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_create_with_assistant_id(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
record = await repo.create("t1", assistant_id="agent1")
|
|
assert record["assistant_id"] == "agent1"
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_create_with_owner_and_display_name(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
record = await repo.create("t1", owner_id="user1", display_name="My Thread")
|
|
assert record["owner_id"] == "user1"
|
|
assert record["display_name"] == "My Thread"
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_create_with_metadata(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
record = await repo.create("t1", metadata={"key": "value"})
|
|
assert record["metadata"] == {"key": "value"}
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_get_nonexistent(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
assert await repo.get("nonexistent") is None
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_list_by_owner(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1", owner_id="user1")
|
|
await repo.create("t2", owner_id="user1")
|
|
await repo.create("t3", owner_id="user2")
|
|
results = await repo.list_by_owner("user1")
|
|
assert len(results) == 2
|
|
assert all(r["owner_id"] == "user1" for r in results)
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_list_by_owner_with_limit_and_offset(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
for i in range(5):
|
|
await repo.create(f"t{i}", owner_id="user1")
|
|
results = await repo.list_by_owner("user1", limit=2, offset=1)
|
|
assert len(results) == 2
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_check_access_no_record_allows(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
assert await repo.check_access("unknown", "user1") is True
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_check_access_owner_matches(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1", owner_id="user1")
|
|
assert await repo.check_access("t1", "user1") is True
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_check_access_owner_mismatch(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1", owner_id="user1")
|
|
assert await repo.check_access("t1", "user2") is False
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_check_access_no_owner_allows_all(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1") # owner_id=None
|
|
assert await repo.check_access("t1", "anyone") is True
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_update_status(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1")
|
|
await repo.update_status("t1", "busy")
|
|
record = await repo.get("t1")
|
|
assert record["status"] == "busy"
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_delete(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.create("t1")
|
|
await repo.delete("t1")
|
|
assert await repo.get("t1") is None
|
|
await _cleanup()
|
|
|
|
@pytest.mark.anyio
|
|
async def test_delete_nonexistent_is_noop(self, tmp_path):
|
|
repo = await _make_repo(tmp_path)
|
|
await repo.delete("nonexistent") # should not raise
|
|
await _cleanup()
|