prefer forEach for test iteration

Co-authored-by: Philippe Rivière <fil@rezo.net>
observablehq · Apr 22, 2024 · 66c01ac · 66c01ac
1 parent 18bbcca
commit 66c01ac
Show file tree

Hide file tree

Showing 7 changed files with 7 additions and 7 deletions.
diff --git a/tests/archives.test.ts b/tests/archives.test.ts
@@ -8,4 +8,4 @@ const archiveTools = [
   { binary: "zstd" },
 ];
 
-archiveTools.map(binaryOnPathTest);
+archiveTools.forEach(binaryOnPathTest);
diff --git a/tests/data-manip.test.ts b/tests/data-manip.test.ts
@@ -6,7 +6,7 @@ const dataManipTools = [
   { binary: "csv2parquet" },
 ];
 
-dataManipTools.map(binaryOnPathTest);
+dataManipTools.forEach(binaryOnPathTest);
 
 binaryVersionTest({
   binary: "duckdb",

diff --git a/tests/dataloader-languages.test.ts b/tests/dataloader-languages.test.ts
@@ -27,4 +27,4 @@ const dataLoaderLanguages = [
   },
 ];
 
-dataLoaderLanguages.map(binaryVersionTest);
+dataLoaderLanguages.forEach(binaryVersionTest);
diff --git a/tests/general-cli.test.ts b/tests/general-cli.test.ts
@@ -10,4 +10,4 @@ const generalCliTools: { binary: string }[] = [
   { binary: "vmstat" },
 ];
 
-generalCliTools.map(binaryOnPathTest);
+generalCliTools.forEach(binaryOnPathTest);
diff --git a/tests/images.test.ts b/tests/images.test.ts
@@ -6,4 +6,4 @@ const imageTools = [
   { binary: "convert", name: "imagemagick" },
 ];
 
-imageTools.map(binaryOnPathTest);
+imageTools.forEach(binaryOnPathTest);
diff --git a/tests/networking.test.ts b/tests/networking.test.ts
@@ -10,4 +10,4 @@ const networkingTools = [
   { binary: "wget" },
 ];
 
-networkingTools.map(binaryOnPathTest);
+networkingTools.forEach(binaryOnPathTest);
diff --git a/tests/text-manip.test.ts b/tests/text-manip.test.ts
@@ -12,4 +12,4 @@ const textManipTools: { binary: string }[] = [
   { binary: "vim" },
 ];
 
-textManipTools.map(binaryOnPathTest);
+textManipTools.forEach(binaryOnPathTest);
-Original file line number
+Diff line change
@@ Expand Up / @@ -27,4 +27,4 @@ const dataLoaderLanguages = [ @@
       },
     ];
-    dataLoaderLanguages.map(binaryVersionTest);
+    dataLoaderLanguages.forEach(binaryVersionTest);