tests: TEMP_DIR replaced with getTempDir()

Author: dg

tests/Bridges.DI/CacheExtension.phpt

@@ -16,7 +16,7 @@ require __DIR__ . '/../bootstrap.php';
 
 test(function () {
 	$compiler = new DI\Compiler;
-	$compiler->addExtension('cache', new CacheExtension(TEMP_DIR));
+	$compiler->addExtension('cache', new CacheExtension(getTempDir()));
 
 	eval($compiler->compile());
 

tests/Bridges.Latte/CacheMacro.cache.phpt

@@ -14,7 +14,7 @@ require __DIR__ . '/../bootstrap.php';
 
 
 $latte = new Latte\Engine;
-$latte->setTempDirectory(TEMP_DIR);
+$latte->setTempDirectory(getTempDir());
 $latte->addMacro('cache', new CacheMacro($latte->getCompiler()));
 $latte->addProvider('cacheStorage', new Nette\Caching\Storages\DevNullStorage);
 

tests/Storages/FileStorage.basic.phpt

@@ -18,7 +18,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = [1, true];
 $value = range("\x00", "\xFF");
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 Assert::null($cache->load($key));
 

tests/Storages/FileStorage.call.phpt

@@ -30,7 +30,7 @@ class Mock
 }
 
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 $mock = new Mock;
 
 $called = false;

tests/Storages/FileStorage.callbacks.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 function dependency($val)

tests/Storages/FileStorage.clean-all.phpt

@@ -14,7 +14,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$storage = new FileStorage(TEMP_DIR);
+$storage = new FileStorage(getTempDir());
 $cacheA = new Cache($storage);
 $cacheB = new Cache($storage, 'B');
 

tests/Storages/FileStorage.clean-namespace.phpt

@@ -13,7 +13,7 @@ use Tester\Assert;
 
 require __DIR__ . '/../bootstrap.php';
 
-$storage = new FileStorage(TEMP_DIR);
+$storage = new FileStorage(getTempDir());
 
 /*
  * Create filestorage cache without namespace and some with namespaces

tests/Storages/FileStorage.closure.phpt

@@ -18,7 +18,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = '../' . implode('', range("\x00", "\x1F"));
 $value = range("\x00", "\xFF");
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 Assert::null($cache->load($key));
 

tests/Storages/FileStorage.const.001.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 define('ANY_CONST', 10);

tests/Storages/FileStorage.const.002.phpt

@@ -18,7 +18,7 @@ $key = 'nette';
 $value = 'rulez';
 
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 // Deleting dependent const

tests/Storages/FileStorage.deadlock.phpt

@@ -14,7 +14,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$storage = new FileStorage(TEMP_DIR);
+$storage = new FileStorage(getTempDir());
 $cache = new Cache($storage);
 
 try {

tests/Storages/FileStorage.derive.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR), 'ns1');
+$cache = new Cache(new FileStorage(getTempDir()), 'ns1');
 $cache = $cache->derive('ns2');
 
 $cache->save($key, $value);

tests/Storages/FileStorage.exceptions.phpt

@@ -15,11 +15,11 @@ require __DIR__ . '/../bootstrap.php';
 
 
 Assert::exception(function () {
-	new FileStorage(TEMP_DIR . '/missing');
+	new FileStorage(getTempDir() . '/missing');
 }, Nette\DirectoryNotFoundException::class, "Directory '%a%' not found.");
 
 
 Assert::exception(function () {
-	$storage = new FileStorage(TEMP_DIR);
+	$storage = new FileStorage(getTempDir());
 	$storage->write('a', 'b', [Cache::TAGS => 'c']);
 }, Nette\InvalidStateException::class, 'CacheJournal has not been provided.');

tests/Storages/FileStorage.expiration.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 // Writing cache...

tests/Storages/FileStorage.files.phpt

@@ -17,10 +17,10 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
-$dependentFile = TEMP_DIR . '/spec.file';
+$dependentFile = getTempDir() . '/spec.file';
 @unlink($dependentFile);
 
 // Writing cache...

tests/Storages/FileStorage.int.phpt

@@ -18,7 +18,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 0;
 $value = range("\x00", "\xFF");
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 Assert::null($cache->load($key));
 

tests/Storages/FileStorage.items.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 // Writing cache...

tests/Storages/FileStorage.loadOrSave.phpt

@@ -18,7 +18,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = '../' . implode('', range("\x00", "\x1F"));
 $value = range("\x00", "\xFF");
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 Assert::null($cache->load($key));
 

tests/Storages/FileStorage.namespace.phpt

@@ -14,7 +14,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$storage = new FileStorage(TEMP_DIR);
+$storage = new FileStorage(getTempDir());
 $cacheA = new Cache($storage, 'a');
 $cacheB = new Cache($storage, 'b');
 

tests/Storages/FileStorage.priority.phpt

@@ -16,7 +16,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$storage = new FileStorage(TEMP_DIR, new SQLiteJournal(TEMP_DIR . '/journal.s3db'));
+$storage = new FileStorage(getTempDir(), new SQLiteJournal(getTempDir() . '/journal.s3db'));
 $cache = new Cache($storage);
 
 

tests/Storages/FileStorage.sliding.phpt

@@ -17,7 +17,7 @@ require __DIR__ . '/../bootstrap.php';
 $key = 'nette';
 $value = 'rulez';
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 // Writing cache...

tests/Storages/FileStorage.start.phpt

@@ -14,7 +14,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 
 ob_start();

tests/Storages/FileStorage.stress.phpt

@@ -33,7 +33,7 @@ function checkStr($s)
 define('COUNT_FILES', 3);
 
 
-$storage = new FileStorage(TEMP_DIR);
+$storage = new FileStorage(getTempDir());
 
 
 // clear playground

tests/Storages/FileStorage.tags.phpt

@@ -16,7 +16,7 @@ use Tester\Assert;
 require __DIR__ . '/../bootstrap.php';
 
 
-$storage = new FileStorage(TEMP_DIR, new SQLiteJournal(TEMP_DIR . '/journal.s3db'));
+$storage = new FileStorage(getTempDir(), new SQLiteJournal(getTempDir() . '/journal.s3db'));
 $cache = new Cache($storage);
 
 

tests/Storages/FileStorage.wrap.phpt

@@ -31,7 +31,7 @@ class Test
 }
 
 
-$cache = new Cache(new FileStorage(TEMP_DIR));
+$cache = new Cache(new FileStorage(getTempDir()));
 
 $called = false;
 Assert::same(55, $cache->wrap('mockFunction')(5, 50));

tests/Storages/Memcached.bulkRead.phpt

@@ -18,7 +18,7 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-files', TEMP_DIR);
+Tester\Environment::lock('memcached-files', getTempDir());
 
 
 

tests/Storages/Memcached.expiration.phpt

@@ -18,7 +18,7 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-expiration', TEMP_DIR);
+Tester\Environment::lock('memcached-expiration', getTempDir());
 
 
 $key = 'nette-memcached-expiration-key';

tests/Storages/Memcached.files.phpt

@@ -18,7 +18,7 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-files', TEMP_DIR);
+Tester\Environment::lock('memcached-files', getTempDir());
 
 
 $key = 'nette-memcached-files-key';
@@ -27,7 +27,7 @@ $value = 'rulez';
 $cache = new Cache(new MemcachedStorage('localhost'));
 
 
-$dependentFile = TEMP_DIR . '/spec-memcached.file';
+$dependentFile = getTempDir() . '/spec-memcached.file';
 @unlink($dependentFile);
 
 // Writing cache...

tests/Storages/Memcached.priority.phpt

@@ -19,10 +19,10 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-priority', TEMP_DIR);
+Tester\Environment::lock('memcached-priority', getTempDir());
 
 
-$storage = new MemcachedStorage('localhost', 11211, '', new SQLiteJournal(TEMP_DIR . '/journal-memcached.s3db'));
+$storage = new MemcachedStorage('localhost', 11211, '', new SQLiteJournal(getTempDir() . '/journal-memcached.s3db'));
 $cache = new Cache($storage);
 
 

tests/Storages/Memcached.sliding.phpt

@@ -18,7 +18,7 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-sliding', TEMP_DIR);
+Tester\Environment::lock('memcached-sliding', getTempDir());
 
 
 $key = 'nette-memcached-sliding-key';

tests/Storages/Memcached.tags.phpt

@@ -19,10 +19,10 @@ if (!MemcachedStorage::isAvailable()) {
 	Tester\Environment::skip('Requires PHP extension Memcached.');
 }
 
-Tester\Environment::lock('memcached-tags', TEMP_DIR);
+Tester\Environment::lock('memcached-tags', getTempDir());
 
 
-$storage = new MemcachedStorage('localhost', 11211, '', new SQLiteJournal(TEMP_DIR . '/journal-memcached.s3db'));
+$storage = new MemcachedStorage('localhost', 11211, '', new SQLiteJournal(getTempDir() . '/journal-memcached.s3db'));
 $cache = new Cache($storage);
 
 

tests/Storages/SQLiteJournal.permissions.phpt

@@ -20,7 +20,7 @@ if (defined('PHP_WINDOWS_VERSION_BUILD')) {
 
 
 test(function () {
-	$file = TEMP_DIR . '/sqlitejournal.permissions.1.sqlite';
+	$file = getTempDir() . '/sqlitejournal.permissions.1.sqlite';
 	Assert::false(file_exists($file));
 
 	umask(0);
@@ -31,7 +31,7 @@ test(function () {
 
 
 test(function () {
-	$file = TEMP_DIR . '/sqlitejournal.permissions.2.sqlite';
+	$file = getTempDir() . '/sqlitejournal.permissions.2.sqlite';
 	Assert::false(file_exists($file));
 
 	umask(0077);

tests/Storages/SQLiteJournal.phpt

@@ -19,7 +19,7 @@ class SQLiteJournalTest extends IJournalTestCase
 	public function createJournal()
 	{
 		static $id = 0;
-		return new SQLiteJournal(TEMP_DIR . '/sqlitejournal_' . ++$id . '.sqlite');
+		return new SQLiteJournal(getTempDir() . '/sqlitejournal_' . ++$id . '.sqlite');
 	}
 }
 

tests/Storages/SQLiteStorage.permissions.phpt

@@ -20,7 +20,7 @@ if (defined('PHP_WINDOWS_VERSION_BUILD')) {
 
 
 test(function () {
-	$file = TEMP_DIR . '/sqlitestorage.permissions.1.sqlite';
+	$file = getTempDir() . '/sqlitestorage.permissions.1.sqlite';
 	Assert::false(file_exists($file));
 
 	umask(0);
@@ -31,7 +31,7 @@ test(function () {
 
 
 test(function () {
-	$file = TEMP_DIR . '/sqlitestorage.permissions.2.sqlite';
+	$file = getTempDir() . '/sqlitestorage.permissions.2.sqlite';
 	Assert::false(file_exists($file));
 
 	umask(0077);

tests/bootstrap.php

@@ -16,21 +16,25 @@
 date_default_timezone_set('Europe/Prague');
 
 
-// create temporary directory
-(function () {
-	define('TEMP_DIR', __DIR__ . '/tmp/' . getmypid());
-
-	// garbage collector
-	$GLOBALS['\\lock'] = $lock = fopen(__DIR__ . '/lock', 'w');
-	if (rand(0, 100)) {
-		flock($lock, LOCK_SH);
-		@mkdir(dirname(TEMP_DIR));
-	} elseif (flock($lock, LOCK_EX)) {
-		Tester\Helpers::purge(dirname(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);
 	}
 
-	@mkdir(TEMP_DIR);
-})();
+	return $dir;
+}
 
 
 function test(\Closure $function): void