Merge pull request #265 from sensiolabs-de/bugfix/resovle-class-names…

…cope

Bugfix: classes in other namespaces are resolved in same namespace

src/AstRunner/Resolver/NameScope.php

@@ -31,7 +31,7 @@ public function resolveStringName(string $name): string
         }
 
         $nameParts = explode('\\', $name);
-        $firstNamePart = strtolower($nameParts[0]);
+        $firstNamePart = $nameParts[0];
 
         if (isset($this->uses[$firstNamePart])) {
             if (1 === count($nameParts)) {

tests/AstRunner/NameScopeTest.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Tests\SensioLabs\Deptrac\AstRunner;
+
+use PHPUnit\Framework\TestCase;
+use SensioLabs\Deptrac\AstRunner\AstMap\AstDependency;
+use SensioLabs\Deptrac\AstRunner\AstMap\AstFileReference;
+use SensioLabs\Deptrac\AstRunner\Resolver\NameScope;
+
+class NameScopeTest extends TestCase
+{
+    public function testResolveStringName(): void
+    {
+        $fileReference = new AstFileReference('baz.php');
+        $fileReference->addDependency(AstDependency::useStmt('FooBar\OtherNamespace\OtherNamespaceClass', 1));
+        $classReference = $fileReference->addClassReference('FooBar\Baz');
+
+        $nameScope = new NameScope($classReference);
+
+        static::assertSame('FooBar\SameNamespaceClass', $nameScope->resolveStringName('SameNamespaceClass'));
+        static::assertSame('FooBar\OtherNamespace\OtherNamespaceClass', $nameScope->resolveStringName('OtherNamespaceClass'));
+    }
+}