CodeSniffer Corrections

Signed-off-by: Justin Yost <justin@loadsys.com>

Author: justinyost

src/Shell/ConfigReadShell.php

@@ -393,13 +393,13 @@ public function getOptionParser() {
 				'short' => 'b',
 				'boolean' => true,
 				'default' => false,
-				'help' => __('Always use bash variable deinfition formatting. When enabled, output will be formatted as `KEY_NAME=\'value\'`. This option is auto-enabled if multiple keys are provided on the command line, or if the value for the requested key is itself an array. When multiple values are returned, each will be output on its own line.')
+				'help' => __('Always use bash variable deinfition formatting. When enabled, output will be formatted as `KEY_NAME=\'value\'`. This option is auto-enabled if multiple keys are provided on the command line, or if the value for the requested key is itself an array. When multiple values are returned, each will be output on its own line.'),
 			])
 			->addOption('serialize', [
 				'short' => 's',
 				'boolean' => true,
 				'default' => false,
-				'help' => __('Encode all output using PHP\'s `serialize()` method. Makes the Shell\'s output suitable for consumption by other PHP console scripts. Always overrides the --bash option. A single requested key will be serialized directly. Multiple requested keys will be combined into an associative array with the provided arguments as key names and then serialized.')
+				'help' => __('Encode all output using PHP\'s `serialize()` method. Makes the Shell\'s output suitable for consumption by other PHP console scripts. Always overrides the --bash option. A single requested key will be serialized directly. Multiple requested keys will be combined into an associative array with the provided arguments as key names and then serialized.'),
 			])
 			->description(
 				__('Provides CLI access to variables defined in the Configure class of the host CakePHP application. Will output the value of any keys passed as arguments. Equivelant to `Configure::read(\'Key.Name\')`. Unrecognized keys will produce empty string or `null` output.')

tests/TestCase/Shell/ConfigReadShellTest.php

@@ -8,8 +8,11 @@
 use Cake\Console\ConsoleOptionParser;
 use Cake\Console\Shell;
 use Cake\Core\Configure;
+use Cake\Datasource\ConnectionManager;
 use Cake\TestSuite\TestCase;
+use Cake\Utility\Security;
 use ConfigRead\Shell\ConfigReadShell;
+use \StdClass;
 
 /**
  * TestConfigReadShell class
@@ -43,23 +46,20 @@ public function printVal($key, $val) {
  *
  */
 class ConfigReadShellTest extends TestCase {
-
 	/**
 	 * Fixtures used in this test case
 	 *
 	 * @var array
 	 */
-	public $fixtures = [
-	];
+	public $fixtures = [];
 
 	/**
 	 * Acts as an accumulator for output produced by the Shell.
 	 *
 	 * @var array
 	 * @see ::initSUIT()
 	 */
-	public $output = [
-	];
+	public $output = [];
 
 	public static $datasources = [
 		'sample1' => [
@@ -88,10 +88,10 @@ class ConfigReadShellTest extends TestCase {
 	 */
 	public static function setUpBeforeClass() {
 		// Prime the ConnectionManager with some configs.
-		\Cake\Datasource\ConnectionManager::config(self::$datasources);
+		ConnectionManager::config(self::$datasources);
 
 		// Prime the security salt.
-		\Cake\Utility\Security::salt(self::$salt);
+		Security::salt(self::$salt);
 	}
 
 	/**
@@ -101,10 +101,10 @@ public static function setUpBeforeClass() {
 	 */
 	public static function tearDownAfterClass() {
 		foreach (self::$datasources as $ds => $configs) {
-			\Cake\Datasource\ConnectionManager::drop($ds);
+			ConnectionManager::drop($ds);
 		}
 
-		\Cake\Utility\Security::salt('');
+		Security::salt('');
 	}
 
 	/**
@@ -430,7 +430,7 @@ public function testMainSerializedIntegration($args, $expected, $msg = '') {
 				'one' => 1,
 				'two' => '2',
 				3 => 'three',
-				'stdClass' => (new \StdClass()),
+				'stdClass' => (new StdClass()),
 			],
 		];