Enable TS strict mode by default (#3899)

Fixes #3324 

Co-authored-by: Kitson Kelly <me@kitsonkelly.com>

1 files changed, 15 insertions, 2 deletions

diff --git a/cli/js/form_data_test.ts b/cli/js/form_data_test.ts
index a40326fba..10147d8b9 100644
--- a/cli/js/form_data_test.ts
+++ b/cli/js/form_data_test.ts
@@ -36,7 +36,9 @@ test(function formDataParamsGetSuccess(): void {
   formData.append("a", "true");
   formData.append("b", "false");
   formData.append("a", "null");
+  // @ts-ignore
   formData.append("d", undefined);
+  // @ts-ignore
   formData.append("e", null);
   assertEquals(formData.get("a"), "true");
   assertEquals(formData.get("b"), "false");
@@ -63,8 +65,10 @@ test(function formDataParamsSetSuccess(): void {
   assertEquals(formData.getAll("b"), ["false"]);
   formData.set("a", "false");
   assertEquals(formData.getAll("a"), ["false"]);
+  // @ts-ignore
   formData.set("d", undefined);
   assertEquals(formData.get("d"), "undefined");
+  // @ts-ignore
   formData.set("e", null);
   assertEquals(formData.get("e"), "null");
 });
@@ -101,15 +105,22 @@ test(function formDataParamsForEachSuccess(): void {
 });
 
 test(function formDataParamsArgumentsCheck(): void {
-  const methodRequireOneParam = ["delete", "getAll", "get", "has", "forEach"];
+  const methodRequireOneParam = [
+    "delete",
+    "getAll",
+    "get",
+    "has",
+    "forEach"
+  ] as const;
 
-  const methodRequireTwoParams = ["append", "set"];
+  const methodRequireTwoParams = ["append", "set"] as const;
 
   methodRequireOneParam.forEach((method): void => {
     const formData = new FormData();
     let hasThrown = 0;
     let errMsg = "";
     try {
+      // @ts-ignore
       formData[method]();
       hasThrown = 1;
     } catch (err) {
@@ -133,6 +144,7 @@ test(function formDataParamsArgumentsCheck(): void {
     let errMsg = "";
 
     try {
+      // @ts-ignore
       formData[method]();
       hasThrown = 1;
     } catch (err) {
@@ -152,6 +164,7 @@ test(function formDataParamsArgumentsCheck(): void {
     hasThrown = 0;
     errMsg = "";
     try {
+      // @ts-ignore
       formData[method]("foo");
       hasThrown = 1;
     } catch (err) {