comgate-payments / sdk-php

90%

master: 90%

LAST BUILD ON BRANCH fix/payment-notification-json

branch: fix/payment-notification-json

CHANGE BRANCH

x

Reset

Sync Branches

• fix/payment-notification-json
• feature/functional-coverage-tests
• master

Build # 37613364678

Build Type

push

github

Committed by web-flow

Commit Message

Feature/functional coverage tests (#115)

* Add fixture-backed functional coverage for SDK workflows

* Remove internal coverage analysis report

* Make fixture router readable; drop env-mutating Cloudflare test

Rewrite tests/Support/Fixture/router.php for maintainability (named
helpers, sectioned routes, header comment explaining the fixture's
purpose) with identical behaviour - all fixture tests still pass.

Remove FixtureTransportCest::sendsCloudflareAccessHeaders and its
$_ENV unset in _after(): it mutated process-global $_ENV to force the
APPLICATION_ENV=sdk-github branch, which leaked across suites and broke
HelpersCest's env read.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

* Replace fixture HTTP server with in-process transport mocks

Drop the fixture API server (IntegrationApi + router.php) that booted a
real php -S process to return canned Comgate responses. Each functional
test now drives Client/ClientTerminal through a small FakeTransport
double that queues its own responses and records what the SDK sent.

- Add tests/Support/FakeTransport.php (ITransport double)
- Rewrite the fixture Cests as *FunctionalCest using per-test mocks
- MOTO test owns its RSA key pair and decrypts what the SDK sent
- Remove the transport-level and download-subprocess tests that could
  not be meaningfully mocked

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

2135 of 2363 relevant lines covered (90.35%)

8.12 hits per line