Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: dependencies are analyzed multiple times #3154

Merged
merged 1 commit into from Apr 27, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
@@ -0,0 +1,9 @@
import { getColor, untilUpdated } from '../../testUtils'

test('should have css applied on second dynamic import', async () => {
await untilUpdated(() => page.textContent('.content'), 'Initial', true)
await page.click('.b')

await untilUpdated(() => page.textContent('.content'), 'Reference', true)
expect(await getColor('.content')).toBe('red')
})
1 change: 1 addition & 0 deletions packages/playground/multiple-entrypoints/deps.json
@@ -0,0 +1 @@
{"a0":"./a0.js","a1":"./a1.js","a2":"./a2.js","a3":"./a3.js","a4":"./a4.js","a5":"./a5.js","a6":"./a6.js","a7":"./a7.js","a8":"./a8.js","a9":"./a9.js","a10":"./a10.js","a11":"./a11.js","a12":"./a12.js","a13":"./a13.js","a14":"./a14.js","a15":"./a15.js","a16":"./a16.js","a17":"./a17.js","a18":"./a18.js","a19":"./a19.js","a20":"./a20.js","a21":"./a21.js","a22":"./a22.js","a23":"./a23.js","a24":"./a24.js"}
1 change: 1 addition & 0 deletions packages/playground/multiple-entrypoints/dynamic-a.js
@@ -0,0 +1 @@
import './reference'
1 change: 1 addition & 0 deletions packages/playground/multiple-entrypoints/dynamic-b.js
@@ -0,0 +1 @@
import './reference'
57 changes: 57 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a0.js
@@ -0,0 +1,57 @@
import a1 from './a1'
import a2 from './a2'
import a3 from './a3'
import a4 from './a4'
import a5 from './a5'
import a6 from './a6'
import a7 from './a7'
import a8 from './a8'
import a9 from './a9'
import a10 from './a10'
import a11 from './a11'
import a12 from './a12'
import a13 from './a13'
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export function other() {
return (
a1() +
a2() +
a3() +
a4() +
a5() +
a6() +
a7() +
a8() +
a9() +
a10() +
a11() +
a12() +
a13() +
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
57 changes: 57 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a1.js
@@ -0,0 +1,57 @@
import a2 from './a2'
import a3 from './a3'
import a4 from './a4'
import a5 from './a5'
import a6 from './a6'
import a7 from './a7'
import a8 from './a8'
import a9 from './a9'
import a10 from './a10'
import a11 from './a11'
import a12 from './a12'
import a13 from './a13'
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a0.js')

export function other() {
return (
a2() +
a3() +
a4() +
a5() +
a6() +
a7() +
a8() +
a9() +
a10() +
a11() +
a12() +
a13() +
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
39 changes: 39 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a10.js
@@ -0,0 +1,39 @@
import a11 from './a11'
import a12 from './a12'
import a13 from './a13'
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a9.js')

export function other() {
return (
a11() +
a12() +
a13() +
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
37 changes: 37 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a11.js
@@ -0,0 +1,37 @@
import a12 from './a12'
import a13 from './a13'
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a10.js')

export function other() {
return (
a12() +
a13() +
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
35 changes: 35 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a12.js
@@ -0,0 +1,35 @@
import a13 from './a13'
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a11.js')

export function other() {
return (
a13() +
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
33 changes: 33 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a13.js
@@ -0,0 +1,33 @@
import a14 from './a14'
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a12.js')

export function other() {
return (
a14() +
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
31 changes: 31 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a14.js
@@ -0,0 +1,31 @@
import a15 from './a15'
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a13.js')

export function other() {
return (
a15() +
a16() +
a17() +
a18() +
a19() +
a20() +
a21() +
a22() +
a23() +
a24()
)
}

export default function () {
return 123
}
19 changes: 19 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a15.js
@@ -0,0 +1,19 @@
import a16 from './a16'
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a14.js')

export function other() {
return a16() + a17() + a18() + a19() + a20() + a21() + a22() + a23() + a24()
}

export default function () {
return 123
}
18 changes: 18 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a16.js
@@ -0,0 +1,18 @@
import a17 from './a17'
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a15.js')

export function other() {
return a17() + a18() + a19() + a20() + a21() + a22() + a23() + a24()
}

export default function () {
return 123
}
17 changes: 17 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a17.js
@@ -0,0 +1,17 @@
import a18 from './a18'
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a16.js')

export function other() {
return a18() + a19() + a20() + a21() + a22() + a23() + a24()
}

export default function () {
return 123
}
16 changes: 16 additions & 0 deletions packages/playground/multiple-entrypoints/entrypoints/a18.js
@@ -0,0 +1,16 @@
import a19 from './a19'
import a20 from './a20'
import a21 from './a21'
import a22 from './a22'
import a23 from './a23'
import a24 from './a24'

export const that = () => import('./a17.js')

export function other() {
return a19() + a20() + a21() + a22() + a23() + a24()
}

export default function () {
return 123
}