Reputation: 36001
I have some junit tests which create some resources which should also be closed.
One way to implement this logic is using the @Before
and @After
approach.
What I did was to encapsulate the creation in some utility class to be reused. For example:
class UserCreatorTestUtil implements AutoClosable {
User create() {...}
void close() {...}
}
The whole point is for the object to close itself, rather than needing to remember to close it in @After
.
The usage should be:
@Test
void test() {
try (UserCreatorTestUtil userCreatorTestUtil = new UserCreatorTestUtil()) {
User user = userCreatorTestUtil.create();
// Do some stuff regarding the user's phone
Assert.assertEquals("123456789", user.getPhone());
}
}
The problem is that junit's assert keyword throws an Error
- not Exception
.
Will the try-with-resource "catch" the Error
and invoke the close method?
* Couldn't find the answer in the try-with-resources documentation.
Upvotes: 55
Views: 6856
Reputation: 44965
The pseudo-code of a basic try-with-resources statement is (cf Java Language Specification §14.20.3.1
):
final VariableModifiers_minus_final R Identifier = Expression;
Throwable #primaryExc = null;
try ResourceSpecification_tail
Block
catch (Throwable #t) {
#primaryExc = #t;
throw #t;
} finally {
if (Identifier != null) {
if (#primaryExc != null) {
try {
Identifier.close();
} catch (Throwable #suppressedExc) {
#primaryExc.addSuppressed(#suppressedExc);
}
} else {
Identifier.close();
}
}
}
As you can see it catches Throwable
not Exception
which includes Error
but only to get the primary exception in order to add as suppressed exceptions any exceptions that occurred while closing the resources.
You can also notice that your resources are closed in the finally
block which means that they will be closed whatever happens (except in case of a System.exit
of course as it terminates the currently running Java Virtual Machine) even in case an Error
or any sub class of Throwable
is thrown.
Upvotes: 38
Reputation: 31851
The idea behind try-with-resources
is to make sure that the resources should be closed.
The problem with conventional try-catch-finally
statements is that let's suppose your try
block throws an exception; now usually you'll handle that exception in finally
block.
Now suppose an exception occurs in finally block as well. In such a case, the exception thrown by try catch is lost and the exception generated in finally
block gets propagated.
try {
// use something that's using resource
// e.g., streams
} catch(IOException e) {
// handle
} finally {
stream.close();
//if any exception occurs in the above line, than that exception
//will be propagated and the original exception that occurred
//in try block is lost.
}
In try-with-resources
the close()
method of the resource will get automatically called, and if the close()
throws any exception, the rest of the finally
isn't reached, and the original exception is lost.
Contrast that with this:
try (InputStream inputStream= new FileInputStream("C://test.txt")){
// ... use stream
} catch(IOException e) {
// handle exception
}
in the above code snippet, the close()
method automatically gets called and if that close()
method also generated any exception, than that exception will automatically get suppressed.
See also: Java Language Specification 14.20.3
Upvotes: 8
Reputation: 140457
Misconception on your end: try-with-resources does not do a catch.
It does a final finally, therefore the kind of "problem" doesn't matter.
See the JLS for further information!
Upvotes: 4
Reputation: 140318
Try-with-resources don't catch anything in and of themselves.
However, you can attach a catch
block to the end of the try-with-resources block, to catch whatever types of Throwable
you like:
try (UserCreatorTestUtil userCreatorTestUtil = new UserCreatorTestUtil()) {
// ... Whatever
} catch (RuntimeException e) {
// Handle e.
} catch (Exception | Throwable t) {
// Handle t.
}
Upvotes: 13
Reputation: 262534
It does not catch
anything. But it does finally
close all resources.
finally
blocks are run even when an Error is thrown.
Upvotes: 73