From 647805a7478ba679435efb7c48ed9ac6fe164a77 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Micha=C3=ABl=20Zasso?= <targos@protonmail.com>
Date: Fri, 18 Feb 2022 16:46:32 +0100
Subject: [PATCH] lib: add FormData global when fetch is enabled

PR-URL: https://github.com/nodejs/node/pull/41956
Backport-PR-URL: https://github.com/nodejs/node/pull/42727
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Robert Nagy <ronagy@icloud.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
---
 .eslintrc.js                            |  1 +
 doc/api/globals.md                      | 11 +++++++++++
 lib/.eslintrc.yaml                      |  4 ++++
 lib/internal/bootstrap/pre_execution.js |  1 +
 test/common/index.js                    |  1 +
 test/parallel/test-fetch.mjs            |  1 +
 test/wpt/test-url.js                    |  9 +++++++++
 tools/doc/type-parser.mjs               |  1 +
 8 files changed, 29 insertions(+)

diff --git a/.eslintrc.js b/.eslintrc.js
index 82579a335d178b..ed78f927782f3f 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -335,6 +335,7 @@ module.exports = {
     atob: 'readable',
     performance: 'readable',
     fetch: 'readable',
+    FormData: 'readable',
     Headers: 'readable',
     Request: 'readable',
     Response: 'readable',
diff --git a/doc/api/globals.md b/doc/api/globals.md
index 2e3b1782f80794..9291a6dfe1636f 100644
--- a/doc/api/globals.md
+++ b/doc/api/globals.md
@@ -306,6 +306,17 @@ added: REPLACEME
 
 A browser-compatible implementation of the [`fetch()`][] function.
 
+## Class `FormData`
+
+<!-- YAML
+added: REPLACEME
+-->
+
+> Stability: 1 - Experimental. Enable this API with the [`--experimental-fetch`][]
+> CLI flag.
+
+A browser-compatible implementation of {FormData}.
+
 ## `global`
 
 <!-- YAML
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
index 91d40ec006edcf..1f2061cfbf09d2 100644
--- a/lib/.eslintrc.yaml
+++ b/lib/.eslintrc.yaml
@@ -43,6 +43,10 @@ rules:
       message: "Use `const { Event } = require('internal/event_target');` instead of the global."
     - name: EventTarget
       message: "Use `const { EventTarget } = require('internal/event_target');` instead of the global."
+    - name: FormData
+      message: "Use `const { FormData } = require('internal/deps/undici/undici');` instead of the global."
+    - name: Headers
+      message: "Use `const { Headers } = require('internal/deps/undici/undici');` instead of the global."
       # Intl is not available in primordials because it can be
       # disabled with --without-intl build flag.
     - name: Intl
diff --git a/lib/internal/bootstrap/pre_execution.js b/lib/internal/bootstrap/pre_execution.js
index 1fe56478e0ab3f..9785393bcf967f 100644
--- a/lib/internal/bootstrap/pre_execution.js
+++ b/lib/internal/bootstrap/pre_execution.js
@@ -157,6 +157,7 @@ function setupFetch() {
 
   const undici = require('internal/deps/undici/undici');
   defineOperation(globalThis, 'fetch', undici.fetch);
+  exposeInterface(globalThis, 'FormData', undici.FormData);
   exposeInterface(globalThis, 'Headers', undici.Headers);
   exposeInterface(globalThis, 'Request', undici.Request);
   exposeInterface(globalThis, 'Response', undici.Response);
diff --git a/test/common/index.js b/test/common/index.js
index 4e671c5a512a1f..e8e90534a939b9 100644
--- a/test/common/index.js
+++ b/test/common/index.js
@@ -296,6 +296,7 @@ if (global.PerformanceMeasure) {
 if (global.fetch) {
   knownGlobals.push(
     global.fetch,
+    global.FormData,
     global.Request,
     global.Response,
     global.Headers,
diff --git a/test/parallel/test-fetch.mjs b/test/parallel/test-fetch.mjs
index fa433c78fc78ef..d435ec3fa6e2b5 100644
--- a/test/parallel/test-fetch.mjs
+++ b/test/parallel/test-fetch.mjs
@@ -7,6 +7,7 @@ import events from 'events';
 import http from 'http';
 
 assert.strictEqual(typeof globalThis.fetch, 'function');
+assert.strictEqual(typeof globalThis.FormData, 'function');
 assert.strictEqual(typeof globalThis.Headers, 'function');
 assert.strictEqual(typeof globalThis.Request, 'function');
 assert.strictEqual(typeof globalThis.Response, 'function');
diff --git a/test/wpt/test-url.js b/test/wpt/test-url.js
index f64189ea3f66d6..18101990a0b2af 100644
--- a/test/wpt/test-url.js
+++ b/test/wpt/test-url.js
@@ -21,6 +21,15 @@ runner.setScriptModifier((obj) => {
     // created via `document.createElement`. So we need to ignore them and just
     // test `URL`.
     obj.code = obj.code.replace(/\["url", "a", "area"\]/, '[ "url" ]');
+  } else if (typeof FormData === 'undefined' &&
+      obj.filename.includes('urlsearchparams-constructor.any.js')) {
+    // TODO(XadillaX): Remove this `else if` after `FormData` is supported.
+
+    // Ignore test named `URLSearchParams constructor, FormData.` because we do
+    // not have `FormData`.
+    obj.code = obj.code.replace(
+      /('URLSearchParams constructor, object\.'\);[\w\W]+)test\(function\(\) {[\w\W]*?}, 'URLSearchParams constructor, FormData\.'\);/,
+      '$1');
   }
 });
 runner.pretendGlobalThisAs('Window');
diff --git a/tools/doc/type-parser.mjs b/tools/doc/type-parser.mjs
index e009b228c9c8da..54426f3ad1602e 100644
--- a/tools/doc/type-parser.mjs
+++ b/tools/doc/type-parser.mjs
@@ -266,6 +266,7 @@ const customTypesMap = {
   'TextDecoderStream':
     'webstreams.md#class-textdecoderstream',
 
+  'FormData': 'https://developer.mozilla.org/en-US/docs/Web/API/FormData',
   'Headers': 'https://developer.mozilla.org/en-US/docs/Web/API/Headers',
   'Response': 'https://developer.mozilla.org/en-US/docs/Web/API/Response',
   'Request': 'https://developer.mozilla.org/en-US/docs/Web/API/Request',