Skip to content

Commit

Permalink
chore: revert unrelative codes
Browse files Browse the repository at this point in the history
  • Loading branch information
ygj6 committed Nov 13, 2021
1 parent a953c78 commit 44fb17c
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 69 deletions.
78 changes: 25 additions & 53 deletions packages/vite/src/node/__tests__/build.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -69,56 +69,40 @@ describe('resolveLibFilename', () => {

describe('resolvePaths', () => {
test('resolve build.rollupOptions.input', async () => {
const config = await resolveConfig(
{
build: {
rollupOptions: {
input: 'index.html'
}
const config = await resolveConfig({
build: {
rollupOptions: {
input: 'index.html'
}
},
'build',
'production'
)
}
}, 'build', 'production')

expect(config.build.rollupOptions.input).toBe(resolve('index.html'))
})
test('resolve build.rollupOptions.input{}', async () => {
const config = await resolveConfig(
{
build: {
rollupOptions: {
input: {
index: 'index.html'
}
const config = await resolveConfig({
build: {
rollupOptions: {
input: {
index: 'index.html'
}
}
},
'build',
'production'
)
}
}, 'build', 'production')

expect(config.build.rollupOptions.input['index']).toBe(
resolve('index.html')
)
expect(config.build.rollupOptions.input['index']).toBe(resolve('index.html'))
})

test('resolve build.rollupOptions.input[]', async () => {
const config = await resolveConfig(
{
build: {
rollupOptions: {
input: ['index.html']
}
const config = await resolveConfig({
build: {
rollupOptions: {
input: ['index.html']
}
},
'build',
'production'
)
}
}, 'build', 'production')

expect(config.build.rollupOptions.input).toStrictEqual([
resolve('index.html')
])
expect(config.build.rollupOptions.input).toStrictEqual([resolve('index.html')])
})

test('resolve index.html', async () => {
Expand All @@ -128,11 +112,7 @@ describe('resolvePaths', () => {
})

test('resolve build.outdir', async () => {
const config = await resolveConfig(
{ build: { outDir: 'outDir' } },
'build',
'production'
)
const config = await resolveConfig({ build: { outDir: 'outDir' } }, 'build', 'production')

expect(config.build.outDir).toBe(resolve('outDir'))
})
Expand All @@ -144,22 +124,14 @@ describe('resolvePaths', () => {
})

test('resolve build.lib.entry', async () => {
const config = await resolveConfig(
{ build: { lib: { entry: 'index.html' } } },
'build',
'production'
)
const config = await resolveConfig({ build: { lib: { entry: 'index.html' } } }, 'build', 'production')

expect(config.build.rollupOptions.input).toBe(resolve('index.html'))
})

test('resolve build.ssr', async () => {
const config = await resolveConfig(
{ build: { ssr: 'ssr.ts' } },
'build',
'production'
)
const config = await resolveConfig({ build: { ssr: 'ssr.ts' } }, 'build', 'production')

expect(config.build.rollupOptions.input).toBe(resolve('ssr.ts'))
})
})
})
16 changes: 5 additions & 11 deletions packages/vite/src/node/build.ts
Original file line number Diff line number Diff line change
Expand Up @@ -233,10 +233,7 @@ export type ResolvedBuildOptions = Required<
>
>

export function resolveBuildOptions(
root: string,
raw?: BuildOptions
): ResolvedBuildOptions {
export function resolveBuildOptions(root: string, raw?: BuildOptions): ResolvedBuildOptions {
const resolved: ResolvedBuildOptions = {
target: 'modules',
polyfillModulePreload: true,
Expand Down Expand Up @@ -272,22 +269,19 @@ export function resolveBuildOptions(
}
}

const resolve = (p: string) =>
p.startsWith('\0') ? p : path.resolve(root, p)
const resolve = (p: string) => p.startsWith('\0') ? p : path.resolve(root, p)

resolved.outDir = resolve(resolved.outDir)

let input

if (raw?.rollupOptions?.input) {
input = Array.isArray(raw.rollupOptions.input)
? raw.rollupOptions.input.map((input) => resolve(input))
? raw.rollupOptions.input.map(input => resolve(input))
: typeof raw.rollupOptions.input === 'object'
? Object.assign(
// @ts-ignore
...Object.keys(raw.rollupOptions.input).map((key) => ({
[key]: resolve(raw.rollupOptions.input[key])
}))
...Object.keys(raw.rollupOptions.input).map(key => ({ [key]: resolve(raw.rollupOptions.input[key]) }))
)
: resolve(raw.rollupOptions.input)
} else {
Expand Down Expand Up @@ -842,4 +836,4 @@ function wrapSsrTransform(fn: Function | undefined) {

function injectSsrFlag(options: any = {}) {
return { ...options, ssr: true }
}
}
7 changes: 2 additions & 5 deletions packages/vite/src/node/plugins/css.ts
Original file line number Diff line number Diff line change
Expand Up @@ -218,10 +218,7 @@ export function cssPlugin(config: ResolvedConfig): Plugin {
: await moduleGraph.ensureEntryFromUrl(
(
await fileToUrl(file, config, this)
).replace(
(config.server?.origin ?? '') + config.base,
'/'
)
).replace((config.server?.origin ?? '') + config.base, '/')
)
)
}
Expand Down Expand Up @@ -1279,4 +1276,4 @@ const preProcessors = Object.freeze({

function isPreProcessor(lang: any): lang is PreprocessLang {
return lang && lang in preProcessors
}
}

0 comments on commit 44fb17c

Please sign in to comment.