Test: Improve FileHandler coverage

Author: jim-parry

tests/system/Cache/Handlers/FileHandlerTest.php

@@ -1,18 +1,17 @@
 <?php
 namespace CodeIgniter\Cache\Handlers;
 
-set_error_handler(function (int $errno, string $errstr, string $errfile, int $errline, array $errcontext)
-{
+set_error_handler(function (int $errno, string $errstr, string $errfile, int $errline, array $errcontext) {
 	//throw new \ErrorException($errstr, $errno, 0, $errfile, $errline);
 });
 
 class FileHandlerTest extends \CIUnitTestCase
 {
 
 	private static $directory = 'FileHandler';
-	private static $key1 = 'key1';
-	private static $key2 = 'key2';
-	private static $key3 = 'key3';
+	private static $key1      = 'key1';
+	private static $key2      = 'key2';
+	private static $key3      = 'key3';
 
 	private static function getKeyArray()
 	{
@@ -32,10 +31,10 @@ protected function setUp()
 		parent::setUp();
 
 		//Initialize path
-		$this->config = new \Config\Cache();
+		$this->config             = new \Config\Cache();
 		$this->config->storePath .= self::$directory;
 
-		if ( ! is_dir($this->config->storePath))
+		if (! is_dir($this->config->storePath))
 		{
 			mkdir($this->config->storePath, 0777, true);
 		}
@@ -80,7 +79,7 @@ public function testNewWithNonWritablePath()
 	public function testSetDefaultPath()
 	{
 		//Initialize path
-		$config = new \Config\Cache();
+		$config            = new \Config\Cache();
 		$config->storePath = null;
 
 		$this->fileHandler = new FileHandler($config);
@@ -131,8 +130,8 @@ public function testDecrement()
 		$this->fileHandler->save(self::$key1, 10);
 		$this->fileHandler->save(self::$key2, 'value');
 
-//	Line following commented out to force the cache to add a zero entry for key3		
-//		$this->fileHandler->save(self::$key3, 0);
+		//  Line following commented out to force the cache to add a zero entry for key3
+		//      $this->fileHandler->save(self::$key3, 0);
 
 		$this->assertSame(9, $this->fileHandler->decrement(self::$key1, 1));
 		$this->assertFalse($this->fileHandler->decrement(self::$key2, 1));
@@ -170,7 +169,7 @@ public function testGetCacheInfo()
 
 		$actual = $this->fileHandler->getCacheInfo();
 		$this->assertArrayHasKey(self::$key1, $actual);
-		$this->assertEquals(self::$key1,$actual[self::$key1]['name']);
+		$this->assertEquals(self::$key1, $actual[self::$key1]['name']);
 		$this->assertArrayHasKey('server_path', $actual[self::$key1]);
 	}
 
@@ -206,7 +205,7 @@ final class BaseTestFileHandler extends FileHandler
 
 	public function __construct()
 	{
-		$this->config = new \Config\Cache();
+		$this->config             = new \Config\Cache();
 		$this->config->storePath .= self::$directory;
 
 		parent::__construct($this->config);
@@ -218,14 +217,14 @@ public function getFileInfoTest()
 		stream_get_meta_data($tmp_handle)['uri'];
 
 		return $this->getFileInfo(stream_get_meta_data($tmp_handle)['uri'], [
-					'name',
-					'server_path',
-					'size',
-					'date',
-					'readable',
-					'writable',
-					'executable',
-					'fileperms',
+			'name',
+			'server_path',
+			'size',
+			'date',
+			'readable',
+			'writable',
+			'executable',
+			'fileperms',
 		]);
 	}