minifier changes:

daliusd · daliusd · commit d36f98ad8e57 · 2022-05-20T15:51:29.000+03:00
* test fixed

* html-minifier used instead as it comes with mjml
diff --git a/package.json b/package.json
@@ -47,7 +47,6 @@
     "@babel/runtime": "^7.14.6",
     "babel-runtime": "~6.25.0",
     "color": "^3.1.3",
-    "html-minifier-terser": "*",
     "react-reconciler": "^0.26.1"
   },
   "babel": {
diff --git a/src/index.js b/src/index.js
@@ -1,6 +1,6 @@
-import { minify } from 'html-minifier-terser'
 import ReactDOMServer from 'react-dom/server';
 import mjml2html from 'mjml';
+import { minify as htmlMinify } from 'html-minifier';
 
 import { renderToJSON } from './utils/render-to-json';
 import { renderToJSON2 } from './utils/render-to-json2';
@@ -51,16 +51,22 @@ function render(email, options = {}) {
     validationLevel: 'strict',
   };
 
-  const html = mjml2html(renderToMjml(email), { ...defaults, ...options, minify: false });
+  const html = mjml2html(renderToMjml(email), {
+    ...defaults,
+    ...options,
+    minify: undefined,
+  });
 
   if (options.minify) {
-      return minify(html, {
-          caseSensitive: true,
-          collapseWhitespace: true,
-          minifyCSS: true,
-          removeComments: true,
-          removeEmptyAttributes: true
-      });
+    return {
+      html: htmlMinify(html.html, {
+        caseSensitive: true,
+        collapseWhitespace: true,
+        minifyCSS: true,
+        removeComments: true,
+        removeEmptyAttributes: true,
+      }),
+    };
   }
 
   return html;
diff --git a/test/render.spec.js b/test/render.spec.js
@@ -30,7 +30,7 @@ describe('render()', () => {
         </MjmlBody>
       </Mjml>
     );
-    const { html } = render(email);
+    const { html } = render(email, { minify: true });
     expect(html).to.not.be.undefined;
     expect(html).to.contain('<!doctype html>');
     expect(html).to.contain('<title>Title</title>');
