diff options
author | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-10-14 17:25:49 +0300 |
---|---|---|
committer | Christoph Wurst <christoph@winzerhof-wurst.at> | 2020-10-15 11:39:56 +0300 |
commit | 6f9d48f6ff606031533f229b802c7e893fe35e10 (patch) | |
tree | d3bd310d05eef9d5332a3840c9a3c84cfc902883 /tests | |
parent | 4f89fbf81f91150229f66334dec6dfe5f40709c1 (diff) |
Fix some issues reported by Psalm
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Integration/Service/MailTransmissionIntegrationTest.php | 1 | ||||
-rw-r--r-- | tests/Unit/Service/MailTransmissionTest.php | 10 |
2 files changed, 11 insertions, 0 deletions
diff --git a/tests/Integration/Service/MailTransmissionIntegrationTest.php b/tests/Integration/Service/MailTransmissionIntegrationTest.php index fad4b3f37..7d6f71362 100644 --- a/tests/Integration/Service/MailTransmissionIntegrationTest.php +++ b/tests/Integration/Service/MailTransmissionIntegrationTest.php @@ -76,6 +76,7 @@ class MailTransmissionIntegrationTest extends TestCase { $mapper = OC::$server->query(MailAccountMapper::class); $mailAccount = MailAccount::fromParams([ 'userId' => $this->user->getUID(), + 'name' => 'Test User', 'email' => 'user@domain.tld', 'inboundHost' => 'localhost', 'inboundPort' => '993', diff --git a/tests/Unit/Service/MailTransmissionTest.php b/tests/Unit/Service/MailTransmissionTest.php index 9525a3a26..4aeaf2cda 100644 --- a/tests/Unit/Service/MailTransmissionTest.php +++ b/tests/Unit/Service/MailTransmissionTest.php @@ -103,6 +103,8 @@ class MailTransmissionTest extends TestCase { /** @var Account|MockObject $account */ $account = $this->createMock(Account::class); $account->method('getMailAccount')->willReturn($mailAccount); + $account->method('getName')->willReturn('Test User'); + $account->method('getEMailAddress')->willReturn('test@user'); $messageData = NewMessageData::fromRequest($account, 'to@d.com', '', '', 'sub', 'bod'); $message = new Message(); $account->expects($this->once()) @@ -123,6 +125,8 @@ class MailTransmissionTest extends TestCase { /** @var Account|MockObject $account */ $account = $this->createMock(Account::class); $account->method('getMailAccount')->willReturn($mailAccount); + $account->method('getName')->willReturn('Test User'); + $account->method('getEMailAddress')->willReturn('test@user'); $alias = new Alias(); $alias->setAlias('a@d.com'); $messageData = NewMessageData::fromRequest($account, 'to@d.com', '', '', 'sub', 'bod'); @@ -151,6 +155,8 @@ class MailTransmissionTest extends TestCase { /** @var Account|MockObject $account */ $account = $this->createMock(Account::class); $account->method('getMailAccount')->willReturn($mailAccount); + $account->method('getName')->willReturn('Test User'); + $account->method('getEMailAddress')->willReturn('test@user'); $attachmenst = [ [ 'fileName' => 'cat.jpg', @@ -191,6 +197,8 @@ class MailTransmissionTest extends TestCase { /** @var Account|MockObject $account */ $account = $this->createMock(Account::class); $account->method('getMailAccount')->willReturn($mailAccount); + $account->method('getName')->willReturn('Test User'); + $account->method('getEMailAddress')->willReturn('test@user'); $messageData = NewMessageData::fromRequest($account, 'to@d.com', '', '', 'sub', 'bod'); $folderId = 'INBOX'; $repliedMessageUid = 321; @@ -217,6 +225,8 @@ class MailTransmissionTest extends TestCase { /** @var Account|MockObject $account */ $account = $this->createMock(Account::class); $account->method('getMailAccount')->willReturn($mailAccount); + $account->method('getName')->willReturn('Test User'); + $account->method('getEMailAddress')->willReturn('test@user'); $messageData = NewMessageData::fromRequest($account, 'to@d.com', '', '', 'sub', 'bod'); $message = new Message(); $account->expects($this->once()) |