Fixing Windows path problem

Author: weaverryan

.appveyor/composer.bat

@@ -0,0 +1 @@
+@php "%~dp0composer.phar" %*

appveyor.yml

@@ -5,6 +5,9 @@ cache:
     - composer.phar
     - .phpunit -> phpunit
 
+init:
+    - SET PATH=c:\php;%PATH%
+
 install:
     - mkdir c:\php && cd c:\php
     - appveyor DownloadFile https://github.com/symfony/binary-utils/releases/download/v0.1/php-7.1.3-Win32-VC14-x86.zip
@@ -35,10 +38,12 @@ install:
     - cd c:\php
     - IF NOT EXIST composer.phar (appveyor DownloadFile https://getcomposer.org/download/1.6.3/composer.phar)
     - php composer.phar self-update
-    - echo @php "c:\php\composer.phar" %*>composer.bat
+    # copy instead of using echo because appveyor does not allow "%s" characters
+    - copy /Y c:\projects\maker-bundle\.appveyor\composer.bat composer.bat
 
     - cd c:\projects\maker-bundle
     - php -dmemory_limit=-1 c:\php\composer.phar install --no-progress --no-suggest --ansi
+    - composer dump-autoload
     - ./vendor/bin/simple-phpunit install
 
 test_script:

src/FileManager.php

@@ -33,7 +33,7 @@ class FileManager
     public function __construct(Filesystem $fs, string $rootDirectory)
     {
         $this->fs = $fs;
-        $this->rootDirectory = rtrim($this->realpath($this->normalizeSlashes($rootDirectory)).'/');
+        $this->rootDirectory = rtrim($this->realpath($this->normalizeSlashes($rootDirectory)), '/');
     }
 
     public function setIO(SymfonyStyle $io)
@@ -61,6 +61,15 @@ public function fileExists($path): bool
         return file_exists($this->absolutizePath($path));
     }
 
+    /**
+     * Attempts to make the path relative to the root directory.
+     *
+     * @param string $absolutePath
+     *
+     * @return string
+     *
+     * @throws \Exception
+     */
     public function relativizePath($absolutePath): string
     {
         $absolutePath = $this->normalizeSlashes($absolutePath);
@@ -80,6 +89,15 @@ public function relativizePath($absolutePath): string
         return is_dir($absolutePath) ? rtrim($relativePath, '/').'/' : $relativePath;
     }
 
+    /**
+     * Returns the relative path to where a new class should live.
+     *
+     * @param string $className
+     *
+     * @return null|string
+     *
+     * @throws \Exception
+     */
     public function getPathForFutureClass(string $className)
     {
         // lookup is obviously modeled off of Composer's autoload logic
@@ -140,7 +158,7 @@ private function getClassLoader(): ClassLoader
         return self::$classLoader;
     }
 
