Treat trait properties as definitions (#40)

* Tests: add test for traits with properties

* findVariableScope: consider traits also

Author: sirbrillig

VariableAnalysis/Lib/Helpers.php

@@ -174,7 +174,7 @@ public static function findVariableScope(File $phpcsFile, int $stackPtr) {
         if (($scopeCode === T_FUNCTION) || ($scopeCode === T_CLOSURE)) {
           return $scopePtr;
         }
-        if (($scopeCode === T_CLASS) || ($scopeCode === T_INTERFACE)) {
+        if (in_array($scopeCode, [T_CLASS, T_INTERFACE, T_TRAIT])) {
           $in_class = true;
         }
       }

VariableAnalysis/Tests/CodeAnalysis/VariableAnalysisTest.php

@@ -401,4 +401,16 @@ public function testVariableFunctionCallsCountAsUsage() {
     $expectedErrors = [];
     $this->assertEquals($expectedErrors, $lines);
   }
+
+  public function testTraitsAllowPropertyDefinitions() {
+    $fixtureFile = $this->getFixture('TraitWithPropertiesFixture.php');
+    $phpcsFile = $this->prepareLocalFileForSniffs($this->getSniffFiles(), $fixtureFile);
+    $phpcsFile->process();
+    $lines = $this->getWarningLineNumbersFromFile($phpcsFile);
+    $expectedWarnings = [];
+    $this->assertEquals($expectedWarnings, $lines);
+    $lines = $this->getErrorLineNumbersFromFile($phpcsFile);
+    $expectedErrors = [];
+    $this->assertEquals($expectedErrors, $lines);
+  }
 }

VariableAnalysis/Tests/CodeAnalysis/fixtures/TraitWithPropertiesFixture.php

@@ -0,0 +1,11 @@
+<?php
+
+trait Hello {
+    protected $storedHello;
+    public $helloOptions = [];
+    public function sayHelloWorld() {
+        echo "hello world";
+    }
+    abstract public function getWorld();
+}
+