mirror of
https://github.com/agentscope-ai/ReMe.git
synced 2026-08-28 05:25:04 +00:00
* refactor: derive dialog paths from session directory * fix: normalize configured session paths * fix: align dialog watch paths with writers * fix: reject absolute session directories
149 lines
6.4 KiB
Python
149 lines
6.4 KiB
Python
"""Unit tests for session-chunk merging via ``merge_session_chunk_intervals``.
|
|
|
|
Session chunks (``*.jsonl`` under the dialog dir) whose line ranges overlap,
|
|
contain one another, or are adjacent are merged into their union by the caller
|
|
(``merge_session_chunk_intervals``) before rendering with
|
|
``render_chunk_entries`` + ``join_chunk_entries``. Bodies here are plain
|
|
(non-``Msg``) text lines, which pass through stripped — letting these tests
|
|
assert the union content and line order directly.
|
|
"""
|
|
|
|
from reme.schema import FileChunk
|
|
from reme.steps.index._source_format import join_chunk_entries, merge_session_chunk_intervals, render_chunk_entries
|
|
|
|
_SESSION_DIR = "session"
|
|
|
|
|
|
def _render(chunks: list[FileChunk], session_dir: str, **kwargs) -> str:
|
|
"""Merge session chunks, render entries, then join — mirroring the search steps."""
|
|
return join_chunk_entries(
|
|
render_chunk_entries(merge_session_chunk_intervals(chunks, session_dir), session_dir, **kwargs),
|
|
)
|
|
|
|
|
|
def _chunk(start: int, end: int, text: str, score: float = 1.0, path: str = "session/dialog/s1.jsonl") -> FileChunk:
|
|
return FileChunk(path=path, start_line=start, end_line=end, text=text, scores={"score": score})
|
|
|
|
|
|
def test_overlapping_session_chunks_merge_into_union_without_duplicates():
|
|
"""Overlapping ranges collapse to one passage; the shared line is shown once, in order."""
|
|
a = _chunk(1, 3, "m1\nm2\nm3\n")
|
|
b = _chunk(3, 5, "m3\nm4\nm5\n")
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\nm4\nm5"
|
|
|
|
|
|
def test_contained_session_chunk_is_absorbed_by_the_larger_range():
|
|
"""When one range fully contains another, only the union (the larger) is shown."""
|
|
big = _chunk(1, 5, "m1\nm2\nm3\nm4\nm5\n")
|
|
small = _chunk(2, 4, "m2\nm3\nm4\n")
|
|
|
|
answer = _render([big, small], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\nm4\nm5"
|
|
|
|
|
|
def test_adjacent_session_chunks_merge_end_plus_one_equals_next_start():
|
|
"""Gap-free consecutive chunks (prev.end + 1 == next.start) merge into one union."""
|
|
a = _chunk(1, 3, "m1\nm2\nm3\n")
|
|
b = _chunk(4, 6, "m4\nm5\nm6\n")
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\nm4\nm5\nm6"
|
|
|
|
|
|
def test_session_chunks_with_a_gap_are_not_merged():
|
|
"""A missing line between ranges (start > end + 1) keeps the chunks separate."""
|
|
a = _chunk(1, 3, "m1\nm2\nm3\n")
|
|
b = _chunk(5, 6, "m5\nm6\n") # line 4 missing -> not adjacent
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\n\nm5\nm6"
|
|
|
|
|
|
def test_session_chunks_from_different_files_are_not_merged():
|
|
"""Overlapping ranges in different session files must stay separate."""
|
|
a = _chunk(1, 3, "A1\nA2\nA3\n", path="session/dialog/s1.jsonl")
|
|
b = _chunk(2, 4, "B2\nB3\nB4\n", path="session/dialog/s2.jsonl")
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "A1\nA2\nA3\n\nB2\nB3\nB4"
|
|
|
|
|
|
def test_non_session_chunks_are_never_merged():
|
|
"""Non-transcript chunks (not ``*.jsonl`` under the dialog dir) pass through untouched."""
|
|
a = _chunk(1, 3, "M1\nM2\nM3\n", path="daily/a.md")
|
|
b = _chunk(2, 4, "M2\nM3\nM4\n", path="daily/a.md")
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "M1\nM2\nM3\n\nM2\nM3\nM4"
|
|
|
|
|
|
def test_merge_preserves_line_order_regardless_of_input_rank_order():
|
|
"""A later, higher-ranked chunk does not reorder union content; lines stay chronological."""
|
|
later = _chunk(3, 5, "m3\nm4\nm5\n", score=9.0)
|
|
earlier = _chunk(1, 3, "m1\nm2\nm3\n", score=1.0)
|
|
|
|
# Higher-scored later-range chunk is listed first (as a ranker would).
|
|
answer = _render([later, earlier], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\nm4\nm5"
|
|
|
|
|
|
def test_merged_header_spans_the_union_range_and_keeps_best_score():
|
|
"""With source headers, the merged entry reports the union range and the top score."""
|
|
a = _chunk(1, 3, "m1\nm2\nm3\n", score=2.0)
|
|
b = _chunk(3, 5, "m3\nm4\nm5\n", score=7.0)
|
|
|
|
answer = _render([a, b], _SESSION_DIR, include_source=True)
|
|
|
|
assert answer.count("==========") == 2 # exactly one header (open + close markers)
|
|
assert "session/dialog/s1.jsonl:1-5" in answer
|
|
assert "score=7.0000" in answer
|
|
|
|
|
|
def test_separate_intervals_in_same_file_stay_separate():
|
|
"""Two disjoint interval clusters in one file yield two merged units, ordered by line."""
|
|
a = _chunk(1, 2, "m1\nm2\n")
|
|
b = _chunk(3, 4, "m3\nm4\n") # adjacent to a -> merges with a into 1-4
|
|
c = _chunk(10, 11, "m10\nm11\n") # far away -> separate
|
|
|
|
answer = _render([a, b, c], _SESSION_DIR, include_source=False)
|
|
|
|
assert answer == "m1\nm2\nm3\nm4\n\nm10\nm11"
|
|
|
|
|
|
def test_same_file_units_stay_adjacent_and_sorted_even_when_interleaved_by_rank():
|
|
"""Two disjoint units of one session file are grouped together and ordered by
|
|
``start_line``, even when a different file is ranked between them and the
|
|
lower interval was ranked last."""
|
|
s1_high = _chunk(10, 12, "S1x\nS1y\nS1z\n", score=9.0, path="session/dialog/s1.jsonl")
|
|
s2_mid = _chunk(1, 3, "S2a\nS2b\nS2c\n", score=5.0, path="session/dialog/s2.jsonl")
|
|
s1_low = _chunk(1, 3, "S1a\nS1b\nS1c\n", score=1.0, path="session/dialog/s1.jsonl")
|
|
|
|
# Rank order (as a ranker would emit, by score desc): s1[10-12], s2[1-3], s1[1-3].
|
|
answer = _render([s1_high, s2_mid, s1_low], _SESSION_DIR, include_source=False)
|
|
|
|
# s1's two units are adjacent and sorted by start_line (1-3 before 10-12),
|
|
# placed at s1's earliest rank (0), so the whole s1 block precedes s2.
|
|
assert answer == "S1a\nS1b\nS1c\n\nS1x\nS1y\nS1z\n\nS2a\nS2b\nS2c"
|
|
|
|
|
|
def test_same_file_units_adjacency_with_source_headers():
|
|
"""Header view: same-file units are contiguous and ascending; other files follow."""
|
|
s1_high = _chunk(10, 12, "S1x\nS1y\nS1z\n", score=9.0, path="session/dialog/s1.jsonl")
|
|
s2_mid = _chunk(1, 3, "S2a\nS2b\nS2c\n", score=5.0, path="session/dialog/s2.jsonl")
|
|
s1_low = _chunk(1, 3, "S1a\nS1b\nS1c\n", score=1.0, path="session/dialog/s1.jsonl")
|
|
|
|
answer = _render([s1_high, s2_mid, s1_low], _SESSION_DIR, include_source=True)
|
|
|
|
headers = [line for line in answer.splitlines() if line.startswith("==========")]
|
|
assert headers[0].split(" [")[0] == "========== session/dialog/s1.jsonl:1-3"
|
|
assert headers[1].split(" [")[0] == "========== session/dialog/s1.jsonl:10-12"
|
|
assert headers[2].split(" [")[0] == "========== session/dialog/s2.jsonl:1-3"
|