Mark methods of abstract classes final where possible

src/Message/AbstractMessage.php

@@ -17,7 +17,7 @@ abstract class AbstractMessage extends AbstractPart
      *
      * @return string
      */
-    public function getId(): string
+    final public function getId(): string
     {
         return $this->getHeaders()->get('message_id');
     }
@@ -27,7 +27,7 @@ public function getId(): string
      *
      * @return EmailAddress
      */
-    public function getFrom(): EmailAddress
+    final public function getFrom(): EmailAddress
     {
         return $this->getHeaders()->get('from');
     }
@@ -37,7 +37,7 @@ public function getFrom(): EmailAddress
      *
      * @return EmailAddress[] Empty array in case message has no To: recipients
      */
-    public function getTo(): array
+    final public function getTo(): array
     {
         return $this->getHeaders()->get('to') ?: [];
     }
@@ -47,7 +47,7 @@ public function getTo(): array
      *
      * @return EmailAddress[] Empty array in case message has no CC: recipients
      */
-    public function getCc(): array
+    final public function getCc(): array
     {
         return $this->getHeaders()->get('cc') ?: [];
     }
@@ -57,7 +57,7 @@ public function getCc(): array
      *
      * @return EmailAddress[] Empty array in case message has no BCC: recipients
      */
-    public function getBcc(): array
+    final public function getBcc(): array
     {
         return $this->getHeaders()->get('bcc') ?: [];
     }
@@ -67,7 +67,7 @@ public function getBcc(): array
      *
      * @return EmailAddress[] Empty array in case message has no Reply-To: recipients
      */
-    public function getReplyTo(): array
+    final public function getReplyTo(): array
     {
         return $this->getHeaders()->get('reply_to') ?: [];
     }
@@ -77,7 +77,7 @@ public function getReplyTo(): array
      *
      * @return EmailAddress[] Empty array in case message has no Sender: recipients
      */
-    public function getSender(): array
+    final public function getSender(): array
     {
         return $this->getHeaders()->get('sender') ?: [];
     }
@@ -87,7 +87,7 @@ public function getSender(): array
      *
      * @return EmailAddress[] Empty array in case message has no Return-Path: recipients
      */
-    public function getReturnPath(): array
+    final public function getReturnPath(): array
     {
         return $this->getHeaders()->get('return_path') ?: [];
     }
@@ -97,7 +97,7 @@ public function getReturnPath(): array
      *
      * @return \DateTimeImmutable
      */
-    public function getDate(): \DateTimeImmutable
+    final public function getDate(): \DateTimeImmutable
     {
         return $this->getHeaders()->get('date');
     }
@@ -107,7 +107,7 @@ public function getDate(): \DateTimeImmutable
      *
      * @return int
      */
-    public function getSize()
+    final public function getSize()
     {
         return $this->getHeaders()->get('size');
     }
@@ -117,7 +117,7 @@ public function getSize()
      *
      * @return string
      */
-    public function getSubject()
+    final public function getSubject()
     {
         return $this->getHeaders()->get('subject');
     }
@@ -127,7 +127,7 @@ public function getSubject()
      *
      * @return string | null Null if message has no HTML message part
      */
