Fix an issue where missing .env files result in an endless loop on Wi…

…ndows, refs #34

src/Environment/LoaderFactory.php

@@ -8,13 +8,11 @@
 use FFraenz\PrivateComposerInstaller\Environment\LoaderInterface;
 
 use function class_exists;
+use function count;
 use function dirname;
 use function getcwd;
-use function in_array;
 use function realpath;
 
-use const DIRECTORY_SEPARATOR;
-
 class LoaderFactory
 {
     /**
@@ -38,13 +36,14 @@ public static function create(?string $path = null, ?string $name = null): Loade
      *
      * @return string[]
      */
-    private static function computePaths(string $path): array
+    public static function computePaths(string $path): array
     {
         $paths = [$path];
+        $path  = dirname($path);
 
-        while (! in_array($path, ['.', DIRECTORY_SEPARATOR], true)) {
-            $path    = dirname($path);
+        while ($paths[count($paths) - 1] !== $path) {
             $paths[] = $path;
+            $path    = dirname($path);
         }
 
         return $paths;

test/unit/Environment/LoaderFactoryTest.php

@@ -0,0 +1,31 @@
+<?php
+
+namespace FFraenz\PrivateComposerInstaller\Test\Environment;
+
+use FFraenz\PrivateComposerInstaller\Environment\LoaderFactory;
+use PHPUnit\Framework\TestCase;
+
+use function implode;
+
+class LoaderFactoryTest extends TestCase
+{
+    public function testComputePaths()
+    {
+        self::assertSame(
+            '.',
+            implode(',', LoaderFactory::computePaths('.'))
+        );
+        self::assertSame(
+            '/',
+            implode(',', LoaderFactory::computePaths('/'))
+        );
+        self::assertSame(
+            'foo/bar,foo,.',
+            implode(',', LoaderFactory::computePaths('foo/bar'))
+        );
+        self::assertSame(
+            '/foo/bar,/foo,/',
+            implode(',', LoaderFactory::computePaths('/foo/bar'))
+        );
+    }
+}