Skip to content

PR: test: supplement coverage for LLM env resolution and control paths - #20

Merged
metasmile merged 3 commits into
mainfrom
15-ante-aux
Sep 6, 2026
Merged

PR: test: supplement coverage for LLM env resolution and control paths#20
metasmile merged 3 commits into
mainfrom
15-ante-aux

Conversation

@metasmile

Copy link
Copy Markdown
Contributor

Final coverage supplement for the actus session work (issues #15-#19).

Scope

  • Extract the LLM endpoint resolution (env precedence, quote stripping, empty defaults) from main.rs into agent::config::resolve_llm_settings/unquote_env_value and cover it with unit tests, so the issue actus: rename hardcoded deepseek references to OpenAI-compatible API naming #16 rename semantics (env-driven provider/model/base URL, no vendor defaults) are pinned.
  • Assert the injected language_models.openai_compatible entry values in the settings-schema test (neutral label, base URL, model).
  • Extend the ext_cli adapter tests: agent-wide cancel kills all running turns, cancel-after-completion is a no-op, the launch probe accepts a hanging binary within the grace window, and status reports a binary removed after construction.
  • Extend the HTTP control tests: policy-denied controller cancel, and human submits without identity never record a parent thread.

Status

Verification

  • cargo test: 109 passed, 0 failed
  • ./run.sh --test: all checks passed (telos connected, agent ready)
  • CI: the standard test workflow runs this branch head on push.

@metasmile metasmile self-assigned this Sep 6, 2026
@metasmile
metasmile merged commit a6d17db into main Sep 6, 2026
3 of 4 checks passed
@metasmile
metasmile deleted the 15-ante-aux branch September 6, 2026 17:29
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