================================= Rule ``php_unit_dedicate_assert`` ================================= PHPUnit assertions like ``assertInternalType``, ``assertFileExists``, should be used over ``assertTrue``. Warning ------- Using this rule is risky ~~~~~~~~~~~~~~~~~~~~~~~~ Fixer could be risky if one is overriding PHPUnit's native methods. Configuration ------------- ``target`` ~~~~~~~~~~ Target version of PHPUnit. Allowed values: ``'3.0'``, ``'3.5'``, ``'5.0'``, ``'5.6'`` and ``'newest'`` Default value: ``'newest'`` Examples -------- Example #1 ~~~~~~~~~~ *Default* configuration. .. code-block:: diff --- Original +++ New assertTrue(is_float( $a), "my message"); - $this->assertTrue(is_nan($a)); + $this->assertInternalType('float', $a, "my message"); + $this->assertNan($a); } } Example #2 ~~~~~~~~~~ With configuration: ``['target' => '5.6']``. .. code-block:: diff --- Original +++ New assertTrue(is_dir($a)); - $this->assertTrue(is_writable($a)); - $this->assertTrue(is_readable($a)); + $this->assertDirectoryExists($a); + $this->assertIsWritable($a); + $this->assertIsReadable($a); } } Rule sets --------- The rule is part of the following rule sets: - `@PHPUnit30Migration:risky <./../../ruleSets/PHPUnit30MigrationRisky.rst>`_ with config: ``['target' => '3.0']`` - `@PHPUnit32Migration:risky <./../../ruleSets/PHPUnit32MigrationRisky.rst>`_ with config: ``['target' => '3.0']`` - `@PHPUnit35Migration:risky <./../../ruleSets/PHPUnit35MigrationRisky.rst>`_ with config: ``['target' => '3.5']`` - `@PHPUnit43Migration:risky <./../../ruleSets/PHPUnit43MigrationRisky.rst>`_ with config: ``['target' => '3.5']`` - `@PHPUnit48Migration:risky <./../../ruleSets/PHPUnit48MigrationRisky.rst>`_ with config: ``['target' => '3.5']`` - `@PHPUnit50Migration:risky <./../../ruleSets/PHPUnit50MigrationRisky.rst>`_ with config: ``['target' => '5.0']`` - `@PHPUnit52Migration:risky <./../../ruleSets/PHPUnit52MigrationRisky.rst>`_ with config: ``['target' => '5.0']`` - `@PHPUnit54Migration:risky <./../../ruleSets/PHPUnit54MigrationRisky.rst>`_ with config: ``['target' => '5.0']`` - `@PHPUnit55Migration:risky <./../../ruleSets/PHPUnit55MigrationRisky.rst>`_ with config: ``['target' => '5.0']`` - `@PHPUnit56Migration:risky <./../../ruleSets/PHPUnit56MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit57Migration:risky <./../../ruleSets/PHPUnit57MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit60Migration:risky <./../../ruleSets/PHPUnit60MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit75Migration:risky <./../../ruleSets/PHPUnit75MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit84Migration:risky <./../../ruleSets/PHPUnit84MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit91Migration:risky <./../../ruleSets/PHPUnit91MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` - `@PHPUnit100Migration:risky <./../../ruleSets/PHPUnit100MigrationRisky.rst>`_ with config: ``['target' => '5.6']`` References ---------- - Fixer class: `PhpCsFixer\\Fixer\\PhpUnit\\PhpUnitDedicateAssertFixer <./../../../src/Fixer/PhpUnit/PhpUnitDedicateAssertFixer.php>`_ - Test class: `PhpCsFixer\\Tests\\Fixer\\PhpUnit\\PhpUnitDedicateAssertFixerTest <./../../../tests/Fixer/PhpUnit/PhpUnitDedicateAssertFixerTest.php>`_ The test class defines officially supported behaviour. Each test case is a part of our backward compatibility promise.