pass a filepath to svgo.optimize()

in order to support prefixIds svgo rule. @see: svg/svgo#700
evoactivity · Aug 2, 2018 · c5f59f0 · c5f59f0
1 parent 9131619
commit c5f59f0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/packages/broccoli-svg-optimizer/index.js b/packages/broccoli-svg-optimizer/index.js
@@ -52,9 +52,9 @@ class SVGOFilter extends PersistentFilter {
     this.optionsHash = stringify(options);
   }
 
-  processString(svg) {
+  processString(svg, relativePath) {
     return svg
-      ? this.optimize(svg).then(({ data }) => data)
+      ? this.optimize(svg, { path: relativePath }).then(({ data }) => data)
       : Promise.resolve('');
   }
 

diff --git a/packages/broccoli-svg-optimizer/tests/index-test.js b/packages/broccoli-svg-optimizer/tests/index-test.js
@@ -133,4 +133,24 @@ describe('broccoli-svg-optimizer', () => {
     let outputNode = fixture.build(new SVGOptimizer(inputNode, options));
     return expect(outputNode).to.be.rejectedWith('promise error');
   });
+
+  it('passes file to optimize', () => {
+    class CustomSVGO {
+      optimize(svg, options) {
+        return Promise.resolve({ data: options && options.path });
+      }
+    }
+
+    let options = {
+      svgoConfig: {},
+      svgoModule: CustomSVGO,
+      persist: false
+    };
+
+    let outputNode = fixture.build(new SVGOptimizer(inputNode, options));
+
+    return expect(outputNode).to.eventually.deep.equal({
+      'test.svg': 'test.svg'
+    });
+  });
 });