fix: Fixes scss-compiler and adds unit test.

Author: jeremyzahner

lib/compilers/scss-compiler.js

@@ -27,7 +27,7 @@ module.exports = (content, filePath, config) => {
   }
   let result
   try {
-    sass.renderSync({
+    result = sass.renderSync({
       data: scssResources + rewriteImports(content, filePath),
       outputStyle: 'compressed'
     }).css.toString()

test/resources/Sass.vue

@@ -4,8 +4,7 @@
 
 <style lang="scss">
   @import "./styles/transitions";
-  @import '~@design'
-
+  @import "~@design";
 
   .testA {
     background-color: red;

test/resources/SassModule.vue

@@ -1,5 +1,10 @@
 <template>
-  <div :class="$style.testA"></div>
+  <div>
+    <div :class="$style.testA"></div>
+    <div :class="[ $style.testA ]"></div>
+    <div :class="{[$style.testB]: true }"></div>
+    <div :class="[$style.testA, { [$style.testB]: true }]"></div>
+  </div>
 </template>
 
 <style module lang="scss">
@@ -8,6 +13,9 @@
   .testA {
     background-color: red;
   }
+  .testB {
+    background-color: blue;
+  }
 </style>
 
 <style module lang="sass">

test/scss.spec.js

@@ -3,12 +3,20 @@ import Sass from './resources/Sass.vue'
 import SassModule from './resources/SassModule.vue'
 
 describe('processes .vue file with scss style', () => {
-  it('does not error on scss or sass', () => {
-    const wrapper = shallow(Sass)
-    expect(wrapper.classes()).toContain('testA')
+  it('does not error on scss/sass', () => {
+    expect(() => shallow(Sass)).not.toThrow()
   })
-
   it('does not error on scss/sass module', () => {
     expect(() => shallow(SassModule)).not.toThrow()
   })
 })
+
+describe('processes .vue files which combine scss/sass and modules', () => {
+  it('does inject classes to $style', () => {
+    const wrapper = shallow(SassModule)
+    expect(wrapper.exists()).toBe(true)
+    expect(wrapper.vm.$style).toBeDefined()
+    expect(wrapper.vm.$style.testA).toBeDefined()
+    expect(wrapper.vm.$style.testB).toBeDefined()
+  })
+})