Add explainer for weird fail case in mailtemplatetest and added alternate test

Signed-off-by: Claudio Cambra <claudio.cambra@gmail.com>

src/mail/mime/tests/mailtemplatetest.cpp

@@ -593,12 +593,21 @@ private Q_SLOTS:
 
         QCOMPARE(result->contentType()->mimeType(), QByteArray{"multipart/alternative"});
 
-        QCOMPARE(result->attachments().size(), 1);
+        // FIXME: KMime should return 0 for this multipart but it doesn't sometimes(?)
+        // It is an attachment as the ical part:
+        // A) Has a name
+        // B) Has a content disposition of CDattachment
+        //
+        // Check the result of QVERIFY(KMime::isAttachment(result->contents()[1])); if
+        // it is true then this confirms that result->attachments() is misreporting
+        QEXPECT_FAIL("", "Should be recognised as an attachment", TestFailMode::Continue);
+        QCOMPARE(result->attachments().count(), 1);
 
         QCOMPARE(result->contents().size(), 2);
         QVERIFY(result->contents()[0]->contentType()->isMimeType("text/plain"));
         QVERIFY(result->contents()[1]->contentType()->isMimeType("text/calendar"));
         QCOMPARE(result->contents()[1]->contentType()->name(), QLatin1String{"event.ics"});
+        QVERIFY(KMime::isAttachment(result->contents()[1]));
     }
 
     void testEncryptedWithProtectedHeadersReply()