Add test for hidden form fields at the beginning of a Server Action f…

…orm (#50978) Fixes #50087. The root cause was fixed in upstream React and here's a test to ensure it works end to end. fix NEXT-1228
vercel · Jun 8, 2023 · b8e308f · b8e308f
1 parent 9a9d095
commit b8e308f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/test/e2e/app-dir/actions/app-action.test.ts b/test/e2e/app-dir/actions/app-action.test.ts
@@ -114,7 +114,7 @@ createNextDescribe(
 
       await check(() => {
         return browser.eval('window.location.pathname + window.location.search')
-      }, '/header?name=test&constructor=FormData')
+      }, '/header?name=test&constructor=FormData&hidden-info=hi')
     })
 
     it('should support .bind', async () => {

diff --git a/test/e2e/app-dir/actions/app/server/form.js b/test/e2e/app-dir/actions/app/server/form.js
@@ -6,7 +6,9 @@ async function action(formData) {
     '/header?name=' +
       formData.get('name') +
       '&constructor=' +
-      formData.constructor.name
+      formData.constructor.name +
+      '&hidden-info=' +
+      formData.get('hidden-info')
   )
 }
 
@@ -43,6 +45,7 @@ export default function Form() {
     <>
       <hr />
       <form action={action}>
+        <input type="text" name="hidden-info" value="hi" hidden />
         <input type="text" name="name" id="name" required />
         <button type="submit" id="submit">
           Submit