Issue #3463648: Save emails to state immediately, instead of on destruct().

Closes #3463648

tests/modules/symfony_mailer_test/src/MailerTestService.php

@@ -2,7 +2,6 @@
 
 namespace Drupal\symfony_mailer_test;
 
-use Drupal\Core\DestructableInterface;
 use Drupal\Core\State\StateInterface;
 use Drupal\symfony_mailer\EmailInterface;
 use Drupal\symfony_mailer\Processor\EmailProcessorInterface;
@@ -11,7 +10,7 @@ use Drupal\symfony_mailer\Processor\EmailProcessorTrait;
 /**
  * Tracks sent emails for testing.
  */
-class MailerTestService implements MailerTestServiceInterface, EmailProcessorInterface, DestructableInterface {
+class MailerTestService implements MailerTestServiceInterface, EmailProcessorInterface {
 
   use EmailProcessorTrait;
 
@@ -42,15 +41,6 @@ class MailerTestService implements MailerTestServiceInterface, EmailProcessorInt
     }
   }
 
-  /**
-   * {@inheritdoc}
-   */
-  public function destruct() {
-    if ($this->emails) {
-      $this->state->set(self::STATE_KEY, $this->emails);
-    }
-  }
-
   /**
    * {@inheritdoc}
    */
@@ -78,6 +68,7 @@ class MailerTestService implements MailerTestServiceInterface, EmailProcessorInt
    */
   public function postSend(EmailInterface $email) {
     $this->emails[] = $email;
+    $this->state->set(self::STATE_KEY, $this->emails);
   }
 
 }