test: refactor tests

Author: yyx990803

Diff for: test/core.spec.js

@@ -10,7 +10,7 @@ const normalizeNewline = require('normalize-newline')
 test('basic', done => {
   mockBundleAndRun({
     entry: 'basic.vue'
-  }, ({ window, module, rawModule }) => {
+  }, ({ window, module }) => {
     const vnode = mockRender(module, {
       msg: 'hi'
     })

Diff for: test/edgeCases.spec.js

@@ -95,3 +95,31 @@ test('normalize multiple use + options', done => {
     }
   }, () => done(), true)
 })
+
+test('should not duplicate css modules value imports', done => {
+  mockBundleAndRun({
+    entry: './test/fixtures/duplicate-cssm.js',
+    modify: config => {
+      config.module.rules[1] = {
+        test: /\.css$/,
+        use: [
+          'vue-style-loader',
+          {
+            loader: 'css-loader',
+            options: {
+              modules: true
+            }
+          }
+        ]
+      }
+    }
+  }, ({ window, module, code }) => {
+    const localsRE = /exports.locals = {\s+"color": "red"\s+};/
+    const matches = code.match(localsRE)
+    expect(matches.length).toBe(1)
+
+    const styles = window.getComputedStyle(window.document.querySelector('h1'))
+    console.log(styles)
+    done()
+  })
+})

Diff for: test/fixtures/entry.js

@@ -1,7 +1,9 @@
-const Component = require('~target')
+import Component from '~target'
+import * as exports from '~target'
 
 if (typeof window !== 'undefined') {
-  window.vueModule = Component
+  window.module = Component
+  window.exports = exports
 }
 
-module.exports = Component
+export default Component

Diff for: test/script.spec.js

@@ -22,9 +22,9 @@ test('allow exporting extended constructor', done => {
 test('named exports', done => {
   mockBundleAndRun({
     entry: 'named-exports.vue'
-  }, ({ window, rawModule }) => {
-    expect(rawModule.default.name).toBe('named-exports')
-    expect(rawModule.foo()).toBe(1)
+  }, ({ exports }) => {
+    expect(exports.default.name).toBe('named-exports')
+    expect(exports.foo()).toBe(1)
     done()
   })
 })

Diff for: test/template.spec.js

@@ -8,7 +8,7 @@ const {
 test('template with comments', done => {
   mockBundleAndRun({
     entry: 'template-comment.vue'
-  }, ({ window, module, rawModule }) => {
+  }, ({ window, module }) => {
     expect(module.comments).toBe(true)
     const vnode = mockRender(module, {
       msg: 'hi'
@@ -113,7 +113,7 @@ test('transform srcset', done => {
 test('functional component with styles', done => {
   mockBundleAndRun({
     entry: 'functional-style.vue'
-  }, ({ window, module, rawModule }) => {
+  }, ({ window, module }) => {
     expect(module.functional).toBe(true)
     const vnode = mockRender(module)
     // <div class="foo">hi</div>

Diff for: test/utils.js

@@ -108,16 +108,15 @@ function mockBundleAndRun (options, assert, wontThrowError) {
     }
 
     const { window } = dom
-    const rawModule = window.vueModule
-    const module = interopDefault(window.vueModule)
+    const { module, exports } = window
     const instance = {}
     if (module && module.beforeCreate) {
       module.beforeCreate.forEach(hook => hook.call(instance))
     }
     assert({
       window,
       module,
-      rawModule,
+      exports,
       instance,
       code,
       jsdomError,