ForbiddenNegativeBitshiftSniffTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	74
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	testForbiddenNegativeBitshift()	8	8	1
A	dataForbiddenNegativeBitshift()	6	6	1
A	testNoViolation()	5	5	1
A	dataNoViolation()	10	10	1

<?php
/**
 * Bitwise shifts by negative number will throw an ArithmeticError in PHP 7.0.
 *
 * @package PHPCompatibility
 */


/**
 * Bitwise shifts by negative number will throw an ArithmeticError in PHP 7.0.
 *
 * @uses BaseSniffTest
 * @package PHPCompatibility
 * @author Wim Godden <wim@cu.be>
 */
class ForbiddenNegativeBitshiftSniffTest extends BaseSniffTest

This class seems to be duplicated in your project.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    const TEST_FILE = 'sniff-examples/forbidden_negative_bitshift.php';

    /**
     * testForbiddenNegativeBitshift
     *
     * @group forbiddenNegativeBitshift
     *
     * @dataProvider dataForbiddenNegativeBitshift
     *
     * @param int $line Line number where the error should occur.
     *
     * @return void
     */
    public function testForbiddenNegativeBitshift($line)
    {
        $file = $this->sniffFile(self::TEST_FILE, '5.6');
        $this->assertNoViolation($file, $line);

        $file = $this->sniffFile(self::TEST_FILE, '7.0');
        $this->assertError($file, $line, 'Bitwise shifts by negative number will throw an ArithmeticError in PHP 7.0');
    }

    /**
     * dataForbiddenNegativeBitshift
     *
     * @see testForbiddenNegativeBitshift()
     *
     * @return array
     */
    public function dataForbiddenNegativeBitshift() {
        return array(
            array(3),
            array(4),
        );
    }


    /**
     * testNoViolation
     *
     * @group forbiddenNegativeBitshift
     *
     * @dataProvider dataNoViolation
     *
     * @param int $line The line number.
     *
     * @return void
     */
    public function testNoViolation($line)
    {
        $file = $this->sniffFile(self::TEST_FILE, '7.0');
        $this->assertNoViolation($file, $line);
    }

    /**
     * Data provider.
     *
     * @see testNoViolation()
     *
     * @return array
     */
    public function dataNoViolation()
    {
        return array(
            array(6),
            array(7),
            array(8),
            //array(9), - currently gives a false positive, @todo: uncomment when fixed.
            array(12),
        );
    }
}