Skip to content

test: fix CI drift in web plugin and update hangup tests#29224

Open
camaragon wants to merge 1 commit into
NousResearch:mainfrom
camaragon:fix/ci-test-drift
Open

test: fix CI drift in web plugin and update hangup tests#29224
camaragon wants to merge 1 commit into
NousResearch:mainfrom
camaragon:fix/ci-test-drift

Conversation

@camaragon

Copy link
Copy Markdown
Contributor

Summary

  • updates bundled web plugin tests for the registered xai web provider
  • makes the update hangup stream assertion robust across hermes_cli.main reloads in shared test workers

Verification

  • python3 -m pytest tests/plugins/web/test_web_search_provider_plugins.py tests/hermes_cli/test_update_hangup_protection.py -q -o addopts=''
  • python3 -m pytest tests/hermes_cli/test_env_loader.py tests/hermes_cli/test_update_hangup_protection.py -q -o addopts=''
  • python3 -m ruff check tests/plugins/web/test_web_search_provider_plugins.py tests/hermes_cli/test_update_hangup_protection.py
  • git diff --check upstream/main...HEAD

@wesleysimplicio

Copy link
Copy Markdown
Contributor

Confirmed this matches the current baseline failures I saw on #29616: the bundled web-search provider registry now includes xai, and the update hangup stdio wrapper assertion can see a reloaded hermes_cli.main module in the larger suite. This PR is clean and directly covers both without adding another duplicate fix.

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

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have tool/web Web search and extraction type/test Test coverage or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants