Add tests

PHP-CS-Fixer · Dec 5, 2017 · 74fe243 · 74fe243
1 parent 9eb9cc8
commit 74fe243
Show file tree

Hide file tree

Showing 17 changed files with 381 additions and 174 deletions.
diff --git a/src/Fixer/Import/NoUnusedImportsFixer.php b/src/Fixer/Import/NoUnusedImportsFixer.php
@@ -96,7 +96,7 @@ protected function applyFix(\SplFileInfo $file, Tokens $tokens)
     }
 
     /**
-     * @param string $content
+     * @param string                 $content
      * @param NamespaceUseAnalysis[] $useDeclarations
      *
      * @return array
@@ -113,7 +113,7 @@ private function detectUseUsages($content, array $useDeclarations)
     }
 
     /**
-     * @param Tokens $tokens
+     * @param Tokens                       $tokens
      * @param StartEndTokenAwareAnalysis[] $partials
      *
      * @return string
@@ -217,9 +217,9 @@ private function removeUseDeclaration(Tokens $tokens, NamespaceUseAnalysis $useD
     }
 
     /**
-     * @param Tokens $tokens
+     * @param Tokens                 $tokens
      * @param NamespaceUseAnalysis[] $useDeclarations
-     * @param NamespaceAnalysis[] $namespaceDeclarations
+     * @param NamespaceAnalysis[]    $namespaceDeclarations
      */
     private function removeUsesInSameNamespace(Tokens $tokens, array $useDeclarations, array $namespaceDeclarations)
     {

diff --git a/src/Tokenizer/Analyzer/Analysis/ArgumentAnalysis.php b/src/Tokenizer/Analyzer/Analysis/ArgumentAnalysis.php
@@ -17,11 +17,10 @@
  */
 final class ArgumentAnalysis
 {
-
     /**
      * The default value of the argument.
      *
-     * @var string|null
+     * @var null|string
      */
     private $default;
 
@@ -42,28 +41,29 @@ final class ArgumentAnalysis
     /**
      * The type of the argument.
      *
-     * @var string|null
+     * @var null|string
      */
     private $type;
 
     /**
-     * @var int|null
+     * @var null|int
      */
     private $typeIndexStart;
 
     /**
-     * @var int|null
+     * @var null|int
      */
     private $typeIndexEnd;
 
     /**
      * ArgumentAnalysis constructor.
-     * @param string $name
-     * @param int $nameIndex
-     * @param string|null $default
-     * @param string|null $type
-     * @param int|null $typeIndexStart
-     * @param int|null $typeIndexEnd
+     *
+     * @param string      $name
+     * @param int         $nameIndex
+     * @param null|string $default
+     * @param null|string $type
+     * @param null|int    $typeIndexStart
+     * @param null|int    $typeIndexEnd
      */
     public function __construct($name, $nameIndex, $default, $type, $typeIndexStart, $typeIndexEnd)
     {
@@ -124,15 +124,15 @@ public function hasType()
     }
 
     /**
-     * @return int|null
+     * @return null|int
      */
     public function getTypeIndexStart()
     {
         return $this->typeIndexStart;
     }
 
     /**
-     * @return int|null
+     * @return null|int
      */
     public function getTypeIndexEnd()
     {

diff --git a/src/Tokenizer/Analyzer/Analysis/NamespaceAnalysis.php b/src/Tokenizer/Analyzer/Analysis/NamespaceAnalysis.php
@@ -47,10 +47,11 @@ final class NamespaceAnalysis implements StartEndTokenAwareAnalysis
 
     /**
      * NamespaceAnalysis constructor.
+     *
      * @param string $fullName
      * @param string $shortName
-     * @param int $startIndex
-     * @param int $endIndex
+     * @param int    $startIndex
+     * @param int    $endIndex
      */
     public function __construct($fullName, $shortName, $startIndex, $endIndex)
     {

diff --git a/src/Tokenizer/Analyzer/Analysis/NamespaceUseAnalysis.php b/src/Tokenizer/Analyzer/Analysis/NamespaceUseAnalysis.php
@@ -54,11 +54,12 @@ final class NamespaceUseAnalysis implements StartEndTokenAwareAnalysis
 
     /**
      * NamespaceAnalysis constructor.
+     *
      * @param string $fullName
      * @param string $shortName
-     * @param bool $aliased
-     * @param int $startIndex
-     * @param int $endIndex
+     * @param bool   $aliased
+     * @param int    $startIndex
+     * @param int    $endIndex
      */
     public function __construct($fullName, $shortName, $aliased, $startIndex, $endIndex)
     {

diff --git a/src/Tokenizer/Analyzer/Analysis/StartEndTokenAwareAnalysis.php b/src/Tokenizer/Analyzer/Analysis/StartEndTokenAwareAnalysis.php
@@ -1,6 +1,14 @@
 <?php
 
-declare(strict_types=1);
+/*
+ * This file is part of PHP CS Fixer.
+ *
+ * (c) Fabien Potencier <fabien@symfony.com>
+ *     Dariusz Rumiński <dariusz.ruminski@gmail.com>
+ *
+ * This source file is subject to the MIT license that is bundled
+ * with this source code in the file LICENSE.
+ */
 
 namespace PhpCsFixer\Tokenizer\Analyzer\Analysis;
 

diff --git a/src/Tokenizer/Analyzer/FunctionsAnalyzer.php b/src/Tokenizer/Analyzer/FunctionsAnalyzer.php
@@ -46,7 +46,7 @@ public function getFunctionArguments(Tokens $tokens, $methodIndex)
      * @param Tokens $tokens
      * @param int    $methodIndex
      *
-     * @return FunctionReturnTypeAnalysis|null
+     * @return null|FunctionReturnTypeAnalysis
      */
     public function getFunctionReturnType(Tokens $tokens, $methodIndex)
     {

diff --git a/src/Tokenizer/Analyzer/NamespaceUsesAnalyzer.php b/src/Tokenizer/Analyzer/NamespaceUsesAnalyzer.php
@@ -19,7 +19,7 @@
 /**
  * @internal
  */
-class NamespaceUsesAnalyzer
+final class NamespaceUsesAnalyzer
 {
     /**
      * @param Tokens $tokens

diff --git a/src/Tokenizer/Analyzer/NamespacesAnalyzer.php b/src/Tokenizer/Analyzer/NamespacesAnalyzer.php
@@ -22,6 +22,7 @@ final class NamespacesAnalyzer
 {
     /**
      * @param Tokens $tokens
+     *
      * @return NamespaceAnalysis[]
      */
     public function getDeclarations(Tokens $tokens)
-Original file line number
+Diff line change
@@ Expand Up / @@ -22,6 +22,7 @@ final class NamespacesAnalyzer @@
     {
         /**
          * @param Tokens $tokens
+         *
          * @return NamespaceAnalysis[]
          */
         public function getDeclarations(Tokens $tokens)
@@ Expand Down @@