Fix eslint warnings (#2151)

Co-authored-by: Bartek Iwańczuk <biwanczuk@gmail.com>
Co-authored-by: LE GOFF Vincent <g_n_s@hotmail.fr>

1 files changed, 57 insertions, 51 deletions

diff --git a/js/url_search_params_test.ts b/js/url_search_params_test.ts
index b93e4f4b0..9b20b3b75 100644
--- a/js/url_search_params_test.ts
+++ b/js/url_search_params_test.ts
@@ -1,7 +1,7 @@
 // Copyright 2018-2019 the Deno authors. All rights reserved. MIT license.
 import { test, assert, assertEquals } from "./test_util.ts";
 
-test(function urlSearchParamsInitString() {
+test(function urlSearchParamsInitString(): void {
   const init = "c=4&a=2&b=3&%C3%A1=1";
   const searchParams = new URLSearchParams(init);
   assert(
@@ -10,32 +10,32 @@ test(function urlSearchParamsInitString() {
   );
 });
 
-test(function urlSearchParamsInitIterable() {
+test(function urlSearchParamsInitIterable(): void {
   const init = [["a", "54"], ["b", "true"]];
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.toString(), "a=54&b=true");
 });
 
-test(function urlSearchParamsInitRecord() {
+test(function urlSearchParamsInitRecord(): void {
   const init = { a: "54", b: "true" };
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.toString(), "a=54&b=true");
 });
 
-test(function urlSearchParamsAppendSuccess() {
+test(function urlSearchParamsAppendSuccess(): void {
   const searchParams = new URLSearchParams();
   searchParams.append("a", "true");
   assertEquals(searchParams.toString(), "a=true");
 });
 
-test(function urlSearchParamsDeleteSuccess() {
+test(function urlSearchParamsDeleteSuccess(): void {
   const init = "a=54&b=true";
   const searchParams = new URLSearchParams(init);
   searchParams.delete("b");
   assertEquals(searchParams.toString(), "a=54");
 });
 
-test(function urlSearchParamsGetAllSuccess() {
+test(function urlSearchParamsGetAllSuccess(): void {
   const init = "a=54&b=true&a=true";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.getAll("a"), ["54", "true"]);
@@ -43,7 +43,7 @@ test(function urlSearchParamsGetAllSuccess() {
   assertEquals(searchParams.getAll("c"), []);
 });
 
-test(function urlSearchParamsGetSuccess() {
+test(function urlSearchParamsGetSuccess(): void {
   const init = "a=54&b=true&a=true";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.get("a"), "54");
@@ -51,7 +51,7 @@ test(function urlSearchParamsGetSuccess() {
   assertEquals(searchParams.get("c"), null);
 });
 
-test(function urlSearchParamsHasSuccess() {
+test(function urlSearchParamsHasSuccess(): void {
   const init = "a=54&b=true&a=true";
   const searchParams = new URLSearchParams(init);
   assert(searchParams.has("a"));
@@ -59,62 +59,64 @@ test(function urlSearchParamsHasSuccess() {
   assert(!searchParams.has("c"));
 });
 
-test(function urlSearchParamsSetReplaceFirstAndRemoveOthers() {
+test(function urlSearchParamsSetReplaceFirstAndRemoveOthers(): void {
   const init = "a=54&b=true&a=true";
   const searchParams = new URLSearchParams(init);
   searchParams.set("a", "false");
   assertEquals(searchParams.toString(), "a=false&b=true");
 });
 
-test(function urlSearchParamsSetAppendNew() {
+test(function urlSearchParamsSetAppendNew(): void {
   const init = "a=54&b=true&a=true";
   const searchParams = new URLSearchParams(init);
   searchParams.set("c", "foo");
   assertEquals(searchParams.toString(), "a=54&b=true&a=true&c=foo");
 });
 
-test(function urlSearchParamsSortSuccess() {
+test(function urlSearchParamsSortSuccess(): void {
   const init = "c=4&a=2&b=3&a=1";
   const searchParams = new URLSearchParams(init);
   searchParams.sort();
   assertEquals(searchParams.toString(), "a=2&a=1&b=3&c=4");
 });
 
-test(function urlSearchParamsForEachSuccess() {
+test(function urlSearchParamsForEachSuccess(): void {
   const init = [["a", "54"], ["b", "true"]];
   const searchParams = new URLSearchParams(init);
   let callNum = 0;
-  searchParams.forEach((value, key, parent) => {
-    assertEquals(searchParams, parent);
-    assertEquals(value, init[callNum][1]);
-    assertEquals(key, init[callNum][0]);
-    callNum++;
-  });
+  searchParams.forEach(
+    (value, key, parent): void => {
+      assertEquals(searchParams, parent);
+      assertEquals(value, init[callNum][1]);
+      assertEquals(key, init[callNum][0]);
+      callNum++;
+    }
+  );
   assertEquals(callNum, init.length);
 });
 
-test(function urlSearchParamsMissingName() {
+test(function urlSearchParamsMissingName(): void {
   const init = "=4";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.get(""), "4");
   assertEquals(searchParams.toString(), "=4");
 });
 
-test(function urlSearchParamsMissingValue() {
+test(function urlSearchParamsMissingValue(): void {
   const init = "4=";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.get("4"), "");
   assertEquals(searchParams.toString(), "4=");
 });
 
-test(function urlSearchParamsMissingEqualSign() {
+test(function urlSearchParamsMissingEqualSign(): void {
   const init = "4";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.get("4"), "");
   assertEquals(searchParams.toString(), "4=");
 });
 
-test(function urlSearchParamsMissingPair() {
+test(function urlSearchParamsMissingPair(): void {
   const init = "c=4&&a=54&";
   const searchParams = new URLSearchParams(init);
   assertEquals(searchParams.toString(), "c=4&a=54");
@@ -122,7 +124,7 @@ test(function urlSearchParamsMissingPair() {
 
 // If pair does not contain exactly two items, then throw a TypeError.
 // ref https://url.spec.whatwg.org/#interface-urlsearchparams
-test(function urlSearchParamsShouldThrowTypeError() {
+test(function urlSearchParamsShouldThrowTypeError(): void {
   let hasThrown = 0;
 
   try {
@@ -139,40 +141,44 @@ test(function urlSearchParamsShouldThrowTypeError() {
   assertEquals(hasThrown, 2);
 });
 
-test(function urlSearchParamsAppendArgumentsCheck() {
+test(function urlSearchParamsAppendArgumentsCheck(): void {
   const methodRequireOneParam = ["delete", "getAll", "get", "has", "forEach"];
 
   const methodRequireTwoParams = ["append", "set"];
 
-  methodRequireOneParam.concat(methodRequireTwoParams).forEach(method => {
-    const searchParams = new URLSearchParams();
-    let hasThrown = 0;
-    try {
-      searchParams[method]();
-      hasThrown = 1;
-    } catch (err) {
-      if (err instanceof TypeError) {
-        hasThrown = 2;
-      } else {
-        hasThrown = 3;
+  methodRequireOneParam.concat(methodRequireTwoParams).forEach(
+    (method: string): void => {
+      const searchParams = new URLSearchParams();
+      let hasThrown = 0;
+      try {
+        searchParams[method]();
+        hasThrown = 1;
+      } catch (err) {
+        if (err instanceof TypeError) {
+          hasThrown = 2;
+        } else {
+          hasThrown = 3;
+        }
       }
+      assertEquals(hasThrown, 2);
     }
-    assertEquals(hasThrown, 2);
-  });
-
-  methodRequireTwoParams.forEach(method => {
-    const searchParams = new URLSearchParams();
-    let hasThrown = 0;
-    try {
-      searchParams[method]("foo");
-      hasThrown = 1;
-    } catch (err) {
-      if (err instanceof TypeError) {
-        hasThrown = 2;
-      } else {
-        hasThrown = 3;
+  );
+
+  methodRequireTwoParams.forEach(
+    (method: string): void => {
+      const searchParams = new URLSearchParams();
+      let hasThrown = 0;
+      try {
+        searchParams[method]("foo");
+        hasThrown = 1;
+      } catch (err) {
+        if (err instanceof TypeError) {
+          hasThrown = 2;
+        } else {
+          hasThrown = 3;
+        }
       }
+      assertEquals(hasThrown, 2);
     }
-    assertEquals(hasThrown, 2);
-  });
+  );
 });