From a4b919479a733a6a97b2b35b49f99bed5589f3fa Mon Sep 17 00:00:00 2001 From: Kibana Machine <42973632+kibanamachine@users.noreply.github.com> Date: Thu, 28 Nov 2024 06:59:44 +1100 Subject: [PATCH] [8.17] [Security Solution][Expandable flyout] - remove unused code (#201477) (#202047) # Backport This will backport the following commits from `main` to `8.17`: - [[Security Solution][Expandable flyout] - remove unused code (#201477)](https://github.com/elastic/kibana/pull/201477) ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) Co-authored-by: Philippe Oberti --- .../kbn-expandable-flyout/__mocks__/index.tsx | 29 ------------------ packages/kbn-expandable-flyout/index.ts | 1 - .../src/with_provider.test.tsx | 30 ------------------- .../src/with_provider.tsx | 26 ---------------- .../public/common/mock/expandable_flyout.tsx | 16 ---------- .../table_field_value_cell.test.tsx | 5 ---- .../shared/components/preview_link.test.tsx | 5 ---- 7 files changed, 112 deletions(-) delete mode 100644 packages/kbn-expandable-flyout/src/with_provider.test.tsx delete mode 100644 packages/kbn-expandable-flyout/src/with_provider.tsx diff --git a/packages/kbn-expandable-flyout/__mocks__/index.tsx b/packages/kbn-expandable-flyout/__mocks__/index.tsx index 37d8975351e55..75da56c9eb814 100644 --- a/packages/kbn-expandable-flyout/__mocks__/index.tsx +++ b/packages/kbn-expandable-flyout/__mocks__/index.tsx @@ -7,37 +7,8 @@ * License v3.0 only", or the "Server Side Public License, v 1". */ -import React from 'react'; import type { IStorage } from '@kbn/kibana-utils-plugin/public'; -export const useExpandableFlyoutApi = jest.fn(() => ({ - openFlyout: jest.fn(), - closeFlyout: jest.fn(), - openPanels: jest.fn(), - openRightPanel: jest.fn(), - openLeftPanel: jest.fn(), - openPreviewPanel: jest.fn(), - closeRightPanel: jest.fn(), - closeLeftPanel: jest.fn(), - closePreviewPanel: jest.fn(), - closePanels: jest.fn(), - previousPreviewPanel: jest.fn(), -})); - -export const useExpandableFlyoutState = jest.fn(); - -export const ExpandableFlyoutProvider = jest.fn(({ children }: React.PropsWithChildren<{}>) => { - return <>{children}; -}); - -export const withExpandableFlyoutProvider = ( - Component: React.ComponentType -) => { - return (props: T) => { - return ; - }; -}; - export const ExpandableFlyout = jest.fn(); export const localStorageMock = (): IStorage => { diff --git a/packages/kbn-expandable-flyout/index.ts b/packages/kbn-expandable-flyout/index.ts index f06be45a68914..5816b6673dbc1 100644 --- a/packages/kbn-expandable-flyout/index.ts +++ b/packages/kbn-expandable-flyout/index.ts @@ -15,7 +15,6 @@ export { useExpandableFlyoutState } from './src/hooks/use_expandable_flyout_stat export { type FlyoutPanels as ExpandableFlyoutState } from './src/store/state'; export { ExpandableFlyoutProvider } from './src/provider'; -export { withExpandableFlyoutProvider } from './src/with_provider'; export type { ExpandableFlyoutProps } from './src'; export type { FlyoutPanelProps, PanelPath, ExpandableFlyoutApi } from './src/types'; diff --git a/packages/kbn-expandable-flyout/src/with_provider.test.tsx b/packages/kbn-expandable-flyout/src/with_provider.test.tsx deleted file mode 100644 index de9f831a5db8f..0000000000000 --- a/packages/kbn-expandable-flyout/src/with_provider.test.tsx +++ /dev/null @@ -1,30 +0,0 @@ -/* - * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one - * or more contributor license agreements. Licensed under the "Elastic License - * 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side - * Public License v 1"; you may not use this file except in compliance with, at - * your election, the "Elastic License 2.0", the "GNU Affero General Public - * License v3.0 only", or the "Server Side Public License, v 1". - */ - -import { render } from '@testing-library/react'; -import { useExpandableFlyoutApi } from './hooks/use_expandable_flyout_api'; -import React from 'react'; -import { withExpandableFlyoutProvider } from './with_provider'; - -const TestComponent = () => { - useExpandableFlyoutApi(); - - return
; -}; - -describe('withExpandableFlyoutProvider', () => { - it('should throw when rendered without Expandable Provider', () => { - expect(() => render()).toThrow(); - }); - - it('should not throw when rendered with Expandable Provider', () => { - const TestComponentWithProvider = withExpandableFlyoutProvider(TestComponent); - expect(() => render()).not.toThrow(); - }); -}); diff --git a/packages/kbn-expandable-flyout/src/with_provider.tsx b/packages/kbn-expandable-flyout/src/with_provider.tsx deleted file mode 100644 index d4fcca19d1bac..0000000000000 --- a/packages/kbn-expandable-flyout/src/with_provider.tsx +++ /dev/null @@ -1,26 +0,0 @@ -/* - * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one - * or more contributor license agreements. Licensed under the "Elastic License - * 2.0", the "GNU Affero General Public License v3.0 only", and the "Server Side - * Public License v 1"; you may not use this file except in compliance with, at - * your election, the "Elastic License 2.0", the "GNU Affero General Public - * License v3.0 only", or the "Server Side Public License, v 1". - */ - -import React from 'react'; -import { ComponentType } from 'react'; -import { ExpandableFlyoutContextProviderProps } from './context'; -import { ExpandableFlyoutProvider } from './provider'; - -export const withExpandableFlyoutProvider = ( - Component: ComponentType, - expandableProviderProps?: ExpandableFlyoutContextProviderProps -) => { - return (props: Props) => { - return ( - - - - ); - }; -}; diff --git a/x-pack/plugins/security_solution/public/common/mock/expandable_flyout.tsx b/x-pack/plugins/security_solution/public/common/mock/expandable_flyout.tsx index a987e99a67d0e..69cd24e4ff453 100644 --- a/x-pack/plugins/security_solution/public/common/mock/expandable_flyout.tsx +++ b/x-pack/plugins/security_solution/public/common/mock/expandable_flyout.tsx @@ -5,8 +5,6 @@ * 2.0. */ -import React from 'react'; - export const createExpandableFlyoutApiMock = () => ({ closeFlyout: jest.fn(), closeLeftPanel: jest.fn(), @@ -18,17 +16,3 @@ export const createExpandableFlyoutApiMock = () => ({ openPreviewPanel: jest.fn(), openRightPanel: jest.fn(), }); - -export const createExpandableFlyoutMock = () => { - return { - useExpandableFlyoutApi: jest.fn().mockReturnValue(createExpandableFlyoutApiMock()), - useExpandableFlyoutState: jest.fn(), - ExpandableFlyoutProvider: ({ children }: React.PropsWithChildren<{}>) => <>{children}, - withExpandableFlyoutProvider: (Component: React.ComponentType) => { - return (props: T) => { - return ; - }; - }, - ExpandableFlyout: jest.fn(), - }; -}; diff --git a/x-pack/plugins/security_solution/public/flyout/document_details/right/components/table_field_value_cell.test.tsx b/x-pack/plugins/security_solution/public/flyout/document_details/right/components/table_field_value_cell.test.tsx index 8de7c909548b0..ee680f1061621 100644 --- a/x-pack/plugins/security_solution/public/flyout/document_details/right/components/table_field_value_cell.test.tsx +++ b/x-pack/plugins/security_solution/public/flyout/document_details/right/components/table_field_value_cell.test.tsx @@ -22,11 +22,6 @@ import { createTelemetryServiceMock } from '../../../../common/lib/telemetry/tel jest.mock('@kbn/expandable-flyout', () => ({ useExpandableFlyoutApi: jest.fn(), ExpandableFlyoutProvider: ({ children }: React.PropsWithChildren<{}>) => <>{children}, - withExpandableFlyoutProvider: (Component: React.ComponentType) => { - return (props: T) => { - return ; - }; - }, })); const mockedTelemetry = createTelemetryServiceMock(); diff --git a/x-pack/plugins/security_solution/public/flyout/shared/components/preview_link.test.tsx b/x-pack/plugins/security_solution/public/flyout/shared/components/preview_link.test.tsx index c1dfc456f40ce..f1dedc18d3b1c 100644 --- a/x-pack/plugins/security_solution/public/flyout/shared/components/preview_link.test.tsx +++ b/x-pack/plugins/security_solution/public/flyout/shared/components/preview_link.test.tsx @@ -34,11 +34,6 @@ jest.mock('../../../common/lib/kibana', () => { jest.mock('@kbn/expandable-flyout', () => ({ useExpandableFlyoutApi: jest.fn(), ExpandableFlyoutProvider: ({ children }: React.PropsWithChildren<{}>) => <>{children}, - withExpandableFlyoutProvider: (Component: React.ComponentType) => { - return (props: T) => { - return ; - }; - }, })); const renderPreviewLink = (field: string, value: string, dataTestSuj?: string) =>