Use explicit close rather than try-with-resources for AutoClosable it…

[rbygrave]

…erator

This way the close() is executed before the catch block rather than after

[SentryMan]

In a try-with-resources statement, any catch or finally block is run after the resources declared have been closed.

[rbygrave]

Ah right, thanks.

Yes, ultimately the failing tests were just down to the race conditions in those tests so I started incorrectly there.

Note that if we put it back to try-with-resources its a warning on the empty try block and wants a @SuppressWarnings("EmptyTryBlock") added so I reckon it might be prettier to just leave it like this.