From 8402cf97de0888f11976b9b7c17e43574d1fcc76 Mon Sep 17 00:00:00 2001
From: nixocio <nixocio@gmail.com>
Date: Sun, 16 Aug 2020 14:57:00 -0400
Subject: [PATCH] Add type column to users list

Add type column to users list. Also, update `UserListItem` to be a
functional component.

See: https://github.com/ansible/awx/issues/5684
---
 .../screens/User/UserList/UserListItem.jsx    | 147 ++++++++++--------
 .../User/UserList/UserListItem.test.jsx       |   8 +-
 2 files changed, 89 insertions(+), 66 deletions(-)

diff --git a/awx/ui_next/src/screens/User/UserList/UserListItem.jsx b/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
index 19b8b5b476..b772a47566 100644
--- a/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
+++ b/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
@@ -19,72 +19,89 @@ import DataListCell from '../../../components/DataListCell';
 
 import { User } from '../../../types';
 
-class UserListItem extends React.Component {
-  static propTypes = {
-    user: User.isRequired,
-    detailUrl: string.isRequired,
-    isSelected: bool.isRequired,
-    onSelect: func.isRequired,
-  };
+function UserListItem({ user, isSelected, onSelect, detailUrl, i18n }) {
+  const labelId = `check-action-${user.id}`;
 
-  render() {
-    const { user, isSelected, onSelect, detailUrl, i18n } = this.props;
-    const labelId = `check-action-${user.id}`;
-    return (
-      <DataListItem key={user.id} aria-labelledby={labelId} id={`${user.id}`}>
-        <DataListItemRow>
-          <DataListCheck
-            id={`select-user-${user.id}`}
-            checked={isSelected}
-            onChange={onSelect}
-            aria-labelledby={labelId}
-          />
-          <DataListItemCells
-            dataListCells={[
-              <DataListCell key="username">
-                <Link to={`${detailUrl}`} id={labelId}>
-                  <b>{user.username}</b>
-                </Link>
-              </DataListCell>,
-              <DataListCell key="first-name">
-                {user.first_name && (
-                  <Fragment>
-                    <b css="margin-right: 24px">{i18n._(t`First Name`)}</b>
-                    {user.first_name}
-                  </Fragment>
-                )}
-              </DataListCell>,
-              <DataListCell key="last-name">
-                {user.last_name && (
-                  <Fragment>
-                    <b css="margin-right: 24px">{i18n._(t`Last Name`)}</b>
-                    {user.last_name}
-                  </Fragment>
-                )}
-              </DataListCell>,
-            ]}
-          />
-          <DataListAction
-            aria-label="actions"
-            aria-labelledby={labelId}
-            id={labelId}
-          >
-            {user.summary_fields.user_capabilities.edit && (
-              <Tooltip content={i18n._(t`Edit User`)} position="top">
-                <Button
-                  aria-label={i18n._(t`Edit User`)}
-                  variant="plain"
-                  component={Link}
-                  to={`/users/${user.id}/edit`}
-                >
-                  <PencilAltIcon />
-                </Button>
-              </Tooltip>
-            )}
-          </DataListAction>
-        </DataListItemRow>
-      </DataListItem>
-    );
+  let user_type;
+  if (user.is_superuser) {
+    user_type = i18n._(t`System Administrator`);
+  } else if (user.is_system_auditor) {
+    user_type = i18n._(t`System Auditor`);
+  } else {
+    user_type = i18n._(t`Normal User`);
   }
+
+  return (
+    <DataListItem key={user.id} aria-labelledby={labelId} id={`${user.id}`}>
+      <DataListItemRow>
+        <DataListCheck
+          id={`select-user-${user.id}`}
+          checked={isSelected}
+          onChange={onSelect}
+          aria-labelledby={labelId}
+        />
+        <DataListItemCells
+          dataListCells={[
+            <DataListCell key="username" aria-label={i18n._(t`username`)}>
+              <Link to={`${detailUrl}`} id={labelId}>
+                <b>{user.username}</b>
+              </Link>
+            </DataListCell>,
+            <DataListCell
+              key="first-name"
+              aria-label={i18n._(t`user first name`)}
+            >
+              {user.first_name && (
+                <Fragment>
+                  <b css="margin-right: 24px">{i18n._(t`First Name`)}</b>
+                  {user.first_name}
+                </Fragment>
+              )}
+            </DataListCell>,
+            <DataListCell
+              key="last-name"
+              aria-label={i18n._(t`user last name`)}
+            >
+              {user.last_name && (
+                <Fragment>
+                  <b css="margin-right: 24px">{i18n._(t`Last Name`)}</b>
+                  {user.last_name}
+                </Fragment>
+              )}
+            </DataListCell>,
+            <DataListCell key="user-type" aria-label={i18n._(t`user type`)}>
+              {user_type}
+            </DataListCell>,
+          ]}
+        />
+        <DataListAction
+          aria-label="actions"
+          aria-labelledby={labelId}
+          id={labelId}
+        >
+          {user.summary_fields.user_capabilities.edit && (
+            <Tooltip content={i18n._(t`Edit User`)} position="top">
+              <Button
+                aria-label={i18n._(t`Edit User`)}
+                variant="plain"
+                component={Link}
+                to={`/users/${user.id}/edit`}
+              >
+                <PencilAltIcon />
+              </Button>
+            </Tooltip>
+          )}
+        </DataListAction>
+      </DataListItemRow>
+    </DataListItem>
+  );
 }
+
+UserListItem.prototype = {
+  user: User.isRequired,
+  detailUrl: string.isRequired,
+  isSelected: bool.isRequired,
+  onSelect: func.isRequired,
+};
+
 export default withI18n()(UserListItem);
diff --git a/awx/ui_next/src/screens/User/UserList/UserListItem.test.jsx b/awx/ui_next/src/screens/User/UserList/UserListItem.test.jsx
index acdc7eecc0..e11b6da00c 100644
--- a/awx/ui_next/src/screens/User/UserList/UserListItem.test.jsx
+++ b/awx/ui_next/src/screens/User/UserList/UserListItem.test.jsx
@@ -28,12 +28,18 @@ describe('UserListItem with full permissions', () => {
       </I18nProvider>
     );
   });
-  test('initially renders succesfully', () => {
+  test('initially renders successfully', () => {
     expect(wrapper.length).toBe(1);
   });
   test('edit button shown to users with edit capabilities', () => {
     expect(wrapper.find('PencilAltIcon').exists()).toBeTruthy();
   });
+
+  test('should display user type', () => {
+    expect(
+      wrapper.find('DataListCell[aria-label="user type"]').prop('children')
+    ).toEqual('System Administrator');
+  });
 });
 
 describe('UserListItem without full permissions', () => {