Only list failed tests #6619
Only list failed tests #6619
Conversation
…me_includes_initial_messages_and_sends_prior_items
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "content": [{ "type": "input_text", "text": "resumed user message" }] | ||
| }, | ||
| { | ||
| "type": "message", | ||
| "type": "message1", | ||
| "role": "assistant", |
There was a problem hiding this comment.
In resume_includes_initial_messages_and_sends_prior_items the expected request body now looks for an assistant item with "type": "message1", but the client code always serializes both user and assistant conversation items with "type": "message" (there is no message1 variant anywhere in the API). This typo makes the assertion fail on every run, so the test suite can no longer pass and the regression the test guards is no longer exercised.
Useful? React with 👍 / 👎.
Makes output easier to parse