| @@ 12-38 (lines=27) @@ | ||
| 9 | * |
|
| 10 | * @package MS\PHPMD\Tests\Functional\CleanCode |
|
| 11 | */ |
|
| 12 | class MethodOneTryCatchTest extends AbstractProcessTest |
|
| 13 | { |
|
| 14 | /** |
|
| 15 | * @covers MS\PHPMD\Rule\CleanCode\MethodOneTryCatch |
|
| 16 | */ |
|
| 17 | public function testMethodOneTryCatchRule() |
|
| 18 | { |
|
| 19 | $output = $this |
|
| 20 | ->runPhpmd('Utility/TryThings.php', 'cleancode.xml') |
|
| 21 | ->getOutput(); |
|
| 22 | ||
| 23 | $this->assertContains('Utility/TryThings.php:39 This method contains more than one try statement. Swap out the try statement in an extra method. It increase the readability.', $output); |
|
| 24 | $this->assertContains('Utility/TryThings.php:53 This method contains more than one try statement. Swap out the try statement in an extra method. It increase the readability.', $output); |
|
| 25 | } |
|
| 26 | ||
| 27 | /** |
|
| 28 | * @covers MS\PHPMD\Rule\CleanCode\MethodOneTryCatch |
|
| 29 | */ |
|
| 30 | public function testRuleWithoutTry() |
|
| 31 | { |
|
| 32 | $output = $this |
|
| 33 | ->runPhpmd('Entity.php', 'cleancode.xml') |
|
| 34 | ->getOutput(); |
|
| 35 | ||
| 36 | $this->assertEmpty(trim($output)); |
|
| 37 | } |
|
| 38 | } |
|
| 39 | ||
| @@ 12-30 (lines=19) @@ | ||
| 9 | * |
|
| 10 | * @package MS\PHPMD\Tests\Functional\CleanCode |
|
| 11 | */ |
|
| 12 | class SuperfluousCommentTest extends AbstractProcessTest |
|
| 13 | { |
|
| 14 | /** |
|
| 15 | * @covers MS\PHPMD\Rule\CleanCode\SuperfluousComment |
|
| 16 | */ |
|
| 17 | public function testSuperfluousCommentRule() |
|
| 18 | { |
|
| 19 | $output = $this |
|
| 20 | ->runPhpmd('Service/SuperfluousComment.php', 'cleancode.xml') |
|
| 21 | ->getOutput(); |
|
| 22 | ||
| 23 | $this->assertNotContains('SuperfluousComment.php:42 It seems', $output); |
|
| 24 | ||
| 25 | $this->assertContains('SuperfluousComment.php:10 It seems that the class has a superfluous comment description. It fits 88 percent with the name of that.', $output); |
|
| 26 | $this->assertContains('SuperfluousComment.php:10 It seems that the property $name has a superfluous comment description. It fits 73 percent with the name of that.', $output); |
|
| 27 | $this->assertContains('SuperfluousComment.php:24 It seems that the method has a superfluous comment description. It fits 100 percent with the name of that.', $output); |
|
| 28 | $this->assertContains('SuperfluousComment.php:34 It seems that the method has a superfluous comment description. It fits 82 percent with the name of that.', $output); |
|
| 29 | } |
|
| 30 | } |
|
| 31 | ||
| @@ 12-30 (lines=19) @@ | ||
| 9 | * |
|
| 10 | * @package MS\PHPMD\Tests\Functional\Test |
|
| 11 | */ |
|
| 12 | class MethodNameUnderstandableTest extends AbstractProcessTest |
|
| 13 | { |
|
| 14 | /** |
|
| 15 | * @covers MS\PHPMD\Rule\Test\MethodNameUnderstandable |
|
| 16 | * @covers MS\PHPMD\Rule\Test\AbstractTestRule |
|
| 17 | */ |
|
| 18 | public function testMethodNameUnderstandableRule() |
|
| 19 | { |
|
| 20 | $output = $this |
|
| 21 | ->runPhpmd('Tests/Test.php', 'test.xml') |
|
| 22 | ->getOutput(); |
|
| 23 | ||
| 24 | $this->assertNotContains('Test.php:13', $output); |
|
| 25 | $this->assertNotContains('Test.php:25', $output); |
|
| 26 | ||
| 27 | $this->assertContains('Test.php:37 Only 1 words are found in the method name. Try to describe your code as good as you can with 3 or more words.', $output); |
|
| 28 | $this->assertContains('Test.php:52 Only 1 words are found in the method name. Try to describe your code as good as you can with 3 or more words.', $output); |
|
| 29 | } |
|
| 30 | } |
|
| 31 | ||
| @@ 12-38 (lines=27) @@ | ||
| 9 | * |
|
| 10 | * @package PHPMD\Tests\Functional\Symfony2 |
|
| 11 | */ |
|
| 12 | class ControllerMethodNameTest extends AbstractProcessTest |
|
| 13 | { |
|
| 14 | /** |
|
| 15 | * @covers MS\PHPMD\Rule\Symfony2\ControllerMethodName |
|
| 16 | */ |
|
| 17 | public function testControllerMethodNameRule() |
|
| 18 | { |
|
| 19 | $output = $this |
|
| 20 | ->runPhpmd('Controller/FooController.php', 'symfony2.xml') |
|
| 21 | ->getOutput(); |
|
| 22 | ||
| 23 | $this->assertNotContains('Controller/FooController.php:18 The method name should end with Action in this controller.', $output); |
|
| 24 | $this->assertContains('Controller/FooController.php:33 The method name should end with Action in this controller.', $output); |
|
| 25 | } |
|
| 26 | ||
| 27 | /** |
|
| 28 | * @covers MS\PHPMD\Rule\Symfony2\ControllerMethodName |
|
| 29 | */ |
|
| 30 | public function testRuleWithAbstractClass() |
|
| 31 | { |
|
| 32 | $output = $this |
|
| 33 | ->runPhpmd('Controller/AbstractFooController.php', 'symfony2.xml') |
|
| 34 | ->getOutput(); |
|
| 35 | ||
| 36 | $this->assertEmpty(trim($output)); |
|
| 37 | } |
|
| 38 | } |
|
| 39 | ||