Printed message of the result of each testcase/testmethod? is hard to find when browsing through the test result. Example below:
```
:
[ INFO] [1487343289.092146469]: Constructing FaceDetector.
[ INFO] [1487343289.537363058]: You must subscribe to one of FaceDetector's outbound topics or else it will not publish anything.
Error opening rosout log file '/root/.ros/07335a6c-f521-11e6-a2a7-0242ac110002/rosout.log': No such file or directoryError opening rosout log file '/root/.ros/07335a6c-f521-11e6-a2a7-0242ac110002/rosout.log': No such file or directoryWaiting for service /wide_stereo/driver/set_parameters...
hztest_withface_/face_detector/faces_cloud ... ok
[ROSTEST]-----------------------------------------------------------------------
:
```
(This example is easier.)
This PR makes the result line(s) stand out more, also text-searchable.
* Added size option to rosclean purge.
When --size option is specified, it will delete old log files until log
directory reaches specified size (in MB), but never below specified
size. Otherwise, it will delete everything inside log directory.
* Changed help message and restored deleted comment.
* Cleaned up the patch to reflect comments.
* Updated functions based on comments.
- Fixed typo
- Removed _check_user_input and check_delete_file_size functions
* Changed logic from keeping minimum to keeping maximum.
* remove unnecessary parenthesis
* check size before asking, use 1024 instead of 1000
This except-and-raise pattern hides the actual error messages and leaves
us with a generic string "cannot create test results..." instead.
This makes it much harder to track down bugs.
On the other hand the exception text "[Errno 13] Permission denied: 'foobar'"
alone does not tell you that "foobar" is supposed to be the directory for the
tests results.
Thus I fused both exception texts. Because the error message in case of
missing permissions is (given above) "Permission denied", I removed the
"Please check permissions." hint.
I suppose this got moved at some point
or that part of the package.xml was copied over?
Either way, I just spend 5 minutes looking for this package
in the wrong repository...
This fixes a linker issue that can arise on some build environments:
ld: warning: librospack.so, needed by /opt/ros/indigo/lib/libroslib.so, not found (try using -rpath or -rpath-link)
/opt/ros/indigo/lib/libroslib.so: undefined reference to `rospack::ROSPack::run(std::string const&)'
collect2: error: ld returned 1 exit status
The xml output of rosunit has changed such that it no longer
"pretty-prints" the output. These changes update the expected
results to match the actual output of rosunit. Note that these
tests pass even with the currently incorrect serialization of
CDATA described in #112.
The changes in #89 to use an XML framework to generate test reports
introduced a bug caused by variable name collisions in the
_print_error() method. This resolves that bug and corrects the
docstring of the method.
Billy Lando