commercetools / commercetools-paypal-plus-integration

Integration between commercetools API and PayPal Plus API
Apache License 2.0
7 stars 1 forks source link

104 separate integration test #126

Closed andrii-kovalenko-ct closed 6 years ago

andrii-kovalenko-ct commented 6 years ago

Addressing issue #104 (partially): Re-factor unit/integration tests run. Many files changed because most of them just renamed (moved to another directory).

Real changes:

The rest changes are minor renaming

butenkor commented 6 years ago

And also good work on reducing number of runners!

andrii-kovalenko-ct commented 6 years ago

@butenkor

And also good work on reducing number of runners!

Are you talking about SpringRunner? If so - it is an alias to SpringJUnit4ClassRunner, it is just shorter

heshamMassoud commented 6 years ago

@andrii-kovalenko-ct can you please also update this section in the README as it seems it's quite outdated after the directory structure changes of the tests: https://github.com/commercetools/commercetools-paypal-plus-integration/tree/104-separate-integration-test#integration-tests

andrii-kovalenko-ct commented 6 years ago

@heshamMassoud

can you please also update this section in the README as it seems it's quite outdated after the directory structure changes of the tests:

The documentation is temporary separated, so updated in dedicated branch: https://github.com/commercetools/commercetools-paypal-plus-integration/commit/18e30afba50569f308d46f3bf2c6886f4658d367

andrii-kovalenko-ct commented 6 years ago

closes #104