Fix CS

dereuromark · Feb 27, 2016 · ed475a6 · ed475a6
1 parent 0e9800e
commit ed475a6
Show file tree

Hide file tree

Showing 31 changed files with 158 additions and 16 deletions.
diff --git a/tests/Fixture/JsonableCommentsFixture.php b/tests/Fixture/JsonableCommentsFixture.php
@@ -6,6 +6,9 @@
 
 class JsonableCommentsFixture extends TestFixture {
 
+	/**
+	 * @var array
+	 */
 	public $fields = [
 		'id' => ['type' => 'integer'],
 		'comment' => ['type' => 'string', 'length' => 255, 'null' => false],
@@ -15,6 +18,9 @@ class JsonableCommentsFixture extends TestFixture {
 		'_constraints' => ['primary' => ['type' => 'primary', 'columns' => ['id']]]
 	];
 
+	/**
+	 * @var array
+	 */
 	public $records = [
 	];
 

diff --git a/tests/Fixture/StringCommentsFixture.php b/tests/Fixture/StringCommentsFixture.php
@@ -6,6 +6,9 @@
 
 class StringCommentsFixture extends TestFixture {
 
+	/**
+	 * @var array
+	 */
 	public $fields = [
 		'id' => ['type' => 'integer'],
 		'comment' => ['type' => 'string', 'length' => 255, 'null' => false],
@@ -14,6 +17,9 @@ class StringCommentsFixture extends TestFixture {
 		'_constraints' => ['primary' => ['type' => 'primary', 'columns' => ['id']]]
 	];
 
+	/**
+	 * @var array
+	 */
 	public $records = [
 	];
 

diff --git a/tests/TestApp/Controller/Component/TestComponent.php b/tests/TestApp/Controller/Component/TestComponent.php
@@ -7,16 +7,28 @@
 
 class TestComponent extends Component {
 
-	public $Controller;
-
+	/**
+	 * @var bool
+	 */
 	public $isInit = false;
 
+	/**
+	 * @var bool
+	 */
 	public $isStartup = false;
 
+	/**
+	 * @param \Cake\Event\Event $event
+	 * @return void
+	 */
 	public function beforeFilter(Event $event) {
 		$this->isInit = true;
 	}
 
+	/**
+	 * @param \Cake\Event\Event $event
+	 * @return void
+	 */
 	public function startup(Event $event) {
 		$this->isStartup = true;
 	}

diff --git a/tests/TestApp/Model/Entity/BitmaskedComment.php b/tests/TestApp/Model/Entity/BitmaskedComment.php
@@ -6,6 +6,10 @@
 
 class BitmaskedComment extends Entity {
 
+	/**
+	 * @param mixed|null $value
+	 * @return mixed
+	 */
 	public static function statuses($value = null) {
 		$options = [
 			static::STATUS_ACTIVE => __d('tools', 'Active'),

diff --git a/tests/TestApp/Model/Table/BitmaskedCommentsTable.php b/tests/TestApp/Model/Table/BitmaskedCommentsTable.php
@@ -6,6 +6,9 @@
 
 class BitmaskedCommentsTable extends Table {
 
+	/**
+	 * @var array
+	 */
 	public $validate = [
 		'status' => [
 			'notBlank' => [

diff --git a/tests/TestApp/Model/Table/JsonableCommentsTable.php b/tests/TestApp/Model/Table/JsonableCommentsTable.php
@@ -6,6 +6,9 @@
 
 class JsonableCommentsTable extends Table {
 
+	/**
+	 * @var array
+	 */
 	public $validate = [
 	];
 

diff --git a/tests/TestCase/Controller/Component/CommonComponentTest.php b/tests/TestCase/Controller/Component/CommonComponentTest.php
@@ -171,6 +171,9 @@ public function testAutoRedirectReferer() {
  */
 class CommonComponentTestController extends Controller {
 
+	/**
+	 * @var array
+	 */
 	public $components = ['Tools.Common'];
 
 }
diff --git a/tests/TestCase/Controller/Component/FlashComponentTest.php b/tests/TestCase/Controller/Component/FlashComponentTest.php
@@ -121,10 +121,19 @@ public function testAjax() {
  */
 class FlashComponentTestController extends Controller {
 
+	/**
+	 * @var array
+	 */
 	public $components = ['Tools.Flash'];
 
+	/**
+	 * @var bool
+	 */
 	public $failed = false;
 
+	/**
+	 * @var array
+	 */
 	public $testHeaders = [];
 
 	public function fail() {

diff --git a/tests/TestCase/Controller/Component/MobileComponentTest.php b/tests/TestCase/Controller/Component/MobileComponentTest.php
@@ -17,6 +17,9 @@
  */
 class MobileComponentTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['core.sessions'];
 
 	/**

diff --git a/tests/TestCase/Controller/ControllerTest.php b/tests/TestCase/Controller/ControllerTest.php
@@ -16,10 +16,19 @@
  */
 class ControllerTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['plugin.Tools.ToolsUsers'];
 
+	/**
+	 * @var Cake\Controller\Controller
+	 */
 	public $Controller;
 
+	/**
+	 * @return void
+	 */
 	public function setUp() {
 		parent::setUp();
 
@@ -29,6 +38,9 @@ public function setUp() {
 		$this->Controller->startupProcess();
 	}
 
+	/**
+	 * @return void
+	 */
 	public function tearDown() {
 		parent::tearDown();
 

diff --git a/tests/TestCase/Form/ContactFormTest.php b/tests/TestCase/Form/ContactFormTest.php
@@ -19,11 +19,17 @@
 
 class ContactFormTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'core.posts', 'core.authors',
 		'plugin.tools.tools_users', 'plugin.tools.roles',
 	];
 
+	/**
+	 * @var \Tools\Form\ContactForm
+	 */
 	public $Form;
 
 	/**

diff --git a/tests/TestCase/HtmlDom/HtmlDomTest.php b/tests/TestCase/HtmlDom/HtmlDomTest.php
@@ -8,7 +8,10 @@
 
 class HtmlDomTest extends TestCase {
 
-	public $HtmlDom = null;
+	/**
+	 * @var \Tools\HtmlDom\HtmlDom
+	 */
+	public $HtmlDom;
 
 	public function setUp() {
 		parent::setUp();
@@ -22,8 +25,8 @@ public function setUp() {
 	 * @return void
 	 */
 	public function testBasics() {
-		$html = new HtmlDom('<div id="hello">Hello</div><div id="world">World</div>');
-		$result = $html->find('div', 1)->innertext;
+		$this->HtmlDom = new HtmlDom('<div id="hello">Hello</div><div id="world">World</div>');
+		$result = $this->HtmlDom->find('div', 1)->innertext;
 		$this->assertSame('World', $result);
 	}
 

diff --git a/tests/TestCase/Model/Behavior/BitmaskedBehaviorTest.php b/tests/TestCase/Model/Behavior/BitmaskedBehaviorTest.php
@@ -12,10 +12,16 @@
 
 class BitmaskedBehaviorTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'plugin.tools.bitmasked_comments'
 	];
 
+	/**
+	 * @var \Tools\Model\Table\Table
+	 */
 	public $Comments;
 
 	public function setUp() {

diff --git a/tests/TestCase/Model/Behavior/ConfirmableBehaviorTest.php b/tests/TestCase/Model/Behavior/ConfirmableBehaviorTest.php
@@ -8,10 +8,19 @@
 
 class ConfirmableBehaviorTest extends TestCase {
 
+	/**
+	 * @var \Tools\Model\Behavior\ConfirmableBehavior
+	 */
 	public $ConfirmableBehavior;
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['plugin.Tools.SluggedArticles'];
 
+	/**
+	 * @return void
+	 */
 	public function setUp() {
 		parent::setUp();
 	}

diff --git a/tests/TestCase/Model/Behavior/JsonableBehaviorTest.php b/tests/TestCase/Model/Behavior/JsonableBehaviorTest.php
@@ -9,10 +9,16 @@
 
 class JsonableBehaviorTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'plugin.tools.jsonable_comments'
 	];
 
+	/**
+	 * @var \Tools\Model\Table\Table
+	 */
 	public $Comments;
 
 	public function setUp() {

diff --git a/tests/TestCase/Model/Behavior/NeighborBehaviorTest.php b/tests/TestCase/Model/Behavior/NeighborBehaviorTest.php
@@ -10,8 +10,14 @@
 
 class NeighborBehaviorTest extends TestCase {
 
+	/**
+	 * @var \Tools\Model\Table\Table
+	 */
 	public $Table;
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['plugin.tools.stories'];
 
 	public function setUp() {

diff --git a/tests/TestCase/Model/Behavior/PasswordableBehaviorTest.php b/tests/TestCase/Model/Behavior/PasswordableBehaviorTest.php
@@ -17,6 +17,9 @@
 
 class PasswordableBehaviorTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'plugin.tools.tools_users', 'plugin.tools.roles',
 	];

diff --git a/tests/TestCase/Model/Behavior/ResetBehaviorTest.php b/tests/TestCase/Model/Behavior/ResetBehaviorTest.php
@@ -10,10 +10,19 @@
 
 class ResetBehaviorTest extends TestCase {
 
+	/**
+	 * @var \Tools\Model\Behavior\ResetBehavior
+	 */
 	public $ResetBehavior;
 
+	/**
+	 * @var \Tools\Model\Table\Table
+	 */
 	public $Table;
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['plugin.tools.reset_comments'];
 
 	public function setUp() {

diff --git a/tests/TestCase/Model/Behavior/StringBehaviorTest.php b/tests/TestCase/Model/Behavior/StringBehaviorTest.php
@@ -9,10 +9,16 @@
 
 class StringBehaviorTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'plugin.tools.string_comments'
 	];
 
+	/**
+	 * @var \Tools\Model\Table\Table
+	 */
 	public $Comments;
 
 	public function setUp() {

diff --git a/tests/TestCase/Model/Entity/EntityTest.php b/tests/TestCase/Model/Entity/EntityTest.php
@@ -11,10 +11,16 @@
 
 class PasswordableBehaviorTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'plugin.tools.tools_users', 'plugin.tools.roles',
 	];
 
+	/**
+	 * @var \Tools\Model\Table\Table;
+	 */
 	public $Users;
 
 	/**

diff --git a/tests/TestCase/Model/Table/TableTest.php b/tests/TestCase/Model/Table/TableTest.php
@@ -19,11 +19,17 @@
 
 class TableTest extends TestCase {
 
+	/**
+	 * @var array
+	 */
 	public $fixtures = [
 		'core.posts', 'core.authors',
 		'plugin.tools.tools_users', 'plugin.tools.roles',
 	];
 
+	/**
+	 * @var \Tools\Model\Table\Table;
+	 */
 	public $Users;
 
 	/**

diff --git a/tests/TestCase/Model/Table/TokensTableTest.php b/tests/TestCase/Model/Table/TokensTableTest.php
@@ -8,8 +8,14 @@
 
 class TokensTableTest extends TestCase {
 
-	public $Tokens = null;
-
+	/**
+	 * @var \Tools\Model\Table\TokensTable;
+	 */
+	public $Tokens;
+
+	/**
+	 * @var array
+	 */
 	public $fixtures = ['plugin.Tools.Tokens'];
 
 	public function setUp() {

diff --git a/tests/TestCase/TestSuite/IntegrationTestCaseTest.php b/tests/TestCase/TestSuite/IntegrationTestCaseTest.php
@@ -6,8 +6,6 @@
 
 class IntegrationTestCaseTest extends IntegrationTestCase {
 
-	public $TestCase;
-
 	public function setUp() {
 		parent::setUp();
 	}

diff --git a/tests/TestCase/TestSuite/TestCaseTest.php b/tests/TestCase/TestSuite/TestCaseTest.php
@@ -6,8 +6,6 @@
 
 class TestCaseTest extends TestCase {
 
-	public $TestCase;
-
 	public function setUp() {
 		parent::setUp();
 	}