-    public function getBodyHtml()
+    final public function getBodyHtml()
     {
         $iterator = new \RecursiveIteratorIterator($this, \RecursiveIteratorIterator::SELF_FIRST);
         foreach ($iterator as $part) {
@@ -149,7 +149,7 @@ public function getBodyHtml()
      *
      * @return string
      */
-    public function getBodyText()
+    final public function getBodyText()
     {
         $iterator = new \RecursiveIteratorIterator($this, \RecursiveIteratorIterator::SELF_FIRST);
         foreach ($iterator as $part) {
@@ -171,7 +171,7 @@ public function getBodyText()
      *
      * @return AttachmentInterface[]
      */
-    public function getAttachments(): array
+    final public function getAttachments(): array
     {
         if (null === $this->attachments) {
             $this->attachments = [];
@@ -197,7 +197,7 @@ public function getAttachments(): array
      *
      * @return bool
      */
-    public function hasAttachments(): bool
+    final public function hasAttachments(): bool
     {
         return \count($this->getAttachments()) > 0;
     }

src/Message/AbstractPart.php

@@ -94,42 +94,42 @@ public function __construct(
      *
      * @return int
      */
-    public function getNumber(): int
+    final public function getNumber(): int
     {
         return $this->messageNumber;
     }
 
-    public function getCharset(): string
+    final public function getCharset(): string
     {
         return $this->parameters->get('charset');
     }
 
-    public function getType()
+    final public function getType()
     {
         return $this->type;
     }
 
-    public function getSubtype()
+    final public function getSubtype()
     {
         return $this->subtype;
     }
 
-    public function getEncoding()
+    final public function getEncoding()
     {
         return $this->encoding;
     }
 
-    public function getBytes()
+    final public function getBytes()
     {
         return $this->bytes;
     }
 
-    public function getLines()
+    final public function getLines()
     {
         return $this->lines;
     }
 
-    public function getParameters(): Parameters
+    final public function getParameters(): Parameters
     {
         return $this->parameters;
     }
@@ -153,7 +153,7 @@ public function getContent(): string
      *
      * @return string
      */
-    public function getDecodedContent(): string
+    final public function getDecodedContent(): string
     {
         if (null === $this->decodedContent) {
             $content = $this->getContent();
@@ -175,7 +175,7 @@ public function getDecodedContent(): string
         return $this->decodedContent;
     }
 
-    public function getStructure(): \stdClass
+    final public function getStructure(): \stdClass
     {
         return $this->structure;
     }
@@ -232,47 +232,47 @@ final protected function parseStructure(\stdClass $structure)
      *
      * @return PartInterface[]
      */
-    public function getParts(): array
+    final public function getParts(): array
     {
         return $this->parts;
     }
 
-    public function current()
+    final public function current()
     {
         return $this->parts[$this->key];
     }
 
-    public function getChildren()
+    final public function getChildren()
     {
         return $this->current();
     }
 
-    public function hasChildren()
+    final public function hasChildren()
     {
         return \count($this->parts) > 0;
     }
 
-    public function key()
+    final public function key()
     {
         return $this->key;
     }
 
-    public function next()
+    final public function next()
     {
         ++$this->key;
     }
 
-    public function rewind()
+    final public function rewind()
     {
         $this->key = 0;
     }
 
-    public function valid()
+    final public function valid()
     {
         return isset($this->parts[$this->key]);
     }
 
-    public function getDisposition()
+    final public function getDisposition()
     {
         return $this->disposition;
     }

tests/ConnectionTest.php

@@ -15,7 +15,7 @@
  * @covers \Ddeboer\Imap\Connection
  * @covers \Ddeboer\Imap\ImapResource
  */
-class ConnectionTest extends AbstractTest
+final class ConnectionTest extends AbstractTest
 {
     public function testValidResourceStream()
     {

tests/MailboxTest.php

@@ -12,7 +12,7 @@
  * @covers \Ddeboer\Imap\Exception\AbstractException
  * @covers \Ddeboer\Imap\Mailbox
  */
-class MailboxTest extends AbstractTest
+final class MailboxTest extends AbstractTest
 {
     /**
      * @var Mailbox

tests/MessageTest.php

@@ -24,7 +24,7 @@
  * @covers \Ddeboer\Imap\Message\Part
  * @covers \Ddeboer\Imap\Message\Transcoder
  */
-class MessageTest extends AbstractTest
+final class MessageTest extends AbstractTest
 {
     /**
      * @var \Ddeboer\Imap\Mailbox

tests/Search/Date/AbstractDateTest.php

@@ -10,7 +10,7 @@
 /**
  * @covers \Ddeboer\Imap\Search\AbstractDate
  */
-class AbstractDateTest extends AbstractTest
+final class AbstractDateTest extends AbstractTest
 {
     protected function setUp()
     {

tests/ServerTest.php

@@ -10,7 +10,7 @@
 /**
  * @covers \Ddeboer\Imap\Server
  */
-class ServerTest extends AbstractTest
+final class ServerTest extends AbstractTest
 {
     public function testValidConnection()
     {