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

🐸Split unit and integration test commands #82

Merged
merged 1 commit into from
Dec 4, 2019

Conversation

dhet
Copy link

@dhet dhet commented Oct 10, 2019

Addresses #47

@heshamMassoud
Copy link
Contributor

🐸

@lojzatran lojzatran changed the title Split unit and integration test commands 🐸Split unit and integration test commands Oct 11, 2019
@dhet
Copy link
Author

dhet commented Oct 13, 2019

Hopefully this PR will be merged by Hacktoberfest 2023 🙏

@LEQADA
Copy link
Contributor

LEQADA commented Oct 13, 2019

@dhet thanks for PR. The PR will be counted by Hacktoberfest even if it's not merged by 2023 😉

@butenkor butenkor requested review from lojzatran and LEQADA October 15, 2019 09:31
@ahmetoz ahmetoz changed the base branch from master to update-first-transaction-type December 4, 2019 10:16
@ahmetoz ahmetoz merged commit 3c35e06 into Adyen:update-first-transaction-type Dec 4, 2019
ahmetoz added a commit that referenced this pull request Dec 10, 2019
…ing to the adyen best practices. (#85)

* add details for checkout steps.

* update Charge to Authorization.

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: andreas Halberkamp <andreas.halberkamp@commercetools.de>

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: andreas Halberkamp <andreas.halberkamp@commercetools.de>

* Link integration guide with Refund/Cancel guide #71

* additional unit test for the manual capture cases

* add additional integration test for manual capture notification

* Split unit and integration test commands (#82)

* bump version to 3.0.0

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: Roman Butenko <roman.butenko@commercetools.de>

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: Roman Butenko <roman.butenko@commercetools.de>

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: Roman Butenko <roman.butenko@commercetools.de>

* Update extension/docs/IntegrationGuide.md

Co-Authored-By: Roman Butenko <roman.butenko@commercetools.de>

* Update notification/test/unit/notification.handler.spec.js

Co-Authored-By: Hasan <LEQADA@users.noreply.github.com>

* add quotes for the adyen events.

* add nyc to integration tests.

* revert changes for test command.

* add explanation for why originalReference should use the AUTHORIZATION notification event.

* make cancel or refund documentation more clear.

* handle the cancel or refund notification correctly on notification module.
@renovate renovate bot mentioned this pull request Nov 27, 2021
1 task
@renovate renovate bot mentioned this pull request May 5, 2023
1 task
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.

4 participants