Rename subactions

x-pack/plugins/actions/server/builtin_action_types/jira/api.test.ts

@@ -340,9 +340,9 @@ describe('api', () => {
       });
     });
 
-    describe('getIssueTypes', () => {
+    describe('issueTypes', () => {
       test('it returns the issue types correctly', async () => {
-        const res = await api.getIssueTypes({
+        const res = await api.issueTypes({
           externalService,
           params: {},
         });
@@ -359,9 +359,9 @@ describe('api', () => {
       });
     });
 
-    describe('getFieldsByIssueType', () => {
+    describe('fieldsByIssueType', () => {
       test('it returns the issue types correctly', async () => {
-        const res = await api.getFieldsByIssueType({
+        const res = await api.fieldsByIssueType({
           externalService,
           params: { id: '10006' },
         });

x-pack/plugins/actions/server/builtin_action_types/jira/api.ts

@@ -177,6 +177,6 @@ export const api: ExternalServiceApi = {
   handshake: handshakeHandler,
   pushToService: pushToServiceHandler,
   getIncident: getIncidentHandler,
-  getIssueTypes: getIssueTypesHandler,
-  getFieldsByIssueType: getFieldsByIssueTypeHandler,
+  issueTypes: getIssueTypesHandler,
+  fieldsByIssueType: getFieldsByIssueTypeHandler,
 };

x-pack/plugins/actions/server/builtin_action_types/jira/index.ts

@@ -119,17 +119,17 @@ async function executor(
     logger.debug(`response push to service for incident id: ${data.id}`);
   }
 
-  if (subAction === 'getIssueTypes') {
+  if (subAction === 'issueTypes') {
     const getIssueTypesParams = subActionParams as ExecutorSubActionGetIssueTypesParams;
-    data = await api.getIssueTypes({
+    data = await api.issueTypes({
       externalService,
       params: getIssueTypesParams,
     });
   }
 
-  if (subAction === 'getFieldsByIssueType') {
+  if (subAction === 'fieldsByIssueType') {
     const getFieldsByIssueTypeParams = subActionParams as ExecutorSubActionGetFieldsByIssueTypeParams;
-    data = await api.getFieldsByIssueType({
+    data = await api.fieldsByIssueType({
       externalService,
       params: getFieldsByIssueTypeParams,
     });

x-pack/plugins/actions/server/builtin_action_types/jira/schema.ts

@@ -74,11 +74,11 @@ export const ExecutorParamsSchema = schema.oneOf([
     subActionParams: ExecutorSubActionPushParamsSchema,
   }),
   schema.object({
-    subAction: schema.literal('getIssueTypes'),
+    subAction: schema.literal('issueTypes'),
     subActionParams: ExecutorSubActionGetIssueTypesParamsSchema,
   }),
   schema.object({
-    subAction: schema.literal('getFieldsByIssueType'),
+    subAction: schema.literal('fieldsByIssueType'),
     subActionParams: ExecutorSubActionGetFieldsByIssueTypeParamsSchema,
   }),
 ]);

x-pack/plugins/actions/server/builtin_action_types/jira/types.ts

@@ -150,8 +150,8 @@ export interface ExternalServiceApi {
   handshake: (args: HandshakeApiHandlerArgs) => Promise<void>;
   pushToService: (args: PushToServiceApiHandlerArgs) => Promise<PushToServiceResponse>;
   getIncident: (args: GetIncidentApiHandlerArgs) => Promise<void>;
-  getIssueTypes: (args: GetIssueTypesHandlerArgs) => Promise<GetIssueTypesResponse>;
-  getFieldsByIssueType: (
+  issueTypes: (args: GetIssueTypesHandlerArgs) => Promise<GetIssueTypesResponse>;
+  fieldsByIssueType: (
     args: GetFieldsByIssueTypeHandlerArgs
   ) => Promise<GetFieldsByIssueTypeResponse>;
 }

x-pack/plugins/actions/server/builtin_action_types/jira/validators.ts

@@ -26,7 +26,7 @@ export const validateCommonConfig = (
   }
 
   try {
-    configurationUtilities.ensureWhitelistedUri(configObject.apiUrl);
+    configurationUtilities.ensureUriAllowed(configObject.apiUrl);
   } catch (whitelistError) {
     return i18n.WHITE_LISTED_ERROR(whitelistError.message);
   }

x-pack/plugins/triggers_actions_ui/public/application/components/builtin_action_types/jira/api.ts

@@ -7,23 +7,6 @@
 import { HttpSetup } from 'kibana/public';
 import { BASE_ACTION_API_PATH } from '../../../constants';
 
-export async function getCreateIssueMetadata({
-  http,
-  signal,
-  connectorId,
-}: {
-  http: HttpSetup;
-  signal: AbortSignal;
-  connectorId: string;
-}): Promise<Record<string, any>> {
-  return await http.post(`${BASE_ACTION_API_PATH}/action/${connectorId}/_execute`, {
-    body: JSON.stringify({
-      params: { subAction: 'getCreateIssueMetadata', subActionParams: {} },
-    }),
-    signal,
-  });
-}
-
 export async function getIssueTypes({
   http,
   signal,
@@ -35,7 +18,7 @@ export async function getIssueTypes({
 }): Promise<Record<string, any>> {
   return await http.post(`${BASE_ACTION_API_PATH}/action/${connectorId}/_execute`, {
     body: JSON.stringify({
-      params: { subAction: 'getIssueTypes', subActionParams: {} },
+      params: { subAction: 'issueTypes', subActionParams: {} },
     }),
     signal,
   });
@@ -54,7 +37,7 @@ export async function getFieldsByIssueType({
 }): Promise<Record<string, any>> {
   return await http.post(`${BASE_ACTION_API_PATH}/action/${connectorId}/_execute`, {
     body: JSON.stringify({
-      params: { subAction: 'getFieldsByIssueType', subActionParams: { id } },
+      params: { subAction: 'fieldsByIssueType', subActionParams: { id } },
     }),
     signal,
   });

x-pack/plugins/triggers_actions_ui/public/application/components/builtin_action_types/jira/jira_connectors.test.tsx

@@ -6,7 +6,7 @@
 import React from 'react';
 import { mountWithIntl } from 'test_utils/enzyme_helpers';
 import { DocLinksStart } from 'kibana/public';
-import JiraConnectorFields from './Jira_connectors';
+import JiraConnectorFields from './jira_connectors';
 import { JiraActionConnector } from './types';
 
 describe('JiraActionConnectorFields renders', () => {