Merge pull request #206 from Slamdunk/connection_expunge

Remove misleading Mailbox::expunge()
ddeboer · Sep 29, 2017 · fadb08f · fadb08f
2 parents 07581df + e528e7c
commit fadb08f
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 12 deletions.
diff --git a/src/Mailbox.php b/src/Mailbox.php
@@ -158,16 +158,6 @@ public function delete()
         $this->connection->deleteMailbox($this);
     }
 
-    /**
-     * Delete all messages marked for deletion
-     *
-     * @return Mailbox
-     */
-    public function expunge()
-    {
-        $this->connection->expunge();
-    }
-
     /**
      * Add a message to the mailbox
      *

diff --git a/tests/MessageTest.php b/tests/MessageTest.php
@@ -11,7 +11,6 @@
 
 /**
  * @covers \Ddeboer\Imap\Connection::expunge
- * @covers \Ddeboer\Imap\Mailbox::expunge
  * @covers \Ddeboer\Imap\Message
  * @covers \Ddeboer\Imap\MessageIterator
  * @covers \Ddeboer\Imap\Message\Attachment
@@ -221,7 +220,7 @@ public function testDelete()
 
         $message = $this->mailbox->getMessage(3);
         $message->delete();
-        $this->mailbox->expunge();
+        $this->getConnection()->expunge();
 
         $this->assertCount(2, $this->mailbox);
         foreach ($this->mailbox->getMessages() as $message) {