replace faker with @faker-js

elastic · Nov 22, 2024 · 5419268 · 5419268
1 parent 3938acc
commit 5419268
Show file tree

Hide file tree

Showing 23 changed files with 101 additions and 106 deletions.
diff --git a/package.json b/package.json
@@ -1563,7 +1563,6 @@
     "@types/eslint": "^8.44.2",
     "@types/express": "^4.17.21",
     "@types/extract-zip": "^1.6.2",
-    "@types/faker": "^5.1.5",
     "@types/fetch-mock": "^7.3.1",
     "@types/file-saver": "^2.0.0",
     "@types/flot": "^0.0.31",
@@ -1731,7 +1730,6 @@
     "expect": "^29.7.0",
     "expose-loader": "^0.7.5",
     "express": "^4.21.1",
-    "faker": "^5.1.0",
     "fetch-mock": "^7.3.9",
     "file-loader": "^4.2.0",
     "find-cypress-specs": "^1.41.4",

diff --git a/packages/kbn-calculate-width-from-char-count/src/calculate_width_from_entries.test.ts b/packages/kbn-calculate-width-from-char-count/src/calculate_width_from_entries.test.ts
@@ -9,24 +9,22 @@
 
 import { calculateWidthFromEntries } from './calculate_width_from_entries';
 import { MAX_WIDTH } from './calculate_width_from_char_count';
