Skip to content

test: drive tests through @intx/inference-testing, add TEI e2e suite - #11

Merged
TheGreatAxios merged 3 commits into
cl-9155-reranking-replace-spread-built-request-options-with-explicitfrom
cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference
Sep 27, 2026
Merged

TheGreatAxios merged 3 commits into
cl-9155-reranking-replace-spread-built-request-options-with-explicitfrom
cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Stacked on refactor!: default rerank deps and pass request options as a literal #10.
  • Replaces the hand-rolled fetch fake with the @intx/inference-testing harness.
  • One wire test per apiStyle asserts path, body and parsed results; 429-then-200 backoff is asserted on the harness clock. Retry-After parser edge cases stay as unit tests.
  • tests/tei.e2e.test.ts ranks three docs against a real TEI /rerank (TEI_RERANK_URL, default http://localhost:8080) via bun run test:e2e, skipping when unreachable.

Verification

  • bun install --frozen-lockfile, bunx tsc --noEmit, bun test ./src, bunx prettier --check ., bun run build pass.
  • bun run test:e2e skips cleanly: no TEI server available locally.

Closes CL-9190
Closes CL-9200
Closes CL-9196

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-review: diff checked against ticket outcomes; independent critique pass run, findings on README wording, registry example and CONTRIBUTING commands fixed. Typecheck, tests, prettier and build pass.

@TheGreatAxios
TheGreatAxios force-pushed the cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference branch from 72c56e2 to 80ab77b Compare September 25, 2026 14:09
@TheGreatAxios
TheGreatAxios force-pushed the cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference branch 2 times, most recently from 68c1c72 to c21e0a4 Compare September 25, 2026 17:31
@TheGreatAxios
TheGreatAxios force-pushed the cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference branch from c21e0a4 to e6f581f Compare September 26, 2026 01:30
The hand-rolled fetch fake is replaced by the inference-testing harness:
one wire test per apiStyle asserts path, body and parsed results, and
429 backoff is asserted on the harness clock. Unit tests of the internal
Retry-After extractor are dropped.

tests/tei.e2e.test.ts ranks three documents against a real TEI /rerank
(TEI_RERANK_URL, default http://localhost:8080) and skips when it is
unreachable. Run with bun run test:e2e.
@TheGreatAxios
TheGreatAxios force-pushed the cl-9190-reranking-replace-hand-rolled-fetch-fake-with-intxinference branch from e6f581f to feb1847 Compare September 26, 2026 01:30
@TheGreatAxios TheGreatAxios changed the title Drive tests through @intx/inference-testing, add TEI e2e test: drive tests through @intx/inference-testing, add TEI e2e suite Sep 26, 2026
@TheGreatAxios
TheGreatAxios added this pull request to stack #15 September 26, 2026 01:59
@TheGreatAxios
TheGreatAxios merged commit 22d2b2a into main Sep 27, 2026
1 check passed
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.

1 participant