fix(connector): stop cashtocode, tokenio and zen declaring manual capture

[abhisheksharma2411]

Type of Change

• Bugfix

Description

Three connectors advertise a capture method they cannot perform. Each lists CaptureMethod::Manual in supported_capture_methods while its ConnectorIntegration<Capture, ..>::build_request returns FlowNotSupported:

connector	Capture flow	declares Manual
cashtocode	Err(FlowNotSupported { flow: "Capture" })	yes
tokenio	Err(FlowNotSupported { flow: "Capture" })	yes
zen	Err(FlowNotSupported { flow: "Capture" })	yes

validate_connector_against_payment_request decides purely on membership of that list:

payment_method_type_info.supported_capture_methods.contains(&capture_method)

So a manual-capture payment passes validation, authorizes, and then cannot be captured. The failure arrives after the customer has already paid, instead of when the request is validated — which is the wrong end of the transaction for a capability error.

SequentialAutomatic is kept for all three, for the same reason as #14292: a charge never reports Authorized, so no follow-up capture call is made for it.

How this was found

Reviewing #14292, which fixes exactly this for Coinbase. Scanning the connector directory for the same shape — a Capture impl returning FlowNotSupported alongside a CaptureMethod::Manual declaration — returned four connectors: coinbase (covered by #14292), and these three. That is the complete set; every other connector whose Capture flow is unsupported already omits Manual.

Credit to @errmakov for #14292 — this is that fix applied to the rest of the set, deliberately mirroring its shape rather than inventing a different one.

Testing

Each connector gets the regression test from #14292, asserting through validate_connector_against_payment_request rather than reading the constant, so the guard survives a refactor of the list:

running 6 tests
test connectors::cashtocode::tests::rejects_manual_capture_because_capture_flow_is_not_supported ... ok
test connectors::tokenio::tests::rejects_manual_capture_because_capture_flow_is_not_supported ... ok
test connectors::zen::tests::rejects_manual_capture_because_capture_flow_is_not_supported ... ok
test connectors::*::tests::accepts_automatic_and_sequential_automatic_capture ... ok
test result: ok. 6 passed; 0 failed

Watched them fail first. Re-adding CaptureMethod::Manual to all three lists:

test result: FAILED. 3 passed; 3 failed

so each test fails for its own connector rather than passing on someone else's behaviour.

cargo fmt --all -- --check clean, cargo clippy -p hyperswitch_connectors --features "v1,payouts,frm,revenue_recovery" clean.

Not included

No change to the webhook-flow declarations. #14292 also drops EventClass::Refunds for Coinbase; I checked these three separately and did not find the same mismatch, so that half is deliberately not copied across.

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/hyperswitch_connectors/src/connectors/cashtocode.rs	2% smaller
crates/hyperswitch_connectors/src/connectors/zen.rs	2% smaller
crates/hyperswitch_connectors/src/connectors/tokenio.rs	2% smaller