test: add name re-export from client components as page case (#66760)

### What Add new test case where a named export from client component is being exported as page ### Why We found this case while investigating the errors triggered introduced by #66286 , adding this test to avoid future regression
2024-06-11 20:42:10 +02:00 · 2024-06-11 20:42:10 +02:00 · 62e8c9dd45
commit 62e8c9dd45
parent b5d0a67912
7 changed files with 30 additions and 1 deletions
--- a/test/e2e/app-dir/app-routes-client-component/app/layout.js
+++ b/test/e2e/app-dir/app-routes-client-component/app/layout.js
@ -0,0 +1,7 @@
+export default function RootLayout({ children }) {
+  return (
+    <html>
+      <body>{children}</body>
+    </html>
+  )
+}
--- a/test/e2e/app-dir/rsc-basic/app/reexport-named/client.js
+++ b/test/e2e/app-dir/rsc-basic/app/reexport-named/client.js
@ -0,0 +1,9 @@
+'use client'
+
+export function PageComponent() {
+  return (
+    <div>
+      <h1 id="client-title">Client Title</h1>
+    </div>
+  )
+}
--- a/test/e2e/app-dir/rsc-basic/app/reexport-named/page.js
+++ b/test/e2e/app-dir/rsc-basic/app/reexport-named/page.js
@ -0,0 +1,3 @@
+import { PageComponent } from './client'
+
+export default PageComponent
--- a/test/e2e/app-dir/rsc-basic/app/reexport-namespace/client-index.js
+++ b/test/e2e/app-dir/rsc-basic/app/reexport-namespace/client-index.js
--- a/test/e2e/app-dir/rsc-basic/app/reexport-namespace/client.js
+++ b/test/e2e/app-dir/rsc-basic/app/reexport-namespace/client.js
@ -1,5 +1,5 @@
 'use client'

 export function Foo() {
-  return <div>Foo</div>
+  return <div id="foo">Foo</div>
 }
--- a/test/e2e/app-dir/rsc-basic/app/reexport-namespace/page.js
+++ b/test/e2e/app-dir/rsc-basic/app/reexport-namespace/page.js
--- a/test/e2e/app-dir/rsc-basic/rsc-basic.test.ts
+++ b/test/e2e/app-dir/rsc-basic/rsc-basic.test.ts
@ -109,6 +109,16 @@ describe('app dir - rsc basics', () => {
    expect($('#return-undefined-layout').html()).toBeEmpty()
  })

+  it('should handle named client components imported as page', async () => {
+    const $ = await next.render$('/reexport-named')
+    expect($('#client-title').text()).toBe('Client Title')
+  })
+
+  it('should handle client components imported as namespace', async () => {
+    const $ = await next.render$('/reexport-namespace')
+    expect($('#foo').text()).toBe('Foo')
+  })
+
  it('should render server components correctly', async () => {
    const homeHTML = await next.render('/', null, {
      headers: {