kenjis / ci-phpunit-test

application/tests/_ci_phpunit_test/CIPHPUnitTestRequest.php

Doc Comments +3 added lines

@@ -250,6 +250,9 @@
 		return $this->createAndCallController($class, $method, $params);
 	}
 
+	/**
+	 * @param string $hook
+	 */
 	protected function callHook($hook)
 	{
 		if ($this->enableHooks)

Spacing +10 added lines, -10 removed lines

@@ -116,7 +116,7 @@ 
 	public function enableHooks()
 	{
 		$this->enableHooks = true;
-		$this->hooks =& load_class('Hooks', 'core');
+		$this->hooks = & load_class('Hooks', 'core');
 	}
 
 	/**
@@ -162,7 +162,7 @@ 
 			{
 				set_status_header($e->getCode());
 			}
-			$CI =& get_instance();
+			$CI = & get_instance();
 			$CI->output->_status['redirect'] = $e->getMessage();
 		}
 		// show_404()
@@ -224,17 +224,17 @@ 
 		$this->setRawInputStream($request_params);
 
 		// 404 checking
-		if (! class_exists($class) || ! method_exists($class, $method))
+		if ( ! class_exists($class) || ! method_exists($class, $method))
 		{
 			// If 404, CodeIgniter instance is not created yet. So create it here
 			// Because we need CI->output->_status to store info
-			$CI =& get_instance();
+			$CI = & get_instance();
 			if ($CI instanceof CIPHPUnitTestNullCodeIgniter)
 			{
 				CIPHPUnitTest::createCodeIgniterInstance();
 			}
 
-			show_404($class.'::'.$method . '() is not found');
+			show_404($class.'::'.$method.'() is not found');
 		}
 
 		$params = $argv;
@@ -296,7 +296,7 @@ 
 	{
 		if (is_string($string))
 		{
-			$INPUT =& load_class('Input', 'core');
+			$INPUT = & load_class('Input', 'core');
 			CIPHPUnitTestReflection::setPrivateProperty(
 				$INPUT,
 				'_raw_input_stream',
@@ -326,12 +326,12 @@ 
 			new CI();
 		}
 		$controller = new $class;
-		$CI =& get_instance();
+		$CI = & get_instance();
 
 		// Set CodeIgniter instance to TestCase
 		$this->testCase->setCI($CI);
 
-		if (! isset($CI->output->_status))
+		if ( ! isset($CI->output->_status))
 		{
 			// Prevent overwriting, if already set in the $class::__construct()
 			// Set default response code 200
@@ -377,8 +377,8 @@ 
 	 */
 	public function getStatus()
 	{
-		$CI =& get_instance();
-		if (! isset($CI->output->_status))
+		$CI = & get_instance();
+		if ( ! isset($CI->output->_status))
 		{
 			throw new LogicException('Status code is not set. You must call $this->request() first');
 		}

Indentation +3 added lines, -3 removed lines

@@ -10,9 +10,9 @@
 
 class CIPHPUnitTestRequest
 {
-    /**
-     * @var TestCase
-     */
+	/**
+	 * @var TestCase
+	 */
 	protected $testCase;
 
 	/**

Braces +2 added lines, -4 removed lines

@@ -154,8 +154,7 @@ 
 				);
 				$this->testCase->disableStrictErrorCheck();
 				return $ret;
-			}
-			else
+			} else
 			{
 				$ret = $this->requestUri($http_method, $argv, $params);
 				$this->testCase->disableStrictErrorCheck();
@@ -169,8 +168,7 @@ 
 			if ($e->getCode() === 0)
 			{
 				set_status_header(200);
-			}
-			else
+			} else
 			{
 				set_status_header($e->getCode());
 			}

application/tests/_ci_phpunit_test/patcher/Cache.php

Doc Comments +9 added lines

@@ -63,6 +63,9 @@ 
 		return self::$cache_dir;
 	}
 
+	/**
+	 * @param string $path
+	 */
 	public static function getSrcCacheFilePath($path)
 	{
 		$len = strlen(self::$project_root);
@@ -158,6 +161,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $filename
+	 */
 	protected static function writeTmpConfFile($filename, array $list)
 	{
 		$contents = implode("\n", $list);
@@ -194,6 +200,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $filename
+	 */
 	protected static function getTmpConfFile($filename)
 	{
 		if (is_readable(self::$$filename))

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace Kenjis\MonkeyPatch;
 
 use RuntimeException;
-
 use RecursiveIteratorIterator;
 use RecursiveDirectoryIterator;
 

Spacing +15 added lines, -15 removed lines

@@ -45,17 +45,17 @@ 
 			throw new LogicException("No such directory: $dir");
 		}
 		
-		self::$src_cache_dir = self::$cache_dir . '/src';
+		self::$src_cache_dir = self::$cache_dir.'/src';
 		self::$tmp_function_whitelist_file = 
-			self::$cache_dir . '/conf/func_whiltelist.php';
+			self::$cache_dir.'/conf/func_whiltelist.php';
 		self::$tmp_function_blacklist_file = 
-			self::$cache_dir . '/conf/func_blacklist.php';
+			self::$cache_dir.'/conf/func_blacklist.php';
 		self::$tmp_patcher_list_file = 
-			self::$cache_dir . '/conf/patcher_list.php';
+			self::$cache_dir.'/conf/patcher_list.php';
 		self::$tmp_include_paths_file = 
-			self::$cache_dir . '/conf/include_paths.php';
+			self::$cache_dir.'/conf/include_paths.php';
 		self::$tmp_exclude_paths_file = 
-			self::$cache_dir . '/conf/exclude_paths.php';
+			self::$cache_dir.'/conf/exclude_paths.php';
 	}
 
 	public static function getCacheDir()
@@ -73,16 +73,16 @@ 
 			return false;
 		}
 
-		return self::$src_cache_dir . '/' . $relative_path;
+		return self::$src_cache_dir.'/'.$relative_path;
 	}
 
 	protected static function createDir($dir)
 	{
-		if (! is_dir($dir))
+		if ( ! is_dir($dir))
 		{
-			if (! @mkdir($dir, 0777, true))
+			if ( ! @mkdir($dir, 0777, true))
 			{
-				throw new RuntimeException('Failed to create folder: ' . $dir);
+				throw new RuntimeException('Failed to create folder: '.$dir);
 			}
 		}
 	}
@@ -154,7 +154,7 @@ 
 	public static function appendTmpFunctionBlacklist($function)
 	{
 		file_put_contents(
-			self::getTmpFunctionBlacklistFile(), $function . "\n", FILE_APPEND
+			self::getTmpFunctionBlacklistFile(), $function."\n", FILE_APPEND
 		);
 	}
 
@@ -234,20 +234,20 @@ 
 	{
 		$cache = self::getSrcCacheFilePath($orig_file);
 		@unlink($cache);
-		MonkeyPatchManager::log('remove_src_cache: ' . $cache);
+		MonkeyPatchManager::log('remove_src_cache: '.$cache);
 		return $cache;
 	}
 
 	public static function clearSrcCache()
 	{
 		self::recursiveUnlink(self::$src_cache_dir);
-		MonkeyPatchManager::log('clear_src_cache: cleared ' . self::$src_cache_dir);
+		MonkeyPatchManager::log('clear_src_cache: cleared '.self::$src_cache_dir);
 	}
 
 	public static function clearCache()
 	{
 		self::recursiveUnlink(self::$cache_dir);
-		MonkeyPatchManager::log('clear_cache: cleared ' . self::$cache_dir);
+		MonkeyPatchManager::log('clear_cache: cleared '.self::$cache_dir);
 	}
 
 	/**
@@ -257,7 +257,7 @@ 
 	*/
 	protected static function recursiveUnlink($dir)
 	{
-		if (! is_dir($dir))
+		if ( ! is_dir($dir))
 		{
 			return;
 		}

Indentation +4 added lines, -4 removed lines

@@ -251,10 +251,10 @@
 	}
 
 	/**
-	* Recursive Unlink
-	*
-	* @param string $dir
-	*/
+	 * Recursive Unlink
+	 *
+	 * @param string $dir
+	 */
 	protected static function recursiveUnlink($dir)
 	{
 		if (! is_dir($dir))

application/tests/_ci_phpunit_test/patcher/MonkeyPatch.php

Doc Comments +4 added lines, -1 removed lines

@@ -21,7 +21,7 @@ 
 	 * 
 	 * @param string $function     function name
 	 * @param mixed  $return_value return value
-	 * @param string $class_name   class::method to apply this patch
+	 * @param string $class_method   class::method to apply this patch
 	 */
 	public static function patchFunction($function, $return_value, $class_method = null)
 	{
@@ -55,6 +55,9 @@ 
 		PatchManager::clear();
 	}
 
+	/**
+	 * @param string $class_method
+	 */
 	protected static function getClassname($class_method)
 	{
 		if (strpos($class_method, '::') === false)

Braces +1 added lines, -2 removed lines

@@ -60,8 +60,7 @@
 		if (strpos($class_method, '::') === false)
 		{
 			return 'Kenjis\MonkeyPatch\Patcher\FunctionPatcher\Proxy';
-		}
-		else
+		} else
 		{
 			return 'Kenjis\MonkeyPatch\Patcher\MethodPatcher\PatchManager';
 		}

Indentation +6 added lines, -6 removed lines

@@ -37,12 +37,12 @@
 	}
 
 	/**
-     * Patch on constant
-     * 
-     * @param type $constant
-     * @param type $value
-     * @param type $class_method
-     */
+	 * Patch on constant
+	 * 
+	 * @param type $constant
+	 * @param type $value
+	 * @param type $class_method
+	 */
 	public static function patchConstant($constant, $value, $class_method = null)
 	{
 		ConstProxy::patch($constant, $value, $class_method);

application/tests/_ci_phpunit_test/replacing/core/Common.php

Doc Comments +8 added lines

@@ -136,6 +136,11 @@ 
 	return $_is_loaded;
 }
 
+/**
+ * @param boolean $return
+ *
+ * @return boolean
+ */
 function is_cli($return = null)
 {
 	static $_return = TRUE;
@@ -148,6 +153,9 @@ 
 	return $_return;
 }
 
+/**
+ * @param string $message
+ */
 function show_error($message, $status_code = 500, $heading = 'An Error Was Encountered')
 {
 	$status_code = abs($status_code);

Spacing +2 added lines, -2 removed lines

@@ -263,7 +263,7 @@ 
 
 	// Save status code in Output object
 	// added by ci-phpunit-test
-	$CI =& get_instance();
+	$CI = & get_instance();
 	$output = $CI->output;
 	$output->_status = [
 		'code'     => $code,
@@ -376,7 +376,7 @@ 
 	if (empty($_config))
 	{
 		// references cannot be directly assigned to static variables, so we use an array
-		$_config[0] =& get_config();
+		$_config[0] = & get_config();
 	}
 
 	return isset($_config[0][$item]) ? $_config[0][$item] : NULL;

Braces +3 added lines, -6 removed lines

@@ -160,8 +160,7 @@ 
 		$exit_status = $status_code + 9; // 9 is EXIT__AUTO_MIN
 
 		$status_code = 500;
-	}
-	else
+	} else
 	{
 		$exit_status = 1; // EXIT_ERROR
 	}
@@ -256,8 +255,7 @@ 
 		if (isset($stati[$code]))
 		{
 			$text = $stati[$code];
-		}
-		else
+		} else
 		{
 			show_error('No status text available. Please check your status code number or supply your own message text.', 500);
 		}
@@ -327,8 +325,7 @@ 
 		if (file_exists($file_path = APPPATH.'config/'.ENVIRONMENT.'/config.php'))
 		{
 			require($file_path);
-		}
-		elseif ( ! $found)
+		} elseif ( ! $found)
 		{
 			set_status_header(503);
 			echo 'The configuration file does not exist.';

application/tests/Bootstrap.php

Switch Indentation +22 added lines, -22 removed lines

@@ -69,28 +69,28 @@
  */
 switch (ENVIRONMENT)
 {
-	case 'testing':
-	case 'development':
-		error_reporting(-1);
-		ini_set('display_errors', 1);
-	break;
-
-	case 'production':
-		ini_set('display_errors', 0);
-		if (version_compare(PHP_VERSION, '5.3', '>='))
-		{
-			error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
-		}
-		else
-		{
-			error_reporting(E_ALL & ~E_NOTICE & ~E_STRICT & ~E_USER_NOTICE);
-		}
-	break;
-
-	default:
-		header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
-		echo 'The application environment is not set correctly.';
-		exit(1); // EXIT_ERROR
+		case 'testing':
+		case 'development':
+			error_reporting(-1);
+			ini_set('display_errors', 1);
+		break;
+
+		case 'production':
+			ini_set('display_errors', 0);
+			if (version_compare(PHP_VERSION, '5.3', '>='))
+			{
+				error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
+			}
+			else
+			{
+				error_reporting(E_ALL & ~E_NOTICE & ~E_STRICT & ~E_USER_NOTICE);
+			}
+		break;
+
+		default:
+			header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
+			echo 'The application environment is not set correctly.';
+			exit(1); // EXIT_ERROR
 }
 
 /*

Braces +9 added lines, -18 removed lines

@@ -80,8 +80,7 @@ 
 		if (version_compare(PHP_VERSION, '5.3', '>='))
 		{
 			error_reporting(E_ALL & ~E_NOTICE & ~E_DEPRECATED & ~E_STRICT & ~E_USER_NOTICE & ~E_USER_DEPRECATED);
-		}
-		else
+		} else
 		{
 			error_reporting(E_ALL & ~E_NOTICE & ~E_STRICT & ~E_USER_NOTICE);
 		}
@@ -204,8 +203,7 @@ 
 	if (($_temp = realpath($system_path)) !== FALSE)
 	{
 		$system_path = $_temp.DIRECTORY_SEPARATOR;
-	}
-	else
+	} else
 	{
 		// Ensure there's a trailing slash
 		$system_path = strtr(
@@ -247,8 +245,7 @@ 
 		if (($_temp = realpath($application_folder)) !== FALSE)
 		{
 			$application_folder = $_temp;
-		}
-		else
+		} else
 		{
 			$application_folder = strtr(
 				rtrim($application_folder, '/\\'),
@@ -256,16 +253,14 @@ 
 				DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 			);
 		}
-	}
-	elseif (is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR))
+	} elseif (is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR))
 	{
 		$application_folder = BASEPATH.strtr(
 			trim($application_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
-	}
-	else
+	} else
 	{
 		header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 		echo 'Your application folder path does not appear to be set correctly. Please open the following file and correct this: '.SELF;
@@ -278,14 +273,12 @@ 
 	if ( ! isset($view_folder[0]) && is_dir(APPPATH.'views'.DIRECTORY_SEPARATOR))
 	{
 		$view_folder = APPPATH.'views';
-	}
-	elseif (is_dir($view_folder))
+	} elseif (is_dir($view_folder))
 	{
 		if (($_temp = realpath($view_folder)) !== FALSE)
 		{
 			$view_folder = $_temp;
-		}
-		else
+		} else
 		{
 			$view_folder = strtr(
 				rtrim($view_folder, '/\\'),
@@ -293,16 +286,14 @@ 
 				DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 			);
 		}
-	}
-	elseif (is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR))
+	} elseif (is_dir(APPPATH.$view_folder.DIRECTORY_SEPARATOR))
 	{
 		$view_folder = APPPATH.strtr(
 			trim($view_folder, '/\\'),
 			'/\\',
 			DIRECTORY_SEPARATOR.DIRECTORY_SEPARATOR
 		);
-	}
-	else
+	} else
 	{
 		header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
 		echo 'Your view folder path does not appear to be set correctly. Please open the following file and correct this: '.SELF;

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
  * NOTE: If you change these, also change the error_reporting() code below
  */
 // This `if` statemant is needed for @runInSeparateProcess
-if (! defined('ENVIRONMENT'))
+if ( ! defined('ENVIRONMENT'))
 {
 	define('ENVIRONMENT', 'testing');
 }
@@ -233,7 +233,7 @@ 
 	define('SELF', pathinfo(__FILE__, PATHINFO_BASENAME));
 
 	// Path to the test directory containing all the test files.
-	define('TESTPATH', __dir__.DIRECTORY_SEPARATOR);  // Should be the folder this `Bootstrap.php` file is in.
+	define('TESTPATH', __dir__.DIRECTORY_SEPARATOR); // Should be the folder this `Bootstrap.php` file is in.
 
 	// Path to the system directory
 	define('BASEPATH', $system_path);
@@ -366,7 +366,7 @@ 
 define('TESTPATH', APPPATH.'tests'.DIRECTORY_SEPARATOR);
 */
 
-require __DIR__ . '/_ci_phpunit_test/CIPHPUnitTest.php';
+require __DIR__.'/_ci_phpunit_test/CIPHPUnitTest.php';
 
 CIPHPUnitTest::init();
 // Or you can set directories for autoloading

application/tests/_ci_phpunit_test/CIPHPUnitTestAutoloader.php

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
 	{
 		if (in_array($class, $this->alias))
 		{
-			$dir = __DIR__ . '/alias';
+			$dir = __DIR__.'/alias';
 			$this->loadClassFile($dir, $class);
 		}
 	}
@@ -88,7 +88,7 @@ 
 		}
 		else
 		{
-			$dir = __DIR__ . '/exceptions';
+			$dir = __DIR__.'/exceptions';
 			$this->loadClassFile($dir, $class);
 		}
 	}
