fix error when use alias for nuxt-edge (#6051)

This commit is contained in:
Xin Du (Clark) 2019-07-10 10:57:31 +01:00 committed by Pooya Parsa
parent 2b86e31ab2
commit e63923419a
2 changed files with 13 additions and 8 deletions

View File

@ -1,20 +1,25 @@
import fs from 'fs' import fs from 'fs'
import path from 'path'
import execa from 'execa' import execa from 'execa'
import { name as pkgName } from '../package.json' import { name as pkgName } from '../package.json'
import NuxtCommand from './command' import NuxtCommand from './command'
import setup from './setup' import setup from './setup'
import getCommand from './commands' import getCommand from './commands'
function checkDuplicateNuxt() { function packageExists(name) {
const dupPkg = pkgName === '@nuxt/cli' ? 'cli-edge' : 'cli' try {
if (fs.existsSync(path.resolve(__dirname, '..', '..', dupPkg))) { require.resolve(name)
throw new Error('Both `nuxt` and `nuxt-edge` are installed! This is unsupported, please choose one and remove the other one from dependencies.') return true
} catch (e) {
return false
} }
} }
export default async function run(_argv) { export default async function run(_argv) {
checkDuplicateNuxt() // Check for not installing both nuxt and nuxt-edge
const dupPkg = '@nuxt/' + (pkgName === '@nuxt/cli-edge' ? 'cli' : 'cli-edge')
if (packageExists(dupPkg)) {
throw new Error('Both `nuxt` and `nuxt-edge` dependencies are installed! This is unsupported, please choose one and remove the other one from dependencies.')
}
// Read from process.argv // Read from process.argv
const argv = _argv ? Array.from(_argv) : process.argv.slice(2) const argv = _argv ? Array.from(_argv) : process.argv.slice(2)

View File

@ -1,12 +1,12 @@
import run from '../../src/run' import run from '../../src/run'
jest.mock('../../package.json', () => ({ jest.mock('../../package.json', () => ({
name: 'cli-edge' name: '@nuxt/cli-edge'
})) }))
describe('run in edge', () => { describe('run in edge', () => {
test('throws error if nuxt and nuxt-edge are installed', async () => { test('throws error if nuxt and nuxt-edge are installed', async () => {
await expect(run()) await expect(run())
.rejects.toThrow('Both `nuxt` and `nuxt-edge` are installed! This is unsupported, please choose one and remove the other one from dependencies.') .rejects.toThrow('Both `nuxt` and `nuxt-edge` dependencies are installed! This is unsupported, please choose one and remove the other one from dependencies.')
}) })
}) })