Neither PaymentAccessControlHandler nor PaymentMethodAccessControlHandler have any test coverage.

This is hiding bugs, and making work on #2842579: Payments should gracefully handle their gateway disappearing. tedious.

CommentFileSizeAuthor
#2 3087921-2-payment-access-tests.patch10.42 KBbojanz

Comments

bojanz created an issue. See original summary.

bojanz’s picture

bojanz commented
Status: Active » Needs review
StatusFileSize
new 3087921-2-payment-access-tests.patch10.42 KB

Naturally, the tests found two bugs along the way.

bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Committed.

  • bojanz committed e984022 on 8.x-2.x 
    Issue #3087921: There is no test coverage for payment's access control...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.