@@ -129,7 +129,7 @@ 
 
 	protected function loadClassFile($dir, $class)
 	{
-		$class_file = $dir . '/' . $class . '.php';
+		$class_file = $dir.'/'.$class.'.php';
 		if (file_exists($class_file))
 		{
 			require $class_file;

Braces +3 added lines, -6 removed lines

@@ -44,8 +44,7 @@ 
 				APPPATH.'modules',
 				APPPATH.'hooks',
 			];
-		}
-		else
+		} else
 		{
 			$this->dirs = $dirs;
 		}
@@ -86,8 +85,7 @@ 
 		{
 			$dir = __DIR__;
 			$this->loadClassFile($dir, $class);
-		}
-		else
+		} else
 		{
 			$dir = __DIR__ . '/exceptions';
 			$this->loadClassFile($dir, $class);
@@ -152,8 +150,7 @@ 
 			{
 				require $filename;
 				return true;
-			}
-			else
+			} else
 			{
 				unset($this->cache[$class]);
 			}

application/tests/_ci_phpunit_test/CIPHPUnitTestFileCache.php

Spacing +3 added lines, -3 removed lines

@@ -24,18 +24,18 @@
 		}
 
 		$dir = dirname($this->file);
-		if (! is_dir($dir))
+		if ( ! is_dir($dir))
 		{
 			if (@mkdir($dir, 0777, true) === false)
 			{
-				throw new RuntimeException('Failed to create folder: ' . $dir);
+				throw new RuntimeException('Failed to create folder: '.$dir);
 			}
 		}
 
 		if (file_put_contents($this->file, '') === false)
 		{
 			throw new RuntimeException(
-				'Failed to write to cache file: ' . $this->file
+				'Failed to write to cache file: '.$this->file
 			);
 		}
 	}

