-
Notifications
You must be signed in to change notification settings - Fork 5
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
Automated tests #8
Comments
I'd like to ask if you need matplotlib charts inside the python scripts located in the debug or tests directories. |
Here is how a test looks like: Note 3 functions at the end of the script with the test_ prefix. |
Probably two scripts are needed:
Is the 3D matplotlib chart needed for debugging? What else is needed for debugging. I can add the required stuff to the generator of the scripts for debugging. |
A script like this example is OK for me. I use all the debugging stuff in a development version of bpypolyskel.py, where I am able to visualize the intermediate results of the process. The only thing I need is the execution of polygonize() with the test data. |
I suggest generating a ready to debug Python script that doesn't require any editing or copy and pasting. Could you please provide an example script? |
I do not need any edit using your example. The only thing that would simplify things a little bit would be the possibility to start it from the test folder. But then I end always in the ImportError "attempted relative import with no known parent package", when trying to import from the parent folder. But don't do too much. Finding the bug is normally a matter of hours, I do not win much time when I don't need to move the script. |
The following works for me: It means executing the script as a Python module from the top-level directory of bpypolyskel library. |
OK, I see what you mean. But I like to execute it in my development environment (I use VS Code), so that I can debug it in detail. There I could not find a similar soltution. But don't care, I have no problem as it is. |
Added more automated tests. |
Add the automated tests for two special case: rectangle and square. |
Finished the documentation for testing: |
I am going to introduce automated tests. That will require some changes in python scripts located in the debug or tests directories.
The text was updated successfully, but these errors were encountered: