Updated EnablementModalCallout name to AdditionalChargesMessage (#203061

) ## Summary This PR renames the enablementModalCallout component to AdditionalChargesMessage for better clarity and alignment with its purpose, addressing [this comment](#201920 (review)) from the original PR. ### Before and After (the output has not changed) ![image](https://github.com/user-attachments/assets/c07ac4a4-51db-4d5c-9040-3cde4fbd199e) ### Checklist Check the PR satisfies following conditions. Reviewers should verify this PR satisfies this list as well. - [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios Co-authored-by: Mark Hopkin <[email protected]>
elastic · Dec 19, 2024 · a21fb6c · a21fb6c
1 parent f881df1
commit a21fb6c
Show file tree

Hide file tree

Showing 7 changed files with 16 additions and 16 deletions.
diff --git a/x-pack/solutions/security/plugins/security_solution/public/contract_components.ts b/x-pack/solutions/security/plugins/security_solution/public/contract_components.ts
@@ -11,7 +11,7 @@ import type { Observable } from 'rxjs';
 export type ContractComponents = Partial<{
   GetStarted: React.ComponentType<{ indicesExist?: boolean }>;
   DashboardsLandingCallout: React.ComponentType<{}>;
-  EnablementModalCallout: React.ComponentType<{}>;
+  AdditionalChargesMessage: React.ComponentType<{}>;
 }>;
 
 export type SetComponents = (components: ContractComponents) => void;

diff --git a/...tion/public/entity_analytics/components/entity_store/components/enablement_modal.test.tsx b/...tion/public/entity_analytics/components/entity_store/components/enablement_modal.test.tsx
@@ -182,9 +182,9 @@ describe('EntityStoreEnablementModal', () => {
     });
 
     it('should render additional charges message when available', async () => {
-      const EnablementModalCalloutMock = () => <span data-test-subj="enablement-modal-test" />;
+      const AdditionalChargesMessageMock = () => <span data-test-subj="enablement-modal-test" />;
       mockUseContractComponents.mockReturnValue({
-        EnablementModalCallout: EnablementModalCalloutMock,
+        AdditionalChargesMessage: AdditionalChargesMessageMock,
       });
 
       await renderComponent();

diff --git a/..._solution/public/entity_analytics/components/entity_store/components/enablement_modal.tsx b/..._solution/public/entity_analytics/components/entity_store/components/enablement_modal.tsx
@@ -73,7 +73,7 @@ export const EntityStoreEnablementModal: React.FC<EntityStoreEnablementModalProp
     useEntityEnginePrivileges();
   const riskEnginePrivileges = useMissingRiskEnginePrivileges();
   const enablementOptions = enablements.riskScore || enablements.entityStore;
-  const { EnablementModalCallout } = useContractComponents();
+  const { AdditionalChargesMessage } = useContractComponents();
 
   if (!visible) {
     return null;
@@ -102,7 +102,7 @@ export const EntityStoreEnablementModal: React.FC<EntityStoreEnablementModalProp
 
       <EuiModalBody>
         <EuiFlexGroup direction="column">
-          <EuiFlexItem>{EnablementModalCallout && <EnablementModalCallout />}</EuiFlexItem>
+          <EuiFlexItem>{AdditionalChargesMessage && <AdditionalChargesMessage />}</EuiFlexItem>
           <EuiFlexItem>
             <EuiSwitch
               label={

diff --git a/...odal_callout/enablement_modal_callout.tsx → ...es_message/additional_charges_message.tsx b/...odal_callout/enablement_modal_callout.tsx → ...es_message/additional_charges_message.tsx
@@ -9,7 +9,7 @@ import React from 'react';
 import { EuiText } from '@elastic/eui';
 import { ADDITIONAL_CHARGES_MESSAGE } from '../../upselling/translations';
 
-export const EnablementModalCallout: React.FC = () => {
+export const AdditionalChargesMessage: React.FC = () => {
   return (
     <div>
       <EuiText>{ADDITIONAL_CHARGES_MESSAGE}</EuiText>
@@ -18,4 +18,4 @@ export const EnablementModalCallout: React.FC = () => {
 };
 
 // eslint-disable-next-line import/no-default-export
-export default EnablementModalCallout;
+export default AdditionalChargesMessage;
diff --git a/...onents/enablement_modal_callout/index.tsx → ...ents/additional_charges_message/index.tsx b/...onents/enablement_modal_callout/index.tsx → ...ents/additional_charges_message/index.tsx
@@ -7,13 +7,13 @@
 import React from 'react';
 import type { Services } from '../../common/services';
 import { ServicesProvider } from '../../common/services';
-import { EnablementModalCallout } from './lazy';
+import { AdditionalChargesMessage } from './lazy';
 
-export const getEnablementModalCallout = (services: Services): React.ComponentType =>
-  function EnablementModalCalloutComponent() {
+export const getAdditionalChargesMessage = (services: Services): React.ComponentType =>
+  function AdditionalChargesMessageComponent() {
     return (
       <ServicesProvider services={services}>
-        <EnablementModalCallout />
+        <AdditionalChargesMessage />
       </ServicesProvider>
     );
   };
diff --git a/...ponents/enablement_modal_callout/lazy.tsx → ...nents/additional_charges_message/lazy.tsx b/...ponents/enablement_modal_callout/lazy.tsx → ...nents/additional_charges_message/lazy.tsx
@@ -8,10 +8,10 @@
 import React, { lazy, Suspense } from 'react';
 import { EuiLoadingSpinner } from '@elastic/eui';
 
-const EnablementModalCalloutLazy = lazy(() => import('./enablement_modal_callout'));
+const AdditionalChargesMessageLazy = lazy(() => import('./additional_charges_message'));
 
-export const EnablementModalCallout = () => (
+export const AdditionalChargesMessage = () => (
   <Suspense fallback={<EuiLoadingSpinner size="s" />}>
-    <EnablementModalCalloutLazy />
+    <AdditionalChargesMessageLazy />
   </Suspense>
 );
diff --git a/x-pack/solutions/security/plugins/security_solution_serverless/public/plugin.ts b/x-pack/solutions/security/plugins/security_solution_serverless/public/plugin.ts
@@ -23,7 +23,7 @@ import {
   type ExperimentalFeatures,
 } from '../common/experimental_features';
 import { setOnboardingSettings } from './onboarding';
-import { getEnablementModalCallout } from './components/enablement_modal_callout';
+import { getAdditionalChargesMessage } from './components/additional_charges_message';
 
 export class SecuritySolutionServerlessPlugin
   implements
@@ -70,7 +70,7 @@ export class SecuritySolutionServerlessPlugin
 
     securitySolution.setComponents({
       DashboardsLandingCallout: getDashboardsLandingCallout(services),
-      EnablementModalCallout: getEnablementModalCallout(services),
+      AdditionalChargesMessage: getAdditionalChargesMessage(services),
     });
 
     setOnboardingSettings(services);