chore: run rectorrector-phpunit10

Signed-off-by: Robin Appelman <robin@icewind.nl>

1 files changed, 28 insertions, 28 deletions

diff --git a/tests/lib/DB/QueryBuilder/QueryBuilderTest.php b/tests/lib/DB/QueryBuilder/QueryBuilderTest.php
index 538ec637099..990191a9ff5 100644
--- a/tests/lib/DB/QueryBuilder/QueryBuilderTest.php
+++ b/tests/lib/DB/QueryBuilder/QueryBuilderTest.php
@@ -100,11 +100,11 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataFirstResult
 	 *
 	 * @param int|null $firstResult
 	 * @param array $expectedSet
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataFirstResult')]
 	public function testFirstResult($firstResult, $expectedSet): void {
 		$this->deleteTestingRows();
 		$this->createTestingRows();
@@ -137,11 +137,11 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataMaxResults
 	 *
 	 * @param int $maxResult
 	 * @param array $expectedSet
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataMaxResults')]
 	public function testMaxResults($maxResult, $expectedSet): void {
 		$this->deleteTestingRows();
 		$this->createTestingRows();
@@ -189,12 +189,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataSelect
 	 *
 	 * @param array $selectArguments
 	 * @param array $expected
 	 * @param string $expectedLiteral
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataSelect')]
 	public function testSelect($selectArguments, $expected, $expectedLiteral = ''): void {
 		$this->deleteTestingRows();
 		$this->createTestingRows();
@@ -242,12 +242,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataSelectAlias
 	 *
 	 * @param mixed $select
 	 * @param array $alias
 	 * @param array $expected
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataSelectAlias')]
 	public function testSelectAlias($select, $alias, $expected): void {
 		$this->deleteTestingRows();
 		$this->createTestingRows();
@@ -366,12 +366,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataAddSelect
 	 *
 	 * @param array $selectArguments
 	 * @param array $expected
 	 * @param string $expectedLiteral
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataAddSelect')]
 	public function testAddSelect($selectArguments, $expected, $expectedLiteral = ''): void {
 		$this->deleteTestingRows();
 		$this->createTestingRows();
@@ -418,13 +418,13 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataDelete
 	 *
 	 * @param string $tableName
 	 * @param string $tableAlias
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataDelete')]
 	public function testDelete($tableName, $tableAlias, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->delete($tableName, $tableAlias);
 
@@ -447,13 +447,13 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataUpdate
 	 *
 	 * @param string $tableName
 	 * @param string $tableAlias
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataUpdate')]
 	public function testUpdate($tableName, $tableAlias, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->update($tableName, $tableAlias);
 
@@ -475,12 +475,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataInsert
 	 *
 	 * @param string $tableName
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataInsert')]
 	public function testInsert($tableName, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->insert($tableName);
 
@@ -517,7 +517,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataFrom
 	 *
 	 * @param string|IQueryFunction $table1Name
 	 * @param string $table1Alias
@@ -526,6 +525,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataFrom')]
 	public function testFrom($table1Name, $table1Alias, $table2Name, $table2Alias, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->from($table1Name, $table1Alias);
 		if ($table2Name !== null) {
@@ -565,7 +565,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataJoin
 	 *
 	 * @param string $fromAlias
 	 * @param string $tableName
@@ -574,6 +573,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataJoin')]
 	public function testJoin($fromAlias, $tableName, $tableAlias, $condition, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->from('data1', 'd1');
 		$this->queryBuilder->join(
@@ -595,7 +595,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataJoin
 	 *
 	 * @param string $fromAlias
 	 * @param string $tableName
@@ -604,6 +603,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataJoin')]
 	public function testInnerJoin($fromAlias, $tableName, $tableAlias, $condition, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->from('data1', 'd1');
 		$this->queryBuilder->innerJoin(
@@ -645,7 +645,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataLeftJoin
 	 *
 	 * @param string $fromAlias
 	 * @param string $tableName
@@ -654,6 +653,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataLeftJoin')]
 	public function testLeftJoin($fromAlias, $tableName, $tableAlias, $condition, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->from('data1', 'd1');
 		$this->queryBuilder->leftJoin(
@@ -695,7 +695,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataRightJoin
 	 *
 	 * @param string $fromAlias
 	 * @param string $tableName
@@ -704,6 +703,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataRightJoin')]
 	public function testRightJoin($fromAlias, $tableName, $tableAlias, $condition, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->from('data1', 'd1');
 		$this->queryBuilder->rightJoin(
@@ -734,7 +734,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataSet
 	 *
 	 * @param string $partOne1
 	 * @param string $partOne2
@@ -743,6 +742,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataSet')]
 	public function testSet($partOne1, $partOne2, $partTwo1, $partTwo2, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->update('data');
 		$this->queryBuilder->set($partOne1, $partOne2);
@@ -769,12 +769,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataWhere
 	 *
 	 * @param array $whereArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataWhere')]
 	public function testWhere($whereArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->select('column');
 		call_user_func_array(
@@ -794,12 +794,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataWhere
 	 *
 	 * @param array $whereArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataWhere')]
 	public function testAndWhere($whereArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->select('column');
 		call_user_func_array(
@@ -826,12 +826,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataOrWhere
 	 *
 	 * @param array $whereArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataOrWhere')]
 	public function testOrWhere($whereArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->select('column');
 		call_user_func_array(
@@ -858,12 +858,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataGroupBy
 	 *
 	 * @param array $groupByArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataGroupBy')]
 	public function testGroupBy($groupByArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->select('column');
 		call_user_func_array(
@@ -890,12 +890,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataAddGroupBy
 	 *
 	 * @param array $groupByArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataAddGroupBy')]
 	public function testAddGroupBy($groupByArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->select('column');
 		$this->queryBuilder->groupBy('column1');
@@ -922,13 +922,13 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataSetValue
 	 *
 	 * @param string $column
 	 * @param string $value
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataSetValue')]
 	public function testSetValue($column, $value, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->insert('data');
 		$this->queryBuilder->setValue($column, $value);
@@ -945,13 +945,13 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataSetValue
 	 *
 	 * @param string $column
 	 * @param string $value
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataSetValue')]
 	public function testValues($column, $value, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->insert('data');
 		$this->queryBuilder->values([
@@ -987,12 +987,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataHaving
 	 *
 	 * @param array $havingArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataHaving')]
 	public function testHaving($havingArguments, $expectedQueryPart, $expectedQuery): void {
 		call_user_func_array(
 			[$this->queryBuilder, 'having'],
@@ -1028,12 +1028,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataAndHaving
 	 *
 	 * @param array $havingArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataAndHaving')]
 	public function testAndHaving($havingArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->having('condition1');
 		call_user_func_array(
@@ -1070,12 +1070,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataOrHaving
 	 *
 	 * @param array $havingArguments
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataOrHaving')]
 	public function testOrHaving($havingArguments, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->having('condition1');
 		call_user_func_array(
@@ -1103,13 +1103,13 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataOrderBy
 	 *
 	 * @param string $sort
 	 * @param string $order
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataOrderBy')]
 	public function testOrderBy($sort, $order, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->orderBy($sort, $order);
 
@@ -1139,7 +1139,6 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataAddOrderBy
 	 *
 	 * @param string $sort2
 	 * @param string $order2
@@ -1147,6 +1146,7 @@ class QueryBuilderTest extends \Test\TestCase {
 	 * @param array $expectedQueryPart
 	 * @param string $expectedQuery
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataAddOrderBy')]
 	public function testAddOrderBy($sort2, $order2, $order1, $expectedQueryPart, $expectedQuery): void {
 		$this->queryBuilder->orderBy('column1', $order1);
 		$this->queryBuilder->addOrderBy($sort2, $order2);
@@ -1219,12 +1219,12 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataGetTableName
 	 *
 	 * @param string|IQueryFunction $tableName
 	 * @param bool $automatic
 	 * @param string $expected
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataGetTableName')]
 	public function testGetTableName($tableName, $automatic, $expected): void {
 		if ($automatic !== null) {
 			$this->queryBuilder->automaticTablePrefix($automatic);
@@ -1244,11 +1244,11 @@ class QueryBuilderTest extends \Test\TestCase {
 	}
 
 	/**
-	 * @dataProvider dataGetColumnName
 	 * @param string $column
 	 * @param string $prefix
 	 * @param string $expected
 	 */
+	#[\PHPUnit\Framework\Attributes\DataProvider('dataGetColumnName')]
 	public function testGetColumnName($column, $prefix, $expected): void {
 		$this->assertSame(
 			$expected,