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 +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))

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;
 		}

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)

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;

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;

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'
 				);
 		}
 	}

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

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/autoloader.php

Spacing +4 added lines, -4 removed lines

@@ -9,10 +9,10 @@ 
  */
 
 // Autoloader for ci-phpunit-test
-require __DIR__ . '/CIPHPUnitTestAutoloader.php';
-require __DIR__ . '/CIPHPUnitTestFileCache.php';
+require __DIR__.'/CIPHPUnitTestAutoloader.php';
+require __DIR__.'/CIPHPUnitTestFileCache.php';
 $cache = new CIPHPUnitTestFileCache(
-	__DIR__ . '/tmp/cache/autoload.php'
+	__DIR__.'/tmp/cache/autoload.php'
 );
 $autoload_dirs = CIPHPUnitTest::getAutoloadDirs();
 $autoloader = new CIPHPUnitTestAutoloader($cache, $autoload_dirs);
@@ -20,5 +20,5 @@ 
 
 // Register CodeIgniter's tests/mocks/autoloader.php
 define('SYSTEM_PATH', BASEPATH);
-require APPPATH .'tests/mocks/autoloader.php';
+require APPPATH.'tests/mocks/autoloader.php';
 spl_autoload_register('autoload');