Skip to content

test: cover reranker basics#1716

Open
snoopuppy582 wants to merge 1 commit into
MemTensor:mainfrom
snoopuppy582:test/reranker
Open

test: cover reranker basics#1716
snoopuppy582 wants to merge 1 commit into
MemTensor:mainfrom
snoopuppy582:test/reranker

Conversation

@snoopuppy582
Copy link
Copy Markdown

Fixes #1697

Summary

  • Added a dedicated tests/reranker package.
  • Covered reranker base interfaces, factories, noop and cosine-local behavior.
  • Added deterministic strategy tests for dialogue tracking, single-turn reconstruction, out-memory aggregation, background concatenation, and document-source concatenation.

Tests

  • uv run --python 3.12 --extra mem-reader --with pytest pytest tests/reranker -q
  • uv run --python 3.12 --extra mem-reader --with ruff ruff check tests/reranker
  • git diff --check

Notes

  • Kept the tests deterministic and local-only; HTTP reranker calls are not made.
  • The contributing guide mentions a dev base branch, but this repository currently exposes main as the active/default base branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests for reranker strategies

1 participant