Skip to content

chore(internal): Improve auth integration tests#10635

Closed
jsklan wants to merge 31 commits intomainfrom
jsklan/update-any-auth-spec
Closed

chore(internal): Improve auth integration tests#10635
jsklan wants to merge 31 commits intomainfrom
jsklan/update-any-auth-spec

Conversation

@jsklan
Copy link
Copy Markdown
Contributor

@jsklan jsklan commented Nov 17, 2025

Description

Before, the any-auth test-definition assumed that

  • request.grant_type parameter was string literal (can be enum)
  • response.expires_in property was required (can be optional)

This updates the test definition to handle these cases.
Also adds any-auth to allowed-failures for typescript since this test change demonstrates that it is broken.

Changes Made

  • GrantType Enum: Changed grant_type from hardcoded string literal to proper enum type with values authorization_code, refresh_token, and client_credentials across Go, Java, and Python SDKs
  • Optional ExpiresIn: Made TokenResponse.expiresIn optional instead of required in all language SDKs
  • TypeScript Allowed Failures: Added any-auth fixture to TypeScript SDK allowed failures list

Testing

  • Updated seed test fixtures for Go, Java, and Python SDKs
  • Verified all code examples and documentation reflect new type signatures
  • Manual testing completed

@jsklan jsklan marked this pull request as ready for review November 17, 2025 21:56
@jsklan jsklan closed this Nov 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants