From 1c2605be8e8f18bf9a0a8ebf6feff2a55e769cd0 Mon Sep 17 00:00:00 2001
From: Marliana Lara <mlara@redhat.com>
Date: Mon, 11 Jan 2021 16:38:32 -0500
Subject: [PATCH] Add ouiaId props to button and alert components

---
 .../ActivityStreamDetail/ActivityStreamDetail.jsx              | 1 +
 .../screens/Setting/AzureAD/AzureADDetail/AzureADDetail.jsx    | 1 +
 .../src/screens/Setting/GitHub/GitHubDetail/GitHubDetail.jsx   | 1 +
 .../GoogleOAuth2/GoogleOAuth2Detail/GoogleOAuth2Detail.jsx     | 1 +
 awx/ui_next/src/screens/Setting/Jobs/JobsDetail/JobsDetail.jsx | 1 +
 awx/ui_next/src/screens/Setting/LDAP/LDAPDetail/LDAPDetail.jsx | 1 +
 .../screens/Setting/License/LicenseDetail/LicenseDetail.jsx    | 1 +
 .../screens/Setting/Logging/LoggingDetail/LoggingDetail.jsx    | 1 +
 .../src/screens/Setting/Logging/LoggingEdit/LoggingEdit.jsx    | 1 +
 .../Setting/MiscSystem/MiscSystemDetail/MiscSystemDetail.jsx   | 1 +
 .../src/screens/Setting/RADIUS/RADIUSDetail/RADIUSDetail.jsx   | 1 +
 awx/ui_next/src/screens/Setting/SAML/SAMLDetail/SAMLDetail.jsx | 1 +
 .../src/screens/Setting/TACACS/TACACSDetail/TACACSDetail.jsx   | 1 +
 awx/ui_next/src/screens/Setting/UI/UIDetail/UIDetail.jsx       | 1 +
 awx/ui_next/src/screens/Setting/shared/LoggingTestAlert.jsx    | 1 +
 awx/ui_next/src/screens/Setting/shared/RevertAllAlert.jsx      | 3 +++
 .../src/screens/Setting/shared/RevertFormActionGroup.jsx       | 3 +++
 awx/ui_next/src/screens/Setting/shared/SharedFields.jsx        | 1 +
 18 files changed, 22 insertions(+)

