Skip to content

Commit

Permalink
test(optimizer): cover pre bundle css require (#12443)
Browse files Browse the repository at this point in the history
  • Loading branch information
sun0day committed Mar 17, 2023
1 parent 535c8c5 commit 1543236
Show file tree
Hide file tree
Showing 7 changed files with 40 additions and 0 deletions.
14 changes: 14 additions & 0 deletions playground/optimize-deps/__tests__/optimize-deps.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ import {
isBuild,
isServe,
page,
viteTestUrl,
} from '~utils'

test('default + named imports from cjs dep (react)', async () => {
Expand Down Expand Up @@ -200,3 +201,16 @@ test.runIf(isServe)('error on builtin modules usage', () => {
]),
)
})

test('pre bundle css require', async () => {
if (isServe) {
const response = page.waitForResponse(/@vitejs_test-dep-css-require\.js/)
await page.goto(viteTestUrl)
const content = await (await response).text()
expect(content).toMatch(
/import\s"\/@fs.+@vitejs\/test-dep-css-require\/style\.css"/,
)
}

expect(await getColor('.css-require')).toBe('red')
})
1 change: 1 addition & 0 deletions playground/optimize-deps/dep-css-require/index.cjs
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
require('./style.css')
6 changes: 6 additions & 0 deletions playground/optimize-deps/dep-css-require/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
{
"name": "@vitejs/test-dep-css-require",
"private": true,
"version": "0.0.0",
"main": "index.cjs"
}
3 changes: 3 additions & 0 deletions playground/optimize-deps/dep-css-require/style.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
.css-require {
color: red;
}
4 changes: 4 additions & 0 deletions playground/optimize-deps/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -96,6 +96,9 @@ <h2>Flatten Id</h2>
<h2>Non Optimized Module isn't duplicated</h2>
<div class="non-optimized-module-is-not-duplicated"></div>

<h2>Pre bundle css require</h2>
<div class="css-require">css require</div>

<script>
function text(el, text) {
document.querySelector(el).textContent = text
Expand Down Expand Up @@ -123,6 +126,7 @@ <h2>Non Optimized Module isn't duplicated</h2>
}

import '@vitejs/test-dep-cjs-with-assets'
import '@vitejs/test-dep-css-require'

import { env } from '@vitejs/test-dep-node-env'
text('.node-env', env)
Expand Down
1 change: 1 addition & 0 deletions playground/optimize-deps/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"@vitejs/test-dep-cjs-compiled-from-cjs": "file:./dep-cjs-compiled-from-cjs",
"@vitejs/test-dep-cjs-compiled-from-esm": "file:./dep-cjs-compiled-from-esm",
"@vitejs/test-dep-cjs-with-assets": "file:./dep-cjs-with-assets",
"@vitejs/test-dep-css-require": "file:./dep-css-require",
"@vitejs/test-dep-esbuild-plugin-transform": "file:./dep-esbuild-plugin-transform",
"@vitejs/test-dep-linked": "link:./dep-linked",
"@vitejs/test-dep-linked-include": "link:./dep-linked-include",
Expand Down
11 changes: 11 additions & 0 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 1543236

Please sign in to comment.