add more unit tests and fixed the output v2

Author: voku

src/DocBlock/Tags/Param.php

@@ -140,11 +140,11 @@ public function isReference() : bool
      */
     public function __toString() : string
     {
-        return ($this->type ? $this->type . ' ' : '')
+        return ($this->type ? $this->type . ($this->variableName ? ' ' : '') : '')
             . ($this->isReference() ? '&' : '')
             . ($this->isVariadic() ? '...' : '')
             . ($this->variableName ? '$' . $this->variableName : '')
-            . ($this->description ? ($this->variableName ? ' ' : '') . $this->description : '');
+            . (('' . $this->description) ? ' ' . $this->description : '');
     }
 
     private static function strStartsWithVariable(string $str) : bool

src/DocBlock/Tags/Property.php

@@ -98,8 +98,8 @@ public function getVariableName() : ?string
      */
     public function __toString() : string
     {
-        return ($this->type ? $this->type . ' ' : '')
+        return ($this->type ? $this->type . ($this->variableName ? ' ' : '') : '')
             . ($this->variableName ? '$' . $this->variableName : '')
-            . ($this->description ? ($this->variableName ? ' ' : '') . $this->description : '');
+            . (('' . $this->description) ? ' ' . $this->description : '');
     }
 }

src/DocBlock/Tags/PropertyRead.php

@@ -98,8 +98,8 @@ public function getVariableName() : ?string
      */
     public function __toString() : string
     {
-        return ($this->type ? $this->type . ' ' : '')
+        return ($this->type ? $this->type . ($this->variableName ? ' ' : '') : '')
             . ($this->variableName ? '$' . $this->variableName : '')
-            . ($this->description ? ($this->variableName ? ' ' : '') . $this->description : '');
+            . (('' . $this->description) ? ' ' . $this->description : '');
     }
 }

src/DocBlock/Tags/PropertyWrite.php

@@ -98,8 +98,8 @@ public function getVariableName() : ?string
      */
     public function __toString() : string
     {
-        return ($this->type ? $this->type . ' ' : '')
+        return ($this->type ? $this->type . ($this->variableName ? ' ' : '') : '')
             . ($this->variableName ? '$' . $this->variableName : '')
-            . ($this->description ? ($this->variableName ? ' ' : '') . $this->description : '');
+            . (('' . $this->description) ? ' ' . $this->description : '');
     }
 }

src/DocBlock/Tags/Var_.php

@@ -99,8 +99,8 @@ public function getVariableName() : ?string
      */
     public function __toString() : string
     {
-        return ($this->type ? $this->type . ' ' : '')
+        return ($this->type ? $this->type . ($this->variableName ? ' ' : '') : '')
             . ($this->variableName ? '$' . $this->variableName : '')
-            . ($this->description ? ($this->variableName ? ' ' : '') . $this->description : '');
+            . (('' . $this->description) ? ' ' . $this->description : '');
     }
 }

tests/unit/DocBlock/Tags/VarTest.php

@@ -237,6 +237,35 @@ public function testFactoryMethodWithType() : void
         $this->assertSame('My Description', $fixture->getDescription() . '');
     }
 
+    /**
+     * @uses \phpDocumentor\Reflection\DocBlock\Tags\Param::<public>
+     * @uses \phpDocumentor\Reflection\DocBlock\DescriptionFactory
+     * @uses \phpDocumentor\Reflection\DocBlock\Description
+     * @uses \phpDocumentor\Reflection\Types\Context
+     *
+     * @covers ::create
+     */
+    public function testFactoryMethodWithTypeWithoutComment() : void
+    {
+        $typeResolver       = new TypeResolver();
+        $fqsenResolver      = new FqsenResolver();
+        $tagFactory         = new StandardTagFactory($fqsenResolver);
+        $descriptionFactory = new DescriptionFactory($tagFactory);
+        $context            = new Context('');
+
+        $fixture = Var_::create(
+            'int',
+            $typeResolver,
+            $descriptionFactory,
+            $context
+        );
+
+        $this->assertSame('int', (string) $fixture);
+        $this->assertSame('', $fixture->getVariableName());
+        $this->assertInstanceOf(Integer::class, $fixture->getType());
+        $this->assertSame('', $fixture->getDescription() . '');
+    }
+
     /**
      * @uses \phpDocumentor\Reflection\DocBlock\Tags\Var_::<public>
      * @uses \phpDocumentor\Reflection\TypeResolver