Noticed a bug with test obj dirs not getting de...

Noticed a bug with test obj dirs not getting deleted. No review.

1 files changed, 6 insertions, 13 deletions

diff --git a/src/partest/scala/tools/partest/Housekeeping.scala b/src/partest/scala/tools/partest/Housekeeping.scala
index fdc04de7e3..3a092a4782 100644
--- a/src/partest/scala/tools/partest/Housekeeping.scala
+++ b/src/partest/scala/tools/partest/Housekeeping.scala
@@ -133,25 +133,18 @@ trait Housekeeping {
       // recreate object dir
       outDir createDirectory true
     }
-    def deleteOutDir() =
-      if (isNoCleanup && outDir.isDirectory) debug("Not deleting " + outDir)
-      else outDir.deleteRecursively()
-
-    def cleanup() {
-      // otherwise, delete obj dir and logs on success
-      deleteOutDir()
-      if (isSuccess)
-        deleteLog()
-    }
+    def deleteOutDir() = outDir.deleteRecursively()
 
     protected def runWrappers[T](body: => T): Option[T] = {
       prepareForTestRun()
 
       withShutdownHook({ debug("Shutdown hook deleting " + outDir) ; deleteOutDir }) {
         loggingOutAndErr {
-          possiblyTimed {
-            body
-          }
+          val result = possiblyTimed { body }
+          if (!isNoCleanup)
+            deleteOutDir()
+
+          result
         }
       }
     }