Browse Source

got all unit tests passing

Clark Stuth 11 years ago
parent
commit
c6133050b3

+ 2 - 0
src/Composer/Repository/Vcs/PerforceDriver.php

@@ -141,6 +141,8 @@ class PerforceDriver extends VcsDriver
         $this->composerInfo = $this->perforce->getComposerInformation('//' . $this->depot . '/' . $identifier);
         $this->composerInfoIdentifier = $identifier;
         $result = false;
+        return !empty($this->composerInfo);
+
         if (!empty($this->composerInfo)) {
             $result = count($this->composerInfo) > 0;
         }

+ 1 - 1
src/Composer/Util/Perforce.php

@@ -47,7 +47,7 @@ class Perforce
         $this->io = $io;
     }
 
-    public static function create($repoConfig, $port, $path, ProcessExecutor $process = null, IOInterface $io)
+    public static function create($repoConfig, $port, $path, ProcessExecutor $process, IOInterface $io)
     {
         if (!isset($process)) {
             $process = new ProcessExecutor;

+ 2 - 2
tests/Composer/Test/Repository/Vcs/PerforceDriverTest.php

@@ -126,7 +126,7 @@ class PerforceDriverTest extends \PHPUnit_Framework_TestCase
         $identifier = 'TEST_IDENTIFIER';
         $formatted_depot_path = '//' . self::TEST_DEPOT . '/' . $identifier;
         $this->driver->setPerforce($this->perforce);
-        $this->perforce->expects($this->at(0))->method('getComposerInformation')->with($this->equalTo($formatted_depot_path))->will($this->returnValue(array()));
+        $this->perforce->expects($this->any())->method('getComposerInformation')->with($this->equalTo($formatted_depot_path))->will($this->returnValue(array()));
         $this->driver->initialize();
         $result = $this->driver->hasComposerFile($identifier);
         $this->assertFalse($result);
@@ -141,7 +141,7 @@ class PerforceDriverTest extends \PHPUnit_Framework_TestCase
         $identifier = 'TEST_IDENTIFIER';
         $formatted_depot_path = '//' . self::TEST_DEPOT . '/' . $identifier;
         $this->driver->setPerforce($this->perforce);
-        $this->perforce->expects($this->at(0))->method('getComposerInformation')->with($this->equalTo($formatted_depot_path))->will($this->returnValue(array('')));
+        $this->perforce->expects($this->any())->method('getComposerInformation')->with($this->equalTo($formatted_depot_path))->will($this->returnValue(array('')));
         $this->driver->initialize();
         $result = $this->driver->hasComposerFile($identifier);
         $this->assertTrue($result);