fix: mime type checking

lykmapipo · Sep 12, 2019 · a47a799 · a47a799
1 parent bcaca97
commit a47a799
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/message.model.js b/lib/message.model.js
@@ -724,7 +724,7 @@ MessageSchema.methods.preValidate = function preValidate(next) {
  * @instance
  */
 MessageSchema.methods.isHtml = function _isHtml() {
-  return (this.mime = MIME_HTML) || isHtml(this.body);
+  return (this.mime === MIME_HTML) || isHtml(this.body);
 };
 
 /**

diff --git a/test/integration/message.send.smtp.spec.js b/test/integration/message.send.smtp.spec.js
@@ -58,6 +58,8 @@ describe('smtp transport', () => {
           expect(sent).to.exist;
           expect(sent._id).to.exist;
           expect(sent.transport).to.be.equal('smtp');
+          expect(sent.mime).to.be.equal('text/plain');
+          expect(sent.isHtml()).to.be.false;
           expect(sent.sentAt).to.exist;
           expect(sent.deliveredAt).to.exist;
           expect(sent.failedAt).to.not.exist;