From 5ba9db391f0cb9cc1212d53bca52cbae8b2c9399 Mon Sep 17 00:00:00 2001 From: weixsun Date: Thu, 13 May 2021 17:06:49 +0800 Subject: [PATCH] Use try-with-resources statements See gh-26449 --- .../boot/loader/jar/HandlerTests.java | 12 ++---------- .../SampleIntegrationParentApplicationTests.java | 16 ++++------------ 2 files changed, 6 insertions(+), 22 deletions(-) diff --git a/spring-boot-project/spring-boot-tools/spring-boot-loader/src/test/java/org/springframework/boot/loader/jar/HandlerTests.java b/spring-boot-project/spring-boot-tools/spring-boot-loader/src/test/java/org/springframework/boot/loader/jar/HandlerTests.java index 0c67f898279..0ab36d6e152 100644 --- a/spring-boot-project/spring-boot-tools/spring-boot-loader/src/test/java/org/springframework/boot/loader/jar/HandlerTests.java +++ b/spring-boot-project/spring-boot-tools/spring-boot-loader/src/test/java/org/springframework/boot/loader/jar/HandlerTests.java @@ -172,13 +172,9 @@ class HandlerTests { TestJarCreator.createTestJar(testJar); URL url = new URL(null, "jar:" + testJar.toURI().toURL() + "!/nested.jar!/3.dat", this.handler); JarURLConnection connection = (JarURLConnection) url.openConnection(); - JarFile jarFile = JarFileWrapper.unwrap(connection.getJarFile()); - try { + try (JarFile jarFile = JarFileWrapper.unwrap(connection.getJarFile())) { assertThat(jarFile.getRootJarFile().getFile()).isEqualTo(testJar); } - finally { - jarFile.close(); - } } @Test @@ -187,13 +183,9 @@ class HandlerTests { TestJarCreator.createTestJar(testJar); URL url = new URL(null, "jar:" + testJar.toURI().toURL() + "!/nested.jar!/3.dat", this.handler); JarURLConnection connection = (JarURLConnection) url.openConnection(); - JarFile jarFile = JarFileWrapper.unwrap(connection.getJarFile()); - try { + try (JarFile jarFile = JarFileWrapper.unwrap(connection.getJarFile())) { assertThat(jarFile.getRootJarFile().getFile()).isEqualTo(testJar); } - finally { - jarFile.close(); - } } private void assertStandardAndCustomHandlerUrlsAreEqual(String context, String spec) throws MalformedURLException { diff --git a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-parent-context/src/test/java/smoketest/parent/consumer/SampleIntegrationParentApplicationTests.java b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-parent-context/src/test/java/smoketest/parent/consumer/SampleIntegrationParentApplicationTests.java index 00557f959d4..d40ba6e91fa 100644 --- a/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-parent-context/src/test/java/smoketest/parent/consumer/SampleIntegrationParentApplicationTests.java +++ b/spring-boot-tests/spring-boot-smoke-tests/spring-boot-smoke-test-parent-context/src/test/java/smoketest/parent/consumer/SampleIntegrationParentApplicationTests.java @@ -49,20 +49,12 @@ class SampleIntegrationParentApplicationTests { void testVanillaExchange(@TempDir Path temp) throws Exception { File inputDir = new File(temp.toFile(), "input"); File outputDir = new File(temp.toFile(), "output"); - ConfigurableApplicationContext app = SpringApplication.run(SampleParentContextApplication.class, - "--service.input-dir=" + inputDir, "--service.output-dir=" + outputDir); - try { - ConfigurableApplicationContext producer = SpringApplication.run(ProducerApplication.class, - "--service.input-dir=" + inputDir, "--service.output-dir=" + outputDir, "World"); - try { + try (ConfigurableApplicationContext app = SpringApplication.run(SampleParentContextApplication.class, + "--service.input-dir=" + inputDir, "--service.output-dir=" + outputDir)) { + try (ConfigurableApplicationContext producer = SpringApplication.run(ProducerApplication.class, + "--service.input-dir=" + inputDir, "--service.output-dir=" + outputDir, "World")) { awaitOutputContaining(outputDir, "Hello World"); } - finally { - producer.close(); - } - } - finally { - app.close(); } }