From 1afeaea987968e737146aaa503af1b1c3e1ad66d Mon Sep 17 00:00:00 2001 From: Sebastiaan Stok Date: Fri, 31 Aug 2012 10:15:46 +0200 Subject: [PATCH] Fixed unresolvable TMP dir on Windows --- tests/ClientTest.php | 46 ++++++++------ tests/InterfaceTest.php | 44 +++++++------- tests/RepositoryTest.php | 126 +++++++++++++++++++++------------------ 3 files changed, 120 insertions(+), 96 deletions(-) diff --git a/tests/ClientTest.php b/tests/ClientTest.php index 945b619..e24ed5d 100644 --- a/tests/ClientTest.php +++ b/tests/ClientTest.php @@ -8,22 +8,32 @@ use Symfony\Component\Filesystem\Filesystem; class ClientTest extends PHPUnit_Framework_TestCase { - const PATH = '/tmp/gitlist'; + protected static $tmpdir = '/tmp/gitlist'; + protected $client; public static function setUpBeforeClass() { - mkdir(ClientTest::PATH); + if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') { + self::$tmpdir = getenv('TMP').'/gitlist'; + } + + $fs = new Filesystem(); + $fs->mkdir(self::$tmpdir); + + if (!is_writable(self::$tmpdir)) { + $this->markTestSkipped('There are no write permissions in order to create test repositories.'); + } } public function setUp() { - if (!is_writable(ClientTest::PATH)) { + if (!is_writable(self::$tmpdir)) { $this->markTestSkipped('There are no write permissions in order to create test repositories.'); } $options['path'] = getenv('GIT_CLIENT') ?: '/usr/bin/git'; - $options['hidden'] = array(ClientTest::PATH . '/hiddenrepo'); + $options['hidden'] = array(self::$tmpdir . '/hiddenrepo'); $this->client = new Client($options); } @@ -32,7 +42,7 @@ class ClientTest extends PHPUnit_Framework_TestCase */ public function testIsNotFindingRepositories() { - $this->client->getRepositories(ClientTest::PATH . '/testrepo'); + $this->client->getRepositories(self::$tmpdir . '/testrepo'); } /** @@ -40,7 +50,7 @@ class ClientTest extends PHPUnit_Framework_TestCase */ public function testIsNotAbleToGetUnexistingRepository() { - $this->client->getRepository(ClientTest::PATH . '/testrepo'); + $this->client->getRepository(self::$tmpdir . '/testrepo'); } /** @@ -50,10 +60,10 @@ class ClientTest extends PHPUnit_Framework_TestCase { $this->client->getRepositories('/tmp'); } - + public function testIsCreatingRepository() { - $repository = $this->client->createRepository(ClientTest::PATH . '/testrepo'); + $repository = $this->client->createRepository(self::$tmpdir . '/testrepo'); $this->assertRegExp("/nothing to commit/", $repository->getClient()->run($repository, 'status')); } @@ -62,14 +72,14 @@ class ClientTest extends PHPUnit_Framework_TestCase */ public function testIsNotAbleToCreateRepositoryDueToExistingOne() { - $this->client->createRepository(ClientTest::PATH . '/testrepo'); + $this->client->createRepository(self::$tmpdir . '/testrepo'); } public function testIsListingRepositories() { - $this->client->createRepository(ClientTest::PATH . '/anothertestrepo'); - $this->client->createRepository(ClientTest::PATH . '/bigbadrepo'); - $repositories = $this->client->getRepositories(ClientTest::PATH); + $this->client->createRepository(self::$tmpdir . '/anothertestrepo'); + $this->client->createRepository(self::$tmpdir . '/bigbadrepo'); + $repositories = $this->client->getRepositories(self::$tmpdir); $this->assertEquals($repositories[0]['name'], 'anothertestrepo'); $this->assertEquals($repositories[1]['name'], 'bigbadrepo'); @@ -78,8 +88,8 @@ class ClientTest extends PHPUnit_Framework_TestCase public function testIsNotListingHiddenRepositories() { - $this->client->createRepository(ClientTest::PATH . '/hiddenrepo'); - $repositories = $this->client->getRepositories(ClientTest::PATH); + $this->client->createRepository(self::$tmpdir . '/hiddenrepo'); + $repositories = $this->client->getRepositories(self::$tmpdir); $this->assertEquals($repositories[0]['name'], 'anothertestrepo'); $this->assertEquals($repositories[1]['name'], 'bigbadrepo'); @@ -92,7 +102,7 @@ class ClientTest extends PHPUnit_Framework_TestCase */ public function testIsNotOpeningHiddenRepositories() { - $this->client->getRepository(ClientTest::PATH . '/hiddenrepo'); + $this->client->getRepository(self::$tmpdir . '/hiddenrepo'); } /** @@ -100,13 +110,13 @@ class ClientTest extends PHPUnit_Framework_TestCase */ public function testIsCatchingGitCommandErrors() { - $repository = $this->client->getRepository(ClientTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $repository->getClient()->run($repository, 'wrong'); } - + public static function tearDownAfterClass() { $fs = new Filesystem(); - $fs->remove(ClientTest::PATH); + $fs->remove(self::$tmpdir); } } diff --git a/tests/InterfaceTest.php b/tests/InterfaceTest.php index 45db2b1..c1ce201 100644 --- a/tests/InterfaceTest.php +++ b/tests/InterfaceTest.php @@ -8,26 +8,30 @@ use GitList\Component\Git\Client; class InterfaceTest extends WebTestCase { - const PATH = '/tmp/gitlist/'; + protected static $tmpdir = '/tmp/gitlist'; public static function setUpBeforeClass() { - $fs = new Filesystem(); - $fs->mkdir(InterfaceTest::PATH); + if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') { + self::$tmpdir = getenv('TMP').'/gitlist'; + } - if (!is_writable(InterfaceTest::PATH)) { + $fs = new Filesystem(); + $fs->mkdir(self::$tmpdir); + + if (!is_writable(self::$tmpdir)) { $this->markTestSkipped('There are no write permissions in order to create test repositories.'); } $options['path'] = getenv('GIT_CLIENT') ?: '/usr/bin/git'; - $options['hidden'] = array(InterfaceTest::PATH . '/hiddenrepo'); + $options['hidden'] = array(self::$tmpdir . '/hiddenrepo'); $git = new Client($options); // GitTest repository fixture - $git->createRepository(InterfaceTest::PATH . 'GitTest'); - $repository = $git->getRepository(InterfaceTest::PATH . 'GitTest'); - file_put_contents(InterfaceTest::PATH . 'GitTest/README.md', "## GitTest\nGitTest is a *test* repository!"); - file_put_contents(InterfaceTest::PATH . 'GitTest/test.php', "createRepository(self::$tmpdir . 'GitTest'); + $repository = $git->getRepository(self::$tmpdir . 'GitTest'); + file_put_contents(self::$tmpdir . 'GitTest/README.md', "## GitTest\nGitTest is a *test* repository!"); + file_put_contents(self::$tmpdir . 'GitTest/test.php', "setConfig('user.name', 'Luke Skywalker'); $repository->setConfig('user.email', 'luke@rebel.org'); $repository->addAll(); @@ -36,14 +40,14 @@ class InterfaceTest extends WebTestCase $repository->createBranch('issue42'); // foobar repository fixture - $git->createRepository(InterfaceTest::PATH . 'foobar'); - $repository = $git->getRepository(InterfaceTest::PATH . '/foobar'); - file_put_contents(InterfaceTest::PATH . 'foobar/bar.json', "{\n\"name\": \"foobar\"\n}"); - file_put_contents(InterfaceTest::PATH . 'foobar/.git/description', 'This is a test repo!'); - $fs->mkdir(InterfaceTest::PATH . 'foobar/myfolder'); - $fs->mkdir(InterfaceTest::PATH . 'foobar/testfolder'); - file_put_contents(InterfaceTest::PATH . 'foobar/myfolder/mytest.php', "createRepository(self::$tmpdir . 'foobar'); + $repository = $git->getRepository(self::$tmpdir . '/foobar'); + file_put_contents(self::$tmpdir . 'foobar/bar.json', "{\n\"name\": \"foobar\"\n}"); + file_put_contents(self::$tmpdir . 'foobar/.git/description', 'This is a test repo!'); + $fs->mkdir(self::$tmpdir . 'foobar/myfolder'); + $fs->mkdir(self::$tmpdir . 'foobar/testfolder'); + file_put_contents(self::$tmpdir . 'foobar/myfolder/mytest.php', "setConfig('user.name', 'Luke Skywalker'); $repository->setConfig('user.email', 'luke@rebel.org'); $repository->addAll(); @@ -54,7 +58,7 @@ class InterfaceTest extends WebTestCase { $app = require 'boot.php'; $app['debug'] = true; - $app['git.repos'] = InterfaceTest::PATH; + $app['git.repos'] = self::$tmpdir; return $app; } @@ -193,6 +197,6 @@ class InterfaceTest extends WebTestCase public static function tearDownAfterClass() { $fs = new Filesystem(); - $fs->remove(InterfaceTest::PATH); + $fs->remove(self::$tmpdir); } -} \ No newline at end of file +} diff --git a/tests/RepositoryTest.php b/tests/RepositoryTest.php index fa12a1f..428176f 100644 --- a/tests/RepositoryTest.php +++ b/tests/RepositoryTest.php @@ -8,17 +8,27 @@ use Symfony\Component\Filesystem\Filesystem; class RepositoryTest extends PHPUnit_Framework_TestCase { - const PATH = '/tmp/gitlist'; + protected static $tmpdir = '/tmp/gitlist'; + protected $client; public static function setUpBeforeClass() { - mkdir(RepositoryTest::PATH); + if (strtoupper(substr(PHP_OS, 0, 3)) === 'WIN') { + self::$tmpdir = getenv('TMP').'/gitlist'; + } + + $fs = new Filesystem(); + $fs->mkdir(self::$tmpdir); + + if (!is_writable(self::$tmpdir)) { + $this->markTestSkipped('There are no write permissions in order to create test repositories.'); + } } public function setUp() { - if (!is_writable(RepositoryTest::PATH)) { + if (!is_writable(self::$tmpdir)) { $this->markTestSkipped('There are no write permissions in order to create test repositories.'); } @@ -29,9 +39,9 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsCreatingRepositoryFixtures() { - $a = $this->client->createRepository(RepositoryTest::PATH . '/testrepo'); - $b = $this->client->createRepository(RepositoryTest::PATH . '/anothertestrepo'); - $c = $this->client->createRepository(RepositoryTest::PATH . '/bigbadrepo'); + $a = $this->client->createRepository(self::$tmpdir . '/testrepo'); + $b = $this->client->createRepository(self::$tmpdir . '/anothertestrepo'); + $c = $this->client->createRepository(self::$tmpdir . '/bigbadrepo'); $this->assertRegExp("/nothing to commit/", $a->getClient()->run($a, 'status')); $this->assertRegExp("/nothing to commit/", $b->getClient()->run($b, 'status')); $this->assertRegExp("/nothing to commit/", $c->getClient()->run($c, 'status')); @@ -39,18 +49,18 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsConfiguratingRepository() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $repository->setConfig('user.name', 'Luke Skywalker'); $repository->setConfig('user.email', 'luke@rebel.org'); $this->assertEquals($repository->getConfig('user.name'), 'Luke Skywalker'); $this->assertEquals($repository->getConfig('user.email'), 'luke@rebel.org'); } - + public function testIsAdding() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); + file_put_contents(self::$tmpdir . '/testrepo/test_file.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); $repository->add('test_file.txt'); $this->assertRegExp("/new file: test_file.txt/", $repository->getClient()->run($repository, 'status')); } @@ -60,11 +70,11 @@ class RepositoryTest extends PHPUnit_Framework_TestCase */ public function testIsAddingDot() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file1.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file2.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file3.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file1.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file2.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file3.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); $repository->add(); @@ -78,14 +88,14 @@ class RepositoryTest extends PHPUnit_Framework_TestCase */ public function testIsAddingAll() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file4.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file5.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file6.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file4.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file5.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file6.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); $repository->addAll(); - + $this->assertRegExp("/new file: test_file4.txt/", $repository->getClient()->run($repository, 'status')); $this->assertRegExp("/new file: test_file5.txt/", $repository->getClient()->run($repository, 'status')); $this->assertRegExp("/new file: test_file6.txt/", $repository->getClient()->run($repository, 'status')); @@ -96,11 +106,11 @@ class RepositoryTest extends PHPUnit_Framework_TestCase */ public function testIsAddingArrayOfFiles() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file7.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file8.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); - file_put_contents(RepositoryTest::PATH . '/testrepo/test_file9.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file7.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file8.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); + file_put_contents(self::$tmpdir . '/testrepo/test_file9.txt', 'Your mother is so ugly, glCullFace always returns TRUE.'); $repository->add(array('test_file7.txt', 'test_file8.txt', 'test_file9.txt')); @@ -108,20 +118,20 @@ class RepositoryTest extends PHPUnit_Framework_TestCase $this->assertRegExp("/new file: test_file8.txt/", $repository->getClient()->run($repository, 'status')); $this->assertRegExp("/new file: test_file9.txt/", $repository->getClient()->run($repository, 'status')); } - + /** * @depends testIsAddingArrayOfFiles */ public function testIsCommiting() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $repository->commit("The truth unveiled"); $this->assertRegExp("/The truth unveiled/", $repository->getClient()->run($repository, 'log')); } public function testIsCreatingBranches() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $repository->createBranch('issue12'); $repository->createBranch('issue42'); $branches = $repository->getBranches(); @@ -132,7 +142,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingCurrentBranch() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $branch = $repository->getCurrentBranch(); $this->assertTrue($branch === 'master'); } @@ -142,9 +152,9 @@ class RepositoryTest extends PHPUnit_Framework_TestCase */ public function testIsGettingCommits() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $commits = $repository->getCommits(); - + foreach ($commits as $commit) { $this->assertInstanceOf('GitList\Component\Git\Commit\Commit', $commit); $this->assertTrue($commit->getMessage() === 'The truth unveiled'); @@ -167,9 +177,9 @@ class RepositoryTest extends PHPUnit_Framework_TestCase */ public function testIsGettingCommitsFromSpecificFile() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $commits = $repository->getCommits('test_file4.txt'); - + foreach ($commits as $commit) { $this->assertInstanceOf('GitList\Component\Git\Commit\Commit', $commit); $this->assertTrue($commit->getMessage() === 'The truth unveiled'); @@ -181,9 +191,9 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingTree() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $files = $repository->getTree('master'); - + foreach ($files as $file) { $this->assertInstanceOf('GitList\Component\Git\Model\Blob', $file); $this->assertRegExp('/test_file[0-9]*.txt/', $file->getName()); @@ -195,9 +205,9 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingTreeOutput() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $files = $repository->getTree('master')->output(); - + foreach ($files as $file) { $this->assertEquals('blob', $file['type']); $this->assertRegExp('/test_file[0-9]*.txt/', $file['name']); @@ -209,19 +219,19 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingTreesWithinTree() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); // Creating folders - mkdir(RepositoryTest::PATH . '/testrepo/MyFolder'); - mkdir(RepositoryTest::PATH . '/testrepo/MyTest'); - mkdir(RepositoryTest::PATH . '/testrepo/MyFolder/Tests'); + mkdir(self::$tmpdir . '/testrepo/MyFolder'); + mkdir(self::$tmpdir . '/testrepo/MyTest'); + mkdir(self::$tmpdir . '/testrepo/MyFolder/Tests'); // Populating created folders - file_put_contents(RepositoryTest::PATH . '/testrepo/MyFolder/crazy.php', 'Lorem ipsum dolor sit amet'); - file_put_contents(RepositoryTest::PATH . '/testrepo/MyFolder/skywalker.php', 'Lorem ipsum dolor sit amet'); - file_put_contents(RepositoryTest::PATH . '/testrepo/MyTest/fortytwo.php', 'Lorem ipsum dolor sit amet'); - file_put_contents(RepositoryTest::PATH . '/testrepo/MyFolder/Tests/web.php', 'Lorem ipsum dolor sit amet'); - file_put_contents(RepositoryTest::PATH . '/testrepo/MyFolder/Tests/cli.php', 'Lorem ipsum dolor sit amet'); + file_put_contents(self::$tmpdir . '/testrepo/MyFolder/crazy.php', 'Lorem ipsum dolor sit amet'); + file_put_contents(self::$tmpdir . '/testrepo/MyFolder/skywalker.php', 'Lorem ipsum dolor sit amet'); + file_put_contents(self::$tmpdir . '/testrepo/MyTest/fortytwo.php', 'Lorem ipsum dolor sit amet'); + file_put_contents(self::$tmpdir . '/testrepo/MyFolder/Tests/web.php', 'Lorem ipsum dolor sit amet'); + file_put_contents(self::$tmpdir . '/testrepo/MyFolder/Tests/cli.php', 'Lorem ipsum dolor sit amet'); // Adding and commiting $repository->addAll(); @@ -251,7 +261,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingBlobsWithinTrees() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $files = $repository->getTree('master:MyFolder/')->output(); $this->assertEquals('folder', $files[0]['type']); @@ -275,7 +285,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingBlobOutput() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $blob = $repository->getBlob('master:MyFolder/crazy.php')->output(); $this->assertEquals('Lorem ipsum dolor sit amet', $blob); @@ -285,7 +295,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingStatistics() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $stats = $repository->getStatistics('master'); $this->assertEquals('10', $stats['extensions']['.txt']); @@ -296,7 +306,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingAuthorStatistics() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $stats = $repository->getAuthorStatistics(); $this->assertEquals('Luke Skywalker', $stats[0]['name']); @@ -305,7 +315,7 @@ class RepositoryTest extends PHPUnit_Framework_TestCase $repository->setConfig('user.name', 'Princess Leia'); $repository->setConfig('user.email', 'sexyleia@republic.com'); - file_put_contents(RepositoryTest::PATH . '/testrepo/MyFolder/crazy.php', 'Lorem ipsum dolor sit AMET'); + file_put_contents(self::$tmpdir . '/testrepo/MyFolder/crazy.php', 'Lorem ipsum dolor sit AMET'); $repository->addAll(); $repository->commit("Fixing AMET case"); @@ -322,17 +332,17 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingSymlinksWithinTrees() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $fs = new Filesystem(); - $fs->touch(RepositoryTest::PATH . '/testrepo/original_file.txt'); - $fs->symlink(RepositoryTest::PATH . '/testrepo/original_file.txt', RepositoryTest::PATH . '/testrepo/link.txt'); + $fs->touch(self::$tmpdir . '/testrepo/original_file.txt'); + $fs->symlink(self::$tmpdir . '/testrepo/original_file.txt', self::$tmpdir . '/testrepo/link.txt'); $repository->addAll(); $repository->commit("Testing symlinks"); $files = $repository->getTree('master'); foreach ($files as $file) { if ($file instanceof GitList\Component\Git\Model\Symlink) { - $this->assertEquals($file->getPath(), RepositoryTest::PATH . '/testrepo/original_file.txt'); + $this->assertEquals($file->getPath(), self::$tmpdir . '/testrepo/original_file.txt'); $this->assertEquals($file->getName(), 'link.txt'); $this->assertEquals($file->getMode(), '120000'); return; @@ -344,17 +354,17 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public function testIsGettingSymlinksWithinTreesOutput() { - $repository = $this->client->getRepository(RepositoryTest::PATH . '/testrepo'); + $repository = $this->client->getRepository(self::$tmpdir . '/testrepo'); $fs = new Filesystem(); - $fs->touch(RepositoryTest::PATH . '/testrepo/original_file.txt'); - $fs->symlink(RepositoryTest::PATH . '/testrepo/original_file.txt', RepositoryTest::PATH . '/testrepo/link2.txt'); + $fs->touch(self::$tmpdir . '/testrepo/original_file.txt'); + $fs->symlink(self::$tmpdir . '/testrepo/original_file.txt', self::$tmpdir . '/testrepo/link2.txt'); $repository->addAll(); $repository->commit("Testing symlinks"); $files = $repository->getTree('master')->output(); foreach ($files as $file) { if ($file['type'] == 'symlink') { - $this->assertEquals($file['path'], RepositoryTest::PATH . '/testrepo/original_file.txt'); + $this->assertEquals($file['path'], self::$tmpdir . '/testrepo/original_file.txt'); $this->assertEquals($file['name'], 'link.txt'); $this->assertEquals($file['mode'], '120000'); return; @@ -367,6 +377,6 @@ class RepositoryTest extends PHPUnit_Framework_TestCase public static function tearDownAfterClass() { $fs = new Filesystem(); - $fs->remove(RepositoryTest::PATH); + $fs->remove(self::$tmpdir); } }