RESULTS-STATUS tells you if there are any non-"TEST-PASSED" results.
But it doesn't provide any way for you to get your hands on these
results. Originally, I thought that one could simply test for them, but
the results categories aren't actually exported from FiveAM, so this is
difficult.
To make it easier to find "interesting" results, I modified
RESULTS-STATUS to return a second value. The first value is a boolean,
as before, telling you if the tests all passed. If they didn't, the
second value is now a list of the non-passing results.
I believe this to be a benign change from the standpoint of backward
compatibility, since anyone just looking for a single return value will
get the same boolean value as before.
It's possible that the actual value --- not interpreted as a boolean ---
will be different, since the new code uses REMOVE-IF instead of EVERY.
But if a caller takes information out of the results list in a function
whose documentation proclaims it to be boolean, I submit that the caller
is doing so at his/her own risk!
Proposed patch is attached.
Best,
Robert
