refactor: Add void return type to PHPUnit test methods

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>

1 files changed, 17 insertions, 17 deletions

diff --git a/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php b/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
index d66af2925f4..a3151d0b040 100644
--- a/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
+++ b/tests/lib/Contacts/ContactsMenu/ContactsStoreTest.php
@@ -69,7 +69,7 @@ class ContactsStoreTest extends TestCase {
 		);
 	}
 
-	public function testGetContactsWithoutFilter() {
+	public function testGetContactsWithoutFilter(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->contactsManager->expects($this->once())
@@ -99,7 +99,7 @@ class ContactsStoreTest extends TestCase {
 		], $entries[1]->getEMailAddresses());
 	}
 
-	public function testGetContactsHidesOwnEntry() {
+	public function testGetContactsHidesOwnEntry(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->contactsManager->expects($this->once())
@@ -126,7 +126,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertCount(1, $entries);
 	}
 
-	public function testGetContactsWithoutBinaryImage() {
+	public function testGetContactsWithoutBinaryImage(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->urlGenerator->expects($this->any())
@@ -159,7 +159,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertSame('https://urlToNcAvatar.test', $entries[1]->getAvatar());
 	}
 
-	public function testGetContactsWithoutAvatarURI() {
+	public function testGetContactsWithoutAvatarURI(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->contactsManager->expects($this->once())
@@ -188,7 +188,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('https://photo', $entries[1]->getAvatar());
 	}
 
-	public function testGetContactsWhenUserIsInExcludeGroups() {
+	public function testGetContactsWhenUserIsInExcludeGroups(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -233,7 +233,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertCount(0, $entries);
 	}
 
-	public function testGetContactsOnlyShareIfInTheSameGroup() {
+	public function testGetContactsOnlyShareIfInTheSameGroup(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -308,7 +308,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[2]->getProperty('UID'));
 	}
 
-	public function testGetContactsOnlyEnumerateIfInTheSameGroup() {
+	public function testGetContactsOnlyEnumerateIfInTheSameGroup(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -383,7 +383,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[2]->getProperty('UID'));
 	}
 
-	public function testGetContactsOnlyEnumerateIfPhoneBookMatch() {
+	public function testGetContactsOnlyEnumerateIfPhoneBookMatch(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -441,7 +441,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[2]->getProperty('UID'));
 	}
 
-	public function testGetContactsOnlyEnumerateIfPhoneBookMatchWithOwnGroupsOnly() {
+	public function testGetContactsOnlyEnumerateIfPhoneBookMatchWithOwnGroupsOnly(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -523,7 +523,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[2]->getProperty('UID'));
 	}
 
-	public function testGetContactsOnlyEnumerateIfPhoneBookOrSameGroup() {
+	public function testGetContactsOnlyEnumerateIfPhoneBookOrSameGroup(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -595,7 +595,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[3]->getProperty('UID'));
 	}
 
-	public function testGetContactsOnlyEnumerateIfPhoneBookOrSameGroupInOwnGroupsOnly() {
+	public function testGetContactsOnlyEnumerateIfPhoneBookOrSameGroupInOwnGroupsOnly(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -677,7 +677,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals('contact', $entries[2]->getProperty('UID'));
 	}
 
-	public function testGetContactsWithFilter() {
+	public function testGetContactsWithFilter(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -764,7 +764,7 @@ class ContactsStoreTest extends TestCase {
 		], $entry[0]->getEMailAddresses());
 	}
 
-	public function testGetContactsWithFilterWithoutFullMatch() {
+	public function testGetContactsWithFilterWithoutFullMatch(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -848,7 +848,7 @@ class ContactsStoreTest extends TestCase {
 		], $entry[0]->getEMailAddresses());
 	}
 
-	public function testFindOneUser() {
+	public function testFindOneUser(): void {
 		$this->config
 			->method('getAppValue')
 			->willReturnMap([
@@ -891,7 +891,7 @@ class ContactsStoreTest extends TestCase {
 		], $entry->getEMailAddresses());
 	}
 
-	public function testFindOneEMail() {
+	public function testFindOneEMail(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->contactsManager->expects($this->once())
@@ -922,7 +922,7 @@ class ContactsStoreTest extends TestCase {
 		], $entry->getEMailAddresses());
 	}
 
-	public function testFindOneNotSupportedType() {
+	public function testFindOneNotSupportedType(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 
@@ -931,7 +931,7 @@ class ContactsStoreTest extends TestCase {
 		$this->assertEquals(null, $entry);
 	}
 
-	public function testFindOneNoMatches() {
+	public function testFindOneNoMatches(): void {
 		/** @var IUser|MockObject $user */
 		$user = $this->createMock(IUser::class);
 		$this->contactsManager->expects($this->once())