Skip to content

Commit a0e8d94

Browse files
committed
test: Rename parameters to show that they are expected values
Signed-off-by: Daniel Calviño Sánchez <[email protected]>
1 parent 45f2683 commit a0e8d94

File tree

1 file changed

+14
-14
lines changed

1 file changed

+14
-14
lines changed

tests/lib/Collaboration/Collaborators/MailPluginTest.php

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -89,13 +89,13 @@ public function instantiatePlugin() {
8989
* @param string $searchTerm
9090
* @param array $contacts
9191
* @param bool $shareeEnumeration
92-
* @param array $expected
93-
* @param bool $exactIdMatch
94-
* @param bool $reachedEnd
92+
* @param array $expectedResult
93+
* @param bool $expectedExactIdMatch
94+
* @param bool $expectedMoreResults
9595
* @param bool $validEmail
9696
*/
9797
#[\PHPUnit\Framework\Attributes\DataProvider('dataGetEmail')]
98-
public function testSearch($searchTerm, $contacts, $shareeEnumeration, $expected, $exactIdMatch, $reachedEnd, $validEmail): void {
98+
public function testSearch($searchTerm, $contacts, $shareeEnumeration, $expectedResult, $expectedExactIdMatch, $expectedMoreResults, $validEmail): void {
9999
$this->config->expects($this->any())
100100
->method('getAppValue')
101101
->willReturnCallback(
@@ -127,9 +127,9 @@ function ($appName, $key, $default) use ($shareeEnumeration) {
127127
$moreResults = $this->plugin->search($searchTerm, 2, 0, $this->searchResult);
128128
$result = $this->searchResult->asArray();
129129

130-
$this->assertSame($exactIdMatch, $this->searchResult->hasExactIdMatch(new SearchResultType('emails')));
131-
$this->assertEquals($expected, $result);
132-
$this->assertSame($reachedEnd, $moreResults);
130+
$this->assertSame($expectedExactIdMatch, $this->searchResult->hasExactIdMatch(new SearchResultType('emails')));
131+
$this->assertEquals($expectedResult, $result);
132+
$this->assertSame($expectedMoreResults, $moreResults);
133133
}
134134

135135
public static function dataGetEmail(): array {
@@ -569,14 +569,14 @@ public static function dataGetEmail(): array {
569569
*
570570
* @param string $searchTerm
571571
* @param array $contacts
572-
* @param array $expected
573-
* @param bool $exactIdMatch
574-
* @param bool $reachedEnd
572+
* @param array $expectedResult
573+
* @param bool $expectedExactIdMatch
574+
* @param bool $expectedMoreResults
575575
* @param array $userToGroupMapping
576576
* @param bool $validEmail
577577
*/
578578
#[\PHPUnit\Framework\Attributes\DataProvider('dataGetEmailGroupsOnly')]
579-
public function testSearchGroupsOnly($searchTerm, $contacts, $expected, $exactIdMatch, $reachedEnd, $userToGroupMapping, $validEmail): void {
579+
public function testSearchGroupsOnly($searchTerm, $contacts, $expectedResult, $expectedExactIdMatch, $expectedMoreResults, $userToGroupMapping, $validEmail): void {
580580
$this->config->expects($this->any())
581581
->method('getAppValue')
582582
->willReturnCallback(
@@ -627,9 +627,9 @@ function ($appName, $key, $default) {
627627
$moreResults = $this->plugin->search($searchTerm, 2, 0, $this->searchResult);
628628
$result = $this->searchResult->asArray();
629629

630-
$this->assertSame($exactIdMatch, $this->searchResult->hasExactIdMatch(new SearchResultType('emails')));
631-
$this->assertEquals($expected, $result);
632-
$this->assertSame($reachedEnd, $moreResults);
630+
$this->assertSame($expectedExactIdMatch, $this->searchResult->hasExactIdMatch(new SearchResultType('emails')));
631+
$this->assertEquals($expectedResult, $result);
632+
$this->assertSame($expectedMoreResults, $moreResults);
633633
}
634634

635635
public static function dataGetEmailGroupsOnly(): array {

0 commit comments

Comments
 (0)