-
Notifications
You must be signed in to change notification settings - Fork 16
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 tests and/or provide test running documentation #17
Comments
Most of the tests should probably live in compactnesslib, since that provides the underlying functionality to the library. We could either duplicate some of those tests here, or just target the interface between the two for testing. |
Cool! I get how we could duplicate tests. I don't understand the other option yet. Do you have any advice on how we would target the interface? Maybe a best practice blog post, or a link to an exemplary project? I haven't worked on any integration between Python and native extensions before, so I'm all about the learns :) |
I'm not sure what the best way forward with this is either, other than that any critical testing of the internal calculations should probably be in compactnesslib rather than the wrapper. If folks have test ideas, one option might be to write them in Python and I could then backport them to compactnesslib. |
Would it be valuable for us to duplicate a happy-path subset of the c++ lib's tests in Python? It would permit the benefit of some quality assurance on each pull request to python-mander without the burden of duplicating every test in the C++ lib. On the other hand, if python-mander is going to be a simple auto-generated thin wrapper around compactnesslib then those tests wouldn't bring much value because they'd really be testing whatever auto-generates the Python based on compactnesslib. |
Sorry, I just glanced up to your older post and saw that you are on board:
|
As mentioned in #14 and #16, I haven't been able to execute the tests successfully. To ensure continuous code quality, it would be awesome if we could accomplish some combination of:
The text was updated successfully, but these errors were encountered: