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.

Sign up for GitHub

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

Jump to bottom

chore(processor) - add unit test cases for adyen-payment service #50

Merged
merged 14 commits into from
Mar 21, 2024
Merged

chore(processor) - add unit test cases for adyen-payment service #50

merged 14 commits into from
Mar 21, 2024

Conversation

leungkinghin-ct
Copy link
Contributor

@leungkinghin-ct leungkinghin-ct commented Mar 12, 2024 •
edited
Loading

  • Migrate payment service unit test case from template repo to adyen repo
  • Add unit test cases for payment refund/capture/create, create session, get payment methods
  • Fix status response for Adyen payment
  • Enhance error-handler.spec.ts

@leungkinghin-ct leungkinghin-ct changed the title Add unit test cases for adyen-payment service chore(processor) - add unit test cases for adyen-payment service Mar 12, 2024
@leungkinghin-ct leungkinghin-ct marked this pull request as draft March 12, 2024 16:05
@leungkinghin-ct leungkinghin-ct marked this pull request as ready for review March 19, 2024 12:48
@leungkinghin-ct leungkinghin-ct requested review from a team as code owners March 19, 2024 12:48
@leungkinghin-ct leungkinghin-ct removed the request for review from dario-commercetools March 21, 2024 08:01
@leungkinghin-ct leungkinghin-ct merged commit ee7c450 into main Mar 21, 2024
2 checks passed
@leungkinghin-ct leungkinghin-ct deleted the kinghin.leung/add-test-cases branch March 21, 2024 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dasanorct dasanorct dasanorct approved these changes

@prateek-ct prateek-ct prateek-ct approved these changes

@ddeliziact ddeliziact Awaiting requested review from ddeliziact

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@leungkinghin-ct @prateek-ct @dasanorct