test: add comprehensive structured output support coverage#1140
Merged
homanp merged 1 commit intoApr 11, 2026
Merged
Conversation
Add 37 unit tests verifying structured output (responseFormat) behavior across all supported providers and model combinations. Covers: OpenAI, Anthropic, Google, Bedrock, Vercel, Groq, Fireworks, OpenRouter (with sub-provider variants), and OpenAI-compatible. Tests both positive cases (responseFormat passed) and negative cases (responseFormat omitted) to ensure the supportsStructuredOutput logic is correct for each provider/model combination. Relates to superagent-ai#1111
|
@frankentini is attempting to deploy a commit to the Superagent Team on Vercel. A member of the Team first needs to authorize it. |
Collaborator
|
@frankentini thanks! |
richardokonicha
pushed a commit
to fugoku/superagent
that referenced
this pull request
May 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds 37 unit tests covering the
supportsStructuredOutputlogic across all supported providers and model combinations. Previously only 2 tests existed for this surface (OpenAI + one Groq negative case).What's tested
All tests use the existing mock pattern (vi.mock callProvider) and verify whether
responseFormatis passed as the 3rd argument.Motivation
Relates to #1111 — moving test coverage toward mocked unit tests. The structured output routing logic is complex with many provider-specific branches, and was previously under-tested.
Test results
All 37 new tests pass. No changes to source code.