Skip to content

Add tests for BalancePlatformWebhooksTest - #2018

Merged
poojah-adyen merged 1 commit into
mainfrom
configurationwebhooks-balanceplatformwebhook-tests
Jul 24, 2026
Merged

poojah-adyen merged 1 commit into
mainfrom
configurationwebhooks-balanceplatformwebhook-tests

Conversation

@poojah-adyen

Copy link
Copy Markdown
Contributor

Description
This PR adds unit test coverage for the Configuration Webhooks functionality introduced by the latest OpenAPI (Configuration Webhooks v2) model update — specifically the new recurring top-up webhooks (balancePlatform.balanceAccount.recurringTopUp.created and .updated) plus the Mandate.createdAt/updatedAt type change from Object to OffsetDateTime. It also strengthens coverage of the MandateBankAccountAccountIdentification and
PaymentInstrumentAdditionalBankAccountIdentificationsInner oneOf discriminator deserialization (ukLocal and iban mappings) by adding failure-path tests for invalid dates and unknown discriminator types. Mock payloads for the new webhooks are sourced directly from the official OpenAPI spec examples.

Tested scenarios

Fixed issue:

@poojah-adyen
poojah-adyen requested a review from a team as a code owner July 23, 2026 19:47

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds several unit tests to BalancePlatformWebhooksTest to verify the deserialization behavior of various webhook payloads, including mandate notifications with invalid dates, unknown account identification types, payment instruments with IBAN additional bank account identifications, and top-up configuration events (created, updated, and unknown types). It also introduces the corresponding mock JSON files for these tests. There are no review comments, and I have no feedback to provide.

@sonarqubecloud

Copy link
Copy Markdown

@poojah-adyen
poojah-adyen added this pull request to the merge queue Jul 24, 2026
Merged via the queue into main with commit 019f175 Jul 24, 2026
8 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