Merge pull request #4659 from nextcloud/fix/ignore-empty-email-contacts-menu

Do not show an email action for contacts with emtpy email addresses

1 files changed, 24 insertions, 0 deletions

diff --git a/tests/lib/Contacts/ContactsMenu/Providers/EMailproviderTest.php b/tests/lib/Contacts/ContactsMenu/Providers/EMailproviderTest.php
index 2d82fa5d68e..353c8d6f58f 100644
--- a/tests/lib/Contacts/ContactsMenu/Providers/EMailproviderTest.php
+++ b/tests/lib/Contacts/ContactsMenu/Providers/EMailproviderTest.php
@@ -79,4 +79,28 @@ class EMailproviderTest extends TestCase {
 		$this->provider->process($entry);
 	}
 
+	public function testProcessEmptyAddress() {
+		$entry = $this->createMock(IEntry::class);
+		$action = $this->createMock(ILinkAction::class);
+		$iconUrl = 'https://example.com/img/actions/icon.svg';
+		$this->urlGenerator->expects($this->once())
+			->method('imagePath')
+			->willReturn('img/actions/icon.svg');
+		$this->urlGenerator->expects($this->once())
+			->method('getAbsoluteURL')
+			->with('img/actions/icon.svg')
+			->willReturn($iconUrl);
+		$entry->expects($this->once())
+			->method('getEMailAddresses')
+			->willReturn([
+				'',
+		]);
+		$this->actionFactory->expects($this->never())
+			->method('newEMailAction');
+		$entry->expects($this->never())
+			->method('addAction');
+
+		$this->provider->process($entry);
+	}
+
 }