diff --git a/tests/Composer/Test/Autoload/Fixtures/php5.4/traits.php b/tests/Composer/Test/Autoload/Fixtures/php5.4/traits.php index 5d1fd3c52..34257c712 100644 --- a/tests/Composer/Test/Autoload/Fixtures/php5.4/traits.php +++ b/tests/Composer/Test/Autoload/Fixtures/php5.4/traits.php @@ -4,7 +4,8 @@ namespace { } - class CFoo { + class CFoo + { use TFoo; } } @@ -22,7 +23,8 @@ namespace Foo { } - class CBar implements IBar { + class CBar implements IBar + { use TBar, TFooBar; } } diff --git a/tests/Composer/Test/ComposerTest.php b/tests/Composer/Test/ComposerTest.php index 80feb51ba..6ff841919 100644 --- a/tests/Composer/Test/ComposerTest.php +++ b/tests/Composer/Test/ComposerTest.php @@ -59,4 +59,4 @@ class ComposerTest extends TestCase $this->assertSame($manager, $composer->getInstallationManager()); } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/DependencyResolver/RequestTest.php b/tests/Composer/Test/DependencyResolver/RequestTest.php index 969240860..89639bc44 100644 --- a/tests/Composer/Test/DependencyResolver/RequestTest.php +++ b/tests/Composer/Test/DependencyResolver/RequestTest.php @@ -15,7 +15,6 @@ namespace Composer\Test\DependencyResolver; use Composer\DependencyResolver\Request; use Composer\DependencyResolver\Pool; use Composer\Repository\ArrayRepository; -use Composer\DependencyResolver\Literal; use Composer\Test\TestCase; class RequestTest extends TestCase diff --git a/tests/Composer/Test/DependencyResolver/SolverTest.php b/tests/Composer/Test/DependencyResolver/SolverTest.php index df7f32cf4..7638e9509 100644 --- a/tests/Composer/Test/DependencyResolver/SolverTest.php +++ b/tests/Composer/Test/DependencyResolver/SolverTest.php @@ -12,15 +12,12 @@ namespace Composer\Test\DependencyResolver; use Composer\Repository\ArrayRepository; -use Composer\Repository\PlatformRepository; -use Composer\Repository\ComposerRepository; use Composer\DependencyResolver\DefaultPolicy; use Composer\DependencyResolver\Pool; use Composer\DependencyResolver\Request; use Composer\DependencyResolver\Solver; use Composer\DependencyResolver\SolverProblemsException; use Composer\Package\Link; -use Composer\Package\LinkConstraint\VersionConstraint; use Composer\Test\TestCase; class SolverTest extends TestCase diff --git a/tests/Composer/Test/Downloader/ArchiveDownloaderTest.php b/tests/Composer/Test/Downloader/ArchiveDownloaderTest.php index b7f2c0f53..cf5fde68e 100644 --- a/tests/Composer/Test/Downloader/ArchiveDownloaderTest.php +++ b/tests/Composer/Test/Downloader/ArchiveDownloaderTest.php @@ -12,8 +12,6 @@ namespace Composer\Test\Downloader; -use Composer\Util\Filesystem; - class ArchiveDownloaderTest extends \PHPUnit_Framework_TestCase { public function testGetFileName() diff --git a/tests/Composer/Test/Downloader/FileDownloaderTest.php b/tests/Composer/Test/Downloader/FileDownloaderTest.php index 9b898e68c..a6fff5f1b 100644 --- a/tests/Composer/Test/Downloader/FileDownloaderTest.php +++ b/tests/Composer/Test/Downloader/FileDownloaderTest.php @@ -21,6 +21,7 @@ class FileDownloaderTest extends \PHPUnit_Framework_TestCase { $io = $io ?: $this->getMock('Composer\IO\IOInterface'); $rfs = $rfs ?: $this->getMockBuilder('Composer\Util\RemoteFilesystem')->disableOriginalConstructor()->getMock(); + return new FileDownloader($io, $rfs); } @@ -96,6 +97,7 @@ class FileDownloaderTest extends \PHPUnit_Framework_TestCase if (is_file($path.'/script.js')) { unlink($path.'/script.js'); } + return $messages; })) ; @@ -108,7 +110,7 @@ class FileDownloaderTest extends \PHPUnit_Framework_TestCase if (is_dir($path)) { $fs = new Filesystem(); $fs->removeDirectory($path); - } else if (is_file($path)) { + } elseif (is_file($path)) { unset($path); } @@ -146,7 +148,7 @@ class FileDownloaderTest extends \PHPUnit_Framework_TestCase if (is_dir($path)) { $fs = new Filesystem(); $fs->removeDirectory($path); - } else if (is_file($path)) { + } elseif (is_file($path)) { unset($path); } diff --git a/tests/Composer/Test/Downloader/ZipDownloaderTest.php b/tests/Composer/Test/Downloader/ZipDownloaderTest.php index c0fd6deac..b852329b0 100644 --- a/tests/Composer/Test/Downloader/ZipDownloaderTest.php +++ b/tests/Composer/Test/Downloader/ZipDownloaderTest.php @@ -12,7 +12,6 @@ namespace Composer\Test\Downloader; -use Composer\Util\Filesystem; use Composer\Downloader\ZipDownloader; class ZipDownloaderTest extends \PHPUnit_Framework_TestCase diff --git a/tests/Composer/Test/FactoryTest.php b/tests/Composer/Test/FactoryTest.php index 2bb838fef..12c19f865 100644 --- a/tests/Composer/Test/FactoryTest.php +++ b/tests/Composer/Test/FactoryTest.php @@ -33,6 +33,7 @@ class FactoryTest extends \PHPUnit_Framework_TestCase { $f = function() { $repositories = func_get_args(); + return array('repositories' => $repositories); }; diff --git a/tests/Composer/Test/IO/NullIOTest.php b/tests/Composer/Test/IO/NullIOTest.php index e0e07dd76..07577d5a1 100644 --- a/tests/Composer/Test/IO/NullIOTest.php +++ b/tests/Composer/Test/IO/NullIOTest.php @@ -67,4 +67,4 @@ class NullIOTest extends TestCase $this->assertEquals('foo', $io->askAndValidate('question', 'validator', false, 'foo')); } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/Installer/InstallerInstallerTest.php b/tests/Composer/Test/Installer/InstallerInstallerTest.php index b1a0e14d0..33c941600 100644 --- a/tests/Composer/Test/Installer/InstallerInstallerTest.php +++ b/tests/Composer/Test/Installer/InstallerInstallerTest.php @@ -147,6 +147,7 @@ class InstallerInstallerMock extends InstallerInstaller public function getInstallPath(PackageInterface $package) { $version = $package->getVersion(); + return __DIR__.'/Fixtures/installer-v'.$version[0].'/'; } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/Installer/LibraryInstallerTest.php b/tests/Composer/Test/Installer/LibraryInstallerTest.php index 3edc8a477..4fa83dfe9 100644 --- a/tests/Composer/Test/Installer/LibraryInstallerTest.php +++ b/tests/Composer/Test/Installer/LibraryInstallerTest.php @@ -13,7 +13,6 @@ namespace Composer\Test\Installer; use Composer\Installer\LibraryInstaller; -use Composer\DependencyResolver\Operation; use Composer\Util\Filesystem; use Composer\Test\TestCase; diff --git a/tests/Composer/Test/InstallerTest.php b/tests/Composer/Test/InstallerTest.php index f94a6092b..8361af192 100644 --- a/tests/Composer/Test/InstallerTest.php +++ b/tests/Composer/Test/InstallerTest.php @@ -17,7 +17,6 @@ use Composer\Config; use Composer\Json\JsonFile; use Composer\Repository\ArrayRepository; use Composer\Repository\RepositoryManager; -use Composer\Repository\RepositoryInterface; use Composer\Package\PackageInterface; use Composer\Package\Link; use Composer\Package\Locker; diff --git a/tests/Composer/Test/Mock/InstallationManagerMock.php b/tests/Composer/Test/Mock/InstallationManagerMock.php index 88724a7a5..cc587a6f9 100644 --- a/tests/Composer/Test/Mock/InstallationManagerMock.php +++ b/tests/Composer/Test/Mock/InstallationManagerMock.php @@ -13,7 +13,6 @@ namespace Composer\Test\Mock; use Composer\Installer\InstallationManager; use Composer\Repository\RepositoryInterface; -use Composer\DependencyResolver\Operation\OperationInterface; use Composer\DependencyResolver\Operation\InstallOperation; use Composer\DependencyResolver\Operation\UpdateOperation; use Composer\DependencyResolver\Operation\UninstallOperation; diff --git a/tests/Composer/Test/Package/LinkConstraint/VersionConstraintTest.php b/tests/Composer/Test/Package/LinkConstraint/VersionConstraintTest.php index 7f6e24e4b..15c66c8e7 100644 --- a/tests/Composer/Test/Package/LinkConstraint/VersionConstraintTest.php +++ b/tests/Composer/Test/Package/LinkConstraint/VersionConstraintTest.php @@ -16,7 +16,7 @@ use Composer\Package\LinkConstraint\VersionConstraint; class VersionConstraintTest extends \PHPUnit_Framework_TestCase { - static public function successfulVersionMatches() + public static function successfulVersionMatches() { return array( // require provide @@ -41,7 +41,7 @@ class VersionConstraintTest extends \PHPUnit_Framework_TestCase $this->assertTrue($versionRequire->matches($versionProvide)); } - static public function failingVersionMatches() + public static function failingVersionMatches() { return array( // require provide diff --git a/tests/Composer/Test/Repository/PearRepositoryTest.php b/tests/Composer/Test/Repository/PearRepositoryTest.php index 4df45449d..3172c06f0 100644 --- a/tests/Composer/Test/Repository/PearRepositoryTest.php +++ b/tests/Composer/Test/Repository/PearRepositoryTest.php @@ -12,7 +12,6 @@ namespace Composer\Repository; -use Composer\Repository\FilesystemRepository; use Composer\Test\TestCase; /** @@ -33,7 +32,7 @@ class PearRepositoryTest extends TestCase /** * @dataProvider repositoryDataProvider * @param string $url - * @param array $expectedPackages + * @param array $expectedPackages */ public function testRepositoryRead($url, array $expectedPackages) { @@ -126,4 +125,4 @@ class PearRepositoryTest extends TestCase $this->repository = null; $this->remoteFilesystem = null; } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/Repository/Vcs/SvnDriverTest.php b/tests/Composer/Test/Repository/Vcs/SvnDriverTest.php index 569fab69f..f14a36994 100644 --- a/tests/Composer/Test/Repository/Vcs/SvnDriverTest.php +++ b/tests/Composer/Test/Repository/Vcs/SvnDriverTest.php @@ -13,7 +13,6 @@ namespace Composer\Test\Repository\Vcs; use Composer\Repository\Vcs\SvnDriver; -use Composer\IO\NullIO; use Composer\Config; class SvnDriverTest extends \PHPUnit_Framework_TestCase diff --git a/tests/Composer/Test/Repository/VcsRepositoryTest.php b/tests/Composer/Test/Repository/VcsRepositoryTest.php index 8c04c4b0b..8b3272e82 100644 --- a/tests/Composer/Test/Repository/VcsRepositoryTest.php +++ b/tests/Composer/Test/Repository/VcsRepositoryTest.php @@ -15,7 +15,6 @@ namespace Composer\Test\Repository; use Symfony\Component\Process\ExecutableFinder; use Composer\Package\Dumper\ArrayDumper; use Composer\Repository\VcsRepository; -use Composer\Repository\Vcs\GitDriver; use Composer\Util\Filesystem; use Composer\Util\ProcessExecutor; use Composer\IO\NullIO; @@ -37,10 +36,12 @@ class VcsRepositoryTest extends \PHPUnit_Framework_TestCase $locator = new ExecutableFinder(); if (!$locator->find('git')) { $this->skipped = 'This test needs a git binary in the PATH to be able to run'; + return; } if (!mkdir(self::$gitRepo) || !chdir(self::$gitRepo)) { $this->skipped = 'Could not create and move into the temp git repo '.self::$gitRepo; + return; } diff --git a/tests/Composer/Test/Script/EventDispatcherTest.php b/tests/Composer/Test/Script/EventDispatcherTest.php index 2da566907..e23dccf8a 100644 --- a/tests/Composer/Test/Script/EventDispatcherTest.php +++ b/tests/Composer/Test/Script/EventDispatcherTest.php @@ -56,4 +56,4 @@ class EventDispatcherTest extends TestCase { throw new \RuntimeException(); } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/TestCase.php b/tests/Composer/Test/TestCase.php index bbceb3875..a544bdfa9 100644 --- a/tests/Composer/Test/TestCase.php +++ b/tests/Composer/Test/TestCase.php @@ -49,6 +49,7 @@ abstract class TestCase extends \PHPUnit_Framework_TestCase protected function getAliasPackage($package, $version) { $normVersion = self::getVersionParser()->normalize($version); + return new AliasPackage($package, $normVersion, $version); } diff --git a/tests/Composer/Test/Util/RemoteFilesystemTest.php b/tests/Composer/Test/Util/RemoteFilesystemTest.php index 93e42153d..4824e1af9 100644 --- a/tests/Composer/Test/Util/RemoteFilesystemTest.php +++ b/tests/Composer/Test/Util/RemoteFilesystemTest.php @@ -13,7 +13,6 @@ namespace Composer\Test\Util; use Composer\Util\RemoteFilesystem; -use Composer\Test\TestCase; class RemoteFilesystemTest extends \PHPUnit_Framework_TestCase { diff --git a/tests/Composer/Test/Util/SpdxLicenseIdentifierTest.php b/tests/Composer/Test/Util/SpdxLicenseIdentifierTest.php index 4b461d31f..2ed7c1819 100644 --- a/tests/Composer/Test/Util/SpdxLicenseIdentifierTest.php +++ b/tests/Composer/Test/Util/SpdxLicenseIdentifierTest.php @@ -91,4 +91,4 @@ class SpdxLicenseIdentifierTest extends TestCase $validator = new SpdxLicenseIdentifier(); $validator->validate($invalidArgument); } -} \ No newline at end of file +} diff --git a/tests/Composer/Test/Util/SvnTest.php b/tests/Composer/Test/Util/SvnTest.php index 2a1839f67..a29db7cee 100644 --- a/tests/Composer/Test/Util/SvnTest.php +++ b/tests/Composer/Test/Util/SvnTest.php @@ -46,4 +46,4 @@ class SvnTest $svn->getCommand('svn ls', $url) ); } -} \ No newline at end of file +}