Add optional resolve-url-loader options

Author: Diego

lib/WebpackConfig.js

@@ -115,7 +115,14 @@ class WebpackConfig {
         // Features/Loaders options
         this.copyFilesConfigs = [];
         this.sassOptions = {
-            resolveUrlLoader: true
+            resolveUrlLoader: true,
+            resolveUrlLoaderOptions: {
+                engine: 'postcss',
+                keepQuery: false,
+                removeCR: false,
+                debug: false,
+                silent: false
+            }
         };
         this.preactOptions = {
             preactCompat: false

lib/loaders/sass.js

@@ -32,7 +32,7 @@ module.exports = {
                 loader: 'resolve-url-loader',
                 options: {
                     sourceMap: webpackConfig.useSourceMaps,
-                    removeCR: true
+                    ...webpackConfig.sassOptions.resolveUrlLoaderOptions
                 }
             });
         }

package.json

@@ -43,7 +43,7 @@
     "optimize-css-assets-webpack-plugin": "^5.0.1",
     "pkg-up": "^1.0.0",
     "pretty-error": "^2.1.1",
-    "resolve-url-loader": "^3.1.0",
+    "resolve-url-loader": "^3.0.1",
     "semver": "^5.5.0",
     "style-loader": "^0.21.0",
     "terser-webpack-plugin": "^1.1.0",

test/loaders/sass.js

@@ -65,6 +65,26 @@ describe('loaders/sass', () => {
         cssLoader.getLoaders.restore();
     });
 
+    it('getLoaders() with resolve-url-loader options', () => {
+        const config = createConfig();
+        config.enableSassLoader(() => {}, {
+            resolveUrlLoaderOptions: {
+                removeCR: true
+            }
+        });
+
+        // make the cssLoader return nothing
+        sinon.stub(cssLoader, 'getLoaders')
+            .callsFake(() => []);
+
+        const actualLoaders = sassLoader.getLoaders(config);
+        expect(actualLoaders).to.have.lengthOf(2);
+        expect(actualLoaders[0].loader).to.equal('resolve-url-loader');
+        expect(actualLoaders[0].options.removeCR).to.be.true;
+
+        cssLoader.getLoaders.restore();
+    });
+
     it('getLoaders() without resolve-url-loader', () => {
         const config = createConfig();
         config.enableSassLoader(() => {}, {