diff --git a/tests/Composer/Test/Fixtures/installer/install-from-empty-lock.test b/tests/Composer/Test/Fixtures/installer/install-from-empty-lock.test index d754651a0..65b3fe80b 100644 --- a/tests/Composer/Test/Fixtures/installer/install-from-empty-lock.test +++ b/tests/Composer/Test/Fixtures/installer/install-from-empty-lock.test @@ -19,7 +19,7 @@ Requirements from the composer file are not installed if the lock file is presen --LOCK-- { "packages": [ - { "package": "required", "version": "1.0.0" } + { "name": "required", "version": "1.0.0" } ], "packages-dev": null, "aliases": [], diff --git a/tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test b/tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test index 63c01b7d5..d73b93557 100644 --- a/tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test +++ b/tests/Composer/Test/Fixtures/installer/update-whitelist-reads-lock.test @@ -24,9 +24,9 @@ Limited update takes rules from lock if available, and not from the installed re --LOCK-- { "packages": [ - { "package": "old/installed", "version": "1.0.0" }, - { "package": "toupdate/installed", "version": "1.0.0" }, - { "package": "toupdate/notinstalled", "version": "1.0.0" } + { "name": "old/installed", "version": "1.0.0" }, + { "name": "toupdate/installed", "version": "1.0.0" }, + { "name": "toupdate/notinstalled", "version": "1.0.0" } ], "packages-dev": null, "aliases": [], diff --git a/tests/Composer/Test/Package/LockerTest.php b/tests/Composer/Test/Package/LockerTest.php index 35cb90e04..d72a43363 100644 --- a/tests/Composer/Test/Package/LockerTest.php +++ b/tests/Composer/Test/Package/LockerTest.php @@ -13,6 +13,7 @@ namespace Composer\Test\Package; use Composer\Package\Locker; +use Composer\Package\CompletePackage; class LockerTest extends \PHPUnit_Framework_TestCase { @@ -68,57 +69,14 @@ class LockerTest extends \PHPUnit_Framework_TestCase ->method('read') ->will($this->returnValue(array( 'packages' => array( - array('package' => 'pkg1', 'version' => '1.0.0-beta'), - array('package' => 'pkg2', 'version' => '0.1.10') + array('name' => 'pkg1', 'version' => '1.0.0-beta'), + array('name' => 'pkg2', 'version' => '0.1.10') ) ))); - $package1 = $this->createPackageMock(); - $package2 = $this->createPackageMock(); - - $repo->getLocalRepository() - ->expects($this->exactly(2)) - ->method('findPackage') - ->with($this->logicalOr('pkg1', 'pkg2'), $this->logicalOr('1.0.0-beta', '0.1.10')) - ->will($this->onConsecutiveCalls($package1, $package2)); - - $this->assertEquals(array($package1, $package2), $locker->getLockedRepository()->getPackages()); - } - - public function testGetPackagesWithoutRepo() - { - $json = $this->createJsonFileMock(); - $repo = $this->createRepositoryManagerMock(); - $inst = $this->createInstallationManagerMock(); - - $locker = new Locker($json, $repo, $inst, 'md5'); - - $json - ->expects($this->once()) - ->method('exists') - ->will($this->returnValue(true)); - $json - ->expects($this->once()) - ->method('read') - ->will($this->returnValue(array( - 'packages' => array( - array('package' => 'pkg1', 'version' => '1.0.0-beta'), - array('package' => 'pkg2', 'version' => '0.1.10') - ) - ))); - - $package1 = $this->createPackageMock(); - $package2 = $this->createPackageMock(); - - $repo->getLocalRepository() - ->expects($this->exactly(2)) - ->method('findPackage') - ->with($this->logicalOr('pkg1', 'pkg2'), $this->logicalOr('1.0.0-beta', '0.1.10')) - ->will($this->onConsecutiveCalls($package1, null)); - - $this->setExpectedException('LogicException'); - - $locker->getLockedRepository(); + $repo = $locker->getLockedRepository(); + $this->assertNotNull($repo->findPackage('pkg1', '1.0.0-beta')); + $this->assertNotNull($repo->findPackage('pkg2', '0.1.10')); } public function testSetLockData()