Cleaner test output - #58
Conversation
Fix a mixed-up between python's stdout and stderr and the c++ one. Improve the output of the tests for a better readability.
|
so much easier to read! 👏 |
|
|
||
| print("indices are") | ||
| print (c2.indices.value) | ||
| Sofa.Simulation.init(root) |
There was a problem hiding this comment.
not related to the PR right ? could you explain?
There was a problem hiding this comment.
It is related to this PR, there's no reason to print to stdout during tests, it just contribute to bloating the output.
We can print to stdout when we are debugging our tests, but we should remove them once we are ready to merge
| - jean-nicolas.brunet@inria.fr | ||
| - thierry.gaugry@inria.fr | ||
| ********************************************************************/ | ||
|
|
There was a problem hiding this comment.
as Damien said last time, let's not remove contributors mention (but we should put it after the license), since this would be illegal
sorry @jnbrunet I will work on ruling on this topic
There was a problem hiding this comment.
There was a duplicated copyright and contributors mention, I've removed the old one but forgot to adjust the contributors. It should now be ok
| - thierry.gaugry@inria.fr | ||
| ********************************************************************/ | ||
|
|
||
| /****************************************************************************** |
There was a problem hiding this comment.
same, but by the way, you can add your name if you'd like
3 participants
This PR makes it much more easier to read the output of unittests. It does so by:
Test output before this PR:
https://gist.github.com/jnbrunet/7fe9cd97f15a43ba67c7357dd1b929c9
Test output with this PR:
https://gist.github.com/jnbrunet/4e3f9dff27213002f6738f311fef59f4