Merge pull request #38 from stephanoskomnenos/fix-global-bindings

Fix incorrect bindings in `Global`

Author: nojaf

src/Global.res

@@ -505,7 +505,6 @@ let response = await fetch(myRequest)
 
 [Read more on MDN](https://developer.mozilla.org/docs/Web/API/Window/fetch)
 */
-@send
 external fetch_withRequest: (request, ~init: requestInit=?) => promise<response> = "fetch"
 
 /**
@@ -567,7 +566,7 @@ Removes the event listener in target's event listener list with the same type, c
 [Read more on MDN](https://developer.mozilla.org/docs/Web/API/EventTarget/removeEventListener)
 */
 external removeEventListener: (
-  string,
+  eventType,
   eventListener<'event>,
   ~options: eventListenerOptions=?,
 ) => unit = "removeEventListener"

tests/Global__test.js

@@ -14,3 +14,18 @@ let response2 = await fetch(
     body: BodyInit.fromString(`secret=foo&response=bar`),
   },
 )
+
+let response3 = await fetch_withRequest(
+  Request.make2(~input="https://rescript-lang.org/"),
+  ~init={
+    method: "POST",
+    headers: HeadersInit.fromDict(
+      dict{
+        "Content-Type": "application/x-www-form-urlencoded",
+      },
+    ),
+    body: BodyInit.fromString(`secret=foo&response=bar`),
+  },
+)
+
+removeEventListener(Mousedown, MouseEvent.preventDefault, ~options={capture: false})