Merge pull request #110 from Yoast/feature/phpunit-10-add-assertislisttrait

PHPUnit 10 | AssertIsList trait: polyfill the Assert::assertIsList() method

Author: jrfnl

README.md

@@ -406,6 +406,17 @@ These methods were introduced in PHPUnit 10.0.0.
 [`Assert::assertStringEqualsStringIgnoringLineEndings()`]:   https://docs.phpunit.de/en/main/assertions.html#assertstringequalsstringignoringlineendings
 [`Assert::assertStringContainsStringIgnoringLineEndings()`]: https://docs.phpunit.de/en/main/assertions.html#assertstringcontainsstringignoringlineendings
 
+#### PHPUnit < 10.0.0: `Yoast\PHPUnitPolyfills\Polyfills\AssertIsList`
+
+Polyfills the following method:
+|                                 |
+|---------------------------------|
+| [`Assert::assertIsList()`] |
+
+This method was introduced in PHPUnit 10.0.0.
+
+[`Assert::assertIsList()`]: https://docs.phpunit.de/en/main/assertions.html#assertislist
+
 
 ### Helper traits
 

phpunitpolyfills-autoload.php

@@ -91,6 +91,10 @@ public static function load( $className ) {
 					self::loadAssertObjectEquals();
 					return true;
 
+				case 'Yoast\PHPUnitPolyfills\Polyfills\AssertIsList':
+					self::loadAssertIsList();
+					return true;
+
 				case 'Yoast\PHPUnitPolyfills\Polyfills\AssertIgnoringLineEndings':
 					self::loadAssertIgnoringLineEndings();
 					return true;
@@ -293,6 +297,23 @@ public static function loadAssertObjectEquals() {
 			require_once __DIR__ . '/src/Polyfills/AssertObjectEquals_Empty.php';
 		}
 
+		/**
+		 * Load the AssertIsList polyfill or an empty trait with the same name
+		 * if a PHPUnit version is used which already contains this functionality.
+		 *
+		 * @return void
+		 */
+		public static function loadAssertIsList() {
+			if ( \method_exists( Assert::class, 'assertIsList' ) === false ) {
+				// PHPUnit < 10.0.0.
+				require_once __DIR__ . '/src/Polyfills/AssertIsList.php';
+				return;
+			}
+
+			// PHPUnit >= 10.0.0.
+			require_once __DIR__ . '/src/Polyfills/AssertIsList_Empty.php';
+		}
+
 		/**
 		 * Load the AssertIgnoringLineEndings polyfill or an empty trait with the same name
 		 * if a PHPUnit version is used which already contains this functionality.

src/Polyfills/AssertIsList.php

@@ -0,0 +1,100 @@
+<?php
+
+namespace Yoast\PHPUnitPolyfills\Polyfills;
+
+use PHPUnit\Framework\Assert;
+
+/**
+ * Polyfill the Assert::assertIsList() method.
+ *
+ * Introduced in PHPUnit 10.0.0.
+ *
+ * @link https://github.com/sebastianbergmann/phpunit/pull/4818
+ */
+trait AssertIsList {
+
+	/**
+	 * Asserts that an array is list.
+	 *
+	 * @param mixed  $array   The value to test.
+	 * @param string $message Optional failure message to display.
+	 *
+	 * @return void
+	 */
+	final public static function assertIsList( $array, $message = '' ) {
+		$msg = self::assertIsListFailureDescription( $array );
+		if ( $message !== '' ) {
+			$msg = $message . \PHP_EOL . $msg;
+		}
+
+		if ( \is_array( $array ) === false ) {
+			if ( \method_exists( Assert::class, 'assertIsArray' ) ) {
+				static::assertIsArray( $array, $msg );
+				return;
+			}
+
+			static::assertInternalType( 'array', $array, $msg );
+			return;
+		}
+
+		if ( $array === [] ) {
+			static::assertSame( $array, $array, $msg );
+			return;
+		}
+
+		if ( \function_exists( 'array_is_list' ) ) {
+			// phpcs:ignore PHPCompatibility.FunctionUse.NewFunctions.array_is_listFound -- PHP 8.1+.
+			static::assertTrue( \array_is_list( $array ), $msg );
+			return;
+		}
+
+		$expected = \range( 0, ( \count( $array ) - 1 ) );
+
+		static::assertSame( $expected, \array_keys( $array ), $msg );
+	}
+
+	/**
+	 * Returns the description of the failure.
+	 *
+	 * @param mixed $other The value under test.
+	 *
+	 * @return string
+	 */
+	private static function assertIsListFailureDescription( $other ) {
+		$type = \strtolower( \gettype( $other ) );
+
+		switch ( $type ) {
+			case 'double':
+				$description = 'a float';
+				break;
+
+			case 'resource (closed)':
+				$description = 'a closed resource';
+				break;
+
+			case 'array':
+			case 'integer':
+			case 'object':
+				$description = 'an ' . $type;
+				break;
+
+			case 'boolean':
+			case 'closed resource':
+			case 'float':
+			case 'resource':
+			case 'string':
+				$description = 'a ' . $type;
+				break;
+
+			case 'null':
+				$description = 'null';
+				break;
+
+			default:
+				$description = 'a value of ' . $type;
+				break;
+		}
+
+		return \sprintf( 'Failed asserting that %s is a list.', $description );
+	}
+}

src/Polyfills/AssertIsList_Empty.php

@@ -0,0 +1,8 @@
+<?php
+
+namespace Yoast\PHPUnitPolyfills\Polyfills;
+
+/**
+ * Empty trait for use with PHPUnit >= 10.0.0 in which this polyfill is not needed.
+ */
+trait AssertIsList {}

src/TestCases/TestCasePHPUnitGte8.php

@@ -8,6 +8,7 @@
 use Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertIgnoringLineEndings;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;
+use Yoast\PHPUnitPolyfills\Polyfills\AssertIsList;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertObjectEquals;
 use Yoast\PHPUnitPolyfills\Polyfills\EqualToSpecializations;
 use Yoast\PHPUnitPolyfills\Polyfills\ExpectExceptionMessageMatches;
@@ -28,6 +29,7 @@ abstract class TestCase extends PHPUnit_TestCase {
 	use AssertFileEqualsSpecializations;
 	use AssertIgnoringLineEndings;
 	use AssertionRenames;
+	use AssertIsList;
 	use AssertObjectEquals;
 	use EqualToSpecializations;
 	use ExpectExceptionMessageMatches;

src/TestCases/TestCasePHPUnitLte7.php

@@ -9,6 +9,7 @@
 use Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertIgnoringLineEndings;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;
+use Yoast\PHPUnitPolyfills\Polyfills\AssertIsList;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertIsType;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertObjectEquals;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertStringContains;
@@ -33,6 +34,7 @@ abstract class TestCase extends PHPUnit_TestCase {
 	use AssertFileEqualsSpecializations;
 	use AssertIgnoringLineEndings;
 	use AssertionRenames;
+	use AssertIsList;
 	use AssertIsType;
 	use AssertObjectEquals;
 	use AssertStringContains;

src/TestCases/XTestCase.php

@@ -9,6 +9,7 @@
 use Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertIgnoringLineEndings;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames;
+use Yoast\PHPUnitPolyfills\Polyfills\AssertIsList;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertIsType;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertObjectEquals;
 use Yoast\PHPUnitPolyfills\Polyfills\AssertStringContains;
@@ -35,6 +36,7 @@ abstract class XTestCase extends PHPUnit_TestCase {
 	use AssertFileEqualsSpecializations;
 	use AssertIgnoringLineEndings;
 	use AssertionRenames;
+	use AssertIsList;
 	use AssertIsType;
 	use AssertObjectEquals;
 	use AssertStringContains;