bug #639 bug #638 sass-loader should not do css minification (wimhendrikx)

This PR was squashed before being merged into the master branch (closes #639).

Discussion
----------

bug #638 sass-loader should not do css minification

Fixes a BC break from the sass-loader minifying the CSS which is already done with mini-css-extract-plugin.

I chose to use outputStyle 'expanded' so it's also compatible with Dart Sass.

Commits
-------

f55fb90 Adapt test for new sass-loader config
ae5fea1 bug #638 sass-loader should not do css minification

Author: weaverryan

lib/loaders/sass.js

@@ -41,7 +41,9 @@ module.exports = {
 
         const config = Object.assign({}, {
             // needed by the resolve-url-loader
-            sourceMap: (true === webpackConfig.sassOptions.resolveUrlLoader) || webpackConfig.useSourceMaps
+            sourceMap: (true === webpackConfig.sassOptions.resolveUrlLoader) || webpackConfig.useSourceMaps,
+            // CSS minification is handled with mini-css-extract-plugin
+            outputStyle: 'expanded'
         });
 
         sassLoaders.push({

test/loaders/sass.js

@@ -120,6 +120,7 @@ describe('loaders/sass', () => {
 
         expect(actualLoaders[1].options).to.deep.equals({
             sourceMap: true,
+            outputStyle: 'expanded',
             custom_optiona: 'baz',
             other_option: true
         });