Skip to content

[OMEGA-429] Update channel mock suites for the V2 mock - #363

Open
TossSky wants to merge 1 commit into
singnet:MeTTaClaw2-vitalyfrom
sassless:OMEGA-429-channel-mock-v2
Open

TossSky wants to merge 1 commit into
singnet:MeTTaClaw2-vitalyfrom
sassless:OMEGA-429-channel-mock-v2

Conversation

@TossSky

@TossSky TossSky commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

The Slack, Telegram and WebSocket mock suites on the #349 branch still passed mock answers as V1 strings, which the V2 mock turns into an empty response, so their tests timed out.

  • channel tests pass answers as (tool, {param: value}) lists
  • LlmMockAgent.chat strips the Step <time>: prefix before matching, so prompts with a sender prefix match again
  • fixed the syntax error in test_skill_query_slack_mock.py and the import in test_slack_unwrap.py

Tested on an image built from this branch:

  • @run_mandatory: 153 passed
  • @run_optional: 5 passed, 1 skipped
  • mock_websocket/: 13 passed
  • mock_slack/: 32 passed, 1 skipped, 1 failed on a git clone timeout that passed on rerun
  • mock_telegram/: 23 passed, 1 skipped

@vsbogd

vsbogd commented Sep 24, 2026

Copy link
Copy Markdown
Member

@TossSky thanks for fixing it, actually I am not sure to which branch this PR should be raised. Maybe it is better to raise it into #358 it depends on which PR we are going to merge first probably. It depends on which ticket will go to the QA state first. Let this PR hang for some time.

@TossSky

TossSky commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator Author

@vsbogd
Agreed, let's leave it open until it's clear which PR goes first. I tested #349 and posted the results in a comment there.

This branch has not been deployed

No deployments
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.

2 participants