change StaticCodeAnalysis to return line 0 for namespace if no namespace is in file

Author: pscheit

src/main/QafooLabs/Refactoring/Adapters/TokenReflection/StaticCodeAnalysis.php

@@ -22,6 +22,7 @@
 
 use TokenReflection\Broker;
 use TokenReflection\Broker\Backend\Memory;
+use TokenReflection\ReflectionNamespace;
 
 class StaticCodeAnalysis extends CodeAnalysis
 {
@@ -143,11 +144,12 @@ public function findClasses(File $file)
 
         $file = $this->broker->processString($file->getCode(), $file->getRelativePath(), true);
         foreach ($file->getNamespaces() as $namespace) {
+            $noNamespace = ReflectionNamespace::NO_NAMESPACE_NAME === $namespace->getName();
             foreach ($namespace->getClasses() as $class) {
                 $classes[] = new PhpClass(
                     PhpName::createDeclarationName($class->getName()),
                     $class->getStartLine(),
-                    $namespace->getStartLine()
+                    $noNamespace ? 0 : $namespace->getStartLine()
                 );
             }
         }

src/test/QafooLabs/Refactoring/Adapters/TokenReflection/StaticCodeAnalysisTest.php

@@ -0,0 +1,28 @@
+<?php
+
+namespace QafooLabs\Refactoring\Adapters\TokenReflection;
+
+use PHPUnit_Framework_TestCase;
+use QafooLabs\Refactoring\Domain\Model\File;
+
+class StaticCodeAnalysisTest extends PHPUnit_Framework_TestCase 
+{
+    public function testNamespaceDeclarationForFileWithoutNamespace_isInLine0()
+    {
+        $file = new File('without-namespace.php', <<<'PHP'
+<?php
+
+class WithoutNamespace
+{
+
+}
+PHP
+        );
+
+        $analysis = new StaticCodeAnalysis();
+        $classes = $analysis->findClasses($file);
+        $class = $classes[0];
+
+        $this->assertEquals(0, $class->namespaceDeclarationLine(), 'namespace declaration line for file without namespace');
+    }
+}