Braces +1 added lines, -2 removed lines

@@ -67,8 +67,7 @@
 		if ($this->offsetExists($key))
 		{
 			return $this->map[$key];
-		}
-		else
+		} else
 		{
 			return null;
 		}

application/tests/_ci_phpunit_test/CIPHPUnitTestReflection.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 		$ref_method->setAccessible(true);
 		$obj = (gettype($obj) === 'object') ? $obj : null;
 
-		return function () use ($obj, $ref_method) {
+		return function() use ($obj, $ref_method) {
 			$args = func_get_args();
 			return $ref_method->invokeArgs($obj, $args);
 		};

application/tests/_ci_phpunit_test/CIPHPUnitTestRouter.php

Braces +2 added lines, -4 removed lines

@@ -29,16 +29,14 @@
 		if (empty($class) OR ! file_exists(APPPATH.'controllers/'.$RTR->directory.$class.'.php'))
 		{
 			$e404 = TRUE;
-		}
-		else
+		} else
 		{
 			require_once(APPPATH.'controllers/'.$RTR->directory.$class.'.php');
 
 			if ( ! class_exists($class, FALSE) OR $method[0] === '_' OR method_exists('CI_Controller', $method))
 			{
 				$e404 = TRUE;
-			}
-			elseif (method_exists($class, '_remap'))
+			} elseif (method_exists($class, '_remap'))
 			{
 				$params = array($method, array_slice($URI->rsegments, 2));
 				$method = '_remap';

Spacing +3 added lines, -3 removed lines

@@ -19,8 +19,8 @@ 
 	 */
 	public function getRoute()
 	{
-		$RTR =& load_class('Router', 'core');
-		$URI =& load_class('URI', 'core');
+		$RTR = & load_class('Router', 'core');
+		$URI = & load_class('URI', 'core');
 
 		$e404 = FALSE;
 		$class = ucfirst($RTR->class);
@@ -57,7 +57,7 @@ 
 		{
 			// If 404, CodeIgniter instance is not created yet. So create it here.
 			// Because we need CI->output->_status
-			$CI =& get_instance();
+			$CI = & get_instance();
 			if ($CI instanceof CIPHPUnitTestNullCodeIgniter)
 			{
 				CIPHPUnitTest::createCodeIgniterInstance();