✅ Add fieldformatMap field to test edge cases

elastic · Oct 22, 2020 · 7a93683 · 7a93683
1 parent 04b391a
commit 7a93683
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/loader.test.ts b/x-pack/plugins/lens/public/indexpattern_datasource/loader.test.ts
@@ -117,6 +117,7 @@ const indexPattern2 = ({
   title: 'my-fake-restricted-pattern',
   timeFieldName: 'timestamp',
   hasRestrictions: true,
+  fieldFormatMap: {},
   fields: [
     {
       name: 'timestamp',

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/loader.ts b/x-pack/plugins/lens/public/indexpattern_datasource/loader.ts
@@ -106,7 +106,9 @@ export async function loadIndexPatterns({
         fieldFormatMap:
           fieldFormatMap &&
           Object.fromEntries(
-            Object.entries(fieldFormatMap).map(([id, format]) => [id, format.toJSON()])
+            Object.entries(fieldFormatMap)
+              .filter(([id, format]) => format)
+              .map(([id, format]) => [id, format.toJSON()])
           ),
         fields: newFields,
         hasRestrictions: !!typeMeta?.aggs,

diff --git a/x-pack/plugins/lens/public/indexpattern_datasource/mocks.ts b/x-pack/plugins/lens/public/indexpattern_datasource/mocks.ts
@@ -82,6 +82,7 @@ export const createMockedRestrictedIndexPattern = () => ({
   title: 'my-fake-restricted-pattern',
   timeFieldName: 'timestamp',
   hasRestrictions: true,
+  fieldFormatMap: { bytes: null },
   fields: [
     {
       name: 'timestamp',