mirror of
https://github.com/hanzoai/next.js.git
synced 2026-09-24 17:00:51 +00:00
Jiachi LiuandZack Tanner
78c2533e1f
### What Inherit the client boudaries from the found matched target in load barrel ### Why The root cause with the barrel transform, we missed the client boundary directive during the transform. Since the new version of mui's case looks like this: Import path page.js (server module) -> `<module>/index.js` (shared module) -> `<module/subpath>/index.js` (client module) -> `<module/subpath/sub-module.js> (client module) After our transform, we lost the `"use client"` which causes the mismatch of the transform: In `rsc` layer: the file pointing to the sub module entry (`<module/subpath>/index.js`), but without the client boundary. Fixes #64369 Closes NEXT-3101
49 lines
1.5 KiB
TypeScript
49 lines
1.5 KiB
TypeScript
import { join } from 'path'
|
|
import { createNextDescribe } from 'e2e-utils'
|
|
import { hasRedbox } from 'next-test-utils'
|
|
// Skipped in Turbopack, will be added later.
|
|
;(process.env.TURBOPACK ? describe.skip : describe)(
|
|
'Skipped in Turbopack',
|
|
() => {
|
|
createNextDescribe(
|
|
'optimizePackageImports - mui',
|
|
{
|
|
env: {
|
|
NEXT_TEST_MODE: '1',
|
|
},
|
|
files: join(__dirname, 'fixture'),
|
|
|
|
dependencies: {
|
|
'@mui/material': '5.15.15',
|
|
'@emotion/react': '11.11.1',
|
|
'@emotion/styled': '11.11.0',
|
|
},
|
|
},
|
|
({ next }) => {
|
|
it('should support MUI', async () => {
|
|
let logs = ''
|
|
next.on('stdout', (log) => {
|
|
logs += log
|
|
})
|
|
|
|
// Ensure that MUI is working
|
|
const $ = await next.render$('/mui')
|
|
expect(await $('#button').text()).toContain('button')
|
|
expect(await $('#typography').text()).toContain('typography')
|
|
|
|
const browser = await next.browser('/mui')
|
|
expect(await hasRedbox(browser)).toBe(false)
|
|
|
|
const modules = [...logs.matchAll(/\((\d+) modules\)/g)]
|
|
expect(modules.length).toBeGreaterThanOrEqual(1)
|
|
for (const [, moduleCount] of modules) {
|
|
// Ensure that the number of modules is less than 1500 - otherwise we're
|
|
// importing the entire library.
|
|
expect(parseInt(moduleCount)).toBeLessThan(1500)
|
|
}
|
|
})
|
|
}
|
|
)
|
|
}
|
|
)
|