Remove usage of any on registries

x-pack/plugins/triggers_actions_ui/public/application/app.tsx

@@ -18,8 +18,7 @@ import {
 } from 'kibana/public';
 import { Section, routeToAlertDetails } from './constants';
 import { AppContextProvider } from './app_context';
-import { ActionTypeModel, AlertTypeModel } from '../types';
-import { TypeRegistry } from './type_registry';
+import { ActionTypeRegistryContract, AlertTypeRegistryContract } from '../types';
 import { ChartsPluginStart } from '../../../../../src/plugins/charts/public';
 import { DataPublicPluginStart } from '../../../../../src/plugins/data/public';
 import { PluginStartContract as AlertingStart } from '../../../alerts/public';
@@ -42,8 +41,8 @@ export interface AppDeps {
   uiSettings: IUiSettingsClient;
   setBreadcrumbs: (crumbs: ChromeBreadcrumb[]) => void;
   capabilities: ApplicationStart['capabilities'];
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
-  alertTypeRegistry: TypeRegistry<AlertTypeModel>;
+  actionTypeRegistry: ActionTypeRegistryContract;
+  alertTypeRegistry: AlertTypeRegistryContract;
   history: ScopedHistory;
 }
 

x-pack/plugins/triggers_actions_ui/public/application/context/actions_connectors_context.tsx

@@ -6,12 +6,11 @@
 
 import React, { createContext, useContext } from 'react';
 import { HttpSetup, ApplicationStart, DocLinksStart, ToastsSetup } from 'kibana/public';
-import { ActionTypeModel, ActionConnector } from '../../types';
-import { TypeRegistry } from '../type_registry';
+import { ActionTypeRegistryContract, ActionConnector } from '../../types';
 
 export interface ActionsConnectorsContextValue {
   http: HttpSetup;
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
+  actionTypeRegistry: ActionTypeRegistryContract;
   toastNotifications: ToastsSetup;
   capabilities: ApplicationStart['capabilities'];
   reloadConnectors?: () => Promise<ActionConnector[] | void>;

x-pack/plugins/triggers_actions_ui/public/application/context/alerts_context.tsx

@@ -18,14 +18,13 @@ import {
   DataPublicPluginStartUi,
   IndexPatternsContract,
 } from 'src/plugins/data/public';
-import { TypeRegistry } from '../type_registry';
-import { AlertTypeModel, ActionTypeModel } from '../../types';
+import { AlertTypeRegistryContract, ActionTypeRegistryContract } from '../../types';
 
 export interface AlertsContextValue<MetaData = Record<string, any>> {
   reloadAlerts?: () => Promise<void>;
   http: HttpSetup;
-  alertTypeRegistry: TypeRegistry<AlertTypeModel>;
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
+  alertTypeRegistry: AlertTypeRegistryContract;
+  actionTypeRegistry: ActionTypeRegistryContract;
   toastNotifications: ToastsStart;
   uiSettings?: IUiSettingsClient;
   charts?: ChartsPluginSetup;

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_connector_form.test.tsx

@@ -22,7 +22,7 @@ describe('action_connector_form', () => {
     ] = await mocks.getStartServices();
     deps = {
       http: mocks.http,
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
       capabilities,
     };

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_connector_form.tsx

@@ -24,10 +24,9 @@ import { ReducerAction } from './connector_reducer';
 import {
   ActionConnector,
   IErrorObject,
-  ActionTypeModel,
+  ActionTypeRegistryContract,
   UserConfiguredActionConnector,
 } from '../../../types';
-import { TypeRegistry } from '../../type_registry';
 import { hasSaveActionsCapability } from '../../lib/capabilities';
 
 export function validateBaseProperties(actionObject: ActionConnector) {
@@ -61,7 +60,7 @@ interface ActionConnectorProps<
   };
   errors: IErrorObject;
   http: HttpSetup;
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
+  actionTypeRegistry: ActionTypeRegistryContract;
   docLinks: DocLinksStart;
   capabilities: ApplicationStart['capabilities'];
   consumer?: string;

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.test.tsx

@@ -178,7 +178,7 @@ describe('action_form', () => {
           },
         },
         setHasActionsWithBrokenConnector: jest.fn(),
-        actionTypeRegistry: actionTypeRegistry as any,
+        actionTypeRegistry,
         docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
       };
       actionTypeRegistry.list.mockReturnValue([

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.tsx

@@ -34,6 +34,7 @@ import { loadActionTypes, loadAllActions as loadConnectors } from '../../lib/act
 import {
   IErrorObject,
   ActionTypeModel,
+  ActionTypeRegistryContract,
   AlertAction,
   ActionTypeIndex,
   ActionConnector,
@@ -42,7 +43,6 @@ import {
 } from '../../../types';
 import { SectionLoading } from '../../components/section_loading';
 import { ConnectorAddModal } from './connector_add_modal';
-import { TypeRegistry } from '../../type_registry';
 import { actionTypeCompare } from '../../lib/action_type_compare';
 import { checkActionFormActionTypeEnabled } from '../../lib/check_action_type_enabled';
 import { VIEW_LICENSE_OPTIONS_LINK } from '../../../common/constants';
@@ -55,7 +55,7 @@ interface ActionAccordionFormProps {
   setAlertProperty: (actions: AlertAction[]) => void;
   setActionParamsProperty: (key: string, value: any, index: number) => void;
   http: HttpSetup;
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
+  actionTypeRegistry: ActionTypeRegistryContract;
   toastNotifications: ToastsSetup;
   docLinks: DocLinksStart;
   actionTypes?: ActionType[];

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_type_menu.test.tsx

@@ -33,7 +33,7 @@ describe('connector_add_flyout', () => {
           show: true,
         },
       },
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
     };
   });

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_flyout.test.tsx

@@ -34,7 +34,7 @@ describe('connector_add_flyout', () => {
           show: true,
         },
       },
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
     };
   });

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_modal.test.tsx

@@ -32,7 +32,7 @@ describe('connector_add_modal', () => {
           delete: true,
         },
       },
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
     };
   });

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_modal.tsx

@@ -19,20 +19,24 @@ import { EuiOverlayMask } from '@elastic/eui';
 import { i18n } from '@kbn/i18n';
 import { HttpSetup, ToastsApi, ApplicationStart, DocLinksStart } from 'kibana/public';
 import { ActionConnectorForm, validateBaseProperties } from './action_connector_form';
-import { ActionType, ActionConnector, IErrorObject, ActionTypeModel } from '../../../types';
 import { connectorReducer } from './connector_reducer';
 import { createActionConnector } from '../../lib/action_connector_api';
-import { TypeRegistry } from '../../type_registry';
 import './connector_add_modal.scss';
 import { hasSaveActionsCapability } from '../../lib/capabilities';
+import {
+  ActionType,
+  ActionConnector,
+  IErrorObject,
+  ActionTypeRegistryContract,
+} from '../../../types';
 
 interface ConnectorAddModalProps {
   actionType: ActionType;
   addModalVisible: boolean;
   setAddModalVisibility: React.Dispatch<React.SetStateAction<boolean>>;
   postSaveEventHandler?: (savedAction: ActionConnector) => void;
   http: HttpSetup;
-  actionTypeRegistry: TypeRegistry<ActionTypeModel>;
+  actionTypeRegistry: ActionTypeRegistryContract;
   toastNotifications: Pick<
     ToastsApi,
     'get$' | 'add' | 'remove' | 'addSuccess' | 'addWarning' | 'addDanger' | 'addError'

x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_edit_flyout.test.tsx

@@ -35,7 +35,7 @@ describe('connector_edit_flyout', () => {
           show: true,
         },
       },
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       alertTypeRegistry: {} as any,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
     };

x-pack/plugins/triggers_actions_ui/public/application/sections/actions_connectors_list/components/actions_connectors_list.test.tsx

@@ -69,7 +69,7 @@ describe('actions_connectors_list component empty', () => {
       },
       history: scopedHistoryMock.create(),
       setBreadcrumbs: jest.fn(),
-      actionTypeRegistry: actionTypeRegistry as any,
+      actionTypeRegistry,
       alertTypeRegistry: {} as any,
     };
     actionTypeRegistry.has.mockReturnValue(true);

x-pack/plugins/triggers_actions_ui/public/application/sections/alert_form/alert_add.test.tsx

