PHPCSStandards / PHP_CodeSniffer / 11206909767

78%

master: 79%

Build Type

push

github

Committed by jrfnl

Commit Message

Runner::processFile()/error handling: add more defensive coding

Follow up to squizlabs/PHP_CodeSniffer 3716 and PR 524.

PR squizlabs/PHP_CodeSniffer 3716 improved the information provided in the error messages for `Internal.Exception`s with additional information about the source of the problem.
This code uses the `Common::getSniffCode()` method to transform a sniff class name into a sniff code.

PR 524 changes how the `Common::getSniffCode()` method handles invalid input, i.e. it will now throw an exception when a class is passed which isn't a sniff (or sniff test).

In the context of this error handling, however, this new exception can easily be encountered when an `Abstract...Sniff` class contains the code which triggered the error - which would trigger the exception as it is not a sniff class.

This commit now adds some additional defensive coding to ensure that a) the error message will still be informative, while b) making sure that any exception which may be thrown by the  `Common::getSniffCode()` method will be caught and and won't cause an uncaught `InvalidArgumentException` in the error handling code to stop execution of the script.

Includes a test documenting the behaviour of the `Common::getSniffCode()` method for abstract sniff classes.

Run Details

0 of 8 new or added lines in 1 file covered. (0.0%)

1 existing line in 1 file now uncovered.

18039 of 24403 relevant lines covered (73.92%)

71.06 hits per line