Skip to content

feat(http): allow injecting Guzzle handler; assert customer id on the wire [BIL-1509]#2

Open
fjalil7 wants to merge 1 commit into
mainfrom
feature/bil-1509
Open

feat(http): allow injecting Guzzle handler; assert customer id on the wire [BIL-1509]#2
fjalil7 wants to merge 1 commit into
mainfrom
feature/bil-1509

Conversation

@fjalil7
Copy link
Copy Markdown

@fjalil7 fjalil7 commented Jun 1, 2026

Summary

  • HttpClient constructor now accepts an optional ?HandlerStack $handler for testing
  • Wire-level tests added (tests/CustomersTest.php) using MockHandler + Middleware::history to capture the real wire body
  • PHP already sent id correctly — this adds the regression test and the handler seam

Test plan

  • vendor/bin/phpunit — 43 tests pass
  • Wire-level test captures the Guzzle request body after convertKeys (camelCase) and asserts billingEmail and id are present

Closes BIL-1509

@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 1, 2026

BIL-1509

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.

1 participant