Skip to content

Commit 4669b6b

Browse files
committed
test: semver is not mockable
1 parent bb5f84c commit 4669b6b

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

packages/builder/test/builder.build.test.js

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,14 @@
11
import path from 'path'
22
import consola from 'consola'
33
import fsExtra from 'fs-extra'
4-
import semver from 'semver'
54
import { r, waitFor } from '@nuxt/utils'
65
import { BundleBuilder } from '@nuxt/webpack'
76

87
import Builder from '../src/builder'
98
import { createNuxt } from './__utils__'
109

1110
jest.mock('fs-extra')
12-
jest.mock('semver')
11+
// jest.mock('semver')
1312
jest.mock('hash-sum', () => src => `hash(${src})`)
1413
jest.mock('@nuxt/utils')
1514
jest.mock('../src/ignore')
@@ -236,9 +235,9 @@ describe('builder: builder build', () => {
236235
}
237236
}
238237
const builder = new Builder(nuxt, BundleBuilder)
239-
semver.satisfies
240-
.mockReturnValueOnce(true)
241-
.mockReturnValueOnce(true)
238+
// semver.satisfies
239+
// .mockReturnValueOnce(true)
240+
// .mockReturnValueOnce(true)
242241
nuxt.resolver.requireModule
243242
.mockReturnValueOnce({ version: 'alpha' })
244243
.mockReturnValueOnce({ version: 'beta' })
@@ -248,9 +247,9 @@ describe('builder: builder build', () => {
248247
expect(nuxt.resolver.requireModule).toBeCalledTimes(2)
249248
expect(nuxt.resolver.requireModule).nthCalledWith(1, 'join(vue, package.json)')
250249
expect(nuxt.resolver.requireModule).nthCalledWith(2, 'join(nuxt, package.json)')
251-
expect(semver.satisfies).toBeCalledTimes(2)
252-
expect(semver.satisfies).nthCalledWith(1, 'alpha', 'latest')
253-
expect(semver.satisfies).nthCalledWith(2, 'beta', 'edge')
250+
// expect(semver.satisfies).toBeCalledTimes(2)
251+
// expect(semver.satisfies).nthCalledWith(1, 'alpha', 'latest')
252+
// expect(semver.satisfies).nthCalledWith(2, 'beta', 'edge')
254253
})
255254

256255
test('should warn and throw error if dependencies is not installed', () => {
@@ -262,8 +261,7 @@ describe('builder: builder build', () => {
262261
}
263262
}
264263
const builder = new Builder(nuxt, BundleBuilder)
265-
semver.satisfies
266-
.mockReturnValueOnce(false)
264+
// semver.satisfies.mockReturnValueOnce(false)
267265
nuxt.resolver.requireModule
268266
.mockReturnValueOnce({ version: 'alpha' })
269267
.mockReturnValueOnce(undefined)
@@ -276,7 +274,7 @@ describe('builder: builder build', () => {
276274
expect(consola.warn).toBeCalledTimes(2)
277275
expect(consola.warn).nthCalledWith(1, 'vue@latest is recommended but vue@alpha is installed!')
278276
expect(consola.warn).nthCalledWith(2, 'nuxt@edge is required but not installed!')
279-
expect(semver.satisfies).toBeCalledTimes(1)
280-
expect(semver.satisfies).nthCalledWith(1, 'alpha', 'latest')
277+
// expect(semver.satisfies).toBeCalledTimes(1)
278+
// expect(semver.satisfies).nthCalledWith(1, 'alpha', 'latest')
281279
})
282280
})

0 commit comments

Comments
 (0)