tests: TEMP_DIR replaced with getTempDir()

Author: dg

tests/Loaders/RobotLoader.caseSensitivity.phpt

@@ -14,7 +14,7 @@ require __DIR__ . '/../bootstrap.php';
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory(__DIR__ . '/files');
 $loader->register();
 

tests/Loaders/RobotLoader.emptyArrayVariadicArgument.phpt

@@ -13,7 +13,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 
 Assert::noError(
 	function () use ($loader) {

tests/Loaders/RobotLoader.phar.phpt

@@ -15,7 +15,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$pharFile = TEMP_DIR . '/test.phar';
+$pharFile = getTempDir() . '/test.phar';
 
 $phar = new Phar($pharFile);
 $phar['class.A.php'] = '<?php class A {}';
@@ -30,7 +30,7 @@ Phar::loadPhar($pharFile, 'test.phar');
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory("phar://$pharFile/sub");
 $loader->addDirectory("PHAR://$pharFile/class.B.php");
 $loader->addDirectory('phar://test.phar/class.C.php');
@@ -43,15 +43,15 @@ Assert::true(class_exists('D'));
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory("phar://$pharFile/non-dir");
 Assert::exception(function () use ($loader, $pharFile) {
 	$loader->register();
 }, Nette\IOException::class, "File or directory 'phar://$pharFile/non-dir' not found.");
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory("phar://$pharFile/non-file.php");
 Assert::exception(function () use ($loader, $pharFile) {
 	$loader->register();

tests/Loaders/RobotLoader.phpt

@@ -14,7 +14,7 @@ require __DIR__ . '/../bootstrap.php';
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory(__DIR__ . '/files');
 $loader->addDirectory(__DIR__ . '/files/'); // purposely doubled
 $loader->addDirectory(__DIR__ . '/file/interface.php'); // as file

tests/Loaders/RobotLoader.rebuild.phpt

@@ -13,26 +13,26 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-file_put_contents(TEMP_DIR . '/file1.php', '<?php class A {}');
-file_put_contents(TEMP_DIR . '/file2.php', '<?php class B {}');
+file_put_contents(getTempDir() . '/file1.php', '<?php class A {}');
+file_put_contents(getTempDir() . '/file2.php', '<?php class B {}');
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
-$loader->addDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
+$loader->addDirectory(getTempDir());
 $loader->register(); // rebuilds cache
 
-rename(TEMP_DIR . '/file1.php', TEMP_DIR . '/file3.php');
+rename(getTempDir() . '/file1.php', getTempDir() . '/file3.php');
 
 Assert::false(class_exists('A'));
 
 
 $loader2 = new RobotLoader;
-$loader2->setTempDirectory(TEMP_DIR);
-$loader2->addDirectory(TEMP_DIR);
+$loader2->setTempDirectory(getTempDir());
+$loader2->addDirectory(getTempDir());
 $loader2->register();
 
 Assert::true(class_exists('A'));
 
-rename(TEMP_DIR . '/file2.php', TEMP_DIR . '/file4.php');
+rename(getTempDir() . '/file2.php', getTempDir() . '/file4.php');
 
 Assert::false(class_exists('B'));

tests/Loaders/RobotLoader.relative.phpt

@@ -14,7 +14,7 @@ require __DIR__ . '/../bootstrap.php';
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
 $loader->addDirectory(__DIR__ . '/../Loaders/files');
 $loader->excludeDirectory(__DIR__ . '/../Loaders/files/exclude');
 $loader->excludeDirectory(__DIR__ . '/../Loaders/files/exclude2/excluded.php');

tests/Loaders/RobotLoader.renamed.phpt

@@ -14,10 +14,10 @@ require __DIR__ . '/../bootstrap.php';
 
 
 $loader = new RobotLoader;
-$loader->setTempDirectory(TEMP_DIR);
-$loader->addDirectory(TEMP_DIR);
+$loader->setTempDirectory(getTempDir());
+$loader->addDirectory(getTempDir());
 
-$dir = TEMP_DIR . DIRECTORY_SEPARATOR;
+$dir = getTempDir() . DIRECTORY_SEPARATOR;
 file_put_contents($dir . 'file1.php', '<?php class A {}');
 file_put_contents($dir . 'file2.php', '<?php class B {}');
 

tests/bootstrap.php

@@ -16,7 +16,22 @@
 date_default_timezone_set('Europe/Prague');
 
 
-// create temporary directory
-define('TEMP_DIR', __DIR__ . '/tmp/' . lcg_value());
-@mkdir(dirname(TEMP_DIR));
-@mkdir(TEMP_DIR);
+function getTempDir(): string
+{
+	$dir = __DIR__ . '/tmp/' . getmypid();
+
+	if (empty($GLOBALS['\\lock'])) {
+		// garbage collector
+		$GLOBALS['\\lock'] = $lock = fopen(__DIR__ . '/lock', 'w');
+		if (rand(0, 100)) {
+			flock($lock, LOCK_SH);
+			@mkdir(dirname($dir));
+		} elseif (flock($lock, LOCK_EX)) {
+			Tester\Helpers::purge(dirname($dir));
+		}
+
+		@mkdir($dir);
+	}
+
+	return $dir;
+}