From ec08997b63f0c727f6d630df223efa43e78ac878 Mon Sep 17 00:00:00 2001
From: nixocio <nixocio@gmail.com>
Date: Wed, 17 Mar 2021 11:26:10 -0400
Subject: [PATCH 1/2] Improve selectors to ease testing

Improve selectors to ease testing
---
 .../src/components/DeleteButton/DeleteButton.jsx      | 11 +++++++++++
 .../src/components/PaginatedTable/PaginatedTable.jsx  |  5 ++++-
 .../ExecutionEnvironmentList.jsx                      |  1 +
 .../OrganizationExecEnvListItem.jsx                   |  2 +-
 .../src/screens/User/UserTeams/UserTeamListItem.jsx   |  2 +-
 5 files changed, 18 insertions(+), 3 deletions(-)

diff --git a/awx/ui_next/src/components/DeleteButton/DeleteButton.jsx b/awx/ui_next/src/components/DeleteButton/DeleteButton.jsx
index 245e6ec54c..f17bd9ca95 100644
--- a/awx/ui_next/src/components/DeleteButton/DeleteButton.jsx
+++ b/awx/ui_next/src/components/DeleteButton/DeleteButton.jsx
@@ -1,4 +1,5 @@
 import React, { useState } from 'react';
+import PropTypes from 'prop-types';
 import { withI18n } from '@lingui/react';
 import { t } from '@lingui/macro';
 import { Button } from '@patternfly/react-core';
