Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: bulk tests #65

Merged
merged 12 commits into from
Feb 25, 2022
Merged

fix: bulk tests #65

merged 12 commits into from
Feb 25, 2022

Conversation

mdebarros
Copy link
Member

@mdebarros mdebarros commented Feb 25, 2022

  • split up bulk positive test assertions from using JSON.stringify which will fail since ordering of JSON elements are not guaranteed

Added the following test-case scenarios for patch notifications:
- p2p_money_transfer_patch_notifications - payee receives PATCH Notification with COMMITTED after sending callback with RESERVED - mojaloop/project#2676
- p2p_money_transfer_patch_notifications - payee receives PATCH Notification with ABORTED status after sending invalid fulfilment - mojaloop/project#2556
- invalid fsp-destination
- placeholder for invalid fulfilment
- added bugfix test-case for "Central-Ledger Fulfil Handler does not correctly invalidate requests with an incorrect/non-existent FSP-ID in the FSPIOP-Destination header #2697"
- updated "patch_notification" feature test
…es into feat/patch-notification-test-scenarios

# Conflicts:
#	collections/hub/golden_path/feature_tests/patch_notifications/patch_notification.json
- split up bulk positive test assertions from using JSON.stringify which will fail since ordering of JSON elements are not guaranteed
@mdebarros mdebarros self-assigned this Feb 25, 2022
@mdebarros mdebarros changed the title fix: bug fix for the last chore update on bulk handlers fix: bulk tests Feb 25, 2022
@mdebarros mdebarros merged commit d334538 into master Feb 25, 2022
@mdebarros mdebarros deleted the feat/patch-notification-test-scenarios branch August 16, 2022 11:55
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