fix(csv-sample): prevent topKey exception (#28)

SocialGouv · Apr 15, 2022 · 34e3dcc · 34e3dcc
1 parent 9fe4b5d
commit 34e3dcc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/packages/csv-sample/src/index.ts b/packages/csv-sample/src/index.ts
@@ -74,7 +74,7 @@ const guessColumnsTypes = (samples: Sample[], onProgress: onProgressFunction) =>
         const detectedType = topKey(detectedTypes, "type");
         return {
           ...sample,
-          type: detectedType,
+          type: detectedType || "unknown",
         };
       }
       return {

diff --git a/packages/csv-sample/src/utils.ts b/packages/csv-sample/src/utils.ts
@@ -22,7 +22,7 @@ const getRandomItems = (arr: any[], n: number) => {
 };
 
 // extract the most present value for a given list of records and a given key
-export const topKey = (arr: Row[], key: string): string => {
+export const topKey = (arr: Row[], key: string): string | undefined => {
   if (arr.length === 1) {
     return arr[0][key];
   }
@@ -41,8 +41,9 @@ export const topKey = (arr: Row[], key: string): string => {
       return a[1] - b[1];
     })
     .reverse();
-
-  return totals[0][0];
+  if (totals.length && totals[0].length) {
+    return totals[0][0];
+  }
 };
 
 export const wait = (args: any) =>