@@ -12,6 +13,7 @@ function DeleteButton({
   variant,
   children,
   isDisabled,
+  ouiaId,
 }) {
   const [isOpen, setIsOpen] = useState(false);
 
@@ -22,6 +24,7 @@ function DeleteButton({
         aria-label={i18n._(t`Delete`)}
         isDisabled={isDisabled}
         onClick={() => setIsOpen(true)}
+        ouiaId={ouiaId}
       >
         {children || i18n._(t`Delete`)}
       </Button>
@@ -58,4 +61,12 @@ function DeleteButton({
   );
 }
 
+DeleteButton.propTypes = {
+  ouiaId: PropTypes.string,
+};
+
+DeleteButton.defaultProps = {
+  ouiaId: null,
+};
+
 export default withI18n()(DeleteButton);
diff --git a/awx/ui_next/src/components/PaginatedTable/PaginatedTable.jsx b/awx/ui_next/src/components/PaginatedTable/PaginatedTable.jsx
index bf70c2acc6..2d33fe1e66 100644
--- a/awx/ui_next/src/components/PaginatedTable/PaginatedTable.jsx
+++ b/awx/ui_next/src/components/PaginatedTable/PaginatedTable.jsx
@@ -38,6 +38,7 @@ function PaginatedTable({
   i18n,
   renderToolbar,
   emptyContentMessage,
+  ouiaId,
 }) {
   const history = useHistory();
 
@@ -95,7 +96,7 @@ function PaginatedTable({
     Content = (
       <div css="overflow: auto">
         {hasContentLoading && <LoadingSpinner />}
-        <TableComposable aria-label={dataListLabel}>
+        <TableComposable aria-label={dataListLabel} ouiaId={ouiaId}>
           {headerRow}
           <Tbody>{items.map(renderRow)}</Tbody>
         </TableComposable>
@@ -181,6 +182,7 @@ PaginatedTable.propTypes = {
   renderToolbar: PropTypes.func,
   hasContentLoading: PropTypes.bool,
   contentError: PropTypes.shape(),
+  ouiaId: PropTypes.string,
 };
 
 PaginatedTable.defaultProps = {
@@ -192,6 +194,7 @@ PaginatedTable.defaultProps = {
   pluralizedItemName: 'Items',
   showPageSizeOptions: true,
   renderToolbar: props => <DataListToolbar {...props} />,
+  ouiaId: null,
 };
 
 export { PaginatedTable as _PaginatedTable };
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
index 312b18f2cf..278566e632 100644
--- a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentList.jsx
@@ -111,6 +111,7 @@ function ExecutionEnvironmentList({ i18n }) {
       <PageSection>
         <Card>
           <PaginatedTable
+            ouiaId="execution-environment-table"
             contentError={contentError}
             hasContentLoading={isLoading || deleteLoading}
             items={executionEnvironments}
diff --git a/awx/ui_next/src/screens/Organization/OrganizationExecEnvList/OrganizationExecEnvListItem.jsx b/awx/ui_next/src/screens/Organization/OrganizationExecEnvList/OrganizationExecEnvListItem.jsx
index 0d2715d7a6..da292712ea 100644
--- a/awx/ui_next/src/screens/Organization/OrganizationExecEnvList/OrganizationExecEnvListItem.jsx
+++ b/awx/ui_next/src/screens/Organization/OrganizationExecEnvList/OrganizationExecEnvListItem.jsx
@@ -23,7 +23,7 @@ function OrganizationExecEnvListItem({
     <DataListItem
       key={executionEnvironment.id}
       aria-labelledby={labelId}
-      id={`${executionEnvironment.id} `}
+      id={`${executionEnvironment.id}`}
     >
       <DataListItemRow>
         <DataListItemCells
diff --git a/awx/ui_next/src/screens/User/UserTeams/UserTeamListItem.jsx b/awx/ui_next/src/screens/User/UserTeams/UserTeamListItem.jsx
index 6df28bd4a7..d6f09e6761 100644
--- a/awx/ui_next/src/screens/User/UserTeams/UserTeamListItem.jsx
+++ b/awx/ui_next/src/screens/User/UserTeams/UserTeamListItem.jsx
@@ -32,7 +32,7 @@ function UserTeamListItem({ team, isSelected, onSelect, i18n }) {
           dataListCells={[
             <DataListCell key="name">
               <Link to={`/teams/${team.id}/details`} id={`team-${team.id}`}>
-                {team.name}
+                <b>{team.name}</b>
               </Link>
             </DataListCell>,
             <DataListCell key="organization">

From d93ef9f6de9b5bcab6f763421b8a6e75efbd56fb Mon Sep 17 00:00:00 2001
From: Tiago Goes <tiago.goes2009@gmail.com>
Date: Wed, 17 Mar 2021 14:35:00 -0300
Subject: [PATCH 2/2] add ouiaID

---
 awx/ui_next/src/components/FormActionGroup/FormActionGroup.jsx  | 2 ++
 .../ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx   | 1 +
 2 files changed, 3 insertions(+)

diff --git a/awx/ui_next/src/components/FormActionGroup/FormActionGroup.jsx b/awx/ui_next/src/components/FormActionGroup/FormActionGroup.jsx
index 43dc0be9e0..47b64f3ab5 100644
--- a/awx/ui_next/src/components/FormActionGroup/FormActionGroup.jsx
+++ b/awx/ui_next/src/components/FormActionGroup/FormActionGroup.jsx
@@ -10,6 +10,7 @@ const FormActionGroup = ({ onCancel, onSubmit, submitDisabled, i18n }) => {
     <FormFullWidthLayout>
       <ActionGroup>
         <Button
+          ouiaId="Save"
           aria-label={i18n._(t`Save`)}
           variant="primary"
           type="button"
@@ -19,6 +20,7 @@ const FormActionGroup = ({ onCancel, onSubmit, submitDisabled, i18n }) => {
           {i18n._(t`Save`)}
         </Button>
         <Button
+          ouiaId="Cancel"
           aria-label={i18n._(t`Cancel`)}
           variant="link"
           type="button"
diff --git a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
index bb814a1921..1f56b53727 100644
--- a/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
+++ b/awx/ui_next/src/screens/ExecutionEnvironment/ExecutionEnvironmentList/ExecutionEnvironmentListItem.jsx
@@ -56,6 +56,7 @@ function ExecutionEnvironmentListItem({
           tooltip={i18n._(t`Edit Execution Environment`)}
         >
           <Button
+            ouiaId={`edit-ee-${executionEnvironment.id}`}
             aria-label={i18n._(t`Edit Execution Environment`)}
             variant="plain"
             component={Link}