-    private function absolutizePath($path): string
+    public function absolutizePath($path): string
     {
         if (0 === strpos($path, '/')) {
             return $path;

src/Test/MakerTestCase.php

@@ -34,8 +34,8 @@ class MakerTestCase extends TestCase
     public static function setupPaths()
     {
         self::$currentRootDir = __DIR__.'/../../tests/tmp/current_project';
-        self::$flexProjectPath = __DIR__.'/../../tests/tmp/template_project';
         self::$fixturesCachePath = __DIR__.'/../../tests/tmp/cache';
+        self::$flexProjectPath = self::$fixturesCachePath.'/flex_project';
     }
 
     protected function executeMakerCommand(MakerTestDetails $testDetails)
@@ -133,7 +133,7 @@ protected function executeMakerCommand(MakerTestDetails $testDetails)
             // execute the tests that were moved into the project!
             $process = $this->createProcess(
                 // using OUR simple-phpunit for speed (to avoid downloading more deps)
-                '../../../vendor/bin/simple-phpunit',
+                __DIR__.'/../../vendor/bin/simple-phpunit',
                 self::$currentRootDir
             );
             $process->run();
@@ -155,7 +155,11 @@ protected function assertContainsCount(string $needle, string $haystack, int $co
 
     private function buildFlexProject()
     {
-        $process = $this->createProcess('composer create-project symfony/skeleton template_project', dirname(self::$flexProjectPath));
+        $targetFlexDir = dirname(self::$flexProjectPath);
+        if (!file_exists($targetFlexDir)) {
+            self::$fs->mkdir($targetFlexDir);
+        }
+        $process = $this->createProcess('composer create-project symfony/skeleton flex_project', $targetFlexDir);
         $this->runProcess($process);
 
         // processes any changes needed to the Flex project
@@ -168,7 +172,7 @@ private function buildFlexProject()
             [
                 'filename' => 'composer.json',
                 'find' => '"App\\\Tests\\\": "tests/"',
-                'replace' => sprintf('"App\\\Tests\\\": "tests/",'."\n".'            "Symfony\\\Bundle\\\MakerBundle\\\": "%s/src/"', __DIR__.'../../../'),
+                'replace' => sprintf('"App\\\Tests\\\": "tests/",'."\n".'            "Symfony\\\Bundle\\\MakerBundle\\\": "%s/src/"', str_replace('\\', '\\\\', __DIR__.'../../..')),
             ],
         ];
         $this->processReplacements($replacements, self::$flexProjectPath);

tests/DependencyBuilderTest.php

@@ -55,6 +55,8 @@ public function testGetMissingPackagesMessage(array $missingPackages, array $mis
             $depBuilder->addClassDependency('Foo', $missingPackage, true, true);
         }
 
+        // normalize line breaks on Windows for comparison
+        $expectedMessage = str_replace("\r\n", "\n", $expectedMessage);
         $this->assertSame($expectedMessage, $depBuilder->getMissingPackagesMessage('make:something'));
     }
 

tests/FileManagerTest.php

@@ -102,4 +102,34 @@ public function getPathForFutureClassTests()
             'Psr0\Package\Admin\Bar' => 'lib/other/Psr0/Package/Admin/Bar.php'
         ];
     }
+
+    /**
+     * @dataProvider getAbsolutePathTests
+     */
+    public function testAbsolutizePath(string $rootDir, string $path, string $expectedPath)
+    {
+        $fileManager = new FileManager(new Filesystem(), $rootDir);
+        $this->assertSame($expectedPath, $fileManager->absolutizePath($path));
+    }
+
+    public function getAbsolutePathTests()
+    {
+        yield 'normal_path_change' => [
+            '/home/project/',
+            'foo/bar',
+            '/home/project/foo/bar',
+        ];
+
+        yield 'already_absolute_path' => [
+            '/home/project/',
+            '/foo/bar',
+            '/foo/bar',
+        ];
+
+        yield 'windows_already_absolute_path' => [
+            'D:\path\to\project',
+            'D:\foo\bar',
+            'D:\foo\bar',
+        ];
+    }
 }

tests/Maker/FunctionalTest.php

@@ -154,7 +154,7 @@ public function getCommandTests()
                 'mysql://db_user:[email protected]:3306/db_name',
                 'sqlite:///%kernel.project_dir%/var/app.db'
             )
-            ->addPostMakeCommand('./bin/console doctrine:schema:create --env=test')
+            ->addPostMakeCommand('php bin/console doctrine:schema:create --env=test')
         ];
 
         yield 'fixtures' => [MakerTestDetails::createTest(
@@ -293,7 +293,7 @@ public function getCommandTests()
             )
             ->addExtraDependencies('doctrine/orm')
             // sync the database, so no changes are needed
-            ->addPreMakeCommand('./bin/console doctrine:schema:create --env=test')
+            ->addPreMakeCommand('php bin/console doctrine:schema:create --env=test')
             ->assert(function(string $output, string $directory) {
                 $this->assertNotContains('Success', $output);