-import faker from 'faker';
-
-const generateLabel = (length: number) => faker.random.alpha({ count: length });
+import { faker } from '@faker-js/faker';
 
 const generateObjectWithLabelOfLength = (length: number, propOverrides?: Record<string, any>) => ({
-  label: generateLabel(length),
+  label: faker.string.alpha(length),
   ...propOverrides,
 });
 
 describe('calculateWidthFromEntries', () => {
   it('calculates width for array of strings', () => {
-    const shortLabels = [10, 20].map(generateLabel);
+    const shortLabels = [10, 20].map(faker.string.alpha);
     expect(calculateWidthFromEntries(shortLabels)).toBe(256);
 
-    const mediumLabels = [50, 55, 10, 20].map(generateLabel);
+    const mediumLabels = [50, 55, 10, 20].map(faker.string.alpha);
     expect(calculateWidthFromEntries(mediumLabels)).toBe(501);
 
-    const longLabels = [80, 90, 10].map(generateLabel);
+    const longLabels = [80, 90, 10].map(faker.string.alpha);
     expect(calculateWidthFromEntries(longLabels)).toBe(MAX_WIDTH);
   });
 
@@ -42,12 +40,12 @@ describe('calculateWidthFromEntries', () => {
   });
   it('calculates width for array of objects for fallback keys', () => {
     const shortLabels = [10, 20].map((v) =>
-      generateObjectWithLabelOfLength(v, { label: undefined, name: generateLabel(v) })
+      generateObjectWithLabelOfLength(v, { label: undefined, name: faker.string.alpha(v) })
     );
     expect(calculateWidthFromEntries(shortLabels, ['id', 'label', 'name'])).toBe(256);
 
     const mediumLabels = [50, 55, 10, 20].map((v) =>
-      generateObjectWithLabelOfLength(v, { label: undefined, name: generateLabel(v) })
+      generateObjectWithLabelOfLength(v, { label: undefined, name: faker.string.alpha(v) })
     );
     expect(calculateWidthFromEntries(mediumLabels, ['id', 'label', 'name'])).toBe(501);
   });

diff --git a/packages/kbn-dom-drag-drop/src/test_utils.tsx b/packages/kbn-dom-drag-drop/src/test_utils.tsx
@@ -8,7 +8,7 @@
  */
 
 import React, { ReactElement } from 'react';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import { RenderOptions, render } from '@testing-library/react';
 import { DragContextState, RootDragDropProvider } from './providers';
 
@@ -22,7 +22,7 @@ export const dataTransfer = {
 };
 
 export const generateDragDropValue = (label = faker.lorem.word()) => ({
-  id: faker.random.uuid(),
+  id: faker.string.uuid(),
   humanData: {
     label,
     groupLabel: faker.lorem.word(),

diff --git a/packages/kbn-visualization-ui-components/components/field_picker/field_picker.test.tsx b/packages/kbn-visualization-ui-components/components/field_picker/field_picker.test.tsx
@@ -10,15 +10,15 @@
 import React from 'react';
 import { FieldPicker, FieldPickerProps } from './field_picker';
 import { render, screen } from '@testing-library/react';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import userEvent from '@testing-library/user-event';
 import { DataType, FieldOptionValue } from './types';
 
 const generateFieldWithLabelOfLength = (length: number) => ({
-  label: faker.random.alpha({ count: length }),
+  label: faker.string.alpha(length),
   value: {
     type: 'field' as const,
-    field: faker.random.alpha({ count: length }),
+    field: faker.string.alpha(length),
     dataType: 'date' as DataType,
     operationType: 'count',
   },

diff --git a/renovate.json b/renovate.json
@@ -137,12 +137,10 @@
       "groupName": "@elastic/appex-qa dependencies",
       "matchDepNames": [
         "cheerio",
-        "@types/faker",
         "@types/pixelmatch",
         "@types/pngjs",
         "@types/supertest",
         "babel-plugin-istanbul",
-        "faker",
         "nyc",
         "oboe",
         "pixelmatch",
@@ -1303,10 +1301,21 @@
     },
     {
       "groupName": "@elastic/request-converter",
-      "matchDepNames": ["@elastic/request-converter"],
-      "reviewers": ["team:kibana-management"],
-      "matchBaseBranches": ["main"],
-      "labels": ["release_note:skip", "backport:skip", "Team:Kibana Management", "Feature:Console"],
+      "matchDepNames": [
+        "@elastic/request-converter"
+      ],
+      "reviewers": [
+        "team:kibana-management"
+      ],
+      "matchBaseBranches": [
+        "main"
+      ],
+      "labels": [
+        "release_note:skip",
+        "backport:skip",
+        "Team:Kibana Management",
+        "Feature:Console"
+      ],
       "enabled": true
     }
   ],

diff --git a/src/plugins/chart_expressions/expression_metric/public/components/metric_vis.test.tsx b/src/plugins/chart_expressions/expression_metric/public/components/metric_vis.test.tsx
@@ -27,7 +27,7 @@ import { CustomPaletteState } from '@kbn/charts-plugin/common/expressions/palett
 import { DimensionsVisParam, MetricVisParam } from '../../common';
 import { euiThemeVars } from '@kbn/ui-theme';
 import { DEFAULT_TRENDLINE_NAME } from '../../common/constants';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const mockDeserialize = jest.fn(({ id }: { id: string }) => {
   const convertFn = (v: unknown) => `${id}-${v}`;
@@ -825,47 +825,47 @@ describe('MetricVisComponent', function () {
       // Raw values here, not formatted
       const trends: Record<string, MetricWTrend['trend']> = {
         Friday: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         Wednesday: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         Saturday: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         Sunday: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         Thursday: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         __other__: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
         // this one shouldn't show up!
         [DEFAULT_TRENDLINE_NAME]: [
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
-          { x: faker.random.number(), y: faker.random.number() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
+          { x: faker.number.int(), y: faker.number.int() },
         ],
       };
 

diff --git a/src/plugins/chart_expressions/expression_partition_vis/public/utils/filter_helpers.test.ts b/src/plugins/chart_expressions/expression_partition_vis/public/utils/filter_helpers.test.ts
@@ -18,7 +18,7 @@ import {
 import { createMockBucketColumns, createMockVisData, createMockPieParams } from '../mocks';
 import { consolidateMetricColumns } from '../../common/utils';
 import { LayerValue } from '@elastic/charts';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const bucketColumns = createMockBucketColumns();
 const visData = createMockVisData();
@@ -186,26 +186,26 @@ describe('getFilterClickData', () => {
         const clickedLayers: LayerValue[] = [
           {
             groupByRollup: 'circle',
-            value: faker.random.number(),
-            depth: faker.random.number(),
+            value: faker.number.int(),
+            depth: faker.number.int(),
             path: [],
-            sortIndex: faker.random.number(),
+            sortIndex: faker.number.int(),
             smAccessorValue: '',
           },
           {
             groupByRollup: 'green',
-            value: faker.random.number(),
-            depth: faker.random.number(),
+            value: faker.number.int(),
+            depth: faker.number.int(),
             path: [],
-            sortIndex: faker.random.number(),
+            sortIndex: faker.number.int(),
             smAccessorValue: '',
           },
           {
             groupByRollup: 'metric2',
-            value: faker.random.number(),
-            depth: faker.random.number(),
+            value: faker.number.int(),
+            depth: faker.number.int(),
             path: [],
-            sortIndex: faker.random.number(),
+            sortIndex: faker.number.int(),
             smAccessorValue: '',
           },
         ];

diff --git a/x-pack/plugins/lens/public/datasources/form_based/operations/definitions/percentile.test.tsx b/x-pack/plugins/lens/public/datasources/form_based/operations/definitions/percentile.test.tsx
@@ -12,7 +12,7 @@ import { unifiedSearchPluginMock } from '@kbn/unified-search-plugin/public/mocks
 import { IStorageWrapper } from '@kbn/kibana-utils-plugin/public';
 import { dataPluginMock } from '@kbn/data-plugin/public/mocks';
 import { dataViewPluginMocks } from '@kbn/data-views-plugin/public/mocks';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { createMockedIndexPattern } from '../../mocks';
@@ -402,7 +402,7 @@ describe('percentile', () => {
     it('should update order-by references for any terms columns', () => {
       const field1 = 'foo';
       const field2 = 'bar';
-      const percentile = faker.random.number(100);
+      const percentile = faker.number.int(100);
 
       const aggs = [
         makeEsAggBuilder('aggTerms', {

diff --git a/.../public/editor_frame_service/editor_frame/config_panel/chart_switch/chart_switch.test.tsx b/.../public/editor_frame_service/editor_frame/config_panel/chart_switch/chart_switch.test.tsx
@@ -21,7 +21,7 @@ import { DatasourcePublicAPI, SuggestionRequest, DatasourceSuggestion } from '..
 import { ChartSwitchProps } from './chart_switch';
 import { ChartSwitchPopover } from './chart_switch_popover';
 import { LensAppState, applyChanges } from '../../../../state_management';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const mockFrame = (layers: string[]) => ({
   ...createMockFramePublicAPI(),

diff --git a/.../plugins/lens/public/editor_frame_service/editor_frame/config_panel/layer_header.test.tsx b/.../plugins/lens/public/editor_frame_service/editor_frame/config_panel/layer_header.test.tsx
@@ -7,7 +7,7 @@
 
 import React from 'react';
 import { screen } from '@testing-library/react';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import {
   createMockDatasource,
   createMockFramePublicAPI,

diff --git a/...k/plugins/lens/public/editor_frame_service/editor_frame/config_panel/layer_panel.test.tsx b/...k/plugins/lens/public/editor_frame_service/editor_frame/config_panel/layer_panel.test.tsx
@@ -12,7 +12,7 @@ import { ChildDragDropProvider, Droppable, Draggable } from '@kbn/dom-drag-drop'
 import { FramePublicAPI, Visualization, VisualizationConfigProps } from '../../../types';
 import { LayerPanel } from './layer_panel';
 import { coreMock } from '@kbn/core/public/mocks';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import { generateId } from '../../../id_generator';
 import {
   createMockVisualization,
@@ -144,7 +144,7 @@ describe('LayerPanel', () => {
   };
 
   beforeEach(() => {
-    mockVisualization = createMockVisualization(faker.random.alphaNumeric());
+    mockVisualization = createMockVisualization(faker.string.alphanumeric());
     mockVisualization.getLayerIds.mockReturnValue(['first']);
     mockDatasource = createMockDatasource();
   });

diff --git a/...ns/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel.test.tsx b/...ns/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel.test.tsx
@@ -39,7 +39,7 @@ import { getLensInspectorService } from '../../../lens_inspector_service';
 import { inspectorPluginMock } from '@kbn/inspector-plugin/public/mocks';
 import { disableAutoApply, enableAutoApply } from '../../../state_management/lens_slice';
 import { Ast, toExpression } from '@kbn/interpreter';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const defaultPermissions: Record<string, Record<string, boolean | Record<string, boolean>>> = {
   navLinks: { management: true },

diff --git a/...public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.test.tsx b/...public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.test.tsx
@@ -18,7 +18,7 @@ import { updateVisualizationState, LensAppState } from '../../../state_managemen
 import { setChangesApplied } from '../../../state_management/lens_slice';
 import { LensInspector } from '../../../lens_inspector_service';
 import { screen } from '@testing-library/react';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import { SettingsMenu } from '../../../app_plugin/settings_menu';
 
 describe('workspace_panel_wrapper', () => {
@@ -98,7 +98,7 @@ describe('workspace_panel_wrapper', () => {
   });
 
   it('should render its children', async () => {
-    const customElementText = faker.random.word();
+    const customElementText = faker.word.words();
     renderWorkspacePanelWrapper({ children: <span>{customElementText}</span> });
     expect(screen.getByText(customElementText)).toBeInTheDocument();
   });

diff --git a/x-pack/plugins/lens/public/lens_ui_telemetry/color_telemetry_helpers.test.ts b/x-pack/plugins/lens/public/lens_ui_telemetry/color_telemetry_helpers.test.ts
@@ -13,7 +13,7 @@ import {
   DEFAULT_COLOR_MAPPING_CONFIG,
   DEFAULT_OTHER_ASSIGNMENT_INDEX,
 } from '@kbn/coloring';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const exampleAssignment = (
   valuesCount = 1,
@@ -35,7 +35,7 @@ const exampleAssignment = (
   return {
     rule: {
       type: 'matchExactly',
-      values: Array.from({ length: valuesCount }, () => faker.random.alpha()),
+      values: Array.from({ length: valuesCount }, () => faker.string.alpha()),
     },
     color,
     touched: false,

diff --git a/x-pack/plugins/lens/public/mocks/visualization_mock.tsx b/x-pack/plugins/lens/public/mocks/visualization_mock.tsx
@@ -7,7 +7,7 @@
 import React from 'react';
 import { LayerTypes } from '@kbn/expression-xy-plugin/public';
 import { toExpression } from '@kbn/interpreter';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 import { Visualization, VisualizationMap } from '../types';
 
 export function createMockVisualization(

diff --git a/x-pack/plugins/lens/public/state_management/load_initial.test.tsx b/x-pack/plugins/lens/public/state_management/load_initial.test.tsx
@@ -18,7 +18,7 @@ import { act } from 'react-dom/test-utils';
 import { LensEmbeddableInput } from '../embeddable';
 import { loadInitial } from './lens_slice';
 import { Filter } from '@kbn/es-query';
-import faker from 'faker';
+import { faker } from '@faker-js/faker';
 
 const history = {
   location: {