@@ -84,8 +84,8 @@ describe('alert_add', () => {
       uiSettings: mocks.uiSettings,
       dataPlugin: dataPluginMock.createStartContract(),
       charts: chartPluginMock.createStartContract(),
-      actionTypeRegistry: actionTypeRegistry as any,
-      alertTypeRegistry: alertTypeRegistry as any,
+      actionTypeRegistry,
+      alertTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
     };
 

x-pack/plugins/triggers_actions_ui/public/application/sections/alert_form/alert_edit.test.tsx

@@ -36,8 +36,8 @@ describe('alert_edit', () => {
       toastNotifications: mockedCoreSetup.notifications.toasts,
       http: mockedCoreSetup.http,
       uiSettings: mockedCoreSetup.uiSettings,
-      actionTypeRegistry: actionTypeRegistry as any,
-      alertTypeRegistry: alertTypeRegistry as any,
+      actionTypeRegistry,
+      alertTypeRegistry,
       docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
       capabilities,
     };

x-pack/plugins/triggers_actions_ui/public/application/sections/alert_form/alert_form.test.tsx

@@ -98,8 +98,8 @@ describe('alert_form', () => {
         toastNotifications: mocks.notifications.toasts,
         http: mocks.http,
         uiSettings: mocks.uiSettings,
-        actionTypeRegistry: actionTypeRegistry as any,
-        alertTypeRegistry: alertTypeRegistry as any,
+        actionTypeRegistry,
+        alertTypeRegistry,
         docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
         capabilities,
       };
@@ -231,8 +231,8 @@ describe('alert_form', () => {
         toastNotifications: mocks.notifications.toasts,
         http: mocks.http,
         uiSettings: mocks.uiSettings,
-        actionTypeRegistry: actionTypeRegistry as any,
-        alertTypeRegistry: alertTypeRegistry as any,
+        actionTypeRegistry,
+        alertTypeRegistry,
         docLinks: { ELASTIC_WEBSITE_URL: '', DOC_LINK_VERSION: '' },
         capabilities,
       };
@@ -332,8 +332,8 @@ describe('alert_form', () => {
         toastNotifications: mockes.notifications.toasts,
         http: mockes.http,
         uiSettings: mockes.uiSettings,
-        actionTypeRegistry: actionTypeRegistry as any,
-        alertTypeRegistry: alertTypeRegistry as any,
+        actionTypeRegistry,
+        alertTypeRegistry,
       };
       alertTypeRegistry.list.mockReturnValue([alertType]);
       alertTypeRegistry.get.mockReturnValue(alertType);

x-pack/plugins/triggers_actions_ui/public/application/sections/alerts_list/components/alerts_list.test.tsx

@@ -109,8 +109,8 @@ describe('alerts_list component empty', () => {
       capabilities,
       history: scopedHistoryMock.create(),
       setBreadcrumbs: jest.fn(),
-      actionTypeRegistry: actionTypeRegistry as any,
-      alertTypeRegistry: alertTypeRegistry as any,
+      actionTypeRegistry,
+      alertTypeRegistry,
     };
 
     wrapper = mountWithIntl(
@@ -278,8 +278,8 @@ describe('alerts_list component with items', () => {
       capabilities,
       history: scopedHistoryMock.create(),
       setBreadcrumbs: jest.fn(),
-      actionTypeRegistry: actionTypeRegistry as any,
-      alertTypeRegistry: alertTypeRegistry as any,
+      actionTypeRegistry,
+      alertTypeRegistry,
     };
 
     alertTypeRegistry.has.mockReturnValue(true);
@@ -478,8 +478,8 @@ describe('alerts_list with show only capability', () => {
       capabilities,
       history: scopedHistoryMock.create(),
       setBreadcrumbs: jest.fn(),
-      actionTypeRegistry: actionTypeRegistry as any,
-      alertTypeRegistry: alertTypeRegistry as any,
+      actionTypeRegistry,
+      alertTypeRegistry,
     };
 
     alertTypeRegistry.has.mockReturnValue(false);

x-pack/plugins/triggers_actions_ui/public/application/test_utils/index.ts

@@ -35,7 +35,7 @@ export async function getMockedAppDependencies() {
     capabilities,
     history: scopedHistoryMock.create(),
     setBreadcrumbs: jest.fn(),
-    actionTypeRegistry: actionTypeRegistry as any,
-    alertTypeRegistry: alertTypeRegistry as any,
+    actionTypeRegistry,
+    alertTypeRegistry,
   };
 }