MailingPreview - We don't need the [CiviMail Draft] prefix for in-b…

…rowser previews
civicrm · Oct 3, 2021 · 405d20b · 405d20b
1 parent 5744a82
commit 405d20b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/ext/flexmailer/src/API/MailingPreview.php b/ext/flexmailer/src/API/MailingPreview.php
@@ -57,7 +57,6 @@ public function save() {
 
     };
     $job->mailing_id = $mailing->id ?: NULL;
-    $job->is_test = 1;
     $job->status = 'Complete';
 
     $flexMailer = new FlexMailer(array(

diff --git a/ext/flexmailer/tests/phpunit/Civi/FlexMailer/MailingPreviewTest.php b/ext/flexmailer/tests/phpunit/Civi/FlexMailer/MailingPreviewTest.php
@@ -82,7 +82,7 @@ public function testMailerPreview(): void {
     $this->assertMaxIds($maxIDs);
 
     $previewResult = $result['values'][$result['id']]['api.Mailing.preview'];
-    $this->assertEquals("[CiviMail Draft] Hello $displayName",
+    $this->assertEquals("Hello $displayName",
       $previewResult['values']['subject']);
 
     $this->assertStringContainsString("This is $displayName", $previewResult['values']['body_text']);
@@ -111,7 +111,7 @@ public function testMailerPreviewWithoutId(): void {
     $previewResult = $this->callAPISuccess('mailing', 'preview', $params);
     $this->assertMaxIds($maxIDs);
 
-    $this->assertEquals("[CiviMail Draft] Hello $displayName",
+    $this->assertEquals("Hello $displayName",
       $previewResult['values']['subject']);
 
     $this->assertStringContainsString("This is $displayName", $previewResult['values']['body_text']);