Browse Source

Normalize todos (#19045)

tags/2.0.44
Alexey Rogachev 3 years ago committed by GitHub
parent
commit
4a8c72f0c4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      .php_cs
  2. 18
      tests/framework/db/ActiveQueryTest.php
  3. 2
      tests/framework/widgets/ActiveFieldTest.php

2
.php_cs

@ -1,7 +1,7 @@
<?php <?php
if (!class_exists('yii\cs\YiisoftConfig', true)) { if (!class_exists('yii\cs\YiisoftConfig', true)) {
// @todo change error message // TODO: change error message
fwrite(STDERR, "Your php-cs-version is outdated: please upgrade it.\n"); fwrite(STDERR, "Your php-cs-version is outdated: please upgrade it.\n");
die(16); die(16);
} }

18
tests/framework/db/ActiveQueryTest.php

@ -53,7 +53,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of prepare() * @todo tests for internal logic of prepare()
*/ */
public function testPrepare() public function testPrepare()
{ {
@ -72,7 +72,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of populate() * @todo tests for internal logic of populate()
*/ */
public function testPopulate_FilledRows() public function testPopulate_FilledRows()
{ {
@ -83,7 +83,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of one() * @todo tests for internal logic of one()
*/ */
public function testOne() public function testOne()
{ {
@ -93,7 +93,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: test internal logic of createCommand() * @todo test internal logic of createCommand()
*/ */
public function testCreateCommand() public function testCreateCommand()
{ {
@ -103,7 +103,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of queryScalar() * @todo tests for internal logic of queryScalar()
*/ */
public function testQueryScalar() public function testQueryScalar()
{ {
@ -113,7 +113,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of joinWith() * @todo tests for internal logic of joinWith()
*/ */
public function testJoinWith() public function testJoinWith()
{ {
@ -125,7 +125,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of innerJoinWith() * @todo tests for internal logic of innerJoinWith()
*/ */
public function testInnerJoinWith() public function testInnerJoinWith()
{ {
@ -157,7 +157,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for the regex inside getQueryTableName * @todo tests for the regex inside getQueryTableName
*/ */
public function testGetQueryTableName_from_not_set() public function testGetQueryTableName_from_not_set()
{ {
@ -231,7 +231,7 @@ abstract class ActiveQueryTest extends DatabaseTestCase
} }
/** /**
* @todo: tests for internal logic of viaTable() * @todo tests for internal logic of viaTable()
*/ */
public function testViaTable() public function testViaTable()
{ {

2
tests/framework/widgets/ActiveFieldTest.php

@ -78,7 +78,7 @@ EOD;
} }
/** /**
* @todo discuss|review Expected HTML shouldn't be wrapped only by the $content? * @todo discuss|review Expected HTML shouldn't be wrapped only by the $content?
*/ */
public function testRenderWithCallableContent() public function testRenderWithCallableContent()
{ {

Loading…
Cancel
Save