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)

Braces +2 added lines, -4 removed lines

@@ -129,8 +129,7 @@ 
 				return $this->callControllerMethod(
 					$http_method, $argv, $params
 				);
-			}
-			else
+			} else
 			{
 				return $this->requestUri($http_method, $argv, $params);
 			}
@@ -141,8 +140,7 @@ 
 			if ($e->getCode() === 0)
 			{
 				set_status_header(200);
-			}
-			else
+			} else
 			{
 				set_status_header($e->getCode());
 			}

Spacing +9 added lines, -9 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,7 +326,7 @@ 
 			new CI();
 		}
 		$controller = new $class;
-		$CI =& get_instance();
+		$CI = & get_instance();
 
 		// Set CodeIgniter instance to TestCase
 		$this->testCase->setCI($CI);
@@ -372,8 +372,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');
 		}

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 +9 added lines, -10 removed lines

@@ -158,14 +158,16 @@ 
 	if ($status_code < 100)
 	{
 		$exit_status = $status_code + 9; // 9 is EXIT__AUTO_MIN
-		if ($exit_status > 125) // 125 is EXIT__AUTO_MAX
+		if ($exit_status > 125) {
+			// 125 is EXIT__AUTO_MAX
 		{
-			$exit_status = 1; // EXIT_ERROR
+			$exit_status = 1;
+		}
+		// EXIT_ERROR
 		}
 
 		$status_code = 500;
-	}
-	else
+	} else
 	{
 		$exit_status = 1; // EXIT_ERROR
 	}
@@ -254,8 +256,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);
 		}
@@ -281,8 +282,7 @@ 
 	if (strpos(PHP_SAPI, 'cgi') === 0)
 	{
 		header('Status: '.$code.' '.$text, TRUE);
-	}
-	else
+	} else
 	{
 		$server_protocol = isset($_SERVER['SERVER_PROTOCOL']) ? $_SERVER['SERVER_PROTOCOL'] : 'HTTP/1.1';
 		header($server_protocol.' '.$code.' '.$text, TRUE, $code);
@@ -325,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/_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/CIPHPUnitTestDouble.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
 				break;
 			default:
 				throw new RuntimeException(
-					'Sorry, ' . $count . ' params not implemented yet'
+					'Sorry, '.$count.' params not implemented yet'
 				);
 		}
 	}

Braces +1 added lines, -2 removed lines

@@ -48,8 +48,7 @@
 		if ($constructor_params === false)
 		{
 			$mock->disableOriginalConstructor();
-		}
-		elseif (is_array($constructor_params))
+		} elseif (is_array($constructor_params))
 		{
 			$mock->setConstructorArgs($constructor_params);
 		}

Doc Comments +5 added lines, -1 removed lines

@@ -32,7 +32,7 @@ 
 	 *
 	 * @param  string $classname
 	 * @param  array  $params             [method_name => return_value]
-	 * @param  mixed  $constructor_params false: disable construntor, array: construntor params
+	 * @param  boolean  $constructor_params false: disable construntor, array: construntor params
 	 * 
 	 * @return object PHPUnit mock object
 	 */
@@ -72,6 +72,10 @@ 
 		return $mock;
 	}
 
+	/**
+	 * @param string $method
+	 * @param string $with
+	 */
 	protected function _verify($mock, $method, $params = null, $expects, $with)
 	{
 		$invocation = $mock->expects($expects)

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();