ISPN-4029 Reworked NotifyingFutureTest

Hi

Please take a look at reworked NotificationFutureTest. These are the steps I performed

1. Removed Thread.sleep() since it can't be trusted (especially when testing on CI server with many slaves)
2. It turned out that all testDone tests were acting exactly the same without Thread.sleep. So I combined them.
3. I reworked the test a bit and added custom assertions.

Best regards
Sebastian
Hmmm, I see what you are doing here. You're applying a timeout globally. Not sure about this. I'd rather see individual wait's being timed unless you have a very good idea on how long the entire test should take. The other problem I have is that if it the entire test class times out, how do you know at which point did it exactly stopped progressing? If individual parts are timed, e.g. waiting for a particular future, you know exactly at which point the timeout happened.