cool#4535 convert-to compare option: add testcase

Fails with the previous commit reverted. Test the RTF output, because
that has markup for redlines and is text-based, so easier to inspect
than PDF.

Signed-off-by: Miklos Vajna <[email protected]>
Change-Id: Ib3aab16a75d0215f3784dad380ae905210703da7

Author: vmiklos

test/UnitConvert.cpp

@@ -58,10 +58,10 @@ class UnitConvert : public UnitWSD
         config.setBool("storage.filesystem[@allow]", false);
     }
 
-    void sendConvertTo(std::unique_ptr<Poco::Net::HTTPClientSession>& session, const std::string& filename, bool isTemplate = false)
+    void sendConvertTo(std::unique_ptr<Poco::Net::HTTPClientSession>& session, const std::string& filename, bool isTemplate = false, bool isCompare = false)
     {
         std::string uri;
-        if (isTemplate)
+        if (isTemplate || isCompare)
         {
             uri = "/cool/convert-to";
         }
@@ -80,6 +80,14 @@ class UnitConvert : public UnitWSD
             std::string templatePath = Poco::Path(TDOC, "template.docx").toString();
             form.addPart("template", new Poco::Net::FilePartSource(templatePath));
         }
+        else if (isCompare)
+        {
+            form.set("format", "rtf");
+            std::string dataPath = Poco::Path(TDOC, filename).toString();
+            form.addPart("data", new Poco::Net::FilePartSource(dataPath));
+            std::string comparePath = Poco::Path(TDOC, "old.docx").toString();
+            form.addPart("compare", new Poco::Net::FilePartSource(comparePath));
+        }
         else
         {
             form.set("format", "txt");
@@ -89,7 +97,7 @@ class UnitConvert : public UnitWSD
         form.write(session->sendRequest(request));
     }
 
-    bool checkConvertTo(std::unique_ptr<Poco::Net::HTTPClientSession>& session, bool isTemplate = false)
+    bool checkConvertTo(std::unique_ptr<Poco::Net::HTTPClientSession>& session, bool isTemplate = false, bool isCompare = false)
     {
         Poco::Net::HTTPResponse response;
         std::stringstream stringStream;
@@ -101,7 +109,7 @@ class UnitConvert : public UnitWSD
         }
 
         bool ret = response.getStatus() == Poco::Net::HTTPResponse::HTTPStatus::HTTP_OK;
-        if (!ret || !isTemplate)
+        if (!ret || !(isTemplate || isCompare))
         {
             if (!ret)
             {
@@ -112,16 +120,33 @@ class UnitConvert : public UnitWSD
         }
 
         std::string responseString = stringStream.str();
-        if (responseString.find("DOCTYPE html") == std::string::npos)
+        if (isTemplate)
         {
-            TST_LOG("checkConvertTo: output is not HTML");
-            return false;
-        }
+            if (responseString.find("DOCTYPE html") == std::string::npos)
+            {
+                TST_LOG("checkConvertTo: output is not HTML");
+                return false;
+            }
 
-        if (responseString.find("background: #156082") == std::string::npos)
+            if (responseString.find("background: #156082") == std::string::npos)
+            {
+                TST_LOG("checkConvertTo: no template color in output");
+                return false;
+            }
+        }
+        else if (isCompare)
         {
-            TST_LOG("checkConvertTo: no template color in output");
-            return false;
+            if (!responseString.starts_with("{\\rtf"))
+            {
+                TST_LOG("checkConvertTo: output is not RTF");
+                return false;
+            }
+
+            if (responseString.find("\\deleted") == std::string::npos)
+            {
+                TST_LOG("checkConvertTo: no old content in output");
+                return false;
+            }
         }
 
         return true;
@@ -164,6 +189,18 @@ class UnitConvert : public UnitWSD
                     return;
                 }
 
+                // Given a current docx + old docx:
+                // When comparing those documents and saving the result as RTF:
+                sendConvertTo(session, "new.docx", /*isTemplate=*/false, /*isCompare=*/true);
+                // Then make sure the output has content from the old document, too:
+                // Without the accompanying fix in place, this test would have failed with:
+                // checkConvertTo: no old content in output
+                if(!checkConvertTo(session, /*isTemplate=*/false, /*isCompare=*/true))
+                {
+                    exitTest(TestResult::Failed);
+                    return;
+                }
+
                 exitTest(TestResult::Ok);
             });
     }