From d02f98c63530454700661fc80354d66207ed8b02 Mon Sep 17 00:00:00 2001
From: Joas Schilling <nickvergessen@gmx.de>
Date: Tue, 11 Jun 2013 16:03:34 +0200
Subject: [PATCH] [ticket/11605] Use empty_dir to better delete files and dirs
 of extensions

PHPBB3-11605
---
 .../phpbb_test_case_helpers.php               | 29 ++++---------------
 1 file changed, 6 insertions(+), 23 deletions(-)

diff --git a/tests/test_framework/phpbb_test_case_helpers.php b/tests/test_framework/phpbb_test_case_helpers.php
index 1b08f2368f..8c91d357b8 100644
--- a/tests/test_framework/phpbb_test_case_helpers.php
+++ b/tests/test_framework/phpbb_test_case_helpers.php
@@ -54,14 +54,18 @@ class phpbb_test_case_helpers
 	{
 		global $phpbb_root_path;
 
+		// Remove all of the files we copied from test ext -> board ext
+		$this->empty_dir($phpbb_root_path . 'ext/');
+
 		// Copy back the board installed extensions from the temp directory
 		if (file_exists($phpbb_root_path . 'store/temp_ext/'))
 		{
 			$this->copy_dir($phpbb_root_path . 'store/temp_ext/', $phpbb_root_path . 'ext/');
+
+			// Remove all of the files we copied from board ext -> temp_ext
+			$this->empty_dir($phpbb_root_path . 'store/temp_ext/');
 		}
 
-		// Remove all of the files we copied around (from board ext -> temp_ext, from test ext -> board ext)
-		$this->remove_files($this->copied_files);
 		$this->copied_files = array();
 
 		if (file_exists($phpbb_root_path . 'store/temp_ext/'))
@@ -254,27 +258,6 @@ class phpbb_test_case_helpers
 		return $copied_files;
 	}
 
-	/**
-	* Remove files/directories that are listed in an array
-	* Designed for use with $this->copy_dir()
-	*
-	* @param array $file_list
-	*/
-	public function remove_files($file_list)
-	{
-		foreach ($file_list as $file)
-		{
-			if (is_dir($file))
-			{
-				rmdir($file);
-			}
-			else
-			{
-				unlink($file);
-			}
-		}
-	}
-
 	/**
 	* Empty directory (remove any subdirectories/files below)
 	*