From 515949fd31ff3f7238f0ceb217b115128a3b7267 Mon Sep 17 00:00:00 2001
From: Stefan Bodewig <bodewig@apache.org>
Date: Wed, 1 Dec 2004 07:45:11 +0000
Subject: [PATCH] Please don't modify my working copy during tests, otherwise
 I'm going to commit changed files by accident

git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@277086 13f79535-47bb-0310-9956-ffa450edef68
---
 src/etc/testcases/taskdefs/recorder.xml       | 19 +++++++-----
 .../tools/ant/taskdefs/RecorderTest.java      | 30 ++++++++++++++-----
 2 files changed, 34 insertions(+), 15 deletions(-)

diff --git a/src/etc/testcases/taskdefs/recorder.xml b/src/etc/testcases/taskdefs/recorder.xml
index 6bbec57b7..18829ff47 100644
--- a/src/etc/testcases/taskdefs/recorder.xml
+++ b/src/etc/testcases/taskdefs/recorder.xml
@@ -1,20 +1,25 @@
 <?xml version="1.0"?>
 <project name="recorder-test" default="all" basedir=".">
 
-    <property name="recdir" location="recorder"/>
+    <property name="recin" location="recorder"/>
+    <property name="recdir" location="recorder-out"/>
 
     <target name="all" depends="noappend,append,restart,deleterestart"/>
 
     <target name="prepare">
-        <fixcrlf srcdir="${recdir}" includes="*.result"/>
+      <mkdir dir="${recdir}"/>
+      <copy toDir="${recdir}">
+        <fileset dir="${recin}"/>
+      </copy>
+      <fixcrlf srcdir="${recdir}"/>
     </target>
 
-
     <target name="noappend">
         <copy file="${recdir}/rectest2.result" tofile="${recdir}/rectest1.log"/>
         <record name="${recdir}/rectest1.log" action="start" />
         <echo message="some message1"/>
         <record name="${recdir}/rectest1.log" action="stop" />
+        <fixcrlf srcdir="${recdir}" includes="*.log"/>
     </target>
 
     <target name="append">
@@ -22,6 +27,7 @@
         <record name="${recdir}/rectest2.log" append="true" action="start"/>
         <echo message="some message2"/>
         <record name="${recdir}/rectest2.log" action="stop"/>
+        <fixcrlf srcdir="${recdir}" includes="*.log"/>
     </target>
 
     <target name="restart">
@@ -32,6 +38,7 @@
         <record name="${recdir}/rectest3.log" action="start"/>
         <echo message="some message3"/>
         <record name="${recdir}/rectest3.log" action="stop"/>
+        <fixcrlf srcdir="${recdir}" includes="*.log"/>
     </target>
 
     <target name="deleterestart">
@@ -43,14 +50,12 @@
         <record name="${recdir}/rectest4.log" action="start"/>
         <echo message="some message3"/>
         <record name="${recdir}/rectest4.log" action="stop"/>
+        <fixcrlf srcdir="${recdir}" includes="*.log"/>
     </target>
 
 
     <target name="cleanup">
-        <delete file="${recdir}/rectest1.log"/>
-        <delete file="${recdir}/rectest2.log"/>
-        <delete file="${recdir}/rectest3.log"/>
-        <delete file="${recdir}/rectest4.log"/>
+        <delete dir="${recdir}"/>
     </target>
 
 </project>
diff --git a/src/testcases/org/apache/tools/ant/taskdefs/RecorderTest.java b/src/testcases/org/apache/tools/ant/taskdefs/RecorderTest.java
index 760317f96..c0296ed23 100644
--- a/src/testcases/org/apache/tools/ant/taskdefs/RecorderTest.java
+++ b/src/testcases/org/apache/tools/ant/taskdefs/RecorderTest.java
@@ -27,6 +27,8 @@ import java.io.IOException;
  */
 public class RecorderTest extends BuildFileTest {
 
+    private static final String REC_DIR = "recorder-out";
+
     public RecorderTest(String name) {
         super(name);
     }
@@ -43,29 +45,41 @@ public class RecorderTest extends BuildFileTest {
     public void testNoAppend() throws IOException {
         FileUtils fileUtils = FileUtils.newFileUtils();
         executeTarget("noappend");
-        assertTrue(fileUtils.contentEquals(project.resolveFile("recorder/rectest1.result"),
-                                           project.resolveFile("recorder/rectest1.log")));
+        assertTrue(fileUtils
+                   .contentEquals(project.resolveFile(REC_DIR 
+                                                      + "rectest1.result"),
+                                  project.resolveFile(REC_DIR 
+                                                      + "rectest1.log")));
     }
 
     public void testAppend() throws IOException {
         FileUtils fileUtils = FileUtils.newFileUtils();
         executeTarget("append");
-        assertTrue(fileUtils.contentEquals(project.resolveFile("recorder/rectest2.result"),
-                                           project.resolveFile("recorder/rectest2.log")));
+        assertTrue(fileUtils
+                   .contentEquals(project.resolveFile(REC_DIR 
+                                                      + "rectest2.result"),
+                                  project.resolveFile(REC_DIR 
+                                                      + "rectest2.log")));
     }
 
     public void testRestart() throws IOException {
         FileUtils fileUtils = FileUtils.newFileUtils();
         executeTarget("restart");
-        assertTrue(fileUtils.contentEquals(project.resolveFile("recorder/rectest3.result"),
-                                           project.resolveFile("recorder/rectest3.log")));
+        assertTrue(fileUtils
+                   .contentEquals(project.resolveFile(REC_DIR 
+                                                      + "rectest3.result"),
+                                  project.resolveFile(REC_DIR 
+                                                      + "rectest3.log")));
     }
 
     public void testDeleteRestart() throws IOException {
         FileUtils fileUtils = FileUtils.newFileUtils();
         executeTarget("deleterestart");
-        assertTrue(fileUtils.contentEquals(project.resolveFile("recorder/rectest4.result"),
-                                           project.resolveFile("recorder/rectest4.log")));
+        assertTrue(fileUtils
+                   .contentEquals(project.resolveFile(REC_DIR 
+                                                      + "rectest4.result"),
+                                  project.resolveFile(REC_DIR 
+                                                      + "rectest4.log")));
     }
 
 }