From 8d29ddeb7381d054a45e2fd2718fd2a9efd2fe9c Mon Sep 17 00:00:00 2001 From: Florent Viel Date: Tue, 26 Aug 2014 17:59:05 +0200 Subject: [PATCH 1/3] Add: listMailbox function --- src/Fetch/Server.php | 12 ++++++++++++ tests/Fetch/Test/ServerTest.php | 10 ++++++++++ 2 files changed, 22 insertions(+) diff --git a/src/Fetch/Server.php b/src/Fetch/Server.php index 4fca523..25fc606 100644 --- a/src/Fetch/Server.php +++ b/src/Fetch/Server.php @@ -445,4 +445,16 @@ class Server { return imap_createmailbox($this->getImapStream(), $this->getServerSpecification() . $mailbox); } + + /** + * List available mailboxes + * + * @param string $pattern + * + * @return array + */ + public function listMailbox($pattern = '*') + { + return imap_list($this->getImapStream(), $this->getServerSpecification(), $pattern); + } } diff --git a/tests/Fetch/Test/ServerTest.php b/tests/Fetch/Test/ServerTest.php index 4255ed2..ee0566d 100644 --- a/tests/Fetch/Test/ServerTest.php +++ b/tests/Fetch/Test/ServerTest.php @@ -205,6 +205,16 @@ class ServerTest extends \PHPUnit_Framework_TestCase $this->assertFalse($server->getMessageByUid(12), 'Message successfully expunged'); } + public function testListMailbox() + { + $server = Static::getServer(); + $spec = sprintf('{%s:143/novalidate-cert}', TESTING_SERVER_HOST); + + $list = $server->listMailbox('*'); + $this->assertContains($spec.'Sent', $list); + $this->assertNotContains($spec.'Cheese', $list); + } + public static function getServer() { $server = new Server(TESTING_SERVER_HOST, 143); From 5e78459970d0bb9333b232db940b70b646238ce3 Mon Sep 17 00:00:00 2001 From: Florent Viel Date: Tue, 26 Aug 2014 18:25:15 +0200 Subject: [PATCH 2/3] Fix: launch list test before create test --- tests/Fetch/Test/ServerTest.php | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/tests/Fetch/Test/ServerTest.php b/tests/Fetch/Test/ServerTest.php index ee0566d..128e424 100644 --- a/tests/Fetch/Test/ServerTest.php +++ b/tests/Fetch/Test/ServerTest.php @@ -164,6 +164,16 @@ class ServerTest extends \PHPUnit_Framework_TestCase $this->assertFalse($server->hasMailBox('Cheese'), 'Does not have mailbox "Cheese"'); } + public function testListMailbox() + { + $server = Static::getServer(); + $spec = sprintf('{%s:143/novalidate-cert}', TESTING_SERVER_HOST); + + $list = $server->listMailbox('*'); + $this->assertContains($spec.'Sent', $list, 'Has mailbox "Sent"'); + $this->assertNotContains($spec.'Cheese', $list, 'Does not have mailbox "Cheese"'); + } + public function testCreateMailbox() { $server = Static::getServer(); @@ -205,16 +215,6 @@ class ServerTest extends \PHPUnit_Framework_TestCase $this->assertFalse($server->getMessageByUid(12), 'Message successfully expunged'); } - public function testListMailbox() - { - $server = Static::getServer(); - $spec = sprintf('{%s:143/novalidate-cert}', TESTING_SERVER_HOST); - - $list = $server->listMailbox('*'); - $this->assertContains($spec.'Sent', $list); - $this->assertNotContains($spec.'Cheese', $list); - } - public static function getServer() { $server = new Server(TESTING_SERVER_HOST, 143); From e8843a11f2f933e431c3a5665e7777157f3718c4 Mon Sep 17 00:00:00 2001 From: Florent Viel Date: Tue, 26 Aug 2014 18:41:21 +0200 Subject: [PATCH 3/3] cs fixer --- src/Fetch/Server.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/Fetch/Server.php b/src/Fetch/Server.php index 25fc606..38d2bcb 100644 --- a/src/Fetch/Server.php +++ b/src/Fetch/Server.php @@ -159,8 +159,6 @@ class Server return false; } - - $this->mailbox = $mailbox; if (isset($this->imapStream)) { $this->setImapStream(); @@ -449,7 +447,7 @@ class Server /** * List available mailboxes * - * @param string $pattern + * @param string $pattern * * @return array */