Skip to content

test(payments): re-enable bearer token integ tests (AgentCredentialProviderService fixed)#502

Merged
jariy17 merged 1 commit into
mainfrom
revert-bearer-token-skip
Jun 2, 2026
Merged

test(payments): re-enable bearer token integ tests (AgentCredentialProviderService fixed)#502
jariy17 merged 1 commit into
mainfrom
revert-bearer-token-skip

Conversation

@aidandaly24
Copy link
Copy Markdown
Contributor

Summary

Reverts #499, which added @pytest.mark.skip to three bearer-token (CUSTOM_JWT) integration tests while the AgentCredentialProviderService was returning 500s:

  • TestPaymentManagerBearerTokenAuth::test_bearer_token_real_api_call
  • TestPaymentManagerBearerTokenAuth::test_token_provider_real_api_call
  • TestPaymentManagerBearerTokenAuth::test_bearer_create_session_real_api_call

The underlying service issue is now resolved, so the skips are no longer needed.

Why it's safe to re-enable

The original failure was a service-side defect, not a test or SDK bug: CreatePaymentSession / ListPaymentInstruments with a valid Cognito bearer token against a READY CUSTOM_JWT Payment Manager returned InternalServerException: AgentCredentialProviderServiceException (the post-validation JWT→workload-identity exchange was 5xx-ing). It was reproducible across two independent accounts in us-west-2 and began ~May 29.

Verification that it's fixed:

  • Live repro now succeeds — a fresh CUSTOM_JWT manager + Cognito client_credentials token returns a valid paymentSessionId, with the service correctly deriving userId from the JWT claim.
  • CI is green — the last several Secure Integration test runs on main pass, including the payments job, with no code changes.

Change

Pure revert of #499 — removes the three @pytest.mark.skip decorators. No other changes.

@jariy17 jariy17 merged commit 69954eb into main Jun 2, 2026
35 checks passed
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