Skip to content

Comments

Remove random suffix from raw_response and raw_usage tests#6311

Merged
shuyangli merged 2 commits intomainfrom
aaron/remove-random-suffix
Feb 13, 2026
Merged

Remove random suffix from raw_response and raw_usage tests#6311
shuyangli merged 2 commits intomainfrom
aaron/remove-random-suffix

Conversation

@Aaron1011
Copy link
Member

@Aaron1011 Aaron1011 commented Feb 12, 2026

This will let these tests use provider-proxy


Note

Low Risk
Test-only changes that make inputs deterministic; no production logic or security-sensitive code is modified.

Overview
Removes per-test random UUID suffixes from the user prompt content in the include_raw_response and include_raw_usage E2E suites, switching from format!(...) to fixed strings across chat-completions, responses, best-of-n, mixture-of-n, and DICL cases.

This makes the request payloads deterministic (and more compatible with proxying/caching behavior) without changing the assertions around raw response/usage inclusion.

Written by Cursor Bugbot for commit 9e72b68. This will update automatically on new commits. Configure here.

@shuyangli shuyangli added this pull request to the merge queue Feb 12, 2026
Merged via the queue into main with commit af0bd42 Feb 13, 2026
65 checks passed
@shuyangli shuyangli deleted the aaron/remove-random-suffix branch February 13, 2026 00:55
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