search

braintree / braintree_python

Braintree Python library
https://developer.paypal.com/braintree/docs/start/overview
MIT License
241 stars 155 forks source link

Include tests in release #136

Closed cedk closed 2 years ago

cedk commented 2 years ago

It will be useful to have the tests included in the release archive published, this will help packager to ensure the package installation is correctly working by executing the tests (see https://bugs.gentoo.org/767256).

crookedneighbor commented 2 years ago

This is not going to do what you want it to do. The unit tests may be able to run under that system, but the integration tests will always fail as they rely on a specific version of the Braintree API that runs in development.

cedk commented 2 years ago

So what would you advise to run if we want to ensure that the installed package is working correctly?

crookedneighbor commented 2 years ago

I don't have any advice. I'm not familiar with the practices of the system y'all are using.

I just know that the integration tests in this repo will never pass upstream on your system, so it doesn't accomplish what you want it to. I'm going to close this for now, but happy to continue the conversation if you have thoughts about it.