Skip to content

test(e2e): harden chat checks and CI gating#119

Open
rominf wants to merge 1 commit into
mainfrom
test/e2e-follow-up-pr69
Open

test(e2e): harden chat checks and CI gating#119
rominf wants to merge 1 commit into
mainfrom
test/e2e-follow-up-pr69

Conversation

@rominf

@rominf rominf commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • require chat responses to contain a non-empty choices array
  • apply the existing inference timeout to tool-calling requests
  • run heavy CI when expectations.toml changes

Follow-up to #69, now rebased onto main.

Risk: low — this only tightens E2E assertions and CI selection.

Test plan

  • cargo fmt --all -- --check
  • cargo clippy -p e2e-cucumber --all-targets -- -D warnings
  • cargo test -p e2e-cucumber --lib
  • pre-push hooks: cargo clippy, cargo test, YAML and hygiene checks

@rominf rominf requested a review from fredespi July 15, 2026 15:48
Base automatically changed from test/add-e2e-robot-framework to main July 15, 2026 15:49
Signed-off-by: Roman Inflianskas <Roman.Inflianskas@amd.com>
@rominf rominf force-pushed the test/e2e-follow-up-pr69 branch from a11f4cb to dec0f4b Compare July 15, 2026 16:18
@rominf rominf marked this pull request as ready for review July 15, 2026 16:24
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