Fix PerforceDownloader, fixes #7979
parent
9957807990
commit
0fd74d1cc4
|
@ -78,7 +78,7 @@ class PerforceDownloader extends VcsDownloader
|
||||||
*/
|
*/
|
||||||
public function doUpdate(PackageInterface $initial, PackageInterface $target, $path, $url)
|
public function doUpdate(PackageInterface $initial, PackageInterface $target, $path, $url)
|
||||||
{
|
{
|
||||||
$this->doDownload($target, $path, $url);
|
$this->doInstall($target, $path, $url);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
|
@ -121,7 +121,7 @@ class PerforceDownloaderTest extends TestCase
|
||||||
* @depends testInitPerforceInstantiatesANewPerforceObject
|
* @depends testInitPerforceInstantiatesANewPerforceObject
|
||||||
* @depends testInitPerforceDoesNothingIfPerforceAlreadySet
|
* @depends testInitPerforceDoesNothingIfPerforceAlreadySet
|
||||||
*/
|
*/
|
||||||
public function testDoDownloadWithTag()
|
public function testDoInstallWithTag()
|
||||||
{
|
{
|
||||||
//I really don't like this test but the logic of each Perforce method is tested in the Perforce class. Really I am just enforcing workflow.
|
//I really don't like this test but the logic of each Perforce method is tested in the Perforce class. Really I am just enforcing workflow.
|
||||||
$ref = 'SOURCE_REF@123';
|
$ref = 'SOURCE_REF@123';
|
||||||
|
@ -145,7 +145,7 @@ class PerforceDownloaderTest extends TestCase
|
||||||
* @depends testInitPerforceInstantiatesANewPerforceObject
|
* @depends testInitPerforceInstantiatesANewPerforceObject
|
||||||
* @depends testInitPerforceDoesNothingIfPerforceAlreadySet
|
* @depends testInitPerforceDoesNothingIfPerforceAlreadySet
|
||||||
*/
|
*/
|
||||||
public function testDoDownloadWithNoTag()
|
public function testDoInstallWithNoTag()
|
||||||
{
|
{
|
||||||
$ref = 'SOURCE_REF';
|
$ref = 'SOURCE_REF';
|
||||||
$label = null;
|
$label = null;
|
||||||
|
|
Loading…
Reference in New Issue