From 38fb6599b0c70cac75e48f307f4d10915e141f0a Mon Sep 17 00:00:00 2001
From: Gidi Meir Morris <github@gidi.io>
Date: Thu, 5 Nov 2020 18:32:29 +0000
Subject: [PATCH] fixed export of components

---
 .../action_form.test.tsx                      |   1 -
 .../action_connector_form/action_form.tsx     | 302 +++++++++---------
 .../action_type_form.tsx                      |   3 -
 .../connector_add_inline.tsx                  |   3 -
 4 files changed, 151 insertions(+), 158 deletions(-)

diff --git a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.test.tsx b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.test.tsx
index 5c8d706da83ecf..94452e70e6bfa3 100644
--- a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.test.tsx
+++ b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.test.tsx
@@ -307,7 +307,6 @@ describe('action_form', () => {
       const actionOption = wrapper.find(
         `[data-test-subj="${actionType.id}-ActionTypeSelectOption"]`
       );
-      wrapper.debug();
       expect(actionOption.exists()).toBeTruthy();
       expect(
         wrapper
diff --git a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.tsx b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.tsx
index 5379a7a99f4292..92aa59d2af2e76 100644
--- a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.tsx
+++ b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_form.tsx
@@ -271,165 +271,165 @@ export const ActionForm = ({
       });
   }
 
-  const alertActionsList =
-    actionTypesIndex &&
-    actions.map((actionItem: AlertAction, index: number) => {
-      const actionConnector = connectors.find((field) => field.id === actionItem.id);
-      // connectors doesn't exists
-      if (!actionConnector) {
-        return (
-          <AddConnectorInline
-            actionTypesIndex={actionTypesIndex}
-            actionItem={actionItem}
-            index={index}
-            actionTypeRegistry={actionTypeRegistry}
-            defaultActionGroupId={defaultActionGroupId}
-            capabilities={capabilities}
-            emptyActionsIds={emptyActionsIds}
-            onDeleteConnector={() => {
-              const updatedActions = actions.filter((_item: AlertAction, i: number) => i !== index);
-              setAlertProperty(updatedActions);
-              setIsAddActionPanelOpen(
-                updatedActions.filter((item: AlertAction) => item.id !== actionItem.id).length === 0
-              );
-              setActiveActionItem(undefined);
-            }}
-            onAddConnector={() => {
-              setActiveActionItem({ actionTypeId: actionItem.actionTypeId, index });
-              setAddModalVisibility(true);
-            }}
-          />
-        );
-      }
-
-      const actionParamsErrors: ActionTypeFormProps['actionParamsErrors'] = actionTypeRegistry
-        .get(actionItem.actionTypeId)
-        ?.validateParams(actionItem.params);
-
-      return (
-        <ActionTypeForm
-          actionItem={actionItem}
-          actionConnector={actionConnector}
-          actionParamsErrors={actionParamsErrors}
-          index={index}
-          setActionParamsProperty={setActionParamsProperty}
-          actionTypesIndex={actionTypesIndex}
-          connectors={connectors}
-          http={http}
-          toastNotifications={toastNotifications}
-          docLinks={docLinks}
-          capabilities={capabilities}
-          actionTypeRegistry={actionTypeRegistry}
-          defaultActionGroupId={defaultActionGroupId}
-          defaultActionMessage={defaultActionMessage}
-          messageVariables={messageVariables}
-          actionGroups={actionGroups}
-          setActionGroupIdByIndex={setActionGroupIdByIndex}
-          onAddConnector={() => {
-            setActiveActionItem({ actionTypeId: actionItem.actionTypeId, index });
-            setAddModalVisibility(true);
-          }}
-          onConnectorSelected={(id: string) => {
-            setActionIdByIndex(id, index);
-          }}
-          onDeleteAction={() => {
-            const updatedActions = actions.filter((_item: AlertAction, i: number) => i !== index);
-            setAlertProperty(updatedActions);
-            setIsAddActionPanelOpen(
-              updatedActions.filter((item: AlertAction) => item.id !== actionItem.id).length === 0
-            );
-            setActiveActionItem(undefined);
-          }}
-        />
-      );
-    });
-
-  return (
+  return isLoadingConnectors ? (
+    <SectionLoading>
+      <FormattedMessage
+        id="xpack.triggersActionsUI.sections.alertForm.loadingConnectorsDescription"
+        defaultMessage="Loading connectors…"
+      />
+    </SectionLoading>
+  ) : (
     <Fragment>
-      {isLoadingConnectors ? (
-        <SectionLoading>
+      <EuiTitle size="s">
+        <h4>
           <FormattedMessage
-            id="xpack.triggersActionsUI.sections.alertForm.loadingConnectorsDescription"
-            defaultMessage="Loading connectors…"
+            defaultMessage="Actions"
+            id="xpack.triggersActionsUI.sections.alertForm.actionSectionsTitle"
           />
-        </SectionLoading>
-      ) : (
-        <Fragment>
-          <EuiTitle size="s">
-            <h4>
-              <FormattedMessage
-                defaultMessage="Actions"
-                id="xpack.triggersActionsUI.sections.alertForm.actionSectionsTitle"
+        </h4>
+      </EuiTitle>
+      <EuiSpacer size="m" />
+      {actionTypesIndex &&
+        actions.map((actionItem: AlertAction, index: number) => {
+          const actionConnector = connectors.find((field) => field.id === actionItem.id);
+          // connectors doesn't exists
+          if (!actionConnector) {
+            return (
+              <AddConnectorInline
+                actionTypesIndex={actionTypesIndex}
+                actionItem={actionItem}
+                index={index}
+                key={`action-form-action-at-${index}`}
+                actionTypeRegistry={actionTypeRegistry}
+                defaultActionGroupId={defaultActionGroupId}
+                capabilities={capabilities}
+                emptyActionsIds={emptyActionsIds}
+                onDeleteConnector={() => {
+                  const updatedActions = actions.filter(
+                    (_item: AlertAction, i: number) => i !== index
+                  );
+                  setAlertProperty(updatedActions);
+                  setIsAddActionPanelOpen(
+                    updatedActions.filter((item: AlertAction) => item.id !== actionItem.id)
+                      .length === 0
+                  );
+                  setActiveActionItem(undefined);
+                }}
+                onAddConnector={() => {
+                  setActiveActionItem({ actionTypeId: actionItem.actionTypeId, index });
+                  setAddModalVisibility(true);
+                }}
               />
-            </h4>
-          </EuiTitle>
-          <EuiSpacer size="m" />
-          {alertActionsList}
-          <EuiSpacer size="m" />
-          {isAddActionPanelOpen ? (
-            <Fragment>
-              <EuiFlexGroup id="alertActionTypeTitle" justifyContent="spaceBetween">
-                <EuiFlexItem grow={false}>
-                  <EuiTitle size="xs">
-                    <h5>
-                      <FormattedMessage
-                        defaultMessage="Select an action type"
-                        id="xpack.triggersActionsUI.sections.alertForm.selectAlertActionTypeTitle"
-                      />
-                    </h5>
-                  </EuiTitle>
-                </EuiFlexItem>
-                {hasDisabledByLicenseActionTypes && (
-                  <EuiFlexItem grow={false}>
-                    <EuiTitle size="xs">
-                      <h5>
-                        <EuiLink
-                          href={VIEW_LICENSE_OPTIONS_LINK}
-                          target="_blank"
-                          external
-                          className="actActionForm__getMoreActionsLink"
-                        >
-                          <FormattedMessage
-                            defaultMessage="Get more actions"
-                            id="xpack.triggersActionsUI.sections.actionForm.getMoreActionsTitle"
-                          />
-                        </EuiLink>
-                      </h5>
-                    </EuiTitle>
-                  </EuiFlexItem>
-                )}
-              </EuiFlexGroup>
-              <EuiSpacer />
-              <EuiFlexGroup gutterSize="m" wrap>
-                {isLoadingActionTypes ? (
-                  <SectionLoading>
-                    <FormattedMessage
-                      id="xpack.triggersActionsUI.sections.alertForm.loadingActionTypesDescription"
-                      defaultMessage="Loading action types…"
-                    />
-                  </SectionLoading>
-                ) : (
-                  actionTypeNodes
-                )}
-              </EuiFlexGroup>
-            </Fragment>
-          ) : (
-            <EuiFlexGroup>
-              <EuiFlexItem grow={false}>
-                <EuiButton
-                  size="s"
-                  data-test-subj="addAlertActionButton"
-                  onClick={() => setIsAddActionPanelOpen(true)}
-                >
+            );
+          }
+
+          const actionParamsErrors: ActionTypeFormProps['actionParamsErrors'] = actionTypeRegistry
+            .get(actionItem.actionTypeId)
+            ?.validateParams(actionItem.params);
+
+          return (
+            <ActionTypeForm
+              actionItem={actionItem}
+              actionConnector={actionConnector}
+              actionParamsErrors={actionParamsErrors}
+              index={index}
+              setActionParamsProperty={setActionParamsProperty}
+              actionTypesIndex={actionTypesIndex}
+              connectors={connectors}
+              http={http}
+              toastNotifications={toastNotifications}
+              docLinks={docLinks}
+              capabilities={capabilities}
+              actionTypeRegistry={actionTypeRegistry}
+              defaultActionGroupId={defaultActionGroupId}
+              defaultActionMessage={defaultActionMessage}
+              messageVariables={messageVariables}
+              actionGroups={actionGroups}
+              setActionGroupIdByIndex={setActionGroupIdByIndex}
+              onAddConnector={() => {
+                setActiveActionItem({ actionTypeId: actionItem.actionTypeId, index });
+                setAddModalVisibility(true);
+              }}
+              onConnectorSelected={(id: string) => {
+                setActionIdByIndex(id, index);
+              }}
+              onDeleteAction={() => {
+                const updatedActions = actions.filter(
+                  (_item: AlertAction, i: number) => i !== index
+                );
+                setAlertProperty(updatedActions);
+                setIsAddActionPanelOpen(
+                  updatedActions.filter((item: AlertAction) => item.id !== actionItem.id).length ===
+                    0
+                );
+                setActiveActionItem(undefined);
+              }}
+            />
+          );
+        })}
+      <EuiSpacer size="m" />
+      {isAddActionPanelOpen ? (
+        <Fragment>
+          <EuiFlexGroup id="alertActionTypeTitle" justifyContent="spaceBetween">
+            <EuiFlexItem grow={false}>
+              <EuiTitle size="xs">
+                <h5>
                   <FormattedMessage
-                    id="xpack.triggersActionsUI.sections.alertForm.addActionButtonLabel"
-                    defaultMessage="Add action"
+                    defaultMessage="Select an action type"
+                    id="xpack.triggersActionsUI.sections.alertForm.selectAlertActionTypeTitle"
                   />
-                </EuiButton>
+                </h5>
+              </EuiTitle>
+            </EuiFlexItem>
+            {hasDisabledByLicenseActionTypes && (
+              <EuiFlexItem grow={false}>
+                <EuiTitle size="xs">
+                  <h5>
+                    <EuiLink
+                      href={VIEW_LICENSE_OPTIONS_LINK}
+                      target="_blank"
+                      external
+                      className="actActionForm__getMoreActionsLink"
+                    >
+                      <FormattedMessage
+                        defaultMessage="Get more actions"
+                        id="xpack.triggersActionsUI.sections.actionForm.getMoreActionsTitle"
+                      />
+                    </EuiLink>
+                  </h5>
+                </EuiTitle>
               </EuiFlexItem>
-            </EuiFlexGroup>
-          )}
+            )}
+          </EuiFlexGroup>
+          <EuiSpacer />
+          <EuiFlexGroup gutterSize="m" wrap>
+            {isLoadingActionTypes ? (
+              <SectionLoading>
+                <FormattedMessage
+                  id="xpack.triggersActionsUI.sections.alertForm.loadingActionTypesDescription"
+                  defaultMessage="Loading action types…"
+                />
+              </SectionLoading>
+            ) : (
+              actionTypeNodes
+            )}
+          </EuiFlexGroup>
         </Fragment>
+      ) : (
+        <EuiFlexGroup>
+          <EuiFlexItem grow={false}>
+            <EuiButton
+              size="s"
+              data-test-subj="addAlertActionButton"
+              onClick={() => setIsAddActionPanelOpen(true)}
+            >
+              <FormattedMessage
+                id="xpack.triggersActionsUI.sections.alertForm.addActionButtonLabel"
+                defaultMessage="Add action"
+              />
+            </EuiButton>
+          </EuiFlexItem>
+        </EuiFlexGroup>
       )}
       {actionTypesIndex && activeActionItem && addModalVisible ? (
         <ConnectorAddModal
diff --git a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_type_form.tsx b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_type_form.tsx
index 8d33001ed3d093..38468283b9c197 100644
--- a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_type_form.tsx
+++ b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/action_type_form.tsx
@@ -337,6 +337,3 @@ export const ActionTypeForm = ({
     </Fragment>
   );
 };
-
-// eslint-disable-next-line import/no-default-export
-export { ActionTypeForm as default };
diff --git a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_inline.tsx b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_inline.tsx
index 45ef4ceab461e8..97baf4a36cb4cc 100644
--- a/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_inline.tsx
+++ b/x-pack/plugins/triggers_actions_ui/public/application/sections/action_connector_form/connector_add_inline.tsx
@@ -151,6 +151,3 @@ export const AddConnectorInline = ({
     </Fragment>
   );
 };
-
-// eslint-disable-next-line import/no-default-export
-export { AddConnectorInline as default };