fix(svg): svgo@3 now throws error

- cleanupIDs -> cleanupIds
- https://github.com/svg/svgo/releases/tag/v3.0.0

README.md

@@ -133,7 +133,7 @@ minify:
     # Retain comments
     removeComments: false
     # Do not remove unused ID attributes
-    cleanupIDs: false
+    cleanupIds: false
   ```
   - For more options, see [svgo](https://github.com/svg/svgo).
 - **globOptions** - See [globbing](#globbing) section.

lib/filter.js

@@ -152,12 +152,16 @@ function minifySvg () {
       assetPath.on('data', (chunk) => (assetTxt += chunk))
       assetPath.on('end', async () => {
         if (assetTxt.length) {
-          const { data, error } = svgOptimize(assetTxt, { ...options, plugins })
-          if (data) {
-            if (verbose) logFn.call(this, assetTxt, data, path, 'svg')
-            resolve(route.set(path, data))
-          } else if (error) {
-            reject(new Error(`Path: ${path}\n${error}`))
+          try {
+            const { data } = svgOptimize(assetTxt, { ...options, plugins })
+            if (data) {
+              if (verbose) logFn.call(this, assetTxt, data, path, 'svg')
+              resolve(route.set(path, data))
+            } else {
+              reject(new Error(`Path: ${path}\nEmpty svg input`))
+            }
+          } catch (err) {
+            reject(new Error(`Path: ${path}\n${err}`))
           }
         }
         resolve()

test/svg.test.js

@@ -71,7 +71,7 @@ describe('svg', () => {
 
   test('option', async () => {
     const customOpt = {
-      cleanupIDs: false
+      cleanupIds: false
     }
     hexo.config.minify.svg.plugins = customOpt
     plugins = [{
@@ -117,10 +117,14 @@ describe('svg', () => {
     const input = '{}'
     hexo.route.set(path, input)
 
-    const { error } = svgOptimize(input, { plugins })
-
-    expect(error).toBeDefined()
-    await expect(s()).rejects.toThrow(`Path: ${path}\n${error}`)
+    let expected
+    try {
+      svgOptimize(input, { plugins })
+    } catch (err) {
+      expected = err
+    }
+    expect(expected).toBeDefined()
+    await expect(s()).rejects.toThrow(`Path: ${path}\n${expected}`)
   })
 
   test('include - exclude *.min.svg by default', async () => {