Skip to content

test(openai): Deduplicate by removing node.callspec.id matching#6427

Open
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/openai/remove-node-callspec
Open

test(openai): Deduplicate by removing node.callspec.id matching#6427
alexander-alderman-webb wants to merge 1 commit into
masterfrom
webb/openai/remove-node-callspec

Conversation

@alexander-alderman-webb
Copy link
Copy Markdown
Contributor

@alexander-alderman-webb alexander-alderman-webb commented May 27, 2026

Description

In Chat Completions tests, jointly parametrize on input messages and the expected value of the gen_ai.system_instructions attribute.

In Responses tests, jointly parametrize on instructions, input and the expected value of the gen_ai.system_instructions and gen_ai.request.messages attributes. Only keep one test case for omit and None system instructions.

In Embeddings tests, jointly parametrize on input and the expected value of the gen_ai.embeddings.input attribute.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title test(openai): Deduplicate by removing node.callspec.id matching test(openai): Deduplicate by removing node.callspec.id matching May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

Codecov Results 📊

228 passed | Total: 228 | Pass Rate: 100% | Execution Time: 35.21s

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 14769 uncovered lines.


Generated by Codecov Action

@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review May 27, 2026 10:33
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner May 27, 2026 10:33
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