kenjis / ci-phpunit-test

application/tests/_ci_phpunit_test/patcher/2.x/Patcher/ConstantPatcher/NodeVisitor.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PhpParser\NodeVisitorAbstract;
-
 use Kenjis\MonkeyPatch\Patcher\ConstantPatcher;
 
 class NodeVisitor extends NodeVisitorAbstract

Spacing +8 added lines, -8 removed lines

@@ -24,7 +24,7 @@ 
 
 	public function enterNode(Node $node)
 	{
-		$callback = [$this, 'before' . ucfirst($node->getType())];
+		$callback = [$this, 'before'.ucfirst($node->getType())];
 		if (is_callable($callback)) {
 			call_user_func_array($callback, [$node]);
 		}
@@ -32,9 +32,9 @@ 
 
 	public function leaveNode(Node $node)
 	{
-		if (! ($node instanceof ConstFetch))
+		if ( ! ($node instanceof ConstFetch))
 		{
-			$callback = [$this, 'rewrite' . ucfirst($node->getType())];
+			$callback = [$this, 'rewrite'.ucfirst($node->getType())];
 			if (is_callable($callback)) {
 				call_user_func_array($callback, [$node]);
 			}
@@ -47,26 +47,26 @@ 
 			return;
 		}
 
-		if (! ($node->name instanceof Name))
+		if ( ! ($node->name instanceof Name))
 		{
 			return;
 		}
 
-		if (! $node->name->isUnqualified())
+		if ( ! $node->name->isUnqualified())
 		{
 			return;
 		}
 
-		if (! ConstantPatcher::isBlacklisted((string) $node->name))
+		if ( ! ConstantPatcher::isBlacklisted((string) $node->name))
 		{
 			$replacement = new FullyQualified(array());
 			$replacement->set(
-				'\__ConstProxy__::get(\'' . (string) $node->name . '\')'
+				'\__ConstProxy__::get(\''.(string) $node->name.'\')'
 			);
 
 			$pos = $node->getAttribute('startTokenPos');
 			ConstantPatcher::$replacement[$pos] = 
-				'\__ConstProxy__::get(\'' . (string) $node->name .'\')';
+				'\__ConstProxy__::get(\''.(string) $node->name.'\')';
 
 			$node->name = $replacement;
 		}

application/tests/_ci_phpunit_test/patcher/2.x/Patcher/FunctionPatcher/NodeVisitor.php

Unused Use Statements -4 removed lines

@@ -21,15 +21,11 @@
 
 namespace Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 
-use ReflectionFunction;
-use ReflectionException;
-
 use PhpParser\Node;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PhpParser\NodeVisitorAbstract;
-
 use Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 
 class NodeVisitor extends NodeVisitorAbstract

Spacing +5 added lines, -5 removed lines

@@ -36,17 +36,17 @@ 
 {
 	public function leaveNode(Node $node)
 	{
-		if (! ($node instanceof FuncCall))
+		if ( ! ($node instanceof FuncCall))
 		{
 			return;
 		}
 
-		if (! ($node->name instanceof Name))
+		if ( ! ($node->name instanceof Name))
 		{
 			return;
 		}
 
-		if (! $node->name->isUnqualified())
+		if ( ! $node->name->isUnqualified())
 		{
 			return;
 		}
@@ -57,12 +57,12 @@ 
 		) {
 			$replacement = new FullyQualified(array());
 			$replacement->set(
-				'\__FuncProxy__::' . (string) $node->name
+				'\__FuncProxy__::'.(string) $node->name
 			);
 
 			$pos = $node->getAttribute('startTokenPos');
 			FunctionPatcher::$replacement[$pos] = 
-				'\__FuncProxy__::' . (string) $node->name;
+				'\__FuncProxy__::'.(string) $node->name;
 
 			$node->name = $replacement;
 		}

application/tests/_ci_phpunit_test/patcher/2.x/Patcher/FunctionPatcher/Proxy.php

Doc Comments +10 added lines, -1 removed lines

@@ -37,7 +37,7 @@ 
 	 * 
 	 * @param string $function     function name
 	 * @param mixed  $return_value return value or callable
-	 * @param string $class_name   class::method to apply this patch
+	 * @param string $class_method   class::method to apply this patch
 	 * 
 	 * @throws LogicException
 	 */
@@ -86,6 +86,9 @@ 
 		InvocationVerifier::verify(self::$expected_invocations, self::$invocations);
 	}
 
+	/**
+	 * @param string $function
+	 */
 	protected static function logInvocation($function, $arguments)
 	{
 		if (MonkeyPatchManager::$debug)
@@ -111,6 +114,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $function
+	 */
 	protected static function checkCalledMethod($function)
 	{
 		$trace = debug_backtrace();
@@ -184,6 +190,9 @@ 
 		return call_user_func_array($function, $arguments);
 	}
 
+	/**
+	 * @param string $function
+	 */
 	protected static function checkPassedByReference($function)
 	{
 		$ref_func = new ReflectionFunction($function);

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use LogicException;
 use ReflectionFunction;
 use ReflectionException;
-
 use Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 use Kenjis\MonkeyPatch\Patcher\Backtrace;
 use Kenjis\MonkeyPatch\MonkeyPatchManager;

Spacing +9 added lines, -9 removed lines

@@ -51,7 +51,7 @@ 
 			self::outputMessage($msg);
 			throw new LogicException($msg);
 		}
-		if (! FunctionPatcher::isWhitelisted($function))
+		if ( ! FunctionPatcher::isWhitelisted($function))
 		{
 			$msg = "<red>Can't patch on '$function'. It is not in whitelist. If you want to patch it, please add it to 'functions_to_patch' in 'tests/Bootstrap.php'. But note that there are some limitations. See <https://github.com/kenjis/ci-phpunit-test/blob/master/docs/HowToWriteTests.md#patching-functions> for details.</red>";
 			self::outputMessage($msg);
@@ -97,16 +97,16 @@ 
 			$line = $info['line'];
 			$method = isset($info['class_method']) ? $info['class_method'] : $info['function'];
 
-			$log_args = function () use ($arguments) {
+			$log_args = function() use ($arguments) {
 				$output = '';
 				foreach ($arguments as $arg) {
-					$output .= var_export($arg, true) . ', ';
+					$output .= var_export($arg, true).', ';
 				}
 				$output = rtrim($output, ', ');
 				return $output;
 			};
 			MonkeyPatchManager::log(
-				'invoke_func: ' . $function . '(' . $log_args() . ') on line ' . $line . ' in ' . $file . ' by ' . $method . '()'
+				'invoke_func: '.$function.'('.$log_args().') on line '.$line.' in '.$file.' by '.$method.'()'
 			);
 		}
 	}
@@ -148,10 +148,10 @@ 
 
 		if (isset(self::$patches_to_apply[$function]))
 		{
-			if (! self::checkCalledMethod($function))
+			if ( ! self::checkCalledMethod($function))
 			{
 				MonkeyPatchManager::log(
-					'invoke_func: ' . $function . '() not patched (out of scope)'
+					'invoke_func: '.$function.'() not patched (out of scope)'
 				);
 				self::checkPassedByReference($function);
 				return call_user_func_array($function, $arguments);
@@ -160,7 +160,7 @@ 
 
 		if (array_key_exists($function, self::$patches))
 		{
-			MonkeyPatchManager::log('invoke_func: ' . $function . '() patched');
+			MonkeyPatchManager::log('invoke_func: '.$function.'() patched');
 
 			if (is_callable(self::$patches[$function]))
 			{
@@ -178,7 +178,7 @@ 
 		}
 
 		MonkeyPatchManager::log(
-			'invoke_func: ' . $function . '() not patched (no patch)'
+			'invoke_func: '.$function.'() not patched (no patch)'
 		);
 		self::checkPassedByReference($function);
 		return call_user_func_array($function, $arguments);
@@ -231,7 +231,7 @@ 
 		$msg = str_replace(
 			['<red>', '</red>'], [$red_begin, $red_end], $msg
 		);
-		echo $msg . "\n";
+		echo $msg."\n";
 	}
 
 	/**

application/tests/_ci_phpunit_test/patcher/3.x/MonkeyPatchManager.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace Kenjis\MonkeyPatch;
 
 use LogicException;
-use RuntimeException;
 use PhpParser\ParserFactory;
 use Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 

Spacing +17 added lines, -17 removed lines

@@ -37,7 +37,7 @@ 
 
 	public static function log($message)
 	{
-		if (! self::$debug)
+		if ( ! self::$debug)
 		{
 			return;
 		}
@@ -72,7 +72,7 @@ 
 		}
 		if (self::$debug)
 		{
-			self::$log_file = __DIR__ . '/debug.log';
+			self::$log_file = __DIR__.'/debug.log';
 		}
 	}
 
@@ -85,10 +85,10 @@ 
 		else
 		{
 			// APPPATH is constant in CodeIgniter
-			Cache::setProjectRootDir(APPPATH . '../');
+			Cache::setProjectRootDir(APPPATH.'../');
 		}
 
-		if (! isset($config['cache_dir']))
+		if ( ! isset($config['cache_dir']))
 		{
 			throw new LogicException('You have to set "cache_dir"');
 		}
@@ -97,7 +97,7 @@ 
 
 	protected static function setPaths(array $config)
 	{
-		if (! isset($config['include_paths']))
+		if ( ! isset($config['include_paths']))
 		{
 			throw new LogicException('You have to set "include_paths"');
 		}
@@ -158,7 +158,7 @@ 
 		// Updated?
 		if ($cached !== $current)
 		{
-			MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__);
+			MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__);
 			Cache::clearSrcCache();
 			Cache::writeTmpIncludePaths($current);
 		}
@@ -169,7 +169,7 @@ 
 		// Updated?
 		if ($cached !== $current)
 		{
-			MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__);
+			MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__);
 			Cache::clearSrcCache();
 			Cache::writeTmpExcludePaths($current);
 		}
@@ -182,7 +182,7 @@ 
 		// Updated?
 		if ($cached !== self::$patcher_list)
 		{
-			MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__);
+			MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__);
 			Cache::clearSrcCache();
 			Cache::writeTmpPatcherList(self::$patcher_list);
 		}
@@ -196,7 +196,7 @@ 
 		// Updated?
 		if ($cached !== $current)
 		{
-			MonkeyPatchManager::log('clear_src_cache: from ' . __METHOD__);
+			MonkeyPatchManager::log('clear_src_cache: from '.__METHOD__);
 			Cache::clearSrcCache();
 			Cache::writeTmpFunctionWhitelist($current);
 		}
@@ -258,7 +258,7 @@ 
 	 */
 	public static function patch($path)
 	{
-		if (! is_readable($path))
+		if ( ! is_readable($path))
 		{
 			throw new LogicException("Can't read '$path'");
 		}
@@ -266,17 +266,17 @@ 
 		// Check cache file
 		if ($cache_file = Cache::getValidSrcCachePath($path))
 		{
-			self::log('cache_hit: ' . $path);
+			self::log('cache_hit: '.$path);
 			return fopen($cache_file, 'r');
 		}
 
-		self::log('cache_miss: ' . $path);
+		self::log('cache_miss: '.$path);
 		$source = file_get_contents($path);
 
 		list($new_source, $patched) = self::execPatchers($source);
 
 		// Write to cache file
-		self::log('write_cache: ' . $path);
+		self::log('write_cache: '.$path);
 		Cache::writeSrcCacheFile($path, $new_source);
 
 		$resource = fopen('php://memory', 'rb+');
@@ -292,12 +292,12 @@ 
 			return;
 		}
 
-		require __DIR__ . '/Patcher/AbstractPatcher.php';
-		require __DIR__ . '/Patcher/Backtrace.php';
+		require __DIR__.'/Patcher/AbstractPatcher.php';
+		require __DIR__.'/Patcher/Backtrace.php';
 
 		foreach (self::$patcher_list as $classname)
 		{
-			require __DIR__ . '/Patcher/' . $classname . '.php';
+			require __DIR__.'/Patcher/'.$classname.'.php';
 		}
 
 		self::$load_patchers = true;
@@ -308,7 +308,7 @@ 
 		$patched = false;
 		foreach (self::$patcher_list as $classname)
 		{
-			$classname = 'Kenjis\MonkeyPatch\Patcher\\' . $classname;
+			$classname = 'Kenjis\MonkeyPatch\Patcher\\'.$classname;
 			$patcher = new $classname;
 			list($source, $patched_this) = $patcher->patch($source);
 			$patched = $patched || $patched_this;

Braces +1 added lines, -2 removed lines

@@ -70,8 +70,7 @@
 		if (isset($config['root_dir']))
 		{
 			Cache::setProjectRootDir($config['root_dir']);
-		}
-		else
+		} else
 		{
 			// APPPATH is constant in CodeIgniter
 			Cache::setProjectRootDir(APPPATH . '../');

Doc Comments +6 added lines

@@ -35,6 +35,9 @@ 
 		'ConstantPatcher',
 	];
 
+	/**
+	 * @param string $message
+	 */
 	public static function log($message)
 	{
 		if (! self::$debug)
@@ -303,6 +306,9 @@ 
 		self::$load_patchers = true;
 	}
 
+	/**
+	 * @param string $source
+	 */
 	protected static function execPatchers($source)
 	{
 		$patched = false;

application/tests/_ci_phpunit_test/patcher/3.x/Patcher/Backtrace.php

Doc Comments +3 added lines

@@ -20,6 +20,9 @@
 		'ConstantPatcher' => 0,
 	];
 
+	/**
+	 * @param string $patcher
+	 */
 	public static function getInfo($patcher, $backtrace)
 	{
 		if (! isset(self::$map[$patcher]))

Spacing +6 added lines, -6 removed lines

@@ -21,7 +21,7 @@ 
 
 	public static function getInfo($patcher, $backtrace)
 	{
-		if (! isset(self::$map[$patcher]))
+		if ( ! isset(self::$map[$patcher]))
 		{
 			throw new LogicException("No such a patcher: $patcher");
 		}
@@ -41,12 +41,12 @@ 
 		$line = isset($backtrace[$offset]['line'])
 				? $backtrace[$offset]['line'] : null;
 
-		if (isset($backtrace[$offset+2]))
+		if (isset($backtrace[$offset + 2]))
 		{
-			$class  = isset($backtrace[$offset+2]['class'])
-					? $backtrace[$offset+2]['class']
+			$class = isset($backtrace[$offset + 2]['class'])
+					? $backtrace[$offset + 2]['class']
 					: null;
-			$function = $backtrace[$offset+2]['function'];
+			$function = $backtrace[$offset + 2]['function'];
 		}
 		else
 		{
@@ -57,7 +57,7 @@ 
 		if (isset($class))
 		{
 			$method = $function;
-			$class_method = $class . '::' . $function;
+			$class_method = $class.'::'.$function;
 			$function = null;
 		}
 		else

Braces +2 added lines, -4 removed lines

@@ -47,8 +47,7 @@ 
 					? $backtrace[$offset+2]['class']
 					: null;
 			$function = $backtrace[$offset+2]['function'];
-		}
-		else
+		} else
 		{
 			$class = null;
 			$function = null;
@@ -59,8 +58,7 @@ 
 			$method = $function;
 			$class_method = $class . '::' . $function;
 			$function = null;
-		}
-		else
+		} else
 		{
 			$method = null;
 			$class_method = null;

application/tests/_ci_phpunit_test/patcher/3.x/Patcher/ConstantPatcher.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 require __DIR__ . '/ConstantPatcher/Proxy.php';
 
 use LogicException;
-
 use Kenjis\MonkeyPatch\Patcher\ConstantPatcher\NodeVisitor;
 
 class ConstantPatcher extends AbstractPatcher

Spacing +2 added lines, -2 removed lines

@@ -10,8 +10,8 @@
 
 namespace Kenjis\MonkeyPatch\Patcher;
 
-require __DIR__ . '/ConstantPatcher/NodeVisitor.php';
-require __DIR__ . '/ConstantPatcher/Proxy.php';
+require __DIR__.'/ConstantPatcher/NodeVisitor.php';
+require __DIR__.'/ConstantPatcher/Proxy.php';
 
 use LogicException;
 

Braces +2 added lines, -4 removed lines

@@ -173,13 +173,11 @@
 			if (is_string($token))
 			{
 				$new_source .= $token;
-			}
-			elseif ($i == $replacement['key'])
+			} elseif ($i == $replacement['key'])
 			{
 				$new_source .= $replacement['value'];
 				$replacement = each(self::$replacement);
-			}
-			else
+			} else
 			{
 				$new_source .= $token[1];
 			}

application/tests/_ci_phpunit_test/patcher/3.x/Patcher/ConstantPatcher/Proxy.php

Doc Comments +6 added lines

@@ -51,6 +51,9 @@ 
 		self::$patches_to_apply = [];
 	}
 
+	/**
+	 * @param string $constant
+	 */
 	protected static function logInvocation($constant)
 	{
 		if (MonkeyPatchManager::$debug)
@@ -68,6 +71,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $constant
+	 */
 	protected static function checkCalledMethod($constant)
 	{
 		$trace = debug_backtrace();

Unused Use Statements -5 removed lines

@@ -13,14 +13,9 @@
 class_alias('Kenjis\MonkeyPatch\Patcher\ConstantPatcher\Proxy', '__ConstProxy__');
 
 use LogicException;
-use ReflectionConstant;
-use ReflectionException;
-
 use Kenjis\MonkeyPatch\Patcher\ConstantPatcher;
 use Kenjis\MonkeyPatch\Patcher\Backtrace;
 use Kenjis\MonkeyPatch\MonkeyPatchManager;
-use Kenjis\MonkeyPatch\Cache;
-use Kenjis\MonkeyPatch\InvocationVerifier;
 
 class Proxy
 {

Spacing +5 added lines, -5 removed lines

@@ -65,7 +65,7 @@ 
 			$method = isset($info['class_method']) ? $info['class_method'] : $info['function'];
 
 			MonkeyPatchManager::log(
-				'invoke_const: ' . $constant . ') on line ' . $line . ' in ' . $file . ' by ' . $method . '()'
+				'invoke_const: '.$constant.') on line '.$line.' in '.$file.' by '.$method.'()'
 			);
 		}
 	}
@@ -111,10 +111,10 @@ 
 
 		if (isset(self::$patches_to_apply[$constant]))
 		{
-			if (! self::checkCalledMethod($constant))
+			if ( ! self::checkCalledMethod($constant))
 			{
 				MonkeyPatchManager::log(
-					'invoke_const: ' . $constant . ' not patched (out of scope)'
+					'invoke_const: '.$constant.' not patched (out of scope)'
 				);
 				return constant($constant);
 			}
@@ -122,12 +122,12 @@ 
 
 		if (array_key_exists($constant, self::$patches))
 		{
-			MonkeyPatchManager::log('invoke_const: ' . $constant . ' patched');
+			MonkeyPatchManager::log('invoke_const: '.$constant.' patched');
 			return self::$patches[$constant];
 		}
 
 		MonkeyPatchManager::log(
-			'invoke_const: ' . $constant . ' not patched (no patch)'
+			'invoke_const: '.$constant.' not patched (no patch)'
 		);
 		return constant($constant);
 	}

application/tests/_ci_phpunit_test/patcher/3.x/Patcher/FunctionPatcher.php

Doc Comments +6 added lines

@@ -84,6 +84,9 @@ 
 		$this->node_visitor = new NodeVisitor();
 	}
 
+	/**
+	 * @param string $error_msg
+	 */
 	protected static function checkLock($error_msg)
 	{
 		if (self::$lock_function_list)
@@ -110,6 +113,9 @@ 
 		return self::$whitelist;
 	}
 
+	/**
+	 * @param string $function_name
+	 */
 	public static function addBlacklist($function_name)
 	{
 		self::checkLock("You can't add to blacklist after initialization");

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 require __DIR__ . '/FunctionPatcher/Proxy.php';
 
 use LogicException;
-
 use Kenjis\MonkeyPatch\Patcher\FunctionPatcher\NodeVisitor;
 
 class FunctionPatcher extends AbstractPatcher

Spacing +2 added lines, -2 removed lines

@@ -10,8 +10,8 @@
 
 namespace Kenjis\MonkeyPatch\Patcher;
 
-require __DIR__ . '/FunctionPatcher/NodeVisitor.php';
-require __DIR__ . '/FunctionPatcher/Proxy.php';
+require __DIR__.'/FunctionPatcher/NodeVisitor.php';
+require __DIR__.'/FunctionPatcher/Proxy.php';
 
 use LogicException;
 

Braces +2 added lines, -4 removed lines

@@ -173,13 +173,11 @@
 			if (is_string($token))
 			{
 				$new_source .= $token;
-			}
-			elseif ($i == $replacement['key'])
+			} elseif ($i == $replacement['key'])
 			{
 				$new_source .= $replacement['value'];
 				$replacement = each(self::$replacement);
-			}
-			else
+			} else
 			{
 				$new_source .= $token[1];
 			}

application/tests/_ci_phpunit_test/patcher/3.x/Patcher/FunctionPatcher/NodeVisitor.php

Unused Use Statements -4 removed lines

@@ -21,15 +21,11 @@
 
 namespace Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 
-use ReflectionFunction;
-use ReflectionException;
-
 use PhpParser\Node;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PhpParser\NodeVisitorAbstract;
-
 use Kenjis\MonkeyPatch\Patcher\FunctionPatcher;
 
 class NodeVisitor extends NodeVisitorAbstract

Spacing +5 added lines, -5 removed lines

@@ -36,17 +36,17 @@ 
 {
 	public function leaveNode(Node $node)
 	{
-		if (! ($node instanceof FuncCall))
+		if ( ! ($node instanceof FuncCall))
 		{
 			return;
 		}
 
-		if (! ($node->name instanceof Name))
+		if ( ! ($node->name instanceof Name))
 		{
 			return;
 		}
 
-		if (! $node->name->isUnqualified())
+		if ( ! $node->name->isUnqualified())
 		{
 			return;
 		}
@@ -55,11 +55,11 @@ 
 			FunctionPatcher::isWhitelisted((string) $node->name)
 			&& ! FunctionPatcher::isBlacklisted((string) $node->name)
 		) {
-			$replacement = new FullyQualified('\__FuncProxy__::' . (string) $node->name);
+			$replacement = new FullyQualified('\__FuncProxy__::'.(string) $node->name);
 
 			$pos = $node->getAttribute('startTokenPos');
 			FunctionPatcher::$replacement[$pos] = 
-				'\__FuncProxy__::' . (string) $node->name;
+				'\__FuncProxy__::'.(string) $node->name;
 
 			$node->name = $replacement;
 		}