From 4f55e86485b7fdc583f4948886eb857fd24a6869 Mon Sep 17 00:00:00 2001
From: Kersom <9053044+nixocio@users.noreply.github.com>
Date: Tue, 15 Sep 2020 20:25:23 -0400
Subject: [PATCH] Add list Execution Environments (#8148)

See: https://github.com/ansible/awx/issues/7886
---
 awx/ui_next/src/api/index.js                  |   3 +
 .../src/api/models/ExecutionEnvironments.js   |  10 +
 .../ExecutionEnviromentList.test.jsx          | 103 +++++++++
 .../ExecutionEnvironmentList.jsx              | 206 +++++++++++++++++-
 .../ExecutionEnvironmentListItem.jsx          |  85 ++++++++
 .../ExecutionEnvironmentListItem.test.jsx     |  52 +++++
 awx/ui_next/src/types.js                      |  10 +
 7 files changed, 461 insertions(+), 8 deletions(-)
 create mode 100644 awx/ui_next/src/api/models/ExecutionEnvironments.js
 create mode 100644 awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnviromentList.test.jsx
 create mode 100644 awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
 create mode 100644 awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.test.jsx

diff --git a/awx/ui_next/src/api/index.js b/awx/ui_next/src/api/index.js
index 3160ebd90734..d048237b7430 100644
--- a/awx/ui_next/src/api/index.js
+++ b/awx/ui_next/src/api/index.js
@@ -7,6 +7,7 @@ import CredentialInputSources from './models/CredentialInputSources';
 import CredentialTypes from './models/CredentialTypes';
 import Credentials from './models/Credentials';
 import Dashboard from './models/Dashboard';
+import ExecutionEnvironments from './models/ExecutionEnvironments';
 import Groups from './models/Groups';
 import Hosts from './models/Hosts';
 import InstanceGroups from './models/InstanceGroups';
@@ -50,6 +51,7 @@ const CredentialInputSourcesAPI = new CredentialInputSources();
 const CredentialTypesAPI = new CredentialTypes();
 const CredentialsAPI = new Credentials();
 const DashboardAPI = new Dashboard();
+const ExecutionEnvironmentsAPI = new ExecutionEnvironments();
 const GroupsAPI = new Groups();
 const HostsAPI = new Hosts();
 const InstanceGroupsAPI = new InstanceGroups();
@@ -94,6 +96,7 @@ export {
   CredentialTypesAPI,
   CredentialsAPI,
   DashboardAPI,
+  ExecutionEnvironmentsAPI,
   GroupsAPI,
   HostsAPI,
   InstanceGroupsAPI,
diff --git a/awx/ui_next/src/api/models/ExecutionEnvironments.js b/awx/ui_next/src/api/models/ExecutionEnvironments.js
new file mode 100644
index 000000000000..2df933d53a1a
--- /dev/null
+++ b/awx/ui_next/src/api/models/ExecutionEnvironments.js
@@ -0,0 +1,10 @@
+import Base from '../Base';
+
+class ExecutionEnvironments extends Base {
+  constructor(http) {
+    super(http);
+    this.baseUrl = '/api/v2/execution_environments/';
+  }
+}
+
+export default ExecutionEnvironments;
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnviromentList.test.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnviromentList.test.jsx
new file mode 100644
index 000000000000..8b042334044a
--- /dev/null
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnviromentList.test.jsx
@@ -0,0 +1,103 @@
+import React from 'react';
+import { act } from 'react-dom/test-utils';
+
+import {
+  mountWithContexts,
+  waitForElement,
+} from '../../../../testUtils/enzymeHelpers';
+
+import { ExecutionEnvironmentsAPI } from '../../../api';
+import ExecutionEnvironmentList from './ExecutionEnvironmentList';
+
+jest.mock('../../../api/models/ExecutionEnvironments');
+
+const executionEnvironments = {
+  data: {
+    results: [
+      {
+        id: 1,
+        image: 'https://registry.com/r/image/manifest',
+        organization: null,
+        credential: null,
+      },
+      {
+        id: 2,
+        image: 'https://registry.com/r/image2/manifest',
+        organization: null,
+        credential: null,
+      },
+    ],
+    count: 2,
+  },
+};
+
+const options = { data: { actions: { POST: true } } };
+
+describe('<ExecutionEnvironmentList/>', () => {
+  let wrapper;
+
+  test('should mount successfully', async () => {
+    await act(async () => {
+      wrapper = mountWithContexts(<ExecutionEnvironmentList />);
+    });
+    await waitForElement(
+      wrapper,
+      'ExecutionEnvironmentList',
+      el => el.length > 0
+    );
+  });
+
+  test('should have data fetched and render 2 rows', async () => {
+    ExecutionEnvironmentsAPI.read.mockResolvedValue(executionEnvironments);
+    ExecutionEnvironmentsAPI.readOptions.mockResolvedValue(options);
+
+    await act(async () => {
+      wrapper = mountWithContexts(<ExecutionEnvironmentList />);
+    });
+    await waitForElement(
+      wrapper,
+      'ExecutionEnvironmentList',
+      el => el.length > 0
+    );
+
+    expect(wrapper.find('ExecutionEnvironmentListItem').length).toBe(2);
+    expect(ExecutionEnvironmentsAPI.read).toBeCalled();
+    expect(ExecutionEnvironmentsAPI.readOptions).toBeCalled();
+  });
+
+  test('should thrown content error', async () => {
+    ExecutionEnvironmentsAPI.read.mockRejectedValue(
+      new Error({
+        response: {
+          config: {
+            method: 'GET',
+            url: '/api/v2/execution_environments',
+          },
+          data: 'An error occurred',
+        },
+      })
+    );
+    ExecutionEnvironmentsAPI.readOptions.mockResolvedValue(options);
+    await act(async () => {
+      wrapper = mountWithContexts(<ExecutionEnvironmentList />);
+    });
+    await waitForElement(
+      wrapper,
+      'ExecutionEnvironmentList',
+      el => el.length > 0
+    );
+    expect(wrapper.find('ContentError').length).toBe(1);
+  });
+
+  test('should not render add button', async () => {
+    ExecutionEnvironmentsAPI.read.mockResolvedValue(executionEnvironments);
+    ExecutionEnvironmentsAPI.readOptions.mockResolvedValue({
+      data: { actions: { POST: false } },
+    });
+    await act(async () => {
+      wrapper = mountWithContexts(<ExecutionEnvironmentList />);
+    });
+    waitForElement(wrapper, 'ExecutionEnvironmentList', el => el.length > 0);
+    expect(wrapper.find('ToolbarAddButton').length).toBe(0);
+  });
+});
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
index 96d2d07e1064..452fc4fbdb50 100644
--- a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
@@ -1,14 +1,204 @@
-import React from 'react';
+import React, { useEffect, useCallback } from 'react';
+import { useLocation, useRouteMatch } from 'react-router-dom';
+import { withI18n } from '@lingui/react';
+import { t } from '@lingui/macro';
 import { Card, PageSection } from '@patternfly/react-core';
 
-function ExecutionEnvironmentList() {
+import { ExecutionEnvironmentsAPI } from '../../../api';
+import { getQSConfig, parseQueryString } from '../../../util/qs';
+import useRequest, { useDeleteItems } from '../../../util/useRequest';
+import useSelected from '../../../util/useSelected';
+import PaginatedDataList, {
+  ToolbarDeleteButton,
+  ToolbarAddButton,
+} from '../../../components/PaginatedDataList';
+import ErrorDetail from '../../../components/ErrorDetail';
+import AlertModal from '../../../components/AlertModal';
+import DatalistToolbar from '../../../components/DataListToolbar';
+
+import ExecutionEnvironmentsListItem from './ExecutionEnvironmentListItem';
+
+const QS_CONFIG = getQSConfig('execution_environments', {
+  page: 1,
+  page_size: 20,
+  managed_by_tower: false,
+  order_by: 'image',
+});
+
+function ExecutionEnvironmentList({ i18n }) {
+  const location = useLocation();
+  const match = useRouteMatch();
+
+  const {
+    error: contentError,
+    isLoading,
+    request: fetchExecutionEnvironments,
+    result: {
+      executionEnvironments,
+      executionEnvironmentsCount,
+      actions,
+      relatedSearchableKeys,
+      searchableKeys,
+    },
+  } = useRequest(
+    useCallback(async () => {
+      const params = parseQueryString(QS_CONFIG, location.search);
+
+      const [response, responseActions] = await Promise.all([
+        ExecutionEnvironmentsAPI.read(params),
+        ExecutionEnvironmentsAPI.readOptions(),
+      ]);
+
+      return {
+        executionEnvironments: response.data.results,
+        executionEnvironmentsCount: response.data.count,
+        actions: responseActions.data.actions,
+        relatedSearchableKeys: (
+          responseActions?.data?.related_search_fields || []
+        ).map(val => val.slice(0, -8)),
+        searchableKeys: Object.keys(
+          responseActions.data.actions?.GET || {}
+        ).filter(key => responseActions.data.actions?.GET[key].filterable),
+      };
+    }, [location]),
+    {
+      executionEnvironments: [],
+      executionEnvironmentsCount: 0,
+      actions: {},
+      relatedSearchableKeys: [],
+      searchableKeys: [],
+    }
+  );
+
+  useEffect(() => {
+    fetchExecutionEnvironments();
+  }, [fetchExecutionEnvironments]);
+
+  const { selected, isAllSelected, handleSelect, setSelected } = useSelected(
+    executionEnvironments
+  );
+
+  const {
+    isLoading: deleteLoading,
+    deletionError,
+    deleteItems: deleteExecutionEnvironments,
+    clearDeletionError,
+  } = useDeleteItems(
+    useCallback(async () => {
+      await Promise.all(
+        selected.map(({ id }) => ExecutionEnvironmentsAPI.destroy(id))
+      );
+    }, [selected]),
+    {
+      qsConfig: QS_CONFIG,
+      allItemsSelected: isAllSelected,
+      fetchItems: fetchExecutionEnvironments,
+    }
+  );
+
+  const handleDelete = async () => {
+    await deleteExecutionEnvironments();
+    setSelected([]);
+  };
+
+  const canAdd = actions && actions.POST;
+
   return (
-    <PageSection>
-      <Card>
-        <div>List Execution environments</div>
-      </Card>
-    </PageSection>
+    <>
+      <PageSection>
+        <Card>
+          <PaginatedDataList
+            contentError={contentError}
+            hasContentLoading={isLoading || deleteLoading}
+            items={executionEnvironments}
+            itemCount={executionEnvironmentsCount}
+            pluralizedItemName={i18n._(t`Execution Environments`)}
+            qsConfig={QS_CONFIG}
+            onRowClick={handleSelect}
+            toolbarSearchableKeys={searchableKeys}
+            toolbarRelatedSearchableKeys={relatedSearchableKeys}
+            toolbarSearchColumns={[
+              {
+                name: i18n._(t`Image`),
+                key: 'image__icontains',
+                isDefault: true,
+              },
+            ]}
+            toolbarSortColumns={[
+              {
+                name: i18n._(t`Image`),
+                key: 'image',
+              },
+              {
+                name: i18n._(t`Created`),
+                key: 'created',
+              },
+              {
+                name: i18n._(t`Organization`),
+                key: 'organization',
+              },
+              {
+                name: i18n._(t`Description`),
+                key: 'description',
+              },
+            ]}
+            renderToolbar={props => (
+              <DatalistToolbar
+                {...props}
+                showSelectAll
+                isAllSelected={isAllSelected}
+                onSelectAll={isSelected =>
+                  setSelected(isSelected ? [...executionEnvironments] : [])
+                }
+                qsConfig={QS_CONFIG}
+                additionalControls={[
+                  ...(canAdd
+                    ? [
+                        <ToolbarAddButton
+                          key="add"
+                          linkTo={`${match.url}/add`}
+                        />,
+                      ]
+                    : []),
+                  <ToolbarDeleteButton
+                    key="delete"
+                    onDelete={handleDelete}
+                    itemsToDelete={selected}
+                    pluralizedItemName={i18n._(t`Execution Environments`)}
+                  />,
+                ]}
+              />
+            )}
+            renderItem={executionEnvironment => (
+              <ExecutionEnvironmentsListItem
+                executionEnvironment={executionEnvironment}
+                detailUrl={`${match.url}/${executionEnvironment.id}/details`}
+                onSelect={() => handleSelect(executionEnvironment)}
+                isSelected={selected.some(
+                  row => row.id === executionEnvironment.id
+                )}
+              />
+            )}
+            emptyStateControls={
+              canAdd && (
+                <ToolbarAddButton key="add" linkTo={`${match.url}/add`} />
+              )
+            }
+          />
+        </Card>
+      </PageSection>
+      <AlertModal
+        aria-label={i18n._(t`Deletion error`)}
+        isOpen={deletionError}
+        onClose={clearDeletionError}
+        title={i18n._(t`Error`)}
+        variant="error"
+      >
+        {i18n._(t`Failed to delete one or more execution environments`)}
+        <ErrorDetail error={deletionError} />
+      </AlertModal>
+    </>
   );
 }
 
-export default ExecutionEnvironmentList;
+export default withI18n()(ExecutionEnvironmentList);
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
new file mode 100644
index 000000000000..7789facfbb3f
--- /dev/null
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
@@ -0,0 +1,85 @@
+import React from 'react';
+import { string, bool, func } from 'prop-types';
+import { withI18n } from '@lingui/react';
+import { t } from '@lingui/macro';
+import { Link } from 'react-router-dom';
+import {
+  Button,
+  DataListAction,
+  DataListCheck,
+  DataListItem,
+  DataListItemRow,
+  DataListItemCells,
+  Tooltip,
+} from '@patternfly/react-core';
+import { PencilAltIcon } from '@patternfly/react-icons';
+
+import DataListCell from '../../../components/DataListCell';
+import { ExecutionEnvironment } from '../../../types';
+
+function ExecutionEnvironmentListItem({
+  executionEnvironment,
+  detailUrl,
+  isSelected,
+  onSelect,
+  i18n,
+}) {
+  const labelId = `check-action-${executionEnvironment.id}`;
+
+  return (
+    <DataListItem
+      key={executionEnvironment.id}
+      aria-labelledby={labelId}
+      id={`${executionEnvironment.id} `}
+    >
+      <DataListItemRow>
+        <DataListCheck
+          id={`select-execution-environment-${executionEnvironment.id}`}
+          checked={isSelected}
+          onChange={onSelect}
+          aria-labelledby={labelId}
+        />
+        <DataListItemCells
+          dataListCells={[
+            <DataListCell
+              key="image"
+              aria-label={i18n._(t`execution environment image`)}
+            >
+              <Link to={`${detailUrl}`}>
+                <b>{executionEnvironment.image}</b>
+              </Link>
+            </DataListCell>,
+          ]}
+        />
+        <DataListAction
+          aria-label="actions"
+          aria-labelledby={labelId}
+          id={labelId}
+        >
+          <Tooltip
+            content={i18n._(t`Edit execution environment`)}
+            position="top"
+          >
+            <Button
+              aria-label={i18n._(t`Edit execution environment`)}
+              variant="plain"
+              component={Link}
+              to={`/execution_environments/${executionEnvironment.id}/edit`}
+            >
+              <PencilAltIcon />
+            </Button>
+          </Tooltip>
+        </DataListAction>
+      </DataListItemRow>
+    </DataListItem>
+  );
+}
+
+ExecutionEnvironmentListItem.prototype = {
+  executionEnvironment: ExecutionEnvironment.isRequired,
+  detailUrl: string.isRequired,
+  isSelected: bool.isRequired,
+  onSelect: func.isRequired,
+};
+
+export default withI18n()(ExecutionEnvironmentListItem);
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.test.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.test.jsx
new file mode 100644
index 000000000000..4f51a5167284
--- /dev/null
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.test.jsx
@@ -0,0 +1,52 @@
+import React from 'react';
+import { act } from 'react-dom/test-utils';
+
+import { mountWithContexts } from '../../../../testUtils/enzymeHelpers';
+
+import ExecutionEnvironmentListItem from './ExecutionEnvironmentListItem';
+
+describe('<ExecutionEnvironmentListItem/>', () => {
+  let wrapper;
+  const executionEnvironment = {
+    id: 1,
+    image: 'https://registry.com/r/image/manifest',
+    organization: null,
+    credential: null,
+  };
+
+  test('should mount successfully', async () => {
+    await act(async () => {
+      wrapper = mountWithContexts(
+        <ExecutionEnvironmentListItem
+          executionEnvironment={executionEnvironment}
+          detailUrl="execution_environments/1/details"
+          isSelected={false}
+          onSelect={() => {}}
+        />
+      );
+    });
+    expect(wrapper.find('ExecutionEnvironmentListItem').length).toBe(1);
+  });
+
+  test('should render the proper data', async () => {
+    await act(async () => {
+      wrapper = mountWithContexts(
+        <ExecutionEnvironmentListItem
+          executionEnvironment={executionEnvironment}
+          detailUrl="execution_environments/1/details"
+          isSelected={false}
+          onSelect={() => {}}
+        />
+      );
+    });
+    expect(
+      wrapper
+        .find('DataListCell[aria-label="execution environment image"]')
+        .text()
+    ).toBe(executionEnvironment.image);
+    expect(wrapper.find('PencilAltIcon').length).toBe(1);
+    expect(
+      wrapper.find('input#select-execution-environment-1').prop('checked')
+    ).toBe(false);
+  });
+});
diff --git a/awx/ui_next/src/types.js b/awx/ui_next/src/types.js
index 760ecf4ed200..53c7ed3b3d17 100644
--- a/awx/ui_next/src/types.js
+++ b/awx/ui_next/src/types.js
@@ -407,3 +407,13 @@ export const WorkflowApproval = shape({
   approval_expiration: string,
   timed_out: bool,
 });
+
+export const ExecutionEnvironment = shape({
+  id: number.isRequired,
+  organization: number,
+  credential: number,
+  image: string.isRequired,
+  url: string,
+  summary_fields: shape({}),
+  description: string,
+});