add test coverage

Author: rajumsys

tests/specs/test-mailer.http.class.php

@@ -203,7 +203,8 @@ function test_get_request_body_without_template() {
 
     NSA::setProperty($this->mailer, 'settings', [
       'enable_tracking' => true,
-      'transactional' => false
+      'transactional' => false,
+      'template' => ''
     ]);
 
     $header_to = 'abc <abc@xyz.com>';
@@ -259,6 +260,27 @@ function test_get_request_body_without_template() {
     $this->assertTrue($expected_request_body == $actual);
   }
 
+  function test_get_request_body_template_in_hook_but_not_in_settings() {
+    $this->mailer->addAddress('abc@xyz.com', 'abc');
+    $this->mailer->setFrom( 'me@hello.com', 'me');
+
+    $callback = function(){
+      return 'test-template'; 
+    };
+
+    add_filter('wpsp_template_id', $callback);
+
+    NSA::setProperty($this->mailer, 'settings', [
+      'enable_tracking' => true,
+      'transactional' => false,
+      'template' => ''
+    ]);
+
+    $body = NSA::invokeMethod($this->mailer, 'get_request_body');
+    remove_filter('wpsp_template_id', $callback);
+    $this->assertTrue($body['content']['template_id'] == 'test-template');
+  }
+
   function test_get_request_body_with_template() {
     $this->mailer->addAddress('abc@xyz.com', 'abc');
     $this->mailer->addBcc('bcc@xyz.com', 'bcc');
@@ -304,7 +326,7 @@ function test_get_request_body_with_template() {
         'content' => '',
         'subject' => '',
         'from_name' => 'me',
-        'from' => 'me <me@hello.com>',
+        'from' => 'me@hello.com',
         'from_localpart'  => 'me'
       ]
     ];