gravityview / GravityView

vendor/squizlabs/php_codesniffer/tests/TestSuite.php

Indentation +14 added lines, -14 removed lines

@@ -16,20 +16,20 @@
 {
 
 
-    /**
-     * Runs the tests and collects their result in a TestResult.
-     *
-     * @param \PHPUnit\Framework\TestResult $result A test result.
-     *
-     * @return \PHPUnit\Framework\TestResult
-     */
-    public function run(TestResult $result=null)
-    {
-        $result = parent::run($result);
-        printPHPCodeSnifferTestOutput();
-        return $result;
-
-    }//end run()
+	/**
+	 * Runs the tests and collects their result in a TestResult.
+	 *
+	 * @param \PHPUnit\Framework\TestResult $result A test result.
+	 *
+	 * @return \PHPUnit\Framework\TestResult
+	 */
+	public function run(TestResult $result=null)
+	{
+		$result = parent::run($result);
+		printPHPCodeSnifferTestOutput();
+		return $result;
+
+	}//end run()
 
 
 }//end class

Spacing +2 added lines, -2 removed lines

@@ -23,9 +23,9 @@
      *
      * @return \PHPUnit\Framework\TestResult
      */
-    public function run(TestResult $result=null)
+    public function run( TestResult $result = null )
     {
-        $result = parent::run($result);
+        $result = parent::run( $result );
         printPHPCodeSnifferTestOutput();
         return $result;
 

Braces +2 added lines, -4 removed lines

@@ -12,8 +12,7 @@ 
 use PHPUnit\Framework\TestSuite as PHPUnit_TestSuite;
 use PHPUnit\Framework\TestResult;
 
-class TestSuite extends PHPUnit_TestSuite
-{
+class TestSuite extends PHPUnit_TestSuite {
 
 
     /**
@@ -23,8 +22,7 @@ 
      *
      * @return \PHPUnit\Framework\TestResult
      */
-    public function run(TestResult $result=null)
-    {
+    public function run(TestResult $result=null) {
         $result = parent::run($result);
         printPHPCodeSnifferTestOutput();
         return $result;

vendor/squizlabs/php_codesniffer/tests/AllTests.php

Indentation +26 added lines, -26 removed lines

@@ -10,11 +10,11 @@ 
 namespace PHP_CodeSniffer\Tests;
 
 if (is_file(__DIR__.'/../autoload.php') === true) {
-    include_once 'Core/AllTests.php';
-    include_once 'Standards/AllSniffs.php';
+	include_once 'Core/AllTests.php';
+	include_once 'Standards/AllSniffs.php';
 } else {
-    include_once 'CodeSniffer/Core/AllTests.php';
-    include_once 'CodeSniffer/Standards/AllSniffs.php';
+	include_once 'CodeSniffer/Core/AllTests.php';
+	include_once 'CodeSniffer/Standards/AllSniffs.php';
 }
 
 // PHPUnit 7 made the TestSuite run() method incompatible with
@@ -22,42 +22,42 @@ 
 // two different suite objects.
 $phpunit7 = false;
 if (class_exists('\PHPUnit\Runner\Version') === true) {
-    $version = \PHPUnit\Runner\Version::id();
-    if ($version[0] === '7') {
-        $phpunit7 = true;
-    }
+	$version = \PHPUnit\Runner\Version::id();
+	if ($version[0] === '7') {
+		$phpunit7 = true;
+	}
 }
 
 if ($phpunit7 === true) {
-    include_once 'TestSuite7.php';
+	include_once 'TestSuite7.php';
 } else {
-    include_once 'TestSuite.php';
+	include_once 'TestSuite.php';
 }
 
 class PHP_CodeSniffer_AllTests
 {
 
 
-    /**
-     * Add all PHP_CodeSniffer test suites into a single test suite.
-     *
-     * @return \PHPUnit\Framework\TestSuite
-     */
-    public static function suite()
-    {
-        $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'] = [];
-        $GLOBALS['PHP_CODESNIFFER_TEST_DIRS']     = [];
+	/**
+	 * Add all PHP_CodeSniffer test suites into a single test suite.
+	 *
+	 * @return \PHPUnit\Framework\TestSuite
+	 */
+	public static function suite()
+	{
+		$GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'] = [];
+		$GLOBALS['PHP_CODESNIFFER_TEST_DIRS']     = [];
 
-        // Use a special PHP_CodeSniffer test suite so that we can
-        // unset our autoload function after the run.
-        $suite = new TestSuite('PHP CodeSniffer');
+		// Use a special PHP_CodeSniffer test suite so that we can
+		// unset our autoload function after the run.
+		$suite = new TestSuite('PHP CodeSniffer');
 
-        $suite->addTest(Core\AllTests::suite());
-        $suite->addTest(Standards\AllSniffs::suite());
+		$suite->addTest(Core\AllTests::suite());
+		$suite->addTest(Standards\AllSniffs::suite());
 
-        return $suite;
+		return $suite;
 
-    }//end suite()
+	}//end suite()
 
 
 }//end class

Spacing +9 added lines, -9 removed lines

@@ -9,7 +9,7 @@ 
 
 namespace PHP_CodeSniffer\Tests;
 
-if (is_file(__DIR__.'/../autoload.php') === true) {
+if ( is_file( __DIR__ . '/../autoload.php' ) === true ) {
     include_once 'Core/AllTests.php';
     include_once 'Standards/AllSniffs.php';
 } else {
@@ -21,14 +21,14 @@ 
 // older PHPUnit versions due to return type hints, so maintain
 // two different suite objects.
 $phpunit7 = false;
-if (class_exists('\PHPUnit\Runner\Version') === true) {
+if ( class_exists( '\PHPUnit\Runner\Version' ) === true ) {
     $version = \PHPUnit\Runner\Version::id();
-    if ($version[0] === '7') {
+    if ( $version[ 0 ] === '7' ) {
         $phpunit7 = true;
     }
 }
 
-if ($phpunit7 === true) {
+if ( $phpunit7 === true ) {
     include_once 'TestSuite7.php';
 } else {
     include_once 'TestSuite.php';
@@ -45,15 +45,15 @@ 
      */
     public static function suite()
     {
-        $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'] = [];
-        $GLOBALS['PHP_CODESNIFFER_TEST_DIRS']     = [];
+        $GLOBALS[ 'PHP_CODESNIFFER_STANDARD_DIRS' ] = [ ];
+        $GLOBALS[ 'PHP_CODESNIFFER_TEST_DIRS' ]     = [ ];
 
         // Use a special PHP_CodeSniffer test suite so that we can
         // unset our autoload function after the run.
-        $suite = new TestSuite('PHP CodeSniffer');
+        $suite = new TestSuite( 'PHP CodeSniffer' );
 
-        $suite->addTest(Core\AllTests::suite());
-        $suite->addTest(Standards\AllSniffs::suite());
+        $suite->addTest( Core\AllTests::suite() );
+        $suite->addTest( Standards\AllSniffs::suite() );
 
         return $suite;
 

Braces +2 added lines, -4 removed lines

@@ -34,8 +34,7 @@ 
     include_once 'TestSuite.php';
 }
 
-class PHP_CodeSniffer_AllTests
-{
+class PHP_CodeSniffer_AllTests {
 
 
     /**
@@ -43,8 +42,7 @@ 
      *
      * @return \PHPUnit\Framework\TestSuite
      */
-    public static function suite()
-    {
+    public static function suite() {
         $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'] = [];
         $GLOBALS['PHP_CODESNIFFER_TEST_DIRS']     = [];
 

vendor/squizlabs/php_codesniffer/tests/Standards/AbstractSniffUnitTest.php

Indentation +416 added lines, -416 removed lines

@@ -23,428 +23,428 @@
 abstract class AbstractSniffUnitTest extends TestCase
 {
 
-    /**
-     * Enable or disable the backup and restoration of the $GLOBALS array.
-     * Overwrite this attribute in a child class of TestCase.
-     * Setting this attribute in setUp() has no effect!
-     *
-     * @var boolean
-     */
-    protected $backupGlobals = false;
-
-    /**
-     * The path to the standard's main directory.
-     *
-     * @var string
-     */
-    public $standardsDir = null;
-
-    /**
-     * The path to the standard's test directory.
-     *
-     * @var string
-     */
-    public $testsDir = null;
-
-
-    /**
-     * Sets up this unit test.
-     *
-     * @return void
-     */
-    protected function setUp()
-    {
-        $class = get_class($this);
-        $this->standardsDir = $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$class];
-        $this->testsDir     = $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$class];
-
-    }//end setUp()
-
-
-    /**
-     * Get a list of all test files to check.
-     *
-     * These will have the same base as the sniff name but different extensions.
-     * We ignore the .php file as it is the class.
-     *
-     * @param string $testFileBase The base path that the unit tests files will have.
-     *
-     * @return string[]
-     */
-    protected function getTestFiles($testFileBase)
-    {
-        $testFiles = [];
-
-        $dir = substr($testFileBase, 0, strrpos($testFileBase, DIRECTORY_SEPARATOR));
-        $di  = new \DirectoryIterator($dir);
-
-        foreach ($di as $file) {
-            $path = $file->getPathname();
-            if (substr($path, 0, strlen($testFileBase)) === $testFileBase) {
-                if ($path !== $testFileBase.'php' && substr($path, -5) !== 'fixed' && substr($path, -4) !== '.bak') {
-                    $testFiles[] = $path;
-                }
-            }
-        }
-
-        // Put them in order.
-        sort($testFiles);
-
-        return $testFiles;
-
-    }//end getTestFiles()
-
-
-    /**
-     * Should this test be skipped for some reason.
-     *
-     * @return boolean
-     */
-    protected function shouldSkipTest()
-    {
-        return false;
-
-    }//end shouldSkipTest()
-
-
-    /**
-     * Tests the extending classes Sniff class.
-     *
-     * @return void
-     * @throws \PHPUnit\Framework\Exception
-     */
-    final public function testSniff()
-    {
-        // Skip this test if we can't run in this environment.
-        if ($this->shouldSkipTest() === true) {
-            $this->markTestSkipped();
-        }
-
-        $sniffCode = Common::getSniffCode(get_class($this));
-        list($standardName, $categoryName, $sniffName) = explode('.', $sniffCode);
-
-        $testFileBase = $this->testsDir.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'UnitTest.';
-
-        // Get a list of all test files to check.
-        $testFiles = $this->getTestFiles($testFileBase);
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'][] = $testFiles;
-
-        if (isset($GLOBALS['PHP_CODESNIFFER_CONFIG']) === true) {
-            $config = $GLOBALS['PHP_CODESNIFFER_CONFIG'];
-        } else {
-            $config        = new Config();
-            $config->cache = false;
-            $GLOBALS['PHP_CODESNIFFER_CONFIG'] = $config;
-        }
-
-        $config->standards = [$standardName];
-        $config->sniffs    = [$sniffCode];
-        $config->ignored   = [];
-
-        if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS']) === false) {
-            $GLOBALS['PHP_CODESNIFFER_RULESETS'] = [];
-        }
-
-        if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName]) === false) {
-            $ruleset = new Ruleset($config);
-            $GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName] = $ruleset;
-        }
-
-        $ruleset = $GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName];
-
-        $sniffFile = $this->standardsDir.DIRECTORY_SEPARATOR.'Sniffs'.DIRECTORY_SEPARATOR.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'Sniff.php';
-
-        $sniffClassName = substr(get_class($this), 0, -8).'Sniff';
-        $sniffClassName = str_replace('\Tests\\', '\Sniffs\\', $sniffClassName);
-        $sniffClassName = Common::cleanSniffClass($sniffClassName);
-
-        $restrictions = [strtolower($sniffClassName) => true];
-        $ruleset->registerSniffs([$sniffFile], $restrictions, []);
-        $ruleset->populateTokenListeners();
-
-        $failureMessages = [];
-        foreach ($testFiles as $testFile) {
-            $filename  = basename($testFile);
-            $oldConfig = $config->getSettings();
-
-            try {
-                $this->setCliValues($filename, $config);
-                $phpcsFile = new LocalFile($testFile, $ruleset, $config);
-                $phpcsFile->process();
-            } catch (RuntimeException $e) {
-                $this->fail('An unexpected exception has been caught: '.$e->getMessage());
-            }
-
-            $failures        = $this->generateFailureMessages($phpcsFile);
-            $failureMessages = array_merge($failureMessages, $failures);
-
-            if ($phpcsFile->getFixableCount() > 0) {
-                // Attempt to fix the errors.
-                $phpcsFile->fixer->fixFile();
-                $fixable = $phpcsFile->getFixableCount();
-                if ($fixable > 0) {
-                    $failureMessages[] = "Failed to fix $fixable fixable violations in $filename";
-                }
-
-                // Check for a .fixed file to check for accuracy of fixes.
-                $fixedFile = $testFile.'.fixed';
-                if (file_exists($fixedFile) === true) {
-                    $diff = $phpcsFile->fixer->generateDiff($fixedFile);
-                    if (trim($diff) !== '') {
-                        $filename          = basename($testFile);
-                        $fixedFilename     = basename($fixedFile);
-                        $failureMessages[] = "Fixed version of $filename does not match expected version in $fixedFilename; the diff is\n$diff";
-                    }
-                }
-            }
-
-            // Restore the config.
-            $config->setSettings($oldConfig);
-        }//end foreach
-
-        if (empty($failureMessages) === false) {
-            $this->fail(implode(PHP_EOL, $failureMessages));
-        }
-
-    }//end testSniff()
-
-
-    /**
-     * Generate a list of test failures for a given sniffed file.
-     *
-     * @param \PHP_CodeSniffer\Files\LocalFile $file The file being tested.
-     *
-     * @return array
-     * @throws \PHP_CodeSniffer\Exceptions\RuntimeException
-     */
-    public function generateFailureMessages(LocalFile $file)
-    {
-        $testFile = $file->getFilename();
-
-        $foundErrors      = $file->getErrors();
-        $foundWarnings    = $file->getWarnings();
-        $expectedErrors   = $this->getErrorList(basename($testFile));
-        $expectedWarnings = $this->getWarningList(basename($testFile));
-
-        if (is_array($expectedErrors) === false) {
-            throw new RuntimeException('getErrorList() must return an array');
-        }
-
-        if (is_array($expectedWarnings) === false) {
-            throw new RuntimeException('getWarningList() must return an array');
-        }
-
-        /*
+	/**
+	 * Enable or disable the backup and restoration of the $GLOBALS array.
+	 * Overwrite this attribute in a child class of TestCase.
+	 * Setting this attribute in setUp() has no effect!
+	 *
+	 * @var boolean
+	 */
+	protected $backupGlobals = false;
+
+	/**
+	 * The path to the standard's main directory.
+	 *
+	 * @var string
+	 */
+	public $standardsDir = null;
+
+	/**
+	 * The path to the standard's test directory.
+	 *
+	 * @var string
+	 */
+	public $testsDir = null;
+
+
+	/**
+	 * Sets up this unit test.
+	 *
+	 * @return void
+	 */
+	protected function setUp()
+	{
+		$class = get_class($this);
+		$this->standardsDir = $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$class];
+		$this->testsDir     = $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$class];
+
+	}//end setUp()
+
+
+	/**
+	 * Get a list of all test files to check.
+	 *
+	 * These will have the same base as the sniff name but different extensions.
+	 * We ignore the .php file as it is the class.
+	 *
+	 * @param string $testFileBase The base path that the unit tests files will have.
+	 *
+	 * @return string[]
+	 */
+	protected function getTestFiles($testFileBase)
+	{
+		$testFiles = [];
+
+		$dir = substr($testFileBase, 0, strrpos($testFileBase, DIRECTORY_SEPARATOR));
+		$di  = new \DirectoryIterator($dir);
+
+		foreach ($di as $file) {
+			$path = $file->getPathname();
+			if (substr($path, 0, strlen($testFileBase)) === $testFileBase) {
+				if ($path !== $testFileBase.'php' && substr($path, -5) !== 'fixed' && substr($path, -4) !== '.bak') {
+					$testFiles[] = $path;
+				}
+			}
+		}
+
+		// Put them in order.
+		sort($testFiles);
+
+		return $testFiles;
+
+	}//end getTestFiles()
+
+
+	/**
+	 * Should this test be skipped for some reason.
+	 *
+	 * @return boolean
+	 */
+	protected function shouldSkipTest()
+	{
+		return false;
+
+	}//end shouldSkipTest()
+
+
+	/**
+	 * Tests the extending classes Sniff class.
+	 *
+	 * @return void
+	 * @throws \PHPUnit\Framework\Exception
+	 */
+	final public function testSniff()
+	{
+		// Skip this test if we can't run in this environment.
+		if ($this->shouldSkipTest() === true) {
+			$this->markTestSkipped();
+		}
+
+		$sniffCode = Common::getSniffCode(get_class($this));
+		list($standardName, $categoryName, $sniffName) = explode('.', $sniffCode);
+
+		$testFileBase = $this->testsDir.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'UnitTest.';
+
+		// Get a list of all test files to check.
+		$testFiles = $this->getTestFiles($testFileBase);
+		$GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'][] = $testFiles;
+
+		if (isset($GLOBALS['PHP_CODESNIFFER_CONFIG']) === true) {
+			$config = $GLOBALS['PHP_CODESNIFFER_CONFIG'];
+		} else {
+			$config        = new Config();
+			$config->cache = false;
+			$GLOBALS['PHP_CODESNIFFER_CONFIG'] = $config;
+		}
+
+		$config->standards = [$standardName];
+		$config->sniffs    = [$sniffCode];
+		$config->ignored   = [];
+
+		if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS']) === false) {
+			$GLOBALS['PHP_CODESNIFFER_RULESETS'] = [];
+		}
+
+		if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName]) === false) {
+			$ruleset = new Ruleset($config);
+			$GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName] = $ruleset;
+		}
+
+		$ruleset = $GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName];
+
+		$sniffFile = $this->standardsDir.DIRECTORY_SEPARATOR.'Sniffs'.DIRECTORY_SEPARATOR.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'Sniff.php';
+
+		$sniffClassName = substr(get_class($this), 0, -8).'Sniff';
+		$sniffClassName = str_replace('\Tests\\', '\Sniffs\\', $sniffClassName);
+		$sniffClassName = Common::cleanSniffClass($sniffClassName);
+
+		$restrictions = [strtolower($sniffClassName) => true];
+		$ruleset->registerSniffs([$sniffFile], $restrictions, []);
+		$ruleset->populateTokenListeners();
+
+		$failureMessages = [];
+		foreach ($testFiles as $testFile) {
+			$filename  = basename($testFile);
+			$oldConfig = $config->getSettings();
+
+			try {
+				$this->setCliValues($filename, $config);
+				$phpcsFile = new LocalFile($testFile, $ruleset, $config);
+				$phpcsFile->process();
+			} catch (RuntimeException $e) {
+				$this->fail('An unexpected exception has been caught: '.$e->getMessage());
+			}
+
+			$failures        = $this->generateFailureMessages($phpcsFile);
+			$failureMessages = array_merge($failureMessages, $failures);
+
+			if ($phpcsFile->getFixableCount() > 0) {
+				// Attempt to fix the errors.
+				$phpcsFile->fixer->fixFile();
+				$fixable = $phpcsFile->getFixableCount();
+				if ($fixable > 0) {
+					$failureMessages[] = "Failed to fix $fixable fixable violations in $filename";
+				}
+
+				// Check for a .fixed file to check for accuracy of fixes.
+				$fixedFile = $testFile.'.fixed';
+				if (file_exists($fixedFile) === true) {
+					$diff = $phpcsFile->fixer->generateDiff($fixedFile);
+					if (trim($diff) !== '') {
+						$filename          = basename($testFile);
+						$fixedFilename     = basename($fixedFile);
+						$failureMessages[] = "Fixed version of $filename does not match expected version in $fixedFilename; the diff is\n$diff";
+					}
+				}
+			}
+
+			// Restore the config.
+			$config->setSettings($oldConfig);
+		}//end foreach
+
+		if (empty($failureMessages) === false) {
+			$this->fail(implode(PHP_EOL, $failureMessages));
+		}
+
+	}//end testSniff()
+
+
+	/**
+	 * Generate a list of test failures for a given sniffed file.
+	 *
+	 * @param \PHP_CodeSniffer\Files\LocalFile $file The file being tested.
+	 *
+	 * @return array
+	 * @throws \PHP_CodeSniffer\Exceptions\RuntimeException
+	 */
+	public function generateFailureMessages(LocalFile $file)
+	{
+		$testFile = $file->getFilename();
+
+		$foundErrors      = $file->getErrors();
+		$foundWarnings    = $file->getWarnings();
+		$expectedErrors   = $this->getErrorList(basename($testFile));
+		$expectedWarnings = $this->getWarningList(basename($testFile));
+
+		if (is_array($expectedErrors) === false) {
+			throw new RuntimeException('getErrorList() must return an array');
+		}
+
+		if (is_array($expectedWarnings) === false) {
+			throw new RuntimeException('getWarningList() must return an array');
+		}
+
+		/*
             We merge errors and warnings together to make it easier
             to iterate over them and produce the errors string. In this way,
             we can report on errors and warnings in the same line even though
             it's not really structured to allow that.
         */
 
-        $allProblems     = [];
-        $failureMessages = [];
-
-        foreach ($foundErrors as $line => $lineErrors) {
-            foreach ($lineErrors as $column => $errors) {
-                if (isset($allProblems[$line]) === false) {
-                    $allProblems[$line] = [
-                        'expected_errors'   => 0,
-                        'expected_warnings' => 0,
-                        'found_errors'      => [],
-                        'found_warnings'    => [],
-                    ];
-                }
-
-                $foundErrorsTemp = [];
-                foreach ($allProblems[$line]['found_errors'] as $foundError) {
-                    $foundErrorsTemp[] = $foundError;
-                }
-
-                $errorsTemp = [];
-                foreach ($errors as $foundError) {
-                    $errorsTemp[] = $foundError['message'].' ('.$foundError['source'].')';
-
-                    $source = $foundError['source'];
-                    if (in_array($source, $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'], true) === false) {
-                        $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'][] = $source;
-                    }
-
-                    if ($foundError['fixable'] === true
-                        && in_array($source, $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'], true) === false
-                    ) {
-                        $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'][] = $source;
-                    }
-                }
-
-                $allProblems[$line]['found_errors'] = array_merge($foundErrorsTemp, $errorsTemp);
-            }//end foreach
-
-            if (isset($expectedErrors[$line]) === true) {
-                $allProblems[$line]['expected_errors'] = $expectedErrors[$line];
-            } else {
-                $allProblems[$line]['expected_errors'] = 0;
-            }
-
-            unset($expectedErrors[$line]);
-        }//end foreach
-
-        foreach ($expectedErrors as $line => $numErrors) {
-            if (isset($allProblems[$line]) === false) {
-                $allProblems[$line] = [
-                    'expected_errors'   => 0,
-                    'expected_warnings' => 0,
-                    'found_errors'      => [],
-                    'found_warnings'    => [],
-                ];
-            }
-
-            $allProblems[$line]['expected_errors'] = $numErrors;
-        }
-
-        foreach ($foundWarnings as $line => $lineWarnings) {
-            foreach ($lineWarnings as $column => $warnings) {
-                if (isset($allProblems[$line]) === false) {
-                    $allProblems[$line] = [
-                        'expected_errors'   => 0,
-                        'expected_warnings' => 0,
-                        'found_errors'      => [],
-                        'found_warnings'    => [],
-                    ];
-                }
-
-                $foundWarningsTemp = [];
-                foreach ($allProblems[$line]['found_warnings'] as $foundWarning) {
-                    $foundWarningsTemp[] = $foundWarning;
-                }
-
-                $warningsTemp = [];
-                foreach ($warnings as $warning) {
-                    $warningsTemp[] = $warning['message'].' ('.$warning['source'].')';
-                }
-
-                $allProblems[$line]['found_warnings'] = array_merge($foundWarningsTemp, $warningsTemp);
-            }//end foreach
-
-            if (isset($expectedWarnings[$line]) === true) {
-                $allProblems[$line]['expected_warnings'] = $expectedWarnings[$line];
-            } else {
-                $allProblems[$line]['expected_warnings'] = 0;
-            }
-
-            unset($expectedWarnings[$line]);
-        }//end foreach
-
-        foreach ($expectedWarnings as $line => $numWarnings) {
-            if (isset($allProblems[$line]) === false) {
-                $allProblems[$line] = [
-                    'expected_errors'   => 0,
-                    'expected_warnings' => 0,
-                    'found_errors'      => [],
-                    'found_warnings'    => [],
-                ];
-            }
-
-            $allProblems[$line]['expected_warnings'] = $numWarnings;
-        }
-
-        // Order the messages by line number.
-        ksort($allProblems);
-
-        foreach ($allProblems as $line => $problems) {
-            $numErrors        = count($problems['found_errors']);
-            $numWarnings      = count($problems['found_warnings']);
-            $expectedErrors   = $problems['expected_errors'];
-            $expectedWarnings = $problems['expected_warnings'];
-
-            $errors      = '';
-            $foundString = '';
-
-            if ($expectedErrors !== $numErrors || $expectedWarnings !== $numWarnings) {
-                $lineMessage     = "[LINE $line]";
-                $expectedMessage = 'Expected ';
-                $foundMessage    = 'in '.basename($testFile).' but found ';
-
-                if ($expectedErrors !== $numErrors) {
-                    $expectedMessage .= "$expectedErrors error(s)";
-                    $foundMessage    .= "$numErrors error(s)";
-                    if ($numErrors !== 0) {
-                        $foundString .= 'error(s)';
-                        $errors      .= implode(PHP_EOL.' -> ', $problems['found_errors']);
-                    }
-
-                    if ($expectedWarnings !== $numWarnings) {
-                        $expectedMessage .= ' and ';
-                        $foundMessage    .= ' and ';
-                        if ($numWarnings !== 0) {
-                            if ($foundString !== '') {
-                                $foundString .= ' and ';
-                            }
-                        }
-                    }
-                }
-
-                if ($expectedWarnings !== $numWarnings) {
-                    $expectedMessage .= "$expectedWarnings warning(s)";
-                    $foundMessage    .= "$numWarnings warning(s)";
-                    if ($numWarnings !== 0) {
-                        $foundString .= 'warning(s)';
-                        if (empty($errors) === false) {
-                            $errors .= PHP_EOL.' -> ';
-                        }
-
-                        $errors .= implode(PHP_EOL.' -> ', $problems['found_warnings']);
-                    }
-                }
-
-                $fullMessage = "$lineMessage $expectedMessage $foundMessage.";
-                if ($errors !== '') {
-                    $fullMessage .= " The $foundString found were:".PHP_EOL." -> $errors";
-                }
-
-                $failureMessages[] = $fullMessage;
-            }//end if
-        }//end foreach
-
-        return $failureMessages;
-
-    }//end generateFailureMessages()
-
-
-    /**
-     * Get a list of CLI values to set before the file is tested.
-     *
-     * @param string                  $filename The name of the file being tested.
-     * @param \PHP_CodeSniffer\Config $config   The config data for the run.
-     *
-     * @return void
-     */
-    public function setCliValues($filename, $config)
-    {
-        return;
-
-    }//end setCliValues()
-
-
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    abstract protected function getErrorList();
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    abstract protected function getWarningList();
+		$allProblems     = [];
+		$failureMessages = [];
+
+		foreach ($foundErrors as $line => $lineErrors) {
+			foreach ($lineErrors as $column => $errors) {
+				if (isset($allProblems[$line]) === false) {
+					$allProblems[$line] = [
+						'expected_errors'   => 0,
+						'expected_warnings' => 0,
+						'found_errors'      => [],
+						'found_warnings'    => [],
+					];
+				}
+
+				$foundErrorsTemp = [];
+				foreach ($allProblems[$line]['found_errors'] as $foundError) {
+					$foundErrorsTemp[] = $foundError;
+				}
+
+				$errorsTemp = [];
+				foreach ($errors as $foundError) {
+					$errorsTemp[] = $foundError['message'].' ('.$foundError['source'].')';
+
+					$source = $foundError['source'];
+					if (in_array($source, $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'], true) === false) {
+						$GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'][] = $source;
+					}
+
+					if ($foundError['fixable'] === true
+						&& in_array($source, $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'], true) === false
+					) {
+						$GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'][] = $source;
+					}
+				}
+
+				$allProblems[$line]['found_errors'] = array_merge($foundErrorsTemp, $errorsTemp);
+			}//end foreach
+
+			if (isset($expectedErrors[$line]) === true) {
+				$allProblems[$line]['expected_errors'] = $expectedErrors[$line];
+			} else {
+				$allProblems[$line]['expected_errors'] = 0;
+			}
+
+			unset($expectedErrors[$line]);
+		}//end foreach
+
+		foreach ($expectedErrors as $line => $numErrors) {
+			if (isset($allProblems[$line]) === false) {
+				$allProblems[$line] = [
+					'expected_errors'   => 0,
+					'expected_warnings' => 0,
+					'found_errors'      => [],
+					'found_warnings'    => [],
+				];
+			}
+
+			$allProblems[$line]['expected_errors'] = $numErrors;
+		}
+
+		foreach ($foundWarnings as $line => $lineWarnings) {
+			foreach ($lineWarnings as $column => $warnings) {
+				if (isset($allProblems[$line]) === false) {
+					$allProblems[$line] = [
+						'expected_errors'   => 0,
+						'expected_warnings' => 0,
+						'found_errors'      => [],
+						'found_warnings'    => [],
+					];
+				}
+
+				$foundWarningsTemp = [];
+				foreach ($allProblems[$line]['found_warnings'] as $foundWarning) {
+					$foundWarningsTemp[] = $foundWarning;
+				}
+
+				$warningsTemp = [];
+				foreach ($warnings as $warning) {
+					$warningsTemp[] = $warning['message'].' ('.$warning['source'].')';
+				}
+
+				$allProblems[$line]['found_warnings'] = array_merge($foundWarningsTemp, $warningsTemp);
+			}//end foreach
+
+			if (isset($expectedWarnings[$line]) === true) {
+				$allProblems[$line]['expected_warnings'] = $expectedWarnings[$line];
+			} else {
+				$allProblems[$line]['expected_warnings'] = 0;
+			}
+
+			unset($expectedWarnings[$line]);
+		}//end foreach
+
+		foreach ($expectedWarnings as $line => $numWarnings) {
+			if (isset($allProblems[$line]) === false) {
+				$allProblems[$line] = [
+					'expected_errors'   => 0,
+					'expected_warnings' => 0,
+					'found_errors'      => [],
+					'found_warnings'    => [],
+				];
+			}
+
+			$allProblems[$line]['expected_warnings'] = $numWarnings;
+		}
+
+		// Order the messages by line number.
+		ksort($allProblems);
+
+		foreach ($allProblems as $line => $problems) {
+			$numErrors        = count($problems['found_errors']);
+			$numWarnings      = count($problems['found_warnings']);
+			$expectedErrors   = $problems['expected_errors'];
+			$expectedWarnings = $problems['expected_warnings'];
+
+			$errors      = '';
+			$foundString = '';
+
+			if ($expectedErrors !== $numErrors || $expectedWarnings !== $numWarnings) {
+				$lineMessage     = "[LINE $line]";
+				$expectedMessage = 'Expected ';
+				$foundMessage    = 'in '.basename($testFile).' but found ';
+
+				if ($expectedErrors !== $numErrors) {
+					$expectedMessage .= "$expectedErrors error(s)";
+					$foundMessage    .= "$numErrors error(s)";
+					if ($numErrors !== 0) {
+						$foundString .= 'error(s)';
+						$errors      .= implode(PHP_EOL.' -> ', $problems['found_errors']);
+					}
+
+					if ($expectedWarnings !== $numWarnings) {
+						$expectedMessage .= ' and ';
+						$foundMessage    .= ' and ';
+						if ($numWarnings !== 0) {
+							if ($foundString !== '') {
+								$foundString .= ' and ';
+							}
+						}
+					}
+				}
+
+				if ($expectedWarnings !== $numWarnings) {
+					$expectedMessage .= "$expectedWarnings warning(s)";
+					$foundMessage    .= "$numWarnings warning(s)";
+					if ($numWarnings !== 0) {
+						$foundString .= 'warning(s)';
+						if (empty($errors) === false) {
+							$errors .= PHP_EOL.' -> ';
+						}
+
+						$errors .= implode(PHP_EOL.' -> ', $problems['found_warnings']);
+					}
+				}
+
+				$fullMessage = "$lineMessage $expectedMessage $foundMessage.";
+				if ($errors !== '') {
+					$fullMessage .= " The $foundString found were:".PHP_EOL." -> $errors";
+				}
+
+				$failureMessages[] = $fullMessage;
+			}//end if
+		}//end foreach
+
+		return $failureMessages;
+
+	}//end generateFailureMessages()
+
+
+	/**
+	 * Get a list of CLI values to set before the file is tested.
+	 *
+	 * @param string                  $filename The name of the file being tested.
+	 * @param \PHP_CodeSniffer\Config $config   The config data for the run.
+	 *
+	 * @return void
+	 */
+	public function setCliValues($filename, $config)
+	{
+		return;
+
+	}//end setCliValues()
+
+
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	abstract protected function getErrorList();
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	abstract protected function getWarningList();
 
 
 }//end class

Spacing +142 added lines, -142 removed lines

@@ -54,9 +54,9 @@ 
      */
     protected function setUp()
     {
-        $class = get_class($this);
-        $this->standardsDir = $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$class];
-        $this->testsDir     = $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$class];
+        $class = get_class( $this );
+        $this->standardsDir = $GLOBALS[ 'PHP_CODESNIFFER_STANDARD_DIRS' ][ $class ];
+        $this->testsDir     = $GLOBALS[ 'PHP_CODESNIFFER_TEST_DIRS' ][ $class ];
 
     }//end setUp()
 
@@ -71,24 +71,24 @@ 
      *
      * @return string[]
      */
-    protected function getTestFiles($testFileBase)
+    protected function getTestFiles( $testFileBase )
     {
-        $testFiles = [];
+        $testFiles = [ ];
 
-        $dir = substr($testFileBase, 0, strrpos($testFileBase, DIRECTORY_SEPARATOR));
-        $di  = new \DirectoryIterator($dir);
+        $dir = substr( $testFileBase, 0, strrpos( $testFileBase, DIRECTORY_SEPARATOR ) );
+        $di  = new \DirectoryIterator( $dir );
 
-        foreach ($di as $file) {
+        foreach ( $di as $file ) {
             $path = $file->getPathname();
-            if (substr($path, 0, strlen($testFileBase)) === $testFileBase) {
-                if ($path !== $testFileBase.'php' && substr($path, -5) !== 'fixed' && substr($path, -4) !== '.bak') {
-                    $testFiles[] = $path;
+            if ( substr( $path, 0, strlen( $testFileBase ) ) === $testFileBase ) {
+                if ( $path !== $testFileBase . 'php' && substr( $path, -5 ) !== 'fixed' && substr( $path, -4 ) !== '.bak' ) {
+                    $testFiles[ ] = $path;
                 }
             }
         }
 
         // Put them in order.
-        sort($testFiles);
+        sort( $testFiles );
 
         return $testFiles;
 
@@ -116,94 +116,94 @@ 
     final public function testSniff()
     {
         // Skip this test if we can't run in this environment.
-        if ($this->shouldSkipTest() === true) {
+        if ( $this->shouldSkipTest() === true ) {
             $this->markTestSkipped();
         }
 
-        $sniffCode = Common::getSniffCode(get_class($this));
-        list($standardName, $categoryName, $sniffName) = explode('.', $sniffCode);
+        $sniffCode = Common::getSniffCode( get_class( $this ) );
+        list( $standardName, $categoryName, $sniffName ) = explode( '.', $sniffCode );
 
-        $testFileBase = $this->testsDir.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'UnitTest.';
+        $testFileBase = $this->testsDir . $categoryName . DIRECTORY_SEPARATOR . $sniffName . 'UnitTest.';
 
         // Get a list of all test files to check.
-        $testFiles = $this->getTestFiles($testFileBase);
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'][] = $testFiles;
+        $testFiles = $this->getTestFiles( $testFileBase );
+        $GLOBALS[ 'PHP_CODESNIFFER_SNIFF_CASE_FILES' ][ ] = $testFiles;
 
-        if (isset($GLOBALS['PHP_CODESNIFFER_CONFIG']) === true) {
-            $config = $GLOBALS['PHP_CODESNIFFER_CONFIG'];
+        if ( isset( $GLOBALS[ 'PHP_CODESNIFFER_CONFIG' ] ) === true ) {
+            $config = $GLOBALS[ 'PHP_CODESNIFFER_CONFIG' ];
         } else {
             $config        = new Config();
             $config->cache = false;
-            $GLOBALS['PHP_CODESNIFFER_CONFIG'] = $config;
+            $GLOBALS[ 'PHP_CODESNIFFER_CONFIG' ] = $config;
         }
 
-        $config->standards = [$standardName];
-        $config->sniffs    = [$sniffCode];
-        $config->ignored   = [];
+        $config->standards = [ $standardName ];
+        $config->sniffs    = [ $sniffCode ];
+        $config->ignored   = [ ];
 
-        if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS']) === false) {
-            $GLOBALS['PHP_CODESNIFFER_RULESETS'] = [];
+        if ( isset( $GLOBALS[ 'PHP_CODESNIFFER_RULESETS' ] ) === false ) {
+            $GLOBALS[ 'PHP_CODESNIFFER_RULESETS' ] = [ ];
         }
 
-        if (isset($GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName]) === false) {
-            $ruleset = new Ruleset($config);
-            $GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName] = $ruleset;
+        if ( isset( $GLOBALS[ 'PHP_CODESNIFFER_RULESETS' ][ $standardName ] ) === false ) {
+            $ruleset = new Ruleset( $config );
+            $GLOBALS[ 'PHP_CODESNIFFER_RULESETS' ][ $standardName ] = $ruleset;
         }
 
-        $ruleset = $GLOBALS['PHP_CODESNIFFER_RULESETS'][$standardName];
+        $ruleset = $GLOBALS[ 'PHP_CODESNIFFER_RULESETS' ][ $standardName ];
 
-        $sniffFile = $this->standardsDir.DIRECTORY_SEPARATOR.'Sniffs'.DIRECTORY_SEPARATOR.$categoryName.DIRECTORY_SEPARATOR.$sniffName.'Sniff.php';
+        $sniffFile = $this->standardsDir . DIRECTORY_SEPARATOR . 'Sniffs' . DIRECTORY_SEPARATOR . $categoryName . DIRECTORY_SEPARATOR . $sniffName . 'Sniff.php';
 
-        $sniffClassName = substr(get_class($this), 0, -8).'Sniff';
-        $sniffClassName = str_replace('\Tests\\', '\Sniffs\\', $sniffClassName);
-        $sniffClassName = Common::cleanSniffClass($sniffClassName);
+        $sniffClassName = substr( get_class( $this ), 0, -8 ) . 'Sniff';
+        $sniffClassName = str_replace( '\Tests\\', '\Sniffs\\', $sniffClassName );
+        $sniffClassName = Common::cleanSniffClass( $sniffClassName );
 
-        $restrictions = [strtolower($sniffClassName) => true];
-        $ruleset->registerSniffs([$sniffFile], $restrictions, []);
+        $restrictions = [ strtolower( $sniffClassName ) => true ];
+        $ruleset->registerSniffs( [ $sniffFile ], $restrictions, [ ] );
         $ruleset->populateTokenListeners();
 
-        $failureMessages = [];
-        foreach ($testFiles as $testFile) {
-            $filename  = basename($testFile);
+        $failureMessages = [ ];
+        foreach ( $testFiles as $testFile ) {
+            $filename  = basename( $testFile );
             $oldConfig = $config->getSettings();
 
             try {
-                $this->setCliValues($filename, $config);
-                $phpcsFile = new LocalFile($testFile, $ruleset, $config);
+                $this->setCliValues( $filename, $config );
+                $phpcsFile = new LocalFile( $testFile, $ruleset, $config );
                 $phpcsFile->process();
-            } catch (RuntimeException $e) {
-                $this->fail('An unexpected exception has been caught: '.$e->getMessage());
+            } catch ( RuntimeException $e ) {
+                $this->fail( 'An unexpected exception has been caught: ' . $e->getMessage() );
             }
 
-            $failures        = $this->generateFailureMessages($phpcsFile);
-            $failureMessages = array_merge($failureMessages, $failures);
+            $failures        = $this->generateFailureMessages( $phpcsFile );
+            $failureMessages = array_merge( $failureMessages, $failures );
 
-            if ($phpcsFile->getFixableCount() > 0) {
+            if ( $phpcsFile->getFixableCount() > 0 ) {
                 // Attempt to fix the errors.
                 $phpcsFile->fixer->fixFile();
                 $fixable = $phpcsFile->getFixableCount();
-                if ($fixable > 0) {
-                    $failureMessages[] = "Failed to fix $fixable fixable violations in $filename";
+                if ( $fixable > 0 ) {
+                    $failureMessages[ ] = "Failed to fix $fixable fixable violations in $filename";
                 }
 
                 // Check for a .fixed file to check for accuracy of fixes.
-                $fixedFile = $testFile.'.fixed';
-                if (file_exists($fixedFile) === true) {
-                    $diff = $phpcsFile->fixer->generateDiff($fixedFile);
-                    if (trim($diff) !== '') {
-                        $filename          = basename($testFile);
-                        $fixedFilename     = basename($fixedFile);
-                        $failureMessages[] = "Fixed version of $filename does not match expected version in $fixedFilename; the diff is\n$diff";
+                $fixedFile = $testFile . '.fixed';
+                if ( file_exists( $fixedFile ) === true ) {
+                    $diff = $phpcsFile->fixer->generateDiff( $fixedFile );
+                    if ( trim( $diff ) !== '' ) {
+                        $filename          = basename( $testFile );
+                        $fixedFilename     = basename( $fixedFile );
+                        $failureMessages[ ] = "Fixed version of $filename does not match expected version in $fixedFilename; the diff is\n$diff";
                     }
                 }
             }
 
             // Restore the config.
-            $config->setSettings($oldConfig);
+            $config->setSettings( $oldConfig );
         }//end foreach
 
-        if (empty($failureMessages) === false) {
-            $this->fail(implode(PHP_EOL, $failureMessages));
+        if ( empty( $failureMessages ) === false ) {
+            $this->fail( implode( PHP_EOL, $failureMessages ) );
         }
 
     }//end testSniff()
@@ -217,21 +217,21 @@ 
      * @return array
      * @throws \PHP_CodeSniffer\Exceptions\RuntimeException
      */
-    public function generateFailureMessages(LocalFile $file)
+    public function generateFailureMessages( LocalFile $file )
     {
         $testFile = $file->getFilename();
 
         $foundErrors      = $file->getErrors();
         $foundWarnings    = $file->getWarnings();
-        $expectedErrors   = $this->getErrorList(basename($testFile));
-        $expectedWarnings = $this->getWarningList(basename($testFile));
+        $expectedErrors   = $this->getErrorList( basename( $testFile ) );
+        $expectedWarnings = $this->getWarningList( basename( $testFile ) );
 
-        if (is_array($expectedErrors) === false) {
-            throw new RuntimeException('getErrorList() must return an array');
+        if ( is_array( $expectedErrors ) === false ) {
+            throw new RuntimeException( 'getErrorList() must return an array' );
         }
 
-        if (is_array($expectedWarnings) === false) {
-            throw new RuntimeException('getWarningList() must return an array');
+        if ( is_array( $expectedWarnings ) === false ) {
+            throw new RuntimeException( 'getWarningList() must return an array' );
         }
 
         /*
@@ -241,167 +241,167 @@ 
             it's not really structured to allow that.
         */
 
-        $allProblems     = [];
-        $failureMessages = [];
+        $allProblems     = [ ];
+        $failureMessages = [ ];
 
-        foreach ($foundErrors as $line => $lineErrors) {
-            foreach ($lineErrors as $column => $errors) {
-                if (isset($allProblems[$line]) === false) {
-                    $allProblems[$line] = [
+        foreach ( $foundErrors as $line => $lineErrors ) {
+            foreach ( $lineErrors as $column => $errors ) {
+                if ( isset( $allProblems[ $line ] ) === false ) {
+                    $allProblems[ $line ] = [
                         'expected_errors'   => 0,
                         'expected_warnings' => 0,
-                        'found_errors'      => [],
-                        'found_warnings'    => [],
+                        'found_errors'      => [ ],
+                        'found_warnings'    => [ ],
                     ];
                 }
 
-                $foundErrorsTemp = [];
-                foreach ($allProblems[$line]['found_errors'] as $foundError) {
-                    $foundErrorsTemp[] = $foundError;
+                $foundErrorsTemp = [ ];
+                foreach ( $allProblems[ $line ][ 'found_errors' ] as $foundError ) {
+                    $foundErrorsTemp[ ] = $foundError;
                 }
 
-                $errorsTemp = [];
-                foreach ($errors as $foundError) {
-                    $errorsTemp[] = $foundError['message'].' ('.$foundError['source'].')';
+                $errorsTemp = [ ];
+                foreach ( $errors as $foundError ) {
+                    $errorsTemp[ ] = $foundError[ 'message' ] . ' (' . $foundError[ 'source' ] . ')';
 
-                    $source = $foundError['source'];
-                    if (in_array($source, $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'], true) === false) {
-                        $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES'][] = $source;
+                    $source = $foundError[ 'source' ];
+                    if ( in_array( $source, $GLOBALS[ 'PHP_CODESNIFFER_SNIFF_CODES' ], true ) === false ) {
+                        $GLOBALS[ 'PHP_CODESNIFFER_SNIFF_CODES' ][ ] = $source;
                     }
 
-                    if ($foundError['fixable'] === true
-                        && in_array($source, $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'], true) === false
+                    if ( $foundError[ 'fixable' ] === true
+                        && in_array( $source, $GLOBALS[ 'PHP_CODESNIFFER_FIXABLE_CODES' ], true ) === false
                     ) {
-                        $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES'][] = $source;
+                        $GLOBALS[ 'PHP_CODESNIFFER_FIXABLE_CODES' ][ ] = $source;
                     }
                 }
 
-                $allProblems[$line]['found_errors'] = array_merge($foundErrorsTemp, $errorsTemp);
+                $allProblems[ $line ][ 'found_errors' ] = array_merge( $foundErrorsTemp, $errorsTemp );
             }//end foreach
 
-            if (isset($expectedErrors[$line]) === true) {
-                $allProblems[$line]['expected_errors'] = $expectedErrors[$line];
+            if ( isset( $expectedErrors[ $line ] ) === true ) {
+                $allProblems[ $line ][ 'expected_errors' ] = $expectedErrors[ $line ];
             } else {
-                $allProblems[$line]['expected_errors'] = 0;
+                $allProblems[ $line ][ 'expected_errors' ] = 0;
             }
 
-            unset($expectedErrors[$line]);
+            unset( $expectedErrors[ $line ] );
         }//end foreach
 
-        foreach ($expectedErrors as $line => $numErrors) {
-            if (isset($allProblems[$line]) === false) {
-                $allProblems[$line] = [
+        foreach ( $expectedErrors as $line => $numErrors ) {
+            if ( isset( $allProblems[ $line ] ) === false ) {
+                $allProblems[ $line ] = [
                     'expected_errors'   => 0,
                     'expected_warnings' => 0,
-                    'found_errors'      => [],
-                    'found_warnings'    => [],
+                    'found_errors'      => [ ],
+                    'found_warnings'    => [ ],
                 ];
             }
 
-            $allProblems[$line]['expected_errors'] = $numErrors;
+            $allProblems[ $line ][ 'expected_errors' ] = $numErrors;
         }
 
-        foreach ($foundWarnings as $line => $lineWarnings) {
-            foreach ($lineWarnings as $column => $warnings) {
-                if (isset($allProblems[$line]) === false) {
-                    $allProblems[$line] = [
+        foreach ( $foundWarnings as $line => $lineWarnings ) {
+            foreach ( $lineWarnings as $column => $warnings ) {
+                if ( isset( $allProblems[ $line ] ) === false ) {
+                    $allProblems[ $line ] = [
                         'expected_errors'   => 0,
                         'expected_warnings' => 0,
-                        'found_errors'      => [],
-                        'found_warnings'    => [],
+                        'found_errors'      => [ ],
+                        'found_warnings'    => [ ],
                     ];
                 }
 
-                $foundWarningsTemp = [];
-                foreach ($allProblems[$line]['found_warnings'] as $foundWarning) {
-                    $foundWarningsTemp[] = $foundWarning;
+                $foundWarningsTemp = [ ];
+                foreach ( $allProblems[ $line ][ 'found_warnings' ] as $foundWarning ) {
+                    $foundWarningsTemp[ ] = $foundWarning;
                 }
 
-                $warningsTemp = [];
-                foreach ($warnings as $warning) {
-                    $warningsTemp[] = $warning['message'].' ('.$warning['source'].')';
+                $warningsTemp = [ ];
+                foreach ( $warnings as $warning ) {
+                    $warningsTemp[ ] = $warning[ 'message' ] . ' (' . $warning[ 'source' ] . ')';
                 }
 
-                $allProblems[$line]['found_warnings'] = array_merge($foundWarningsTemp, $warningsTemp);
+                $allProblems[ $line ][ 'found_warnings' ] = array_merge( $foundWarningsTemp, $warningsTemp );
             }//end foreach
 
-            if (isset($expectedWarnings[$line]) === true) {
-                $allProblems[$line]['expected_warnings'] = $expectedWarnings[$line];
+            if ( isset( $expectedWarnings[ $line ] ) === true ) {
+                $allProblems[ $line ][ 'expected_warnings' ] = $expectedWarnings[ $line ];
             } else {
-                $allProblems[$line]['expected_warnings'] = 0;
+                $allProblems[ $line ][ 'expected_warnings' ] = 0;
             }
 
-            unset($expectedWarnings[$line]);
+            unset( $expectedWarnings[ $line ] );
         }//end foreach
 
-        foreach ($expectedWarnings as $line => $numWarnings) {
-            if (isset($allProblems[$line]) === false) {
-                $allProblems[$line] = [
+        foreach ( $expectedWarnings as $line => $numWarnings ) {
+            if ( isset( $allProblems[ $line ] ) === false ) {
+                $allProblems[ $line ] = [
                     'expected_errors'   => 0,
                     'expected_warnings' => 0,
-                    'found_errors'      => [],
-                    'found_warnings'    => [],
+                    'found_errors'      => [ ],
+                    'found_warnings'    => [ ],
                 ];
             }
 
-            $allProblems[$line]['expected_warnings'] = $numWarnings;
+            $allProblems[ $line ][ 'expected_warnings' ] = $numWarnings;
         }
 
         // Order the messages by line number.
-        ksort($allProblems);
+        ksort( $allProblems );
 
-        foreach ($allProblems as $line => $problems) {
-            $numErrors        = count($problems['found_errors']);
-            $numWarnings      = count($problems['found_warnings']);
-            $expectedErrors   = $problems['expected_errors'];
-            $expectedWarnings = $problems['expected_warnings'];
+        foreach ( $allProblems as $line => $problems ) {
+            $numErrors        = count( $problems[ 'found_errors' ] );
+            $numWarnings      = count( $problems[ 'found_warnings' ] );
+            $expectedErrors   = $problems[ 'expected_errors' ];
+            $expectedWarnings = $problems[ 'expected_warnings' ];
 
             $errors      = '';
             $foundString = '';
 
-            if ($expectedErrors !== $numErrors || $expectedWarnings !== $numWarnings) {
+            if ( $expectedErrors !== $numErrors || $expectedWarnings !== $numWarnings ) {
                 $lineMessage     = "[LINE $line]";
                 $expectedMessage = 'Expected ';
-                $foundMessage    = 'in '.basename($testFile).' but found ';
+                $foundMessage    = 'in ' . basename( $testFile ) . ' but found ';
 
-                if ($expectedErrors !== $numErrors) {
+                if ( $expectedErrors !== $numErrors ) {
                     $expectedMessage .= "$expectedErrors error(s)";
                     $foundMessage    .= "$numErrors error(s)";
-                    if ($numErrors !== 0) {
+                    if ( $numErrors !== 0 ) {
                         $foundString .= 'error(s)';
-                        $errors      .= implode(PHP_EOL.' -> ', $problems['found_errors']);
+                        $errors      .= implode( PHP_EOL . ' -> ', $problems[ 'found_errors' ] );
                     }
 
-                    if ($expectedWarnings !== $numWarnings) {
+                    if ( $expectedWarnings !== $numWarnings ) {
                         $expectedMessage .= ' and ';
                         $foundMessage    .= ' and ';
-                        if ($numWarnings !== 0) {
-                            if ($foundString !== '') {
+                        if ( $numWarnings !== 0 ) {
+                            if ( $foundString !== '' ) {
                                 $foundString .= ' and ';
                             }
                         }
                     }
                 }
 
-                if ($expectedWarnings !== $numWarnings) {
+                if ( $expectedWarnings !== $numWarnings ) {
                     $expectedMessage .= "$expectedWarnings warning(s)";
                     $foundMessage    .= "$numWarnings warning(s)";
-                    if ($numWarnings !== 0) {
+                    if ( $numWarnings !== 0 ) {
                         $foundString .= 'warning(s)';
-                        if (empty($errors) === false) {
-                            $errors .= PHP_EOL.' -> ';
+                        if ( empty( $errors ) === false ) {
+                            $errors .= PHP_EOL . ' -> ';
                         }
 
-                        $errors .= implode(PHP_EOL.' -> ', $problems['found_warnings']);
+                        $errors .= implode( PHP_EOL . ' -> ', $problems[ 'found_warnings' ] );
                     }
                 }
 
                 $fullMessage = "$lineMessage $expectedMessage $foundMessage.";
-                if ($errors !== '') {
-                    $fullMessage .= " The $foundString found were:".PHP_EOL." -> $errors";
+                if ( $errors !== '' ) {
+                    $fullMessage .= " The $foundString found were:" . PHP_EOL . " -> $errors";
                 }
 
-                $failureMessages[] = $fullMessage;
+                $failureMessages[ ] = $fullMessage;
             }//end if
         }//end foreach
 
@@ -418,7 +418,7 @@ 
      *
      * @return void
      */
-    public function setCliValues($filename, $config)
+    public function setCliValues( $filename, $config )
     {
         return;
 

Braces +7 added lines, -14 removed lines

@@ -20,8 +20,7 @@ 
 use PHP_CodeSniffer\Util\Common;
 use PHPUnit\Framework\TestCase;
 
-abstract class AbstractSniffUnitTest extends TestCase
-{
+abstract class AbstractSniffUnitTest extends TestCase {
 
     /**
      * Enable or disable the backup and restoration of the $GLOBALS array.
@@ -52,8 +51,7 @@ 
      *
      * @return void
      */
-    protected function setUp()
-    {
+    protected function setUp() {
         $class = get_class($this);
         $this->standardsDir = $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$class];
         $this->testsDir     = $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$class];
@@ -71,8 +69,7 @@ 
      *
      * @return string[]
      */
-    protected function getTestFiles($testFileBase)
-    {
+    protected function getTestFiles($testFileBase) {
         $testFiles = [];
 
         $dir = substr($testFileBase, 0, strrpos($testFileBase, DIRECTORY_SEPARATOR));
@@ -100,8 +97,7 @@ 
      *
      * @return boolean
      */
-    protected function shouldSkipTest()
-    {
+    protected function shouldSkipTest() {
         return false;
 
     }//end shouldSkipTest()
@@ -113,8 +109,7 @@ 
      * @return void
      * @throws \PHPUnit\Framework\Exception
      */
-    final public function testSniff()
-    {
+    final public function testSniff() {
         // Skip this test if we can't run in this environment.
         if ($this->shouldSkipTest() === true) {
             $this->markTestSkipped();
@@ -217,8 +212,7 @@ 
      * @return array
      * @throws \PHP_CodeSniffer\Exceptions\RuntimeException
      */
-    public function generateFailureMessages(LocalFile $file)
-    {
+    public function generateFailureMessages(LocalFile $file) {
         $testFile = $file->getFilename();
 
         $foundErrors      = $file->getErrors();
@@ -418,8 +412,7 @@ 
      *
      * @return void
      */
-    public function setCliValues($filename, $config)
-    {
+    public function setCliValues($filename, $config) {
         return;
 
     }//end setCliValues()

vendor/squizlabs/php_codesniffer/tests/Standards/AllSniffs.php

Indentation +100 added lines, -100 removed lines

@@ -18,106 +18,106 @@
 {
 
 
-    /**
-     * Prepare the test runner.
-     *
-     * @return void
-     */
-    public static function main()
-    {
-        TestRunner::run(self::suite());
-
-    }//end main()
-
-
-    /**
-     * Add all sniff unit tests into a test suite.
-     *
-     * Sniff unit tests are found by recursing through the 'Tests' directory
-     * of each installed coding standard.
-     *
-     * @return \PHPUnit\Framework\TestSuite
-     */
-    public static function suite()
-    {
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES']      = [];
-        $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES']    = [];
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'] = [];
-
-        $suite = new TestSuite('PHP CodeSniffer Standards');
-
-        $isInstalled = !is_file(__DIR__.'/../../autoload.php');
-
-        // Optionally allow for ignoring the tests for one or more standards.
-        $ignoreTestsForStandards = getenv('PHPCS_IGNORE_TESTS');
-        if ($ignoreTestsForStandards === false) {
-            $ignoreTestsForStandards = [];
-        } else {
-            $ignoreTestsForStandards = explode(',', $ignoreTestsForStandards);
-        }
-
-        $installedStandards = self::getInstalledStandardDetails();
-
-        foreach ($installedStandards as $standard => $details) {
-            Autoload::addSearchPath($details['path'], $details['namespace']);
-
-            // If the test is running PEAR installed, the built-in standards
-            // are split into different directories; one for the sniffs and
-            // a different file system location for tests.
-            if ($isInstalled === true && is_dir(dirname($details['path']).DIRECTORY_SEPARATOR.'Generic') === true) {
-                $testPath = realpath(__DIR__.'/../../src/Standards/'.$standard);
-            } else {
-                $testPath = $details['path'];
-            }
-
-            if (in_array($standard, $ignoreTestsForStandards, true) === true) {
-                continue;
-            }
-
-            $testsDir = $testPath.DIRECTORY_SEPARATOR.'Tests'.DIRECTORY_SEPARATOR;
-            if (is_dir($testsDir) === false) {
-                // No tests for this standard.
-                continue;
-            }
-
-            $di = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($testsDir));
-
-            foreach ($di as $file) {
-                // Skip hidden files.
-                if (substr($file->getFilename(), 0, 1) === '.') {
-                    continue;
-                }
-
-                // Tests must have the extension 'php'.
-                $parts = explode('.', $file);
-                $ext   = array_pop($parts);
-                if ($ext !== 'php') {
-                    continue;
-                }
-
-                $className = Autoload::loadFile($file->getPathname());
-                $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$className] = $details['path'];
-                $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$className]     = $testsDir;
-                $suite->addTestSuite($className);
-            }
-        }//end foreach
-
-        return $suite;
-
-    }//end suite()
-
-
-    /**
-     * Get the details of all coding standards installed.
-     *
-     * @return array
-     * @see    Standards::getInstalledStandardDetails()
-     */
-    protected static function getInstalledStandardDetails()
-    {
-        return Standards::getInstalledStandardDetails(true);
-
-    }//end getInstalledStandardDetails()
+	/**
+	 * Prepare the test runner.
+	 *
+	 * @return void
+	 */
+	public static function main()
+	{
+		TestRunner::run(self::suite());
+
+	}//end main()
+
+
+	/**
+	 * Add all sniff unit tests into a test suite.
+	 *
+	 * Sniff unit tests are found by recursing through the 'Tests' directory
+	 * of each installed coding standard.
+	 *
+	 * @return \PHPUnit\Framework\TestSuite
+	 */
+	public static function suite()
+	{
+		$GLOBALS['PHP_CODESNIFFER_SNIFF_CODES']      = [];
+		$GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES']    = [];
+		$GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'] = [];
+
+		$suite = new TestSuite('PHP CodeSniffer Standards');
+
+		$isInstalled = !is_file(__DIR__.'/../../autoload.php');
+
+		// Optionally allow for ignoring the tests for one or more standards.
+		$ignoreTestsForStandards = getenv('PHPCS_IGNORE_TESTS');
+		if ($ignoreTestsForStandards === false) {
+			$ignoreTestsForStandards = [];
+		} else {
+			$ignoreTestsForStandards = explode(',', $ignoreTestsForStandards);
+		}
+
+		$installedStandards = self::getInstalledStandardDetails();
+
+		foreach ($installedStandards as $standard => $details) {
+			Autoload::addSearchPath($details['path'], $details['namespace']);
+
+			// If the test is running PEAR installed, the built-in standards
+			// are split into different directories; one for the sniffs and
+			// a different file system location for tests.
+			if ($isInstalled === true && is_dir(dirname($details['path']).DIRECTORY_SEPARATOR.'Generic') === true) {
+				$testPath = realpath(__DIR__.'/../../src/Standards/'.$standard);
+			} else {
+				$testPath = $details['path'];
+			}
+
+			if (in_array($standard, $ignoreTestsForStandards, true) === true) {
+				continue;
+			}
+
+			$testsDir = $testPath.DIRECTORY_SEPARATOR.'Tests'.DIRECTORY_SEPARATOR;
+			if (is_dir($testsDir) === false) {
+				// No tests for this standard.
+				continue;
+			}
+
+			$di = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($testsDir));
+
+			foreach ($di as $file) {
+				// Skip hidden files.
+				if (substr($file->getFilename(), 0, 1) === '.') {
+					continue;
+				}
+
+				// Tests must have the extension 'php'.
+				$parts = explode('.', $file);
+				$ext   = array_pop($parts);
+				if ($ext !== 'php') {
+					continue;
+				}
+
+				$className = Autoload::loadFile($file->getPathname());
+				$GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$className] = $details['path'];
+				$GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$className]     = $testsDir;
+				$suite->addTestSuite($className);
+			}
+		}//end foreach
+
+		return $suite;
+
+	}//end suite()
+
+
+	/**
+	 * Get the details of all coding standards installed.
+	 *
+	 * @return array
+	 * @see    Standards::getInstalledStandardDetails()
+	 */
+	protected static function getInstalledStandardDetails()
+	{
+		return Standards::getInstalledStandardDetails(true);
+
+	}//end getInstalledStandardDetails()
 
 
 }//end class

Spacing +29 added lines, -29 removed lines

@@ -25,7 +25,7 @@ 
      */
     public static function main()
     {
-        TestRunner::run(self::suite());
+        TestRunner::run( self::suite() );
 
     }//end main()
 
@@ -40,65 +40,65 @@ 
      */
     public static function suite()
     {
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES']      = [];
-        $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES']    = [];
-        $GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'] = [];
+        $GLOBALS[ 'PHP_CODESNIFFER_SNIFF_CODES' ]      = [ ];
+        $GLOBALS[ 'PHP_CODESNIFFER_FIXABLE_CODES' ]    = [ ];
+        $GLOBALS[ 'PHP_CODESNIFFER_SNIFF_CASE_FILES' ] = [ ];
 
-        $suite = new TestSuite('PHP CodeSniffer Standards');
+        $suite = new TestSuite( 'PHP CodeSniffer Standards' );
 
-        $isInstalled = !is_file(__DIR__.'/../../autoload.php');
+        $isInstalled = ! is_file( __DIR__ . '/../../autoload.php' );
 
         // Optionally allow for ignoring the tests for one or more standards.
-        $ignoreTestsForStandards = getenv('PHPCS_IGNORE_TESTS');
-        if ($ignoreTestsForStandards === false) {
-            $ignoreTestsForStandards = [];
+        $ignoreTestsForStandards = getenv( 'PHPCS_IGNORE_TESTS' );
+        if ( $ignoreTestsForStandards === false ) {
+            $ignoreTestsForStandards = [ ];
         } else {
-            $ignoreTestsForStandards = explode(',', $ignoreTestsForStandards);
+            $ignoreTestsForStandards = explode( ',', $ignoreTestsForStandards );
         }
 
         $installedStandards = self::getInstalledStandardDetails();
 
-        foreach ($installedStandards as $standard => $details) {
-            Autoload::addSearchPath($details['path'], $details['namespace']);
+        foreach ( $installedStandards as $standard => $details ) {
+            Autoload::addSearchPath( $details[ 'path' ], $details[ 'namespace' ] );
 
             // If the test is running PEAR installed, the built-in standards
             // are split into different directories; one for the sniffs and
             // a different file system location for tests.
-            if ($isInstalled === true && is_dir(dirname($details['path']).DIRECTORY_SEPARATOR.'Generic') === true) {
-                $testPath = realpath(__DIR__.'/../../src/Standards/'.$standard);
+            if ( $isInstalled === true && is_dir( dirname( $details[ 'path' ] ) . DIRECTORY_SEPARATOR . 'Generic' ) === true ) {
+                $testPath = realpath( __DIR__ . '/../../src/Standards/' . $standard );
             } else {
-                $testPath = $details['path'];
+                $testPath = $details[ 'path' ];
             }
 
-            if (in_array($standard, $ignoreTestsForStandards, true) === true) {
+            if ( in_array( $standard, $ignoreTestsForStandards, true ) === true ) {
                 continue;
             }
 
-            $testsDir = $testPath.DIRECTORY_SEPARATOR.'Tests'.DIRECTORY_SEPARATOR;
-            if (is_dir($testsDir) === false) {
+            $testsDir = $testPath . DIRECTORY_SEPARATOR . 'Tests' . DIRECTORY_SEPARATOR;
+            if ( is_dir( $testsDir ) === false ) {
                 // No tests for this standard.
                 continue;
             }
 
-            $di = new \RecursiveIteratorIterator(new \RecursiveDirectoryIterator($testsDir));
+            $di = new \RecursiveIteratorIterator( new \RecursiveDirectoryIterator( $testsDir ) );
 
-            foreach ($di as $file) {
+            foreach ( $di as $file ) {
                 // Skip hidden files.
-                if (substr($file->getFilename(), 0, 1) === '.') {
+                if ( substr( $file->getFilename(), 0, 1 ) === '.' ) {
                     continue;
                 }
 
                 // Tests must have the extension 'php'.
-                $parts = explode('.', $file);
-                $ext   = array_pop($parts);
-                if ($ext !== 'php') {
+                $parts = explode( '.', $file );
+                $ext   = array_pop( $parts );
+                if ( $ext !== 'php' ) {
                     continue;
                 }
 
-                $className = Autoload::loadFile($file->getPathname());
-                $GLOBALS['PHP_CODESNIFFER_STANDARD_DIRS'][$className] = $details['path'];
-                $GLOBALS['PHP_CODESNIFFER_TEST_DIRS'][$className]     = $testsDir;
-                $suite->addTestSuite($className);
+                $className = Autoload::loadFile( $file->getPathname() );
+                $GLOBALS[ 'PHP_CODESNIFFER_STANDARD_DIRS' ][ $className ] = $details[ 'path' ];
+                $GLOBALS[ 'PHP_CODESNIFFER_TEST_DIRS' ][ $className ]     = $testsDir;
+                $suite->addTestSuite( $className );
             }
         }//end foreach
 
@@ -115,7 +115,7 @@ 
      */
     protected static function getInstalledStandardDetails()
     {
-        return Standards::getInstalledStandardDetails(true);
+        return Standards::getInstalledStandardDetails( true );
 
     }//end getInstalledStandardDetails()
 

Braces +4 added lines, -8 removed lines

@@ -14,8 +14,7 @@ 
 use PHPUnit\TextUI\TestRunner;
 use PHPUnit\Framework\TestSuite;
 
-class AllSniffs
-{
+class AllSniffs {
 
 
     /**
@@ -23,8 +22,7 @@ 
      *
      * @return void
      */
-    public static function main()
-    {
+    public static function main() {
         TestRunner::run(self::suite());
 
     }//end main()
@@ -38,8 +36,7 @@ 
      *
      * @return \PHPUnit\Framework\TestSuite
      */
-    public static function suite()
-    {
+    public static function suite() {
         $GLOBALS['PHP_CODESNIFFER_SNIFF_CODES']      = [];
         $GLOBALS['PHP_CODESNIFFER_FIXABLE_CODES']    = [];
         $GLOBALS['PHP_CODESNIFFER_SNIFF_CASE_FILES'] = [];
@@ -113,8 +110,7 @@ 
      * @return array
      * @see    Standards::getInstalledStandardDetails()
      */
-    protected static function getInstalledStandardDetails()
-    {
+    protected static function getInstalledStandardDetails() {
         return Standards::getInstalledStandardDetails(true);
 
     }//end getInstalledStandardDetails()

vendor/squizlabs/php_codesniffer/tests/Core/IsCamelCapsTest.php

Indentation +114 added lines, -114 removed lines

@@ -16,120 +16,120 @@
 {
 
 
-    /**
-     * Test valid public function/method names.
-     *
-     * @return void
-     */
-    public function testValidNotClassFormatPublic()
-    {
-        $this->assertTrue(Common::isCamelCaps('thisIsCamelCaps', false, true, true));
-        $this->assertTrue(Common::isCamelCaps('thisISCamelCaps', false, true, false));
-
-    }//end testValidNotClassFormatPublic()
-
-
-    /**
-     * Test invalid public function/method names.
-     *
-     * @return void
-     */
-    public function testInvalidNotClassFormatPublic()
-    {
-        $this->assertFalse(Common::isCamelCaps('_thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('thisISCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('ThisIsCamelCaps', false, true, true));
-
-        $this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, true, true));
-
-        $this->assertFalse(Common::isCamelCaps('this*IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this-IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this_IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this_is_camel_caps', false, true, true));
-
-    }//end testInvalidNotClassFormatPublic()
-
-
-    /**
-     * Test valid private method names.
-     *
-     * @return void
-     */
-    public function testValidNotClassFormatPrivate()
-    {
-        $this->assertTrue(Common::isCamelCaps('_thisIsCamelCaps', false, false, true));
-        $this->assertTrue(Common::isCamelCaps('_thisISCamelCaps', false, false, false));
-        $this->assertTrue(Common::isCamelCaps('_i18N', false, false, true));
-        $this->assertTrue(Common::isCamelCaps('_i18n', false, false, true));
-
-    }//end testValidNotClassFormatPrivate()
-
-
-    /**
-     * Test invalid private method names.
-     *
-     * @return void
-     */
-    public function testInvalidNotClassFormatPrivate()
-    {
-        $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_thisISCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('__thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('__thisISCamelCaps', false, false, false));
-
-        $this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_this_is_camel_caps', false, false, true));
-
-    }//end testInvalidNotClassFormatPrivate()
-
-
-    /**
-     * Test valid class names.
-     *
-     * @return void
-     */
-    public function testValidClassFormatPublic()
-    {
-        $this->assertTrue(Common::isCamelCaps('ThisIsCamelCaps', true, true, true));
-        $this->assertTrue(Common::isCamelCaps('ThisISCamelCaps', true, true, false));
-        $this->assertTrue(Common::isCamelCaps('This3IsCamelCaps', true, true, false));
-
-    }//end testValidClassFormatPublic()
-
-
-    /**
-     * Test invalid class names.
-     *
-     * @return void
-     */
-    public function testInvalidClassFormat()
-    {
-        $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', true));
-        $this->assertFalse(Common::isCamelCaps('This-IsCamelCaps', true));
-        $this->assertFalse(Common::isCamelCaps('This_Is_Camel_Caps', true));
-
-    }//end testInvalidClassFormat()
-
-
-    /**
-     * Test invalid class names with the private flag set.
-     *
-     * Note that the private flag is ignored if the class format
-     * flag is set, so these names are all invalid.
-     *
-     * @return void
-     */
-    public function testInvalidClassFormatPrivate()
-    {
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, true));
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, false));
-
-    }//end testInvalidClassFormatPrivate()
+	/**
+	 * Test valid public function/method names.
+	 *
+	 * @return void
+	 */
+	public function testValidNotClassFormatPublic()
+	{
+		$this->assertTrue(Common::isCamelCaps('thisIsCamelCaps', false, true, true));
+		$this->assertTrue(Common::isCamelCaps('thisISCamelCaps', false, true, false));
+
+	}//end testValidNotClassFormatPublic()
+
+
+	/**
+	 * Test invalid public function/method names.
+	 *
+	 * @return void
+	 */
+	public function testInvalidNotClassFormatPublic()
+	{
+		$this->assertFalse(Common::isCamelCaps('_thisIsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('thisISCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('ThisIsCamelCaps', false, true, true));
+
+		$this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, true, true));
+
+		$this->assertFalse(Common::isCamelCaps('this*IsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('this-IsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('this_IsCamelCaps', false, true, true));
+		$this->assertFalse(Common::isCamelCaps('this_is_camel_caps', false, true, true));
+
+	}//end testInvalidNotClassFormatPublic()
+
+
+	/**
+	 * Test valid private method names.
+	 *
+	 * @return void
+	 */
+	public function testValidNotClassFormatPrivate()
+	{
+		$this->assertTrue(Common::isCamelCaps('_thisIsCamelCaps', false, false, true));
+		$this->assertTrue(Common::isCamelCaps('_thisISCamelCaps', false, false, false));
+		$this->assertTrue(Common::isCamelCaps('_i18N', false, false, true));
+		$this->assertTrue(Common::isCamelCaps('_i18n', false, false, true));
+
+	}//end testValidNotClassFormatPrivate()
+
+
+	/**
+	 * Test invalid private method names.
+	 *
+	 * @return void
+	 */
+	public function testInvalidNotClassFormatPrivate()
+	{
+		$this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('_thisISCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('__thisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('__thisISCamelCaps', false, false, false));
+
+		$this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, false, true));
+		$this->assertFalse(Common::isCamelCaps('_this_is_camel_caps', false, false, true));
+
+	}//end testInvalidNotClassFormatPrivate()
+
+
+	/**
+	 * Test valid class names.
+	 *
+	 * @return void
+	 */
+	public function testValidClassFormatPublic()
+	{
+		$this->assertTrue(Common::isCamelCaps('ThisIsCamelCaps', true, true, true));
+		$this->assertTrue(Common::isCamelCaps('ThisISCamelCaps', true, true, false));
+		$this->assertTrue(Common::isCamelCaps('This3IsCamelCaps', true, true, false));
+
+	}//end testValidClassFormatPublic()
+
+
+	/**
+	 * Test invalid class names.
+	 *
+	 * @return void
+	 */
+	public function testInvalidClassFormat()
+	{
+		$this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', true));
+		$this->assertFalse(Common::isCamelCaps('This-IsCamelCaps', true));
+		$this->assertFalse(Common::isCamelCaps('This_Is_Camel_Caps', true));
+
+	}//end testInvalidClassFormat()
+
+
+	/**
+	 * Test invalid class names with the private flag set.
+	 *
+	 * Note that the private flag is ignored if the class format
+	 * flag is set, so these names are all invalid.
+	 *
+	 * @return void
+	 */
+	public function testInvalidClassFormatPrivate()
+	{
+		$this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, true));
+		$this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, false));
+
+	}//end testInvalidClassFormatPrivate()
 
 
 }//end class

Spacing +33 added lines, -33 removed lines

@@ -23,8 +23,8 @@ 
      */
     public function testValidNotClassFormatPublic()
     {
-        $this->assertTrue(Common::isCamelCaps('thisIsCamelCaps', false, true, true));
-        $this->assertTrue(Common::isCamelCaps('thisISCamelCaps', false, true, false));
+        $this->assertTrue( Common::isCamelCaps( 'thisIsCamelCaps', false, true, true ) );
+        $this->assertTrue( Common::isCamelCaps( 'thisISCamelCaps', false, true, false ) );
 
     }//end testValidNotClassFormatPublic()
 
@@ -36,18 +36,18 @@ 
      */
     public function testInvalidNotClassFormatPublic()
     {
-        $this->assertFalse(Common::isCamelCaps('_thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('thisISCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('ThisIsCamelCaps', false, true, true));
+        $this->assertFalse( Common::isCamelCaps( '_thisIsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( 'thisISCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( 'ThisIsCamelCaps', false, true, true ) );
 
-        $this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, true, true));
+        $this->assertFalse( Common::isCamelCaps( '3thisIsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( '*thisIsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( '-thisIsCamelCaps', false, true, true ) );
 
-        $this->assertFalse(Common::isCamelCaps('this*IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this-IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this_IsCamelCaps', false, true, true));
-        $this->assertFalse(Common::isCamelCaps('this_is_camel_caps', false, true, true));
+        $this->assertFalse( Common::isCamelCaps( 'this*IsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( 'this-IsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( 'this_IsCamelCaps', false, true, true ) );
+        $this->assertFalse( Common::isCamelCaps( 'this_is_camel_caps', false, true, true ) );
 
     }//end testInvalidNotClassFormatPublic()
 
@@ -59,10 +59,10 @@ 
      */
     public function testValidNotClassFormatPrivate()
     {
-        $this->assertTrue(Common::isCamelCaps('_thisIsCamelCaps', false, false, true));
-        $this->assertTrue(Common::isCamelCaps('_thisISCamelCaps', false, false, false));
-        $this->assertTrue(Common::isCamelCaps('_i18N', false, false, true));
-        $this->assertTrue(Common::isCamelCaps('_i18n', false, false, true));
+        $this->assertTrue( Common::isCamelCaps( '_thisIsCamelCaps', false, false, true ) );
+        $this->assertTrue( Common::isCamelCaps( '_thisISCamelCaps', false, false, false ) );
+        $this->assertTrue( Common::isCamelCaps( '_i18N', false, false, true ) );
+        $this->assertTrue( Common::isCamelCaps( '_i18n', false, false, true ) );
 
     }//end testValidNotClassFormatPrivate()
 
@@ -74,16 +74,16 @@ 
      */
     public function testInvalidNotClassFormatPrivate()
     {
-        $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_thisISCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('__thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('__thisISCamelCaps', false, false, false));
+        $this->assertFalse( Common::isCamelCaps( 'thisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '_thisISCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '_ThisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '__thisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '__thisISCamelCaps', false, false, false ) );
 
-        $this->assertFalse(Common::isCamelCaps('3thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('*thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('-thisIsCamelCaps', false, false, true));
-        $this->assertFalse(Common::isCamelCaps('_this_is_camel_caps', false, false, true));
+        $this->assertFalse( Common::isCamelCaps( '3thisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '*thisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '-thisIsCamelCaps', false, false, true ) );
+        $this->assertFalse( Common::isCamelCaps( '_this_is_camel_caps', false, false, true ) );
 
     }//end testInvalidNotClassFormatPrivate()
 
@@ -95,9 +95,9 @@ 
      */
     public function testValidClassFormatPublic()
     {
-        $this->assertTrue(Common::isCamelCaps('ThisIsCamelCaps', true, true, true));
-        $this->assertTrue(Common::isCamelCaps('ThisISCamelCaps', true, true, false));
-        $this->assertTrue(Common::isCamelCaps('This3IsCamelCaps', true, true, false));
+        $this->assertTrue( Common::isCamelCaps( 'ThisIsCamelCaps', true, true, true ) );
+        $this->assertTrue( Common::isCamelCaps( 'ThisISCamelCaps', true, true, false ) );
+        $this->assertTrue( Common::isCamelCaps( 'This3IsCamelCaps', true, true, false ) );
 
     }//end testValidClassFormatPublic()
 
@@ -109,9 +109,9 @@ 
      */
     public function testInvalidClassFormat()
     {
-        $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', true));
-        $this->assertFalse(Common::isCamelCaps('This-IsCamelCaps', true));
-        $this->assertFalse(Common::isCamelCaps('This_Is_Camel_Caps', true));
+        $this->assertFalse( Common::isCamelCaps( 'thisIsCamelCaps', true ) );
+        $this->assertFalse( Common::isCamelCaps( 'This-IsCamelCaps', true ) );
+        $this->assertFalse( Common::isCamelCaps( 'This_Is_Camel_Caps', true ) );
 
     }//end testInvalidClassFormat()
 
@@ -126,8 +126,8 @@ 
      */
     public function testInvalidClassFormatPrivate()
     {
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, true));
-        $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, false));
+        $this->assertFalse( Common::isCamelCaps( '_ThisIsCamelCaps', true, true ) );
+        $this->assertFalse( Common::isCamelCaps( '_ThisIsCamelCaps', true, false ) );
 
     }//end testInvalidClassFormatPrivate()
 

Braces +8 added lines, -16 removed lines

@@ -12,8 +12,7 @@ 
 use PHP_CodeSniffer\Util\Common;
 use PHPUnit\Framework\TestCase;
 
-class IsCamelCapsTest extends TestCase
-{
+class IsCamelCapsTest extends TestCase {
 
 
     /**
@@ -21,8 +20,7 @@ 
      *
      * @return void
      */
-    public function testValidNotClassFormatPublic()
-    {
+    public function testValidNotClassFormatPublic() {
         $this->assertTrue(Common::isCamelCaps('thisIsCamelCaps', false, true, true));
         $this->assertTrue(Common::isCamelCaps('thisISCamelCaps', false, true, false));
 
@@ -34,8 +32,7 @@ 
      *
      * @return void
      */
-    public function testInvalidNotClassFormatPublic()
-    {
+    public function testInvalidNotClassFormatPublic() {
         $this->assertFalse(Common::isCamelCaps('_thisIsCamelCaps', false, true, true));
         $this->assertFalse(Common::isCamelCaps('thisISCamelCaps', false, true, true));
         $this->assertFalse(Common::isCamelCaps('ThisIsCamelCaps', false, true, true));
@@ -57,8 +54,7 @@ 
      *
      * @return void
      */
-    public function testValidNotClassFormatPrivate()
-    {
+    public function testValidNotClassFormatPrivate() {
         $this->assertTrue(Common::isCamelCaps('_thisIsCamelCaps', false, false, true));
         $this->assertTrue(Common::isCamelCaps('_thisISCamelCaps', false, false, false));
         $this->assertTrue(Common::isCamelCaps('_i18N', false, false, true));
@@ -72,8 +68,7 @@ 
      *
      * @return void
      */
-    public function testInvalidNotClassFormatPrivate()
-    {
+    public function testInvalidNotClassFormatPrivate() {
         $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', false, false, true));
         $this->assertFalse(Common::isCamelCaps('_thisISCamelCaps', false, false, true));
         $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', false, false, true));
@@ -93,8 +88,7 @@ 
      *
      * @return void
      */
-    public function testValidClassFormatPublic()
-    {
+    public function testValidClassFormatPublic() {
         $this->assertTrue(Common::isCamelCaps('ThisIsCamelCaps', true, true, true));
         $this->assertTrue(Common::isCamelCaps('ThisISCamelCaps', true, true, false));
         $this->assertTrue(Common::isCamelCaps('This3IsCamelCaps', true, true, false));
@@ -107,8 +101,7 @@ 
      *
      * @return void
      */
-    public function testInvalidClassFormat()
-    {
+    public function testInvalidClassFormat() {
         $this->assertFalse(Common::isCamelCaps('thisIsCamelCaps', true));
         $this->assertFalse(Common::isCamelCaps('This-IsCamelCaps', true));
         $this->assertFalse(Common::isCamelCaps('This_Is_Camel_Caps', true));
@@ -124,8 +117,7 @@ 
      *
      * @return void
      */
-    public function testInvalidClassFormatPrivate()
-    {
+    public function testInvalidClassFormatPrivate() {
         $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, true));
         $this->assertFalse(Common::isCamelCaps('_ThisIsCamelCaps', true, false));
 

vendor/squizlabs/php_codesniffer/tests/Core/File/FindEndOfStatementTest.inc

Indentation +2 added lines, -2 removed lines

@@ -18,8 +18,8 @@
 
 /* testSwitch */
 switch ($a) {
-    case 1: {break;}
-    default: {break;}
+	case 1: {break;}
+	default: {break;}
 }
 
 /* testStatementAsArrayValue */

Spacing +9 added lines, -9 removed lines

@@ -4,25 +4,25 @@
 $a = false;
 
 /* testControlStructure */
-while(true) {}
+while ( true ) {}
 $a = 1;
 
 /* testClosureAssignment */
-$a = function($b=false;){};
+$a = function( $b = false; ) {};
 
 /* testHeredocFunctionArg */
-myFunction(<<<END
+myFunction( <<<END
 Foo
 END
-, 'bar');
+, 'bar' );
 
 /* testSwitch */
-switch ($a) {
-    case 1: {break;}
-    default: {break;}
+switch ( $a ) {
+    case 1: {break; }
+    default: {break; }
 }
 
 /* testStatementAsArrayValue */
-$a = [new Datetime];
-$a = array(new Datetime);
+$a = [ new Datetime ];
+$a = array( new Datetime );
 $a = new Datetime;

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
 
 /* testHeredocFunctionArg */
 myFunction(<<<END
-Foo
-END
+foo
+end
 , 'bar');
 
 /* testSwitch */

vendor/squizlabs/php_codesniffer/tests/Core/File/FindEndOfStatementTest.php

Indentation +156 added lines, -156 removed lines

@@ -17,187 +17,187 @@
 class FindEndOfStatementTest extends TestCase
 {
 
-    /**
-     * The PHP_CodeSniffer_File object containing parsed contents of the test case file.
-     *
-     * @var \PHP_CodeSniffer\Files\File
-     */
-    private $phpcsFile;
-
-
-    /**
-     * Initialize & tokenize \PHP_CodeSniffer\Files\File with code from the test case file.
-     *
-     * Methods used for these tests can be found in a test case file in the same
-     * directory and with the same name, using the .inc extension.
-     *
-     * @return void
-     */
-    public function setUp()
-    {
-        $config            = new Config();
-        $config->standards = ['Generic'];
+	/**
+	 * The PHP_CodeSniffer_File object containing parsed contents of the test case file.
+	 *
+	 * @var \PHP_CodeSniffer\Files\File
+	 */
+	private $phpcsFile;
+
+
+	/**
+	 * Initialize & tokenize \PHP_CodeSniffer\Files\File with code from the test case file.
+	 *
+	 * Methods used for these tests can be found in a test case file in the same
+	 * directory and with the same name, using the .inc extension.
+	 *
+	 * @return void
+	 */
+	public function setUp()
+	{
+		$config            = new Config();
+		$config->standards = ['Generic'];
 
-        $ruleset = new Ruleset($config);
+		$ruleset = new Ruleset($config);
 
-        $pathToTestFile  = dirname(__FILE__).'/'.basename(__FILE__, '.php').'.inc';
-        $this->phpcsFile = new DummyFile(file_get_contents($pathToTestFile), $ruleset, $config);
-        $this->phpcsFile->process();
-
-    }//end setUp()
-
-
-    /**
-     * Clean up after finished test.
-     *
-     * @return void
-     */
-    public function tearDown()
-    {
-        unset($this->phpcsFile);
-
-    }//end tearDown()
-
-
-    /**
-     * Test a simple assignment.
-     *
-     * @return void
-     */
-    public function testSimpleAssignment()
-    {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSimpleAssignment */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+		$pathToTestFile  = dirname(__FILE__).'/'.basename(__FILE__, '.php').'.inc';
+		$this->phpcsFile = new DummyFile(file_get_contents($pathToTestFile), $ruleset, $config);
+		$this->phpcsFile->process();
+
+	}//end setUp()
+
+
+	/**
+	 * Clean up after finished test.
+	 *
+	 * @return void
+	 */
+	public function tearDown()
+	{
+		unset($this->phpcsFile);
+
+	}//end tearDown()
+
+
+	/**
+	 * Test a simple assignment.
+	 *
+	 * @return void
+	 */
+	public function testSimpleAssignment()
+	{
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSimpleAssignment */') + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 5)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 5)], $tokens[$found]);
 
-    }//end testSimpleAssignment()
-
-
-    /**
-     * Test a direct call to a control structure.
-     *
-     * @return void
-     */
-    public function testControlStructure()
-    {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testControlStructure */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
-
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 6)], $tokens[$found]);
-
-    }//end testControlStructure()
-
-
-    /**
-     * Test the assignment of a closure.
-     *
-     * @return void
-     */
-    public function testClosureAssignment()
-    {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testClosureAssignment */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
-
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 13)], $tokens[$found]);
-
-    }//end testClosureAssignment()
-
-
-    /**
-     * Test using a heredoc in a function argument.
-     *
-     * @return void
-     */
-    public function testHeredocFunctionArg()
-    {
-        // Find the end of the function.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testHeredocFunctionArg */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
-
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 10)], $tokens[$found]);
+	}//end testSimpleAssignment()
+
+
+	/**
+	 * Test a direct call to a control structure.
+	 *
+	 * @return void
+	 */
+	public function testControlStructure()
+	{
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testControlStructure */') + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
+
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 6)], $tokens[$found]);
+
+	}//end testControlStructure()
+
+
+	/**
+	 * Test the assignment of a closure.
+	 *
+	 * @return void
+	 */
+	public function testClosureAssignment()
+	{
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testClosureAssignment */') + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
+
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 13)], $tokens[$found]);
+
+	}//end testClosureAssignment()
+
+
+	/**
+	 * Test using a heredoc in a function argument.
+	 *
+	 * @return void
+	 */
+	public function testHeredocFunctionArg()
+	{
+		// Find the end of the function.
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testHeredocFunctionArg */') + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
+
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 10)], $tokens[$found]);
 
-        // Find the end of the heredoc.
-        $start += 2;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+		// Find the end of the heredoc.
+		$start += 2;
+		$found  = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 4)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 4)], $tokens[$found]);
 
-        // Find the end of the last arg.
-        $start = ($found + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+		// Find the end of the last arg.
+		$start = ($found + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[$start], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[$start], $tokens[$found]);
 
-    }//end testHeredocFunctionArg()
+	}//end testHeredocFunctionArg()
 
 
-    /**
-     * Test parts of a switch statement.
-     *
-     * @return void
-     */
-    public function testSwitch()
-    {
-        // Find the end of the switch.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSwitch */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+	/**
+	 * Test parts of a switch statement.
+	 *
+	 * @return void
+	 */
+	public function testSwitch()
+	{
+		// Find the end of the switch.
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSwitch */') + 2);
+		$found = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 28)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 28)], $tokens[$found]);
 
-        // Find the end of the case.
-        $start += 9;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+		// Find the end of the case.
+		$start += 9;
+		$found  = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 8)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 8)], $tokens[$found]);
 
-        // Find the end of default case.
-        $start += 11;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+		// Find the end of default case.
+		$start += 11;
+		$found  = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 6)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 6)], $tokens[$found]);
 
-    }//end testSwitch()
+	}//end testSwitch()
 
 
-    /**
-     * Test statements that are array values.
-     *
-     * @return void
-     */
-    public function testStatementAsArrayValue()
-    {
-        // Test short array syntax.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testStatementAsArrayValue */') + 7);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+	/**
+	 * Test statements that are array values.
+	 *
+	 * @return void
+	 */
+	public function testStatementAsArrayValue()
+	{
+		// Test short array syntax.
+		$start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testStatementAsArrayValue */') + 7);
+		$found = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 2)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 2)], $tokens[$found]);
 
-        // Test long array syntax.
-        $start += 12;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+		// Test long array syntax.
+		$start += 12;
+		$found  = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 2)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 2)], $tokens[$found]);
 
-        // Test same statement outside of array.
-        $start += 10;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+		// Test same statement outside of array.
+		$start += 10;
+		$found  = $this->phpcsFile->findEndOfStatement($start);
 
-        $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 3)], $tokens[$found]);
+		$tokens = $this->phpcsFile->getTokens();
+		$this->assertSame($tokens[($start + 3)], $tokens[$found]);
 
-    }//end testStatementAsArrayValue()
+	}//end testStatementAsArrayValue()
 
 
 }//end class

Spacing +36 added lines, -36 removed lines

@@ -36,12 +36,12 @@ 
     public function setUp()
     {
         $config            = new Config();
-        $config->standards = ['Generic'];
+        $config->standards = [ 'Generic' ];
 
-        $ruleset = new Ruleset($config);
+        $ruleset = new Ruleset( $config );
 
-        $pathToTestFile  = dirname(__FILE__).'/'.basename(__FILE__, '.php').'.inc';
-        $this->phpcsFile = new DummyFile(file_get_contents($pathToTestFile), $ruleset, $config);
+        $pathToTestFile  = dirname( __FILE__ ) . '/' . basename( __FILE__, '.php' ) . '.inc';
+        $this->phpcsFile = new DummyFile( file_get_contents( $pathToTestFile ), $ruleset, $config );
         $this->phpcsFile->process();
 
     }//end setUp()
@@ -54,7 +54,7 @@ 
      */
     public function tearDown()
     {
-        unset($this->phpcsFile);
+        unset( $this->phpcsFile );
 
     }//end tearDown()
 
@@ -66,11 +66,11 @@ 
      */
     public function testSimpleAssignment()
     {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSimpleAssignment */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testSimpleAssignment */' ) + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 5)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 5 ) ], $tokens[ $found ] );
 
     }//end testSimpleAssignment()
 
@@ -82,11 +82,11 @@ 
      */
     public function testControlStructure()
     {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testControlStructure */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testControlStructure */' ) + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 6)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 6 ) ], $tokens[ $found ] );
 
     }//end testControlStructure()
 
@@ -98,11 +98,11 @@ 
      */
     public function testClosureAssignment()
     {
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testClosureAssignment */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testClosureAssignment */' ) + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 13)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 13 ) ], $tokens[ $found ] );
 
     }//end testClosureAssignment()
 
@@ -115,25 +115,25 @@ 
     public function testHeredocFunctionArg()
     {
         // Find the end of the function.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testHeredocFunctionArg */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testHeredocFunctionArg */' ) + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 10)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 10 ) ], $tokens[ $found ] );
 
         // Find the end of the heredoc.
         $start += 2;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+        $found  = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 4)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 4 ) ], $tokens[ $found ] );
 
         // Find the end of the last arg.
-        $start = ($found + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $found + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[$start], $tokens[$found]);
+        $this->assertSame( $tokens[ $start ], $tokens[ $found ] );
 
     }//end testHeredocFunctionArg()
 
@@ -146,25 +146,25 @@ 
     public function testSwitch()
     {
         // Find the end of the switch.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSwitch */') + 2);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testSwitch */' ) + 2 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 28)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 28 ) ], $tokens[ $found ] );
 
         // Find the end of the case.
         $start += 9;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+        $found  = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 8)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 8 ) ], $tokens[ $found ] );
 
         // Find the end of default case.
         $start += 11;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+        $found  = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 6)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 6 ) ], $tokens[ $found ] );
 
     }//end testSwitch()
 
@@ -177,25 +177,25 @@ 
     public function testStatementAsArrayValue()
     {
         // Test short array syntax.
-        $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testStatementAsArrayValue */') + 7);
-        $found = $this->phpcsFile->findEndOfStatement($start);
+        $start = ( $this->phpcsFile->findNext( T_COMMENT, 0, null, false, '/* testStatementAsArrayValue */' ) + 7 );
+        $found = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 2)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 2 ) ], $tokens[ $found ] );
 
         // Test long array syntax.
         $start += 12;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+        $found  = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 2)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 2 ) ], $tokens[ $found ] );
 
         // Test same statement outside of array.
         $start += 10;
-        $found  = $this->phpcsFile->findEndOfStatement($start);
+        $found  = $this->phpcsFile->findEndOfStatement( $start );
 
         $tokens = $this->phpcsFile->getTokens();
-        $this->assertSame($tokens[($start + 3)], $tokens[$found]);
+        $this->assertSame( $tokens[ ( $start + 3 ) ], $tokens[ $found ] );
 
     }//end testStatementAsArrayValue()
 

Braces +9 added lines, -18 removed lines

@@ -14,8 +14,7 @@ 
 use PHP_CodeSniffer\Files\DummyFile;
 use PHPUnit\Framework\TestCase;
 
-class FindEndOfStatementTest extends TestCase
-{
+class FindEndOfStatementTest extends TestCase {
 
     /**
      * The PHP_CodeSniffer_File object containing parsed contents of the test case file.
@@ -33,8 +32,7 @@ 
      *
      * @return void
      */
-    public function setUp()
-    {
+    public function setUp() {
         $config            = new Config();
         $config->standards = ['Generic'];
 
@@ -52,8 +50,7 @@ 
      *
      * @return void
      */
-    public function tearDown()
-    {
+    public function tearDown() {
         unset($this->phpcsFile);
 
     }//end tearDown()
@@ -64,8 +61,7 @@ 
      *
      * @return void
      */
-    public function testSimpleAssignment()
-    {
+    public function testSimpleAssignment() {
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSimpleAssignment */') + 2);
         $found = $this->phpcsFile->findEndOfStatement($start);
 
@@ -80,8 +76,7 @@ 
      *
      * @return void
      */
-    public function testControlStructure()
-    {
+    public function testControlStructure() {
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testControlStructure */') + 2);
         $found = $this->phpcsFile->findEndOfStatement($start);
 
@@ -96,8 +91,7 @@ 
      *
      * @return void
      */
-    public function testClosureAssignment()
-    {
+    public function testClosureAssignment() {
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testClosureAssignment */') + 2);
         $found = $this->phpcsFile->findEndOfStatement($start);
 
@@ -112,8 +106,7 @@ 
      *
      * @return void
      */
-    public function testHeredocFunctionArg()
-    {
+    public function testHeredocFunctionArg() {
         // Find the end of the function.
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testHeredocFunctionArg */') + 2);
         $found = $this->phpcsFile->findEndOfStatement($start);
@@ -143,8 +136,7 @@ 
      *
      * @return void
      */
-    public function testSwitch()
-    {
+    public function testSwitch() {
         // Find the end of the switch.
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testSwitch */') + 2);
         $found = $this->phpcsFile->findEndOfStatement($start);
@@ -174,8 +166,7 @@ 
      *
      * @return void
      */
-    public function testStatementAsArrayValue()
-    {
+    public function testStatementAsArrayValue() {
         // Test short array syntax.
         $start = ($this->phpcsFile->findNext(T_COMMENT, 0, null, false, '/* testStatementAsArrayValue */') + 7);
         $found = $this->phpcsFile->findEndOfStatement($start);

vendor/squizlabs/php_codesniffer/tests/Core/File/GetMethodParametersTest.inc

Spacing +9 added lines, -9 removed lines

@@ -1,29 +1,29 @@
 <?php
 
 /* testPassByReference */
-function passByReference(&$var) {}
+function passByReference( &$var ) {}
 
 /* testArrayHint */
-function arrayHint(array $var) {}
+function arrayHint( array $var ) {}
 
 /* testVariable */
-function variable($var) {}
+function variable( $var ) {}
 
 /* testSingleDefaultValue */
-function defaultValue($var1=self::CONSTANT) {}
+function defaultValue( $var1 = self::CONSTANT ) {}
 
 /* testDefaultValues */
-function defaultValues($var1=1, $var2='value') {}
+function defaultValues( $var1 = 1, $var2 = 'value' ) {}
 
 /* testTypeHint */
-function typeHint(foo $var1, bar $var2) {}
+function typeHint( foo $var1, bar $var2 ) {}
 
 class MyClass {
-/* testSelfTypeHint */ function typeSelfHint(self $var) {}
+/* testSelfTypeHint */ function typeSelfHint( self $var ) {}
 }
 
 /* testNullableTypeHint */
-function nullableTypeHint(?int $var1, ?\bar $var2) {}
+function nullableTypeHint( ?int $var1, ?\bar $var2 ) {}
 
 /* testBitwiseAndConstantExpressionDefaultValue */
-function myFunction($a = 10 & 20) {}
+function myFunction( $a = 10 & 20 ) {}

vendor/squizlabs/php_codesniffer/tests/Core/File/GetMemberPropertiesTest.inc

Indentation +95 added lines, -95 removed lines

@@ -2,113 +2,113 @@
 
 class TestMemberProperties
 {
-    /* testVar */
-    var $varA = true;
-
-    /* testPublic */
-    public $varB = true;
-
-    /* testProtected */
-    protected $varC = true;
-
-    /* testPrivate */
-    private $varD = true;
-
-    /* testStatic */
-    static $varE = true;
-
-    /* testStaticVar */
-    static var $varF = true;
-
-    /* testVarStatic */
-    var static $varG = true;
-
-    /* testPublicStatic */
-    public static $varH = true;
-
-    /* testProtectedStatic */
-    static protected $varI = true;
-
-    /* testPrivateStatic */
-    private static $varJ = true;
-
-    /* testNoPrefix */
-    $varK = true;
-
-    /* testPublicStaticWithDocblock */
-    /**
-     * Comment here.
-     *
-     * @phpcs:ignore Standard.Category.Sniff -- because
-     * @var boolean
-     */
-    public static $varH = true;
-
-    /* testProtectedStaticWithDocblock */
-    /**
-     * Comment here.
-     *
-     * @phpcs:ignore Standard.Category.Sniff -- because
-     * @var boolean
-     */
-    static protected $varI = true;
-
-    /* testPrivateStaticWithDocblock */
-    /**
-     * Comment here.
-     *
-     * @phpcs:ignore Standard.Category.Sniff -- because
-     * @var boolean
-     */
-    private static $varJ = true;
-
-
-    protected static
-        /* testGroupProtectedStatic 1 */
-        $varL,
-        /* testGroupProtectedStatic 2 */
-        $varM,
-        /* testGroupProtectedStatic 3 */
-        $varN;
-
-    private
-        /* testGroupPrivate 1 */
-        $varO = true,
-        /* testGroupPrivate 2 */
-        $varP = array( 'a' => 'a', 'b' => 'b' ),
-        /* testGroupPrivate 3 */
-        $varQ = 'string',
-        /* testGroupPrivate 4 */
-        $varR = 123,
-        /* testGroupPrivate 5 */
-        $varS = ONE / self::THREE,
-        /* testGroupPrivate 6 */
-        $varT = [
+	/* testVar */
+	var $varA = true;
+
+	/* testPublic */
+	public $varB = true;
+
+	/* testProtected */
+	protected $varC = true;
+
+	/* testPrivate */
+	private $varD = true;
+
+	/* testStatic */
+	static $varE = true;
+
+	/* testStaticVar */
+	static var $varF = true;
+
+	/* testVarStatic */
+	var static $varG = true;
+
+	/* testPublicStatic */
+	public static $varH = true;
+
+	/* testProtectedStatic */
+	static protected $varI = true;
+
+	/* testPrivateStatic */
+	private static $varJ = true;
+
+	/* testNoPrefix */
+	$varK = true;
+
+	/* testPublicStaticWithDocblock */
+	/**
+	 * Comment here.
+	 *
+	 * @phpcs:ignore Standard.Category.Sniff -- because
+	 * @var boolean
+	 */
+	public static $varH = true;
+
+	/* testProtectedStaticWithDocblock */
+	/**
+	 * Comment here.
+	 *
+	 * @phpcs:ignore Standard.Category.Sniff -- because
+	 * @var boolean
+	 */
+	static protected $varI = true;
+
+	/* testPrivateStaticWithDocblock */
+	/**
+	 * Comment here.
+	 *
+	 * @phpcs:ignore Standard.Category.Sniff -- because
+	 * @var boolean
+	 */
+	private static $varJ = true;
+
+
+	protected static
+		/* testGroupProtectedStatic 1 */
+		$varL,
+		/* testGroupProtectedStatic 2 */
+		$varM,
+		/* testGroupProtectedStatic 3 */
+		$varN;
+
+	private
+		/* testGroupPrivate 1 */
+		$varO = true,
+		/* testGroupPrivate 2 */
+		$varP = array( 'a' => 'a', 'b' => 'b' ),
+		/* testGroupPrivate 3 */
+		$varQ = 'string',
+		/* testGroupPrivate 4 */
+		$varR = 123,
+		/* testGroupPrivate 5 */
+		$varS = ONE / self::THREE,
+		/* testGroupPrivate 6 */
+		$varT = [
 			'a' => 'a',
 			'b' => 'b'
 		],
-        /* testGroupPrivate 7 */
-        $varU = __DIR__ . "/base";
+		/* testGroupPrivate 7 */
+		$varU = __DIR__ . "/base";
 
 
-    /* testMethodParam */
-    public function methodName($param) {
-        /* testImportedGlobal */
-        global $importedGlobal = true;
+	/* testMethodParam */
+	public function methodName($param) {
+		/* testImportedGlobal */
+		global $importedGlobal = true;
 
-        /* testLocalVariable */
-        $localVariable = true;
-    }
+		/* testLocalVariable */
+		$localVariable = true;
+	}
 
-    /* testPropertyAfterMethod */
-    private static $varV = true;
+	/* testPropertyAfterMethod */
+	private static $varV = true;
 
 }
 
 interface Base
 {
-    /* testInterfaceProperty */
-    protected $anonymous;
+	/* testInterfaceProperty */
+	protected $anonymous;
 }
 
 /* testGlobalVariable */

Spacing +3 added lines, -4 removed lines

@@ -92,7 +92,7 @@ 
 
 
     /* testMethodParam */
-    public function methodName($param) {
+    public function methodName( $param ) {
         /* testImportedGlobal */
         global $importedGlobal = true;
 
@@ -117,13 +117,12 @@ 
 /* testNotAVariable */
 return;
 
-$a = ( $foo == $bar ? new stdClass() :
-	new class() {
+$a = ( $foo == $bar ? new stdClass() : new class() {
 		/* testNestedProperty 1 */
 		public $var = true;
 
 		/* testNestedMethodParam 1 */
-		public function something($var = false) {}
+		public function something( $var = false ) {}
 	}
 );
 

Braces +2 added lines, -4 removed lines

@@ -1,7 +1,6 @@ 
 <?php
 
-class TestMemberProperties
-{
+class TestMemberProperties {
     /* testVar */
     var $varA = true;
 
@@ -105,8 +104,7 @@ 
 
 }
 
-interface Base
-{
+interface Base {
     /* testInterfaceProperty */
     protected $anonymous;
 }