Skip to content

Commit

Permalink
Add unit test for phpmd/phpmd#914
Browse files Browse the repository at this point in the history
  • Loading branch information
kylekatarnls committed Nov 16, 2023
1 parent bfb52ab commit d1243a1
Show file tree
Hide file tree
Showing 3 changed files with 99 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,68 @@
<?php
/**
* This file is part of PDepend.
*
* PHP Version 5
*
* Copyright (c) 2008-2017 Manuel Pichler <mapi@pdepend.org>.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
* are met:
*
* * Redistributions of source code must retain the above copyright
* notice, this list of conditions and the following disclaimer.
*
* * Redistributions in binary form must reproduce the above copyright
* notice, this list of conditions and the following disclaimer in
* the documentation and/or other materials provided with the
* distribution.
*
* * Neither the name of Manuel Pichler nor the names of his
* contributors may be used to endorse or promote products derived
* from this software without specific prior written permission.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
* "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
* LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
* FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
* COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
* INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
* BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
* LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
* CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN
* ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
*
* @copyright 2008-2017 Manuel Pichler. All rights reserved.
* @license http://www.opensource.org/licenses/bsd-license.php BSD License
*/

namespace PDepend\Bugs\Phpmd;


use PDepend\Bugs\AbstractRegressionTest;
use PDepend\Source\AST\ASTFunction;

/**
* https://github.com/phpmd/phpmd/issues/914
*/
class FunctionDocBlockBugPhpmd914Test extends AbstractRegressionTest
{
public function testFunctionDocBlockCanBeRead()
{
/** @var ASTFunction $function */
$function = $this->getFirstFunctionForTestCase();

$lines = array_map(function ($line) {
return trim($line, " \t\n\r\0\x0B/*");
}, explode("\n", trim($function->getComment(), " \t\n\r\0\x0B/*")));

$this->assertSame(array(
'@SuppressWarnings(PHPMD.CyclomaticComplexity)',
'@SuppressWarnings(PHPMD.NPathComplexity)',
), $lines);
}
}
18 changes: 18 additions & 0 deletions src/test/php/PDepend/fix-php-compatibility.php
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,18 @@
"#[\\ReturnTypeWillChange]\npublic function getIterator(",
),
),
__DIR__ . '/../../../../vendor/phpunit/phpunit/src/Runner/Filter/Test.php' => array(
array(
'public function accept(',
"#[\\ReturnTypeWillChange]\npublic function accept(",
),
),
__DIR__ . '/../../../../vendor/phpunit/php-file-iterator/src/Iterator.php' => array(
array(
'public function accept(',
"#[\\ReturnTypeWillChange]\npublic function accept(",
),
),
__DIR__ . '/../../../../vendor/phpunit/phpunit/src/Util/TestSuiteIterator.php' => array(
array(
'public function hasChildren(',
Expand Down Expand Up @@ -95,6 +107,12 @@
"#[\\ReturnTypeWillChange]\npublic function rewind(",
),
),
__DIR__ . '/../../../../vendor/phpunit/phpunit/src/Util/Getopt.php' => array(
array(
'strlen($opt_arg)',
'strlen((string) $opt_arg)',
),
),
__DIR__ . '/../../../../vendor/phpunit/php-code-coverage/src/CodeCoverage.php' => (PHP_VERSION >= 7) ? array(
array(
'$docblock = $token->getDocblock();',
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
<?php

/**
* @SuppressWarnings(PHPMD.CyclomaticComplexity)
* @SuppressWarnings(PHPMD.NPathComplexity)
*/
function doesNothing() {
if (true AND true AND false) return;
/** This comment breaks SuppressWarnings */
if (true AND true AND false) return;
if (true AND true AND false) return;
if (true AND true AND false) return;
}

0 comments on commit d1243a1

Please sign in to comment.