Skip to content

test(model): isolate custom-provider grouping tests from live discovery (salvages #62421) - #67921

Merged
teknium1 merged 1 commit into
mainfrom
salvage/picker-test-hermeticity
Jul 20, 2026
Merged

test(model): isolate custom-provider grouping tests from live discovery (salvages #62421)#67921
teknium1 merged 1 commit into
mainfrom
salvage/picker-test-hermeticity

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Three custom-provider grouping tests are now deterministic on hosts running a real Ollama at localhost:11434. Salvages #62421 (@virtualex-itv) onto current main with authorship preserved.

Changes

  • tests/hermes_cli/test_model_switch_custom_providers.py: probe_custom_providers=False on the three grouping tests whose fixtures point at http://localhost:11434/v1 — without it, a live Ollama's catalog replaces the fixture models and the tests assert against the host's install instead of the grouping invariant. Dedicated live-probe tests are unchanged.

Validation

Check Result
scripts/run_tests.sh tests/hermes_cli/test_model_switch_custom_providers.py 47/47 pass

Infographic

picker-test-hermeticity

@alt-glitch alt-glitch added type/test Test coverage or test infrastructure P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation needs-decision Awaiting maintainer decision before any implementation labels Jul 20, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Related to open #62421. This is a maintainer salvage of the same three-test hermeticity fix, so it is not marked duplicate; please choose the branch to merge.

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

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard needs-decision Awaiting maintainer decision before any implementation P3 Low — cosmetic, nice to have sweeper:risk-automation Sweeper risk: may affect CI, automerge, label sync, or maintainer automation type/test Test coverage or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants