Tests: make dataproviders static

jrfnl · jrfnl · commit 153072e44bc7 · 2023-09-23T07:58:19.000+02:00
As of PHPUnit 10, data providers are (again) expected to be `static` methods.

This updates the test suite to respect that.
diff --git a/tests/BrainMonkey/TestCaseTest.php b/tests/BrainMonkey/TestCaseTest.php
@@ -184,7 +184,7 @@ public function testMakeDoubleForUnavailableClass( $class_name ) {
 	 *
 	 * @return array
 	 */
-	public function dataMakeDoubleForUnavailableClass() {
+	public static function dataMakeDoubleForUnavailableClass() {
 		return [
 			'Global class name'                        => [ 'GlobalClassName' ],
 			'Global class name with leading backslash' => [ '\BackslashedClassName' ],
@@ -220,7 +220,7 @@ public function testMakeDoubleForUnavailableClassThrowsExceptionWithInvalidName(
 	 *
 	 * @return array
 	 */
-	public function dataMakeDoubleForUnavailableClassThrowsExceptionWithInvalidName() {
+	public static function dataMakeDoubleForUnavailableClassThrowsExceptionWithInvalidName() {
 		return [
 			'Empty string as class name'                       => [ '' ],
 			'Only backslashes'                                 => [ '\\\\\\' ],
diff --git a/tests/BrainMonkey/YoastTestCaseTest.php b/tests/BrainMonkey/YoastTestCaseTest.php
@@ -30,7 +30,7 @@ public function testStubGetBlogInfo( $show, $expected ) {
 	 *
 	 * @return array
 	 */
-	public function dataStubGetBlogInfo() {
+	public static function dataStubGetBlogInfo() {
 		return [
 			// Explicit cases.
 			'charset' => [
@@ -140,7 +140,7 @@ public function testStubWpParseArgs( $settings, $defaults, $expected ) {
 	 *
 	 * @return array
 	 */
-	public function dataStubWpParseArgs() {
+	public static function dataStubWpParseArgs() {
 		return [
 			'two empty arrays' => [
 				'settings' => [],
@@ -221,7 +221,7 @@ public function testStubWpStripAllTags( $text, $remove_breaks, $expected ) {
 	 *
 	 * @return array
 	 */
-	public function dataStubWpStripAllTags() {
+	public static function dataStubWpStripAllTags() {
 		return [
 			'Empty string' => [
 				'text'          => '',
@@ -284,7 +284,7 @@ public function testStubWpSlash( $input ) {
 	 *
 	 * @return array
 	 */
-	public function dataStubWpSlash() {
+	public static function dataStubWpSlash() {
 		return [
 			'string' => [
 				'input' => "O'Reilly?",
@@ -322,7 +322,7 @@ public function testStubWpUnslash( $input, $expected ) {
 	 *
 	 * @return array
 	 */
-	public function dataStubWpUnslash() {
+	public static function dataStubWpUnslash() {
 		return [
 			'string' => [
 				'input'    => "O\'Reilly\?",
diff --git a/tests/Helpers/ExpectOutputHelperTest.php b/tests/Helpers/ExpectOutputHelperTest.php
@@ -125,7 +125,7 @@ public function testMatchingSubstringsWithMismatchedLineEndingsAmount( $expected
 	 *
 	 * @return array
 	 */
-	public function dataMatchingSubstringsMismatchedLineEndings() {
+	public static function dataMatchingSubstringsMismatchedLineEndings() {
 		return [
 			// Actual line ending type.
 			[ "fox\njumps" ],
@@ -233,7 +233,7 @@ public function testNormalizeLineEndings( $input, $expected ) {
 	 *
 	 * @return array
 	 */
-	public function dataNormalizeLineEndings() {
+	public static function dataNormalizeLineEndings() {
 		return [
 			'already-lf'            => [ "foo\n\nbar\n", "foo\n\nbar\n" ],
 			'windows-crlf'          => [ "foo\r\n\r\nbar\r\n", "foo\n\nbar\n" ],
