Testing tools: RetryLoop swallows the interesting exception
-----------------------------------------------------------

                 Key: SLING-2430
                 URL: https://issues.apache.org/jira/browse/SLING-2430
             Project: Sling
          Issue Type: Improvement
          Components: Testing
            Reporter: Alexander Klimetschek
            Priority: Minor


The fail at the end of the RetryLoop() constructor does not report the last 
exception catches inside the loop, which is usually of interest. Instead, these 
are passed to reportException(), which turns out not really useful, since the 
fail will break through the outer code and the code would have to remember the 
exception, catch the AssertionError etc.

It might still make sense to report *all* inner exceptions in the loop, but a 
good default behavior should include the actual exception right away in the 
fail().

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to