Skip to content

Commit 9a52bf4

Browse files
yyx990803yyx990803
committed
refacotr: rename test method name
1 parent f06e3e7 commit 9a52bf4

File tree

7 files changed

+44
-44
lines changed

7 files changed

+44
-44
lines changed

‎test/advanced.js‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const {
66
mfs,
77
loaderPath,
88
bundle,
9-
test,
9+
mockBundleAndRun,
1010
mockRender
1111
} = require('./shared')
1212

@@ -16,7 +16,7 @@ const SourceMapConsumer = require('source-map').SourceMapConsumer
1616

1717
describe('advanced features', () => {
1818
it('pre/post loaders', done => {
19-
test({
19+
mockBundleAndRun({
2020
entry: 'basic.vue',
2121
vue: {
2222
preLoaders: {
@@ -46,7 +46,7 @@ describe('advanced features', () => {
4646

4747
it('support chaining with other loaders', done => {
4848
const mockLoaderPath = require.resolve('./mock-loaders/js')
49-
test({
49+
mockBundleAndRun({
5050
entry: 'basic.vue',
5151
modify: config => {
5252
config.module.rules[0].loader = loaderPath + '!' + mockLoaderPath
@@ -58,7 +58,7 @@ describe('advanced features', () => {
5858
})
5959

6060
it('expose filename', done => {
61-
test({
61+
mockBundleAndRun({
6262
entry: 'basic.vue'
6363
}, (window, module, rawModule) => {
6464
expect(module.__file).to.equal(path.normalize('test/fixtures/basic.vue'))
@@ -125,7 +125,7 @@ describe('advanced features', () => {
125125
})
126126

127127
it('pre-processors with extract css', done => {
128-
test({
128+
mockBundleAndRun({
129129
entry: 'pre.vue',
130130
vue: {
131131
extractCSS: true
@@ -178,7 +178,7 @@ describe('advanced features', () => {
178178
})
179179

180180
it('multiple rule definitions', done => {
181-
test({
181+
mockBundleAndRun({
182182
modify: config => {
183183
// remove default rule
184184
config.module.rules.shift()
@@ -247,7 +247,7 @@ describe('advanced features', () => {
247247
})
248248

249249
it('thread mode', done => {
250-
test({
250+
mockBundleAndRun({
251251
entry: 'basic.vue',
252252
vue: {
253253
threadMode: true

‎test/basic.js‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ process.env.VUE_LOADER_TEST = true
33
const { expect } = require('chai')
44
const {
55
genId,
6-
test,
6+
mockBundleAndRun,
77
mockRender,
88
initStylesForAllSubComponents
99
} = require('./shared')
@@ -12,7 +12,7 @@ const normalizeNewline = require('normalize-newline')
1212

1313
describe('basic usage', () => {
1414
it('basic', done => {
15-
test({
15+
mockBundleAndRun({
1616
entry: 'basic.vue'
1717
}, (window, module, rawModule) => {
1818
const vnode = mockRender(module, {
@@ -33,7 +33,7 @@ describe('basic usage', () => {
3333
})
3434

3535
it('pre-processors', done => {
36-
test({
36+
mockBundleAndRun({
3737
entry: 'pre.vue'
3838
}, (window, module) => {
3939
const vnode = mockRender(module)
@@ -53,7 +53,7 @@ describe('basic usage', () => {
5353
})
5454

5555
it('style import', done => {
56-
test({
56+
mockBundleAndRun({
5757
entry: 'style-import.vue'
5858
}, (window) => {
5959
const styles = window.document.querySelectorAll('style')
@@ -66,7 +66,7 @@ describe('basic usage', () => {
6666
})
6767

6868
it('style import for a same file twice', done => {
69-
test({
69+
mockBundleAndRun({
7070
entry: 'style-import-twice.vue'
7171
}, (window, module) => {
7272
initStylesForAllSubComponents(module)
@@ -83,7 +83,7 @@ describe('basic usage', () => {
8383
})
8484

8585
it('template import', done => {
86-
test({
86+
mockBundleAndRun({
8787
entry: 'template-import.vue'
8888
}, (window, module) => {
8989
const vnode = mockRender(module)
@@ -95,7 +95,7 @@ describe('basic usage', () => {
9595
})
9696

9797
it('script import', done => {
98-
test({
98+
mockBundleAndRun({
9999
entry: 'script-import.vue'
100100
}, (window, module) => {
101101
expect(module.data().msg).to.contain('Hello from Component A!')

‎test/custom.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ const { expect } = require('chai')
44
const {
55
mfs,
66
bundle,
7-
test,
7+
mockBundleAndRun,
88
mockRender
99
} = require('./shared')
1010

@@ -62,7 +62,7 @@ describe('custom block features', () => {
6262

6363
it('passes Component to custom block loaders', done => {
6464
const mockLoaderPath = require.resolve('./mock-loaders/docs')
65-
test({
65+
mockBundleAndRun({
6666
entry: 'custom-language.vue',
6767
vue: {
6868
loaders: {
@@ -85,7 +85,7 @@ describe('custom block features', () => {
8585
})
8686

8787
it('custom blocks with ES module default export', done => {
88-
test({
88+
mockBundleAndRun({
8989
entry: 'custom-blocks.vue',
9090
vue: {
9191
loaders: {
@@ -129,7 +129,7 @@ describe('custom block features', () => {
129129
})
130130

131131
it('pre/post loaders for custom blocks', done => {
132-
test({
132+
mockBundleAndRun({
133133
entry: 'custom-blocks.vue',
134134
vue: {
135135
preLoaders: {

‎test/script.js‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,13 @@ process.env.VUE_LOADER_TEST = true
22

33
const { expect } = require('chai')
44
const {
5-
test,
5+
mockBundleAndRun,
66
mockRender
77
} = require('./shared')
88

99
describe('script block features', () => {
1010
it('allows to export extended constructor', done => {
11-
test({
11+
mockBundleAndRun({
1212
entry: 'extend.vue'
1313
}, (window, Module) => {
1414
// extend.vue should export Vue constructor
@@ -23,7 +23,7 @@ describe('script block features', () => {
2323
})
2424

2525
it('named exports', done => {
26-
test({
26+
mockBundleAndRun({
2727
entry: 'named-exports.vue'
2828
}, (window, _, rawModule) => {
2929
expect(rawModule.default.name).to.equal('named-exports')

‎test/shared.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ function bundle (options, cb, wontThrowError) {
7878
})
7979
}
8080

81-
function test (options, assert, wontThrowError) {
81+
function mockBundleAndRun (options, assert, wontThrowError) {
8282
bundle(options, (code, stats, err) => {
8383
jsdom.env({
8484
html: '<!DOCTYPE html><html><head></head><body></body></html>',
@@ -132,7 +132,7 @@ module.exports = {
132132
globalConfig,
133133
genId,
134134
bundle,
135-
test,
135+
mockBundleAndRun,
136136
mockRender,
137137
interopDefault,
138138
initStylesForAllSubComponents

‎test/style.js‎

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,15 @@ const path = require('path')
55
const { expect } = require('chai')
66
const {
77
genId,
8-
test,
8+
mockBundleAndRun,
99
mockRender
1010
} = require('./shared')
1111

1212
const normalizeNewline = require('normalize-newline')
1313

1414
describe('style block features', () => {
1515
it('scoped style', done => {
16-
test({
16+
mockBundleAndRun({
1717
entry: 'scoped-css.vue'
1818
}, (window, module) => {
1919
const id = 'data-v-' + genId('scoped-css.vue')
@@ -58,7 +58,7 @@ describe('style block features', () => {
5858
})
5959

6060
it('media-query', done => {
61-
test({
61+
mockBundleAndRun({
6262
entry: 'media-query.vue'
6363
}, (window) => {
6464
let style = window.document.querySelector('style').textContent
@@ -70,7 +70,7 @@ describe('style block features', () => {
7070
})
7171

7272
it('supports-query', done => {
73-
test({
73+
mockBundleAndRun({
7474
entry: 'supports-query.vue'
7575
}, (window) => {
7676
let style = window.document.querySelector('style').textContent
@@ -82,7 +82,7 @@ describe('style block features', () => {
8282
})
8383

8484
it('postcss options', done => {
85-
test({
85+
mockBundleAndRun({
8686
entry: 'postcss.vue',
8787
vue: {
8888
postcss: {
@@ -101,7 +101,7 @@ describe('style block features', () => {
101101

102102
it('load postcss config file', done => {
103103
fs.writeFileSync('.postcssrc', JSON.stringify({ parser: 'sugarss' }))
104-
test({
104+
mockBundleAndRun({
105105
entry: 'postcss.vue'
106106
}, (window) => {
107107
let style = window.document.querySelector('style').textContent
@@ -114,7 +114,7 @@ describe('style block features', () => {
114114

115115
it('load cascading postcss config file', done => {
116116
fs.writeFileSync('.postcssrc', JSON.stringify({ parser: 'sugarss' }))
117-
test({
117+
mockBundleAndRun({
118118
entry: 'sub/postcss-cascade.vue',
119119
vue: { postcss: { cascade: true }}
120120
}, (window) => {
@@ -128,7 +128,7 @@ describe('style block features', () => {
128128

129129
it('load postcss config file by path', done => {
130130
fs.writeFileSync('test/.postcssrc', JSON.stringify({ parser: 'sugarss' }))
131-
test({
131+
mockBundleAndRun({
132132
entry: 'postcss.vue',
133133
vue: {
134134
postcss: {
@@ -148,7 +148,7 @@ describe('style block features', () => {
148148

149149
it('load postcss config file with js syntax error', done => {
150150
fs.writeFileSync('.postcssrc.js', 'module.exports = { hello }')
151-
test({
151+
mockBundleAndRun({
152152
entry: 'basic.vue'
153153
}, (window, module, vueModule, instance, jsdomErr, webpackInfo) => {
154154
const { stats: { compilation: { warnings, errors }}, err } = webpackInfo
@@ -163,7 +163,7 @@ describe('style block features', () => {
163163
})
164164

165165
it('postcss lang', done => {
166-
test({
166+
mockBundleAndRun({
167167
entry: 'postcss-lang.vue'
168168
}, (window) => {
169169
let style = window.document.querySelector('style').textContent
@@ -175,7 +175,7 @@ describe('style block features', () => {
175175

176176
it('css-modules', done => {
177177
function testWithIdent (localIdentName, regexToMatch, cb) {
178-
test({
178+
mockBundleAndRun({
179179
entry: 'css-modules.vue',
180180
vue: {
181181
cssModules: localIdentName && {

‎test/template.js‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,15 +3,15 @@ process.env.VUE_LOADER_TEST = true
33
const path = require('path')
44
const { expect } = require('chai')
55
const {
6-
test,
6+
mockBundleAndRun,
77
mockRender
88
} = require('./shared')
99

1010
const normalizeNewline = require('normalize-newline')
1111

1212
describe('template block features', () => {
1313
it('template with comments', done => {
14-
test({
14+
mockBundleAndRun({
1515
entry: 'template-comment.vue'
1616
}, (window, module, rawModule) => {
1717
expect(module.comments).to.equal(true)
@@ -29,7 +29,7 @@ describe('template block features', () => {
2929
})
3030

3131
it('transpile ES2015 features in template', done => {
32-
test({
32+
mockBundleAndRun({
3333
entry: 'es2015.vue'
3434
}, (window, module) => {
3535
const vnode = mockRender(module, {
@@ -45,7 +45,7 @@ describe('template block features', () => {
4545
})
4646

4747
it('translates relative URLs and respects resolve alias', done => {
48-
test({
48+
mockBundleAndRun({
4949
entry: 'resolve.vue',
5050
resolve: {
5151
alias: {
@@ -76,7 +76,7 @@ describe('template block features', () => {
7676
})
7777

7878
it('transformToRequire option', done => {
79-
test({
79+
mockBundleAndRun({
8080
entry: 'transform.vue',
8181
module: {
8282
rules: [
@@ -116,7 +116,7 @@ describe('template block features', () => {
116116
})
117117

118118
it('should allow adding custom html loaders', done => {
119-
test({
119+
mockBundleAndRun({
120120
entry: 'markdown.vue',
121121
vue: {
122122
loaders: {
@@ -136,7 +136,7 @@ describe('template block features', () => {
136136
})
137137

138138
it('custom compiler modules', done => {
139-
test({
139+
mockBundleAndRun({
140140
entry: 'custom-module.vue',
141141
vue: {
142142
compilerModules: [
@@ -168,7 +168,7 @@ describe('template block features', () => {
168168
})
169169

170170
it('custom compiler directives', done => {
171-
test({
171+
mockBundleAndRun({
172172
entry: 'custom-directive.vue',
173173
vue: {
174174
compilerDirectives: {
@@ -191,7 +191,7 @@ describe('template block features', () => {
191191
})
192192

193193
it('functional component with styles', done => {
194-
test({
194+
mockBundleAndRun({
195195
entry: 'functional-style.vue'
196196
}, (window, module, rawModule) => {
197197
expect(module.functional).to.equal(true)
@@ -209,7 +209,7 @@ describe('template block features', () => {
209209
})
210210

211211
it('functional template', done => {
212-
test({
212+
mockBundleAndRun({
213213
entry: 'functional-root.vue',
214214
vue: {
215215
preserveWhitespace: false

0 commit comments

Comments
 (0)