Adyen / adyen-openapi

OpenAPI specification for the Adyen APIs
MIT License
67 stars 50 forks source link

Notification examples do not adhere to defined schema #25

Open LucasRxTx opened 3 years ago

LucasRxTx commented 3 years ago

Describe the bug There are several notification examples that do not adhere to their defined schema.

For example: REPORT_AVAILABLE example notification does not comply with the declared schema for a ReportAvailableNotificationRequestItem. paymentMethod is stated to be required but the example is missing it. It is unclear what schema I should expect the NotificationRequestItem to be.

There are other errors in the schema definitions, for example for success values, though i have reported it as a separate issue: https://github.com/Adyen/adyen-openapi/issues/26

To Reproduce Look at the openAPI specification for a ReportAvailableNotificationRequestItem and see that paymentMethod is required. Look at the openAPI specification example response for REPORT_AVAILABLE and see the paymentMethod is not included in the example.

Expected behavior Both the actual notification, and the example notification should adhere to the declared schema for REPORT_AVAILABLE.

Screenshots None

Desktop (please complete the following information): irrelevant

Smartphone (please complete the following information): irrelevant

Additional context Using your example responses to create a mock server causes errors in our test system. We are now unsure of the actual expected schema for notifications.

LucasRxTx commented 3 years ago

This seems to be the case in other notifications as well

for example:

PPKFS commented 1 year ago

Also encountering this. Plus, the boolean fields in the schema are listed as type: boolean but they are encoded as strings over the enum "true", "false".