diff options
author | Christoph Wurst <christoph@winzerhof-wurst.at> | 2021-11-10 11:50:48 +0300 |
---|---|---|
committer | Christoph Wurst <christoph@winzerhof-wurst.at> | 2021-11-10 12:04:13 +0300 |
commit | d0cd84dabdc8dde0d33024b5a1bc65ad3b988063 (patch) | |
tree | 8a52c57e319d7968646f05caefb054f15e7dec4e /tests | |
parent | 8e5bea02eda13ad148254c225d2d8f6853cc5c5a (diff) |
Update to Nextcloud coding standard v1
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/Unit/Controller/MessagesControllerTest.php | 8 | ||||
-rw-r--r-- | tests/Unit/Listener/AddressCollectionListenerTest.php | 2 | ||||
-rw-r--r-- | tests/Unit/Service/MailManagerTest.php | 6 | ||||
-rw-r--r-- | tests/Unit/Service/MailTransmissionTest.php | 2 |
4 files changed, 9 insertions, 9 deletions
diff --git a/tests/Unit/Controller/MessagesControllerTest.php b/tests/Unit/Controller/MessagesControllerTest.php index f89ed523b..1784003a9 100644 --- a/tests/Unit/Controller/MessagesControllerTest.php +++ b/tests/Unit/Controller/MessagesControllerTest.php @@ -705,7 +705,7 @@ class MessagesControllerTest extends TestCase { ->will($this->returnValue($this->account)); $this->mailManager->expects($this->once()) ->method('getTagByImapLabel') - ->with($imapLabel,$this->userId) + ->with($imapLabel, $this->userId) ->willThrowException(new ClientException('Computer says no')); $this->mailManager->expects($this->never()) ->method('tagMessage'); @@ -740,7 +740,7 @@ class MessagesControllerTest extends TestCase { ->will($this->returnValue($this->account)); $this->mailManager->expects($this->once()) ->method('getTagByImapLabel') - ->with($tag->getImapLabel(),$this->userId) + ->with($tag->getImapLabel(), $this->userId) ->willReturn($tag); $this->mailManager->expects($this->once()) ->method('tagMessage') @@ -806,7 +806,7 @@ class MessagesControllerTest extends TestCase { ->will($this->returnValue($this->account)); $this->mailManager->expects($this->once()) ->method('getTagByImapLabel') - ->with($imapLabel,$this->userId) + ->with($imapLabel, $this->userId) ->willThrowException(new ClientException('Computer says no')); $this->mailManager->expects($this->never()) ->method('tagMessage'); @@ -841,7 +841,7 @@ class MessagesControllerTest extends TestCase { ->will($this->returnValue($this->account)); $this->mailManager->expects($this->once()) ->method('getTagByImapLabel') - ->with($tag->getImapLabel(),$this->userId) + ->with($tag->getImapLabel(), $this->userId) ->willReturn($tag); $this->mailManager->expects($this->once()) ->method('tagMessage') diff --git a/tests/Unit/Listener/AddressCollectionListenerTest.php b/tests/Unit/Listener/AddressCollectionListenerTest.php index c3c4f34d9..10a84b0ce 100644 --- a/tests/Unit/Listener/AddressCollectionListenerTest.php +++ b/tests/Unit/Listener/AddressCollectionListenerTest.php @@ -111,7 +111,7 @@ class AddressCollectionListenerTest extends TestCase { $message = $this->createMock(IMessage::class); $this->preferences->expects($this->once()) ->method('getPreference') - ->with('test','collect-data', 'true') + ->with('test', 'collect-data', 'true') ->willReturn('true'); /** @var Horde_Mime_Mail|MockObject $mail */ $mail = $this->createMock(Horde_Mime_Mail::class); diff --git a/tests/Unit/Service/MailManagerTest.php b/tests/Unit/Service/MailManagerTest.php index ddbfb5a2f..b9b7f60d2 100644 --- a/tests/Unit/Service/MailManagerTest.php +++ b/tests/Unit/Service/MailManagerTest.php @@ -357,7 +357,7 @@ class MailManagerTest extends TestCase { //standard flags foreach ($flags as $k => $flag) { - $this->assertEquals($this->manager->filterFlags($account, $k , 'INBOX'), $flags[$k]); + $this->assertEquals($this->manager->filterFlags($account, $k, 'INBOX'), $flags[$k]); } } @@ -369,7 +369,7 @@ class MailManagerTest extends TestCase { ->method('getClient') ->willReturn($client); - $this->assertEquals([], $this->manager->filterFlags($account, Tag::LABEL_IMPORTANT , 'INBOX')); + $this->assertEquals([], $this->manager->filterFlags($account, Tag::LABEL_IMPORTANT, 'INBOX')); } public function testSetFilterFlagsImportant() { @@ -383,7 +383,7 @@ class MailManagerTest extends TestCase { ->method('status') ->willReturn(['permflags' => [ "11" => "\*" ]]); - $this->assertEquals([Tag::LABEL_IMPORTANT], $this->manager->filterFlags($account, Tag::LABEL_IMPORTANT , 'INBOX')); + $this->assertEquals([Tag::LABEL_IMPORTANT], $this->manager->filterFlags($account, Tag::LABEL_IMPORTANT, 'INBOX')); } public function testIsPermflagsEnabledTrue(): void { diff --git a/tests/Unit/Service/MailTransmissionTest.php b/tests/Unit/Service/MailTransmissionTest.php index cca1ecfef..be2736aa2 100644 --- a/tests/Unit/Service/MailTransmissionTest.php +++ b/tests/Unit/Service/MailTransmissionTest.php @@ -293,7 +293,7 @@ class MailTransmissionTest extends TestCase { $content = ['blablabla']; $this->messageMapper->expects($this->once()) ->method('getRawAttachments') - ->with($this->imapClientFactory->getClient($account), $mailbox->getName(), $attachmentMessage->getUid(),[$originalAttachment[0]['id']]) + ->with($this->imapClientFactory->getClient($account), $mailbox->getName(), $attachmentMessage->getUid(), [$originalAttachment[0]['id']]) ->willReturn($content); $this->transmission->sendMessage($messageData, null); |