Test typos changes

Author: atishhamte

system/Test/CIDatabaseTestCase.php

@@ -37,6 +37,8 @@
 
 namespace CodeIgniter\Test;
 
+use Config\Database;
+use Config\Migrations;
 use Config\Services;
 use CodeIgniter\Database\BaseConnection;
 use CodeIgniter\Database\MigrationRunner;
@@ -123,14 +125,14 @@ public function loadDependencies()
 	{
 		if ($this->db === null)
 		{
-			$this->db = \Config\Database::connect($this->DBGroup);
+			$this->db = Database::connect($this->DBGroup);
 			$this->db->initialize();
 		}
 
 		if ($this->migrations === null)
 		{
 			// Ensure that we can run migrations
-			$config          = new \Config\Migrations();
+			$config          = new Migrations();
 			$config->enabled = true;
 
 			$this->migrations = Services::migrations($config, $this->db);
@@ -139,7 +141,7 @@ public function loadDependencies()
 
 		if ($this->seeder === null)
 		{
-			$this->seeder = \Config\Database::seeder($this->DBGroup);
+			$this->seeder = Database::seeder($this->DBGroup);
 			$this->seeder->setSilent(true);
 		}
 	}
@@ -170,7 +172,7 @@ protected function setUp()
 
 			if (is_array($tables))
 			{
-				$forge = \Config\Database::forge('tests');
+				$forge = Database::forge('tests');
 
 				foreach ($tables as $table)
 				{

system/Test/CIUnitTestCase.php

@@ -76,6 +76,7 @@ protected function setUp()
 	 * @param string $level
 	 * @param null   $expectedMessage
 	 *
+	 * @return bool
 	 * @throws \Exception
 	 */
 	public function assertLogged(string $level, $expectedMessage = null)
@@ -201,6 +202,7 @@ public function assertCloseEnough(int $expected, $actual, string $message = '',
 	 * @param string  $message
 	 * @param integer $tolerance
 	 *
+	 * @return bool
 	 * @throws \Exception
 	 */
 	public function assertCloseEnoughString($expected, $actual, string $message = '', int $tolerance = 1)

system/Test/ControllerTester.php

@@ -45,6 +45,8 @@
 use CodeIgniter\HTTP\URI;
 use Config\App;
 use Config\Services;
+use InvalidArgumentException;
+use Throwable;
 
 /**
  * ControllerTester Trait
@@ -82,7 +84,7 @@ public function controller(string $name)
 	{
 		if (! class_exists($name))
 		{
-			throw new \InvalidArgumentException('Invalid Controller: ' . $name);
+			throw new InvalidArgumentException('Invalid Controller: ' . $name);
 		}
 
 		if (empty($this->appConfig))
@@ -123,7 +125,7 @@ public function execute(string $method, ...$params)
 	{
 		if (! method_exists($this->controller, $method) || ! is_callable([$this->controller, $method]))
 		{
-			throw new \InvalidArgumentException('Method does not exist or is not callable in controller: ' . $method);
+			throw new InvalidArgumentException('Method does not exist or is not callable in controller: ' . $method);
 		}
 
 		// The URL helper is always loaded by the system
@@ -141,7 +143,7 @@ public function execute(string $method, ...$params)
 
 			$response = $this->controller->{$method}(...$params);
 		}
-		catch (\Throwable $e)
+		catch (Throwable $e)
 		{
 			$result->response()
 					->setStatusCode($e->getCode());

system/Test/FeatureResponse.php

@@ -39,6 +39,7 @@
 
 use CodeIgniter\HTTP\RedirectResponse;
 use CodeIgniter\HTTP\Response;
+use Config\Format;
 use PHPUnit\Framework\TestCase;
 
 /**
@@ -226,10 +227,6 @@ public function assertCookie(string $key, $value = null, string $prefix = '')
 	 * Assert the Response does not have the specified cookie set.
 	 *
 	 * @param string $key
-	 * @param null   $value
-	 * @param string $prefix
-	 *
-	 * @throws \Exception
 	 */
 	public function assertCookieMissing(string $key)
 	{
@@ -379,7 +376,7 @@ public function assertJSONExact($test)
 
 		if (is_array($test))
 		{
-			$config    = new \Config\Format();
+			$config    = new Format();
 			$formatter = $config->getFormatter('application/json');
 			$test      = $formatter->format($test);
 		}

system/Test/FeatureTestCase.php

@@ -88,7 +88,7 @@ class FeatureTestCase extends CIDatabaseTestCase
 	 */
 	protected function withRoutes(array $routes = null)
 	{
-		$collection = \Config\Services::routes();
+		$collection = Services::routes();
 
 		if ($routes)
 		{
@@ -297,6 +297,7 @@ protected function setupRequest(string $method, string $path = null, array $para
 	 * @param array|null                $params
 	 *
 	 * @return \CodeIgniter\HTTP\Request
+	 * @throws \ReflectionException
 	 */
 	protected function populateGlobals(string $method, Request $request, array $params = null)
 	{

system/Test/ReflectionHelper.php

@@ -47,13 +47,14 @@
  */
 trait ReflectionHelper
 {
-
 	/**
 	 * Find a private method invoker.
 	 *
-	 * @param  object|string $obj    object or class name
-	 * @param  string        $method method name
+	 * @param object|string $obj    object or class name
+	 * @param string        $method method name
+	 *
 	 * @return \Closure
+	 * @throws \ReflectionException
 	 */
 	public static function getPrivateMethodInvoker($obj, $method)
 	{
@@ -74,6 +75,7 @@ public static function getPrivateMethodInvoker($obj, $method)
 	 * @param string $property
 	 *
 	 * @return \ReflectionProperty
+	 * @throws \ReflectionException
 	 */
 	private static function getAccessibleRefProperty($obj, $property)
 	{
@@ -98,6 +100,8 @@ private static function getAccessibleRefProperty($obj, $property)
 	 * @param object|string $obj      object or class name
 	 * @param string        $property property name
 	 * @param mixed         $value    value
+	 *
+	 * @throws \ReflectionException
 	 */
 	public static function setPrivateProperty($obj, $property, $value)
 	{
@@ -108,9 +112,11 @@ public static function setPrivateProperty($obj, $property, $value)
 	/**
 	 * Retrieve a private property.
 	 *
-	 * @param  object|string $obj      object or class name
-	 * @param  string        $property property name
+	 * @param object|string $obj      object or class name
+	 * @param string        $property property name
+	 *
 	 * @return mixed value
+	 * @throws \ReflectionException
 	 */
 	public static function getPrivateProperty($obj, $property)
 	{