bugfix Factory::create

CHANGELOG.md

@@ -13,6 +13,7 @@ when the `intl` extension is missing or the ICU minimum version is not supported
 ### Fixed
 
 - `Factory::create` now always remove dot segments
+- `Factory::create` create with a relative path
 - Using PHPstan
 - Improve Host parsing according to RFC3986
 - Improve Parsing performance

src/Factory.php

@@ -192,13 +192,21 @@ protected function getClassName($scheme, $base_uri = null): string
      */
     protected function newInstance(array $components, string $className)
     {
-        return (new ReflectionClass($className))
+        $uri = (new ReflectionClass($className))
             ->newInstanceWithoutConstructor()
             ->withHost($components['host'] ?? '')
             ->withPort($components['port'] ?? null)
             ->withUserInfo($components['user'] ?? '', $components['pass'] ?? null)
             ->withScheme($components['scheme'] ?? '')
-            ->withPath($components['path'] ?? '')
+        ;
+
+        $path = $components['path'] ?? '';
+        if ('' !== $uri->getAuthority() && '' !== $path && '/' !== $path[0]) {
+            $path = '/'.$path;
+        }
+
+        return $uri
+            ->withPath($path)
             ->withQuery($components['query'] ?? '')
             ->withFragment($components['fragment'] ?? '')
         ;

tests/FactoryTest.php

@@ -203,6 +203,12 @@ public function uriBaseUriProvider()
                 'uri' => 'https://EXAMPLE.com/../x',
                 'base_uri' => null,
             ],
+            'relative path with absolute path' => [
+                'expected_class' => Uri\File::class,
+                'expected_uri' => 'file://localhost/',
+                'uri' => '..',
+                'base_uri' => 'file://C:/demo',
+            ],
         ];
     }