Merge pull request #4623 from samsonasik/codecoverage-ignore-deprecated

[Test] Use @codeCoverageIgnore for deprecated class/method

phpunit.xml.dist

@@ -11,7 +11,7 @@
 		 failOnWarning="true"
 		 verbose="true">
 
-	<coverage processUncoveredFiles="true">
+	<coverage processUncoveredFiles="true" ignoreDeprecatedCodeUnits="true">
 		<include>
 			<directory suffix=".php">./system</directory>
 		</include>

system/Controller.php

@@ -130,6 +130,8 @@ protected function cachePage(int $time)
 	 * Handles "auto-loading" helper files.
 	 *
 	 * @deprecated Use `helper` function instead of using this method.
+	 *
+	 * @codeCoverageIgnore
 	 */
 	protected function loadHelpers()
 	{

system/Database/ModelFactory.php

@@ -17,6 +17,8 @@
  * Returns new or shared Model instances
  *
  * @deprecated Use CodeIgniter\Config\Factories::models()
+ *
+ * @codeCoverageIgnore
  */
 class ModelFactory
 {

system/Exceptions/CastException.php

@@ -14,7 +14,9 @@
 /**
  * Cast Exceptions.
  *
- * @deprecated use CodeIgniter\Entity\Exceptions\CastException instead. 
+ * @deprecated use CodeIgniter\Entity\Exceptions\CastException instead.
+ *
+ * @codeCoverageIgnore
  */
 class CastException extends CriticalError
 {

system/HTTP/Message.php

@@ -61,6 +61,8 @@ public function getBody()
 	 * @return array<string, Header> An array of the request headers
 	 *
 	 * @deprecated Use Message::headers() to make room for PSR-7
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function getHeaders(): array
 	{
@@ -76,6 +78,8 @@ public function getHeaders(): array
 	 * @return array|Header|null
 	 *
 	 * @deprecated Use Message::header() to make room for PSR-7
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function getHeader(string $name)
 	{

system/HTTP/Request.php

@@ -77,6 +77,8 @@ public function __construct($config = null)
 	 * @return boolean
 	 *
 	 * @deprecated Use Validation instead
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function isValidIP(string $ip = null, string $which = null): bool
 	{
@@ -91,6 +93,8 @@ public function isValidIP(string $ip = null, string $which = null): bool
 	 * @return string
 	 *
 	 * @deprecated The $upper functionality will be removed and this will revert to its PSR-7 equivalent
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function getMethod(bool $upper = false): string
 	{
@@ -105,6 +109,8 @@ public function getMethod(bool $upper = false): string
 	 * @return Request
 	 *
 	 * @deprecated Use withMethod() instead for immutability
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function setMethod(string $method)
 	{

system/HTTP/Response.php

@@ -223,6 +223,8 @@ public function getStatusCode(): int
 	 * @return string
 	 *
 	 * @deprecated Use getReasonPhrase()
+	 *
+	 * @codeCoverageIgnore
 	 */
 	public function getReason(): string
 	{

system/Router/Router.php

@@ -584,6 +584,8 @@ public function autoRoute(string $uri)
 	 * @return array returns an array of remaining uri segments that don't map onto a directory
 	 *
 	 * @deprecated this function name does not properly describe its behavior so it has been deprecated
+	 *
+	 * @codeCoverageIgnore
 	 */
 	protected function validateRequest(array $segments): array
 	{

system/Test/ControllerResponse.php

@@ -18,6 +18,8 @@
  * Testable response from a controller
  *
  * @deprecated Use TestResponse directly
+ *
+ * @codeCoverageIgnore
  */
 class ControllerResponse extends TestResponse
 {

system/Test/ControllerTester.php

@@ -35,6 +35,8 @@
  *       ->execute('methodName');
  *
  * @deprecated Use ControllerTestTrait instead
+ *
+ * @codeCoverageIgnore
  */
 trait ControllerTester
 {

system/Test/FeatureTestCase.php

@@ -30,6 +30,8 @@
  * against your application.
  *
  * @deprecated Use FeatureTestTrait instead
+ *
+ * @codeCoverageIgnore
  */
 class FeatureTestCase extends CIUnitTestCase
 {