chore: tidy up before merge

pkg/message/parser.go

@@ -91,7 +91,6 @@ func Parse(r io.Reader, key, keyName string) (m *pmapi.Message, mimeBody, plainB
 func convertForeignEncodings(p *parser.Parser) error {
 	logrus.Debug("Converting foreign encodings")
 
-	// HELP: Is it correct to only do this to text types?
 	return p.NewWalker().
 		RegisterContentTypeHandler("text/html", func(p *parser.Part) error {
 			if err := p.ConvertToUTF8(); err != nil {

test/features/bridge/smtp/send/plain.feature

@@ -222,8 +222,6 @@ Feature: SMTP sending of plain messages
       }
       """
 
-	# HELP: This is strange... why do we want this behaviour?
-	@ignore
   Scenario: Message without charset and content is detected as HTML
     When SMTP client sends message
       """
@@ -254,7 +252,7 @@ Feature: SMTP sending of plain messages
           ],
           "CCList": [],
           "BCCList": [],
-          "MIMEType": "text/html"
+          "MIMEType": "text/plain"
         }
       }
       """