diff --git a/awx/ui_next/src/screens/Setting/ActivityStream/ActivityStreamDetail/ActivityStreamDetail.jsx b/awx/ui_next/src/screens/Setting/ActivityStream/ActivityStreamDetail/ActivityStreamDetail.jsx
index d5e6afe101..55bcb74db0 100644
--- a/awx/ui_next/src/screens/Setting/ActivityStream/ActivityStreamDetail/ActivityStreamDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/ActivityStream/ActivityStreamDetail/ActivityStreamDetail.jsx
@@ -86,6 +86,7 @@ function ActivityStreamDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/activity_stream/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/AzureAD/AzureADDetail/AzureADDetail.jsx b/awx/ui_next/src/screens/Setting/AzureAD/AzureADDetail/AzureADDetail.jsx
index 889ac19163..aa3528f3a3 100644
--- a/awx/ui_next/src/screens/Setting/AzureAD/AzureADDetail/AzureADDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/AzureAD/AzureADDetail/AzureADDetail.jsx
@@ -78,6 +78,7 @@ function AzureADDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/azure/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/GitHub/GitHubDetail/GitHubDetail.jsx b/awx/ui_next/src/screens/Setting/GitHub/GitHubDetail/GitHubDetail.jsx
index 1fd6da3f0d..5dc76348fb 100644
--- a/awx/ui_next/src/screens/Setting/GitHub/GitHubDetail/GitHubDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/GitHub/GitHubDetail/GitHubDetail.jsx
@@ -114,6 +114,7 @@ function GitHubDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to={`${baseURL}/${category}/edit`}
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/GoogleOAuth2/GoogleOAuth2Detail/GoogleOAuth2Detail.jsx b/awx/ui_next/src/screens/Setting/GoogleOAuth2/GoogleOAuth2Detail/GoogleOAuth2Detail.jsx
index f19f87378f..98a241f801 100644
--- a/awx/ui_next/src/screens/Setting/GoogleOAuth2/GoogleOAuth2Detail/GoogleOAuth2Detail.jsx
+++ b/awx/ui_next/src/screens/Setting/GoogleOAuth2/GoogleOAuth2Detail/GoogleOAuth2Detail.jsx
@@ -78,6 +78,7 @@ function GoogleOAuth2Detail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/google_oauth2/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/Jobs/JobsDetail/JobsDetail.jsx b/awx/ui_next/src/screens/Setting/Jobs/JobsDetail/JobsDetail.jsx
index c25c221518..a29a633a2d 100644
--- a/awx/ui_next/src/screens/Setting/Jobs/JobsDetail/JobsDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/Jobs/JobsDetail/JobsDetail.jsx
@@ -95,6 +95,7 @@ function JobsDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/jobs/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/LDAP/LDAPDetail/LDAPDetail.jsx b/awx/ui_next/src/screens/Setting/LDAP/LDAPDetail/LDAPDetail.jsx
index 120b00fa44..74850dfbe4 100644
--- a/awx/ui_next/src/screens/Setting/LDAP/LDAPDetail/LDAPDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/LDAP/LDAPDetail/LDAPDetail.jsx
@@ -159,6 +159,7 @@ function LDAPDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to={`${baseURL}/${category}/edit`}
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/License/LicenseDetail/LicenseDetail.jsx b/awx/ui_next/src/screens/Setting/License/LicenseDetail/LicenseDetail.jsx
index 233fb40895..05f551c512 100644
--- a/awx/ui_next/src/screens/Setting/License/LicenseDetail/LicenseDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/License/LicenseDetail/LicenseDetail.jsx
@@ -13,6 +13,7 @@ function LicenseDetail({ i18n }) {
         <Button
           aria-label={i18n._(t`Edit`)}
           component={Link}
+          ouiaId="edit-button"
           to="/settings/license/edit"
         >
           {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/Logging/LoggingDetail/LoggingDetail.jsx b/awx/ui_next/src/screens/Setting/Logging/LoggingDetail/LoggingDetail.jsx
index 5ede6a1839..54643f1773 100644
--- a/awx/ui_next/src/screens/Setting/Logging/LoggingDetail/LoggingDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/Logging/LoggingDetail/LoggingDetail.jsx
@@ -99,6 +99,7 @@ function LoggingDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/logging/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/Logging/LoggingEdit/LoggingEdit.jsx b/awx/ui_next/src/screens/Setting/Logging/LoggingEdit/LoggingEdit.jsx
index 074e6ab318..df488c8d3b 100644
--- a/awx/ui_next/src/screens/Setting/Logging/LoggingEdit/LoggingEdit.jsx
+++ b/awx/ui_next/src/screens/Setting/Logging/LoggingEdit/LoggingEdit.jsx
@@ -246,6 +246,7 @@ function LoggingEdit({ i18n }) {
                       <div>
                         <Button
                           aria-label={i18n._(t`Test logging`)}
+                          ouiaId="test-logging-button"
                           variant="secondary"
                           type="button"
                           onClick={handleTest}
diff --git a/awx/ui_next/src/screens/Setting/MiscSystem/MiscSystemDetail/MiscSystemDetail.jsx b/awx/ui_next/src/screens/Setting/MiscSystem/MiscSystemDetail/MiscSystemDetail.jsx
index 146e9d82a7..f92b3e00cf 100644
--- a/awx/ui_next/src/screens/Setting/MiscSystem/MiscSystemDetail/MiscSystemDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/MiscSystem/MiscSystemDetail/MiscSystemDetail.jsx
@@ -140,6 +140,7 @@ function MiscSystemDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/miscellaneous_system/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/RADIUS/RADIUSDetail/RADIUSDetail.jsx b/awx/ui_next/src/screens/Setting/RADIUS/RADIUSDetail/RADIUSDetail.jsx
index 52b313bbe0..d776f177c7 100644
--- a/awx/ui_next/src/screens/Setting/RADIUS/RADIUSDetail/RADIUSDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/RADIUS/RADIUSDetail/RADIUSDetail.jsx
@@ -78,6 +78,7 @@ function RADIUSDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/radius/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/SAML/SAMLDetail/SAMLDetail.jsx b/awx/ui_next/src/screens/Setting/SAML/SAMLDetail/SAMLDetail.jsx
index 0a7fac5806..5d2e54497e 100644
--- a/awx/ui_next/src/screens/Setting/SAML/SAMLDetail/SAMLDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/SAML/SAMLDetail/SAMLDetail.jsx
@@ -78,6 +78,7 @@ function SAMLDetail({ i18n }) {
             <Button
               aria-label={i18n._(t`Edit`)}
               component={Link}
+              ouiaId="edit-button"
               to="/settings/saml/edit"
             >
               {i18n._(t`Edit`)}
diff --git a/awx/ui_next/src/screens/Setting/TACACS/TACACSDetail/TACACSDetail.jsx b/awx/ui_next/src/screens/Setting/TACACS/TACACSDetail/TACACSDetail.jsx
index 6c9f1a85de..094dbf72bd 100644
--- a/awx/ui_next/src/screens/Setting/TACACS/TACACSDetail/TACACSDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/TACACS/TACACSDetail/TACACSDetail.jsx
@@ -79,6 +79,7 @@ function TACACSDetail({ i18n }) {
               aria-label={i18n._(t`Edit`)}
               component={Link}
               to="/settings/tacacs/edit"
+              ouiaId="edit-button"
             >
               {i18n._(t`Edit`)}
             </Button>
diff --git a/awx/ui_next/src/screens/Setting/UI/UIDetail/UIDetail.jsx b/awx/ui_next/src/screens/Setting/UI/UIDetail/UIDetail.jsx
index ef458e5163..e65f176b83 100644
--- a/awx/ui_next/src/screens/Setting/UI/UIDetail/UIDetail.jsx
+++ b/awx/ui_next/src/screens/Setting/UI/UIDetail/UIDetail.jsx
@@ -94,6 +94,7 @@ function UIDetail({ i18n }) {
               aria-label={i18n._(t`Edit`)}
               component={Link}
               to="/settings/ui/edit"
+              ouiaId="edit-button"
             >
               {i18n._(t`Edit`)}
             </Button>
diff --git a/awx/ui_next/src/screens/Setting/shared/LoggingTestAlert.jsx b/awx/ui_next/src/screens/Setting/shared/LoggingTestAlert.jsx
index a1c3903f53..4ec68182fc 100644
--- a/awx/ui_next/src/screens/Setting/shared/LoggingTestAlert.jsx
+++ b/awx/ui_next/src/screens/Setting/shared/LoggingTestAlert.jsx
@@ -33,6 +33,7 @@ function LoggingTestAlert({ i18n, successResponse, errorResponse, onClose }) {
       {testMessage && (
         <Alert
           actionClose={<AlertActionCloseButton onClose={onClose} />}
+          ouiaId="logging-test-alert"
           title={successResponse ? i18n._(t`Success`) : i18n._(t`Error`)}
           variant={successResponse ? 'success' : 'danger'}
         >
diff --git a/awx/ui_next/src/screens/Setting/shared/RevertAllAlert.jsx b/awx/ui_next/src/screens/Setting/shared/RevertAllAlert.jsx
index 55b23437b9..46ed00e8d6 100644
--- a/awx/ui_next/src/screens/Setting/shared/RevertAllAlert.jsx
+++ b/awx/ui_next/src/screens/Setting/shared/RevertAllAlert.jsx
@@ -11,12 +11,14 @@ function RevertAllAlert({ i18n, onClose, onRevertAll }) {
       title={i18n._(t`Revert settings`)}
       variant="info"
       onClose={onClose}
+      ouiaId="revert-all-modal"
       actions={[
         <Button
           key="revert"
           variant="primary"
           aria-label={i18n._(t`Confirm revert all`)}
           onClick={onRevertAll}
+          ouiaId="confirm-revert-all-button"
         >
           {i18n._(t`Revert all`)}
         </Button>,
@@ -25,6 +27,7 @@ function RevertAllAlert({ i18n, onClose, onRevertAll }) {
           variant="secondary"
           aria-label={i18n._(t`Cancel revert`)}
           onClick={onClose}
+          ouiaId="cancel-revert-all-button"
         >
           {i18n._(t`Cancel`)}
         </Button>,
diff --git a/awx/ui_next/src/screens/Setting/shared/RevertFormActionGroup.jsx b/awx/ui_next/src/screens/Setting/shared/RevertFormActionGroup.jsx
index 03e80d4eb1..1e707090e2 100644
--- a/awx/ui_next/src/screens/Setting/shared/RevertFormActionGroup.jsx
+++ b/awx/ui_next/src/screens/Setting/shared/RevertFormActionGroup.jsx
@@ -20,6 +20,7 @@ const RevertFormActionGroup = ({
           variant="primary"
           type="button"
           onClick={onSubmit}
+          ouiaId="save-button"
         >
           {i18n._(t`Save`)}
         </Button>
@@ -28,6 +29,7 @@ const RevertFormActionGroup = ({
           variant="secondary"
           type="button"
           onClick={onRevert}
+          ouiaId="revert-all-button"
         >
           {i18n._(t`Revert all to default`)}
         </Button>
@@ -37,6 +39,7 @@ const RevertFormActionGroup = ({
           variant="secondary"
           type="button"
           onClick={onCancel}
+          ouiaId="cancel-button"
         >
           {i18n._(t`Cancel`)}
         </Button>
diff --git a/awx/ui_next/src/screens/Setting/shared/SharedFields.jsx b/awx/ui_next/src/screens/Setting/shared/SharedFields.jsx
index c23a9da17a..2e27fd0ea3 100644
--- a/awx/ui_next/src/screens/Setting/shared/SharedFields.jsx
+++ b/awx/ui_next/src/screens/Setting/shared/SharedFields.jsx
@@ -90,6 +90,7 @@ const BooleanField = withI18n()(
           labelOff={i18n._(t`Off`)}
           onChange={checked => helpers.setValue(checked)}
           aria-label={ariaLabel || config.label}
+          ouiaId={ariaLabel || config.label}
         />
       </SettingGroup>
     ) : null;