On 21/08/2019 18:30, Alan Bateman wrote:
Daniel - if there are other changes to this test then it might be worth re-examining the ensureXXX methods. The naming is very strange as they return a boolean. Also they print messages to say that the test is skipped but it's the caller that will determinate that, not the ensureXXX methods.
We could make those two methods return void and throw the SkipException themselves? I don't mind doing that change in this patch.
Nick
