Extracted static TestHelper

tests/AbstractTestCase.php

@@ -13,7 +13,7 @@ abstract class AbstractTestCase extends \PHPUnit_Framework_TestCase
 
 	protected function tearDown()
 	{
-		$this->cleanTestDirectory();
+		TestHelper::cleanTestDirectory();
 	}
 
 	protected function mock($className)
@@ -21,32 +21,29 @@ abstract class AbstractTestCase extends \PHPUnit_Framework_TestCase
 		return $this->getMockBuilder($className)->disableOriginalConstructor()->getMock();
 	}
 
-	protected function mockConfig($dataPath = null, $publicDataPath = null)
-	{
-		return new ConfigMock($dataPath, $publicDataPath);
-	}
-
 	protected function mockTransactionManager()
 	{
 		return new TransactionManagerMock($this->mock(DatabaseConnection::class));
 	}
 
+	protected function mockConfig($dataPath = null, $publicDataPath = null)
+	{
+		return TestHelper::mockConfig($dataPath, $publicDataPath);
+	}
+
 	protected function createTestDirectory()
 	{
-		$path = $this->getTestDirectoryPath();
+		return TestHelper::createTestDirectory();
-		if (!file_exists($path))
-			mkdir($path, 0777, true);
-		return $path;
 	}
 
 	protected function getTestFile($fileName)
 	{
-		return file_get_contents($this->getTestFilePath($fileName));
+		return TestHelper::getTestFile($fileName);
 	}
 
 	protected function getTestFilePath($fileName)
 	{
-		return __DIR__ . DIRECTORY_SEPARATOR . 'test_files' . DIRECTORY_SEPARATOR . $fileName;
+		return TestHelper::getTestFilePath($fileName);
 	}
 
 	protected function assertEntitiesEqual($expected, $actual)
@@ -77,31 +74,4 @@ abstract class AbstractTestCase extends \PHPUnit_Framework_TestCase
 			}
 		}
 	}
-
-	private function getTestDirectoryPath()
-	{
-		return __DIR__ . DIRECTORY_SEPARATOR . 'files';
-	}
-
-	private function cleanTestDirectory()
-	{
-		if (!file_exists($this->getTestDirectoryPath()))
-			return;
-
-		$dirIterator = new \RecursiveDirectoryIterator(
-			$this->getTestDirectoryPath(),
-			\RecursiveDirectoryIterator::SKIP_DOTS);
-
-		$files = new \RecursiveIteratorIterator(
-			$dirIterator,
-			\RecursiveIteratorIterator::CHILD_FIRST);
-
-		foreach ($files as $fileInfo)
-		{
-			if ($fileInfo->isDir())
-				rmdir($fileInfo->getRealPath());
-			else
-				unlink($fileInfo->getRealPath());
-		}
-	}
 }

tests/TestHelper.php

@@ -0,0 +1,55 @@
+<?php
+namespace Szurubooru\Tests;
+
+class TestHelper
+{
+	public static function createTestDirectory()
+	{
+		$path = self::getTestDirectoryPath();
+		if (!file_exists($path))
+			mkdir($path, 0777, true);
+		return $path;
+	}
+
+	public static function mockConfig($dataPath = null, $publicDataPath = null)
+	{
+		return new ConfigMock($dataPath, $publicDataPath);
+	}
+
+	public static function getTestFile($fileName)
+	{
+		return file_get_contents(self::getTestFilePath($fileName));
+	}
+
+	public static function getTestFilePath($fileName)
+	{
+		return __DIR__ . DIRECTORY_SEPARATOR . 'test_files' . DIRECTORY_SEPARATOR . $fileName;
+	}
+
+	public static function cleanTestDirectory()
+	{
+		if (!file_exists(self::getTestDirectoryPath()))
+			return;
+
+		$dirIterator = new \RecursiveDirectoryIterator(
+			self::getTestDirectoryPath(),
+			\RecursiveDirectoryIterator::SKIP_DOTS);
+
+		$files = new \RecursiveIteratorIterator(
+			$dirIterator,
+			\RecursiveIteratorIterator::CHILD_FIRST);
+
+		foreach ($files as $fileInfo)
+		{
+			if ($fileInfo->isDir())
+				rmdir($fileInfo->getRealPath());
+			else
+				unlink($fileInfo->getRealPath());
+		}
+	}
+
+	private static function getTestDirectoryPath()
+	{
+		return __DIR__ . DIRECTORY_SEPARATOR . 'files';
+	}
+}