SubmoduleStatusTest: Open TestRepository in try-with-resource

Change-Id: Iebb6abd35fa5b084a4c044e416a448785a3c9291
Signed-off-by: David Pursehouse <david.pursehouse@gmail.com>

org.eclipse.jgit.test/tst/org/eclipse/jgit/submodule/SubmoduleStatusTest.java

@@ -263,8 +263,10 @@ public class SubmoduleStatusTest extends RepositoryTestCase {
 				.getRepository();
 		assertNotNull(subRepo);
 
-		TestRepository<?> subTr = new TestRepository<>(subRepo);
-		ObjectId id = subTr.branch(Constants.HEAD).commit().create().copy();
+		ObjectId id;
+		try (TestRepository<?> subTr = new TestRepository<>(subRepo)) {
+			id = subTr.branch(Constants.HEAD).commit().create().copy();
+		}
 
 		DirCache cache = db.lockDirCache();
 		DirCacheEditor editor = cache.editor();
@@ -315,9 +317,8 @@ public class SubmoduleStatusTest extends RepositoryTestCase {
 				.getRepository();
 		assertNotNull(subRepo);
 
-		TestRepository<?> subTr = new TestRepository<>(subRepo);
+		try (TestRepository<?> subTr = new TestRepository<>(subRepo)) {
 			ObjectId id = subTr.branch(Constants.HEAD).commit().create().copy();
-
 			DirCache cache = db.lockDirCache();
 			DirCacheEditor editor = cache.editor();
 			editor.add(new PathEdit(path) {
@@ -336,22 +337,24 @@ public class SubmoduleStatusTest extends RepositoryTestCase {
 					ConfigConstants.CONFIG_KEY_URL, url);
 			config.save();
 
-		FileBasedConfig modulesConfig = new FileBasedConfig(new File(
-				db.getWorkTree(), Constants.DOT_GIT_MODULES), db.getFS());
-		modulesConfig.setString(ConfigConstants.CONFIG_SUBMODULE_SECTION, path,
-				ConfigConstants.CONFIG_KEY_PATH, path);
-		modulesConfig.setString(ConfigConstants.CONFIG_SUBMODULE_SECTION, path,
-				ConfigConstants.CONFIG_KEY_URL, url);
+			FileBasedConfig modulesConfig = new FileBasedConfig(
+					new File(db.getWorkTree(), Constants.DOT_GIT_MODULES),
+					db.getFS());
+			modulesConfig.setString(ConfigConstants.CONFIG_SUBMODULE_SECTION,
+					path, ConfigConstants.CONFIG_KEY_PATH, path);
+			modulesConfig.setString(ConfigConstants.CONFIG_SUBMODULE_SECTION,
+					path, ConfigConstants.CONFIG_KEY_URL, url);
 			modulesConfig.save();
 
-		ObjectId newId = subTr.branch(Constants.HEAD).commit().create().copy();
+			ObjectId newId = subTr.branch(Constants.HEAD).commit().create()
+					.copy();
 
 			SubmoduleStatusCommand command = new SubmoduleStatusCommand(db);
 			Map<String, SubmoduleStatus> statuses = command.call();
 			assertNotNull(statuses);
 			assertEquals(1, statuses.size());
-		Entry<String, SubmoduleStatus> module = statuses.entrySet().iterator()
-				.next();
+			Entry<String, SubmoduleStatus> module = statuses.entrySet()
+					.iterator().next();
 			assertNotNull(module);
 			assertEquals(path, module.getKey());
 			SubmoduleStatus status = module.getValue();
@@ -362,3 +365,4 @@ public class SubmoduleStatusTest extends RepositoryTestCase {
 			assertEquals(SubmoduleStatusType.REV_CHECKED_OUT, status.getType());
 		}
 	}
+}