From ee90d6afba1946345386efcee8b14a017e0eb01c Mon Sep 17 00:00:00 2001
From: Matthias Sohn <matthias.sohn@sap.com>
Date: Mon, 2 Apr 2012 01:20:31 +0200
Subject: [PATCH] Fix tests for Ant task "git-clone"

If project.init() isn't called GitCloneTaskTest fails when started from
Eclipse, according to [1] calling init() is necessary to properly
initialize the Ant project programmatically.

Always set the destination folder in order to ensure that all test
resources are created under the project's target folder and do not
pollute the project's source tree with test data.

[1]
http://ant.1045680.n5.nabble.com/project-createTask-not-working-with-ant-1-8-2-td3385716.html

Change-Id: Icbeb62680b018a92673faa58828b5e850564c7a8
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
---
 .../org/eclipse/jgit/ant/tasks/GitCloneTaskTest.java  | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/org.eclipse.jgit.ant.test/src/org/eclipse/jgit/ant/tasks/GitCloneTaskTest.java b/org.eclipse.jgit.ant.test/src/org/eclipse/jgit/ant/tasks/GitCloneTaskTest.java
index c7b949c25..760d0dce6 100644
--- a/org.eclipse.jgit.ant.test/src/org/eclipse/jgit/ant/tasks/GitCloneTaskTest.java
+++ b/org.eclipse.jgit.ant.test/src/org/eclipse/jgit/ant/tasks/GitCloneTaskTest.java
@@ -45,6 +45,7 @@ package org.eclipse.jgit.ant.tasks;
 import static org.junit.Assert.assertTrue;
 
 import java.io.File;
+import java.io.IOException;
 
 import org.apache.tools.ant.BuildException;
 import org.apache.tools.ant.DefaultLogger;
@@ -60,13 +61,17 @@ public class GitCloneTaskTest extends LocalDiskRepositoryTestCase {
 
 	private GitCloneTask task;
 	private Project project;
+	private File dest;
 
 	@Before
-	public void before() {
+	public void before() throws IOException {
 		project = new Project();
+		project.init();
 		enableLogging();
 		project.addTaskDefinition("git-clone", GitCloneTask.class);
 		task = (GitCloneTask) project.createTask("git-clone");
+		dest = createTempFile();
+		task.setDest(dest);
 	}
 
 	@Test(expected = BuildException.class)
@@ -97,8 +102,6 @@ public class GitCloneTaskTest extends LocalDiskRepositoryTestCase {
 		FileRepository repo = createBareRepository();
 		File directory = repo.getDirectory();
 		task.setUri("file://" + directory);
-		File dest = createTempFile();
-		task.setDest(dest);
 		task.execute();
 
 		assertTrue(RepositoryCache.FileKey.isGitRepository(new File(dest, ".git"), FS.DETECTED));
@@ -110,8 +113,6 @@ public class GitCloneTaskTest extends LocalDiskRepositoryTestCase {
 		File directory = repo.getDirectory();
 		task.setUri("file://" + directory);
 		task.setBare(true);
-		File dest = createTempFile();
-		task.setDest(dest);
 		task.execute();
 
 		assertTrue(RepositoryCache.FileKey.isGitRepository(dest, FS.DETECTED));