From 77e99ad3555a5e78b74df74f069598db9c510353 Mon Sep 17 00:00:00 2001
From: mabashian <mabashia@redhat.com>
Date: Thu, 26 Mar 2020 12:57:31 -0400
Subject: [PATCH] Adds aria-label to some buttons without text

---
 .../src/components/JobList/JobListItem.jsx    |  6 +++-
 .../PageHeaderToolbar/PageHeaderToolbar.jsx   |  5 ++-
 .../ScheduleList/ScheduleListItem.jsx         |  1 +
 .../src/components/Sparkline/Sparkline.jsx    |  5 ++-
 .../CredentialList/CredentialListItem.jsx     |  1 +
 .../screens/Host/HostList/HostListItem.jsx    |  1 +
 .../InventoryGroupHostListItem.jsx            |  1 +
 .../InventoryGroups/InventoryGroupItem.jsx    |  7 ++++-
 .../InventoryList/InventoryListItem.jsx       |  1 +
 .../screens/Job/JobOutput/JobOutput.test.jsx  |  6 ++--
 .../screens/Job/JobOutput/PageControls.jsx    | 31 +++++++++++++++----
 .../Job/JobOutput/PageControls.test.jsx       |  6 ++--
 .../Job/JobOutput/shared/OutputToolbar.jsx    |  8 +++--
 .../OrganizationList/OrganizationListItem.jsx |  1 +
 .../Project/ProjectList/ProjectListItem.jsx   |  2 ++
 .../screens/Team/TeamList/TeamListItem.jsx    |  1 +
 .../TemplateList/TemplateListItem.jsx         |  2 ++
 .../screens/User/UserList/UserListItem.jsx    |  1 +
 18 files changed, 68 insertions(+), 18 deletions(-)

diff --git a/awx/ui_next/src/components/JobList/JobListItem.jsx b/awx/ui_next/src/components/JobList/JobListItem.jsx
index b38e85b11e..ea0eb86c3e 100644
--- a/awx/ui_next/src/components/JobList/JobListItem.jsx
+++ b/awx/ui_next/src/components/JobList/JobListItem.jsx
@@ -81,7 +81,11 @@ function JobListItem({
             <Tooltip content={i18n._(t`Relaunch Job`)} position="top">
               <LaunchButton resource={job}>
                 {({ handleRelaunch }) => (
-                  <Button variant="plain" onClick={handleRelaunch}>
+                  <Button
+                    variant="plain"
+                    onClick={handleRelaunch}
+                    aria-label={i18n._(t`Relaunch`)}
+                  >
                     <RocketIcon />
                   </Button>
                 )}
diff --git a/awx/ui_next/src/components/PageHeaderToolbar/PageHeaderToolbar.jsx b/awx/ui_next/src/components/PageHeaderToolbar/PageHeaderToolbar.jsx
index 1adce89e4e..229136c23b 100644
--- a/awx/ui_next/src/components/PageHeaderToolbar/PageHeaderToolbar.jsx
+++ b/awx/ui_next/src/components/PageHeaderToolbar/PageHeaderToolbar.jsx
@@ -72,7 +72,10 @@ class PageHeaderToolbar extends Component {
                 position={DropdownPosition.right}
                 onSelect={this.handleHelpSelect}
                 toggle={
-                  <DropdownToggle onToggle={this.handleHelpToggle}>
+                  <DropdownToggle
+                    onToggle={this.handleHelpToggle}
+                    aria-label={i18n._(t`Info`)}
+                  >
                     <QuestionCircleIcon />
                   </DropdownToggle>
                 }
diff --git a/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.jsx b/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.jsx
index 758658a950..6e36a73e16 100644
--- a/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.jsx
+++ b/awx/ui_next/src/components/Schedule/ScheduleList/ScheduleListItem.jsx
@@ -107,6 +107,7 @@ function ScheduleListItem({ i18n, isSelected, onSelect, schedule }) {
               {schedule.summary_fields.user_capabilities.edit ? (
                 <Tooltip content={i18n._(t`Edit Schedule`)} position="top">
                   <Button
+                    aria-label={i18n._(t`Edit Schedule`)}
                     css="grid-column: 2"
                     variant="plain"
                     component={Link}
diff --git a/awx/ui_next/src/components/Sparkline/Sparkline.jsx b/awx/ui_next/src/components/Sparkline/Sparkline.jsx
index 06ec23ae3a..e4faa123df 100644
--- a/awx/ui_next/src/components/Sparkline/Sparkline.jsx
+++ b/awx/ui_next/src/components/Sparkline/Sparkline.jsx
@@ -38,7 +38,10 @@ const Sparkline = ({ i18n, jobs }) => {
 
   const statusIcons = jobs.map(job => (
     <Tooltip position="top" content={generateTooltip(job)} key={job.id}>
-      <Link to={`/jobs/${JOB_TYPE_URL_SEGMENTS[job.type]}/${job.id}`}>
+      <Link
+        aria-label={i18n._(t`View job ${job.id}`)}
+        to={`/jobs/${JOB_TYPE_URL_SEGMENTS[job.type]}/${job.id}`}
+      >
         <StatusIcon status={job.status} />
       </Link>
     </Tooltip>
diff --git a/awx/ui_next/src/screens/Credential/CredentialList/CredentialListItem.jsx b/awx/ui_next/src/screens/Credential/CredentialList/CredentialListItem.jsx
index 17874efb12..5ef3a86c51 100644
--- a/awx/ui_next/src/screens/Credential/CredentialList/CredentialListItem.jsx
+++ b/awx/ui_next/src/screens/Credential/CredentialList/CredentialListItem.jsx
@@ -68,6 +68,7 @@ function CredentialListItem({
           {canEdit ? (
             <Tooltip content={i18n._(t`Edit Credential`)} position="top">
               <Button
+                aria-label={i18n._(t`Edit Credential`)}
                 variant="plain"
                 component={Link}
                 to={`/credentials/${credential.id}/edit`}
diff --git a/awx/ui_next/src/screens/Host/HostList/HostListItem.jsx b/awx/ui_next/src/screens/Host/HostList/HostListItem.jsx
index 08df94becc..fd2c6b0335 100644
--- a/awx/ui_next/src/screens/Host/HostList/HostListItem.jsx
+++ b/awx/ui_next/src/screens/Host/HostList/HostListItem.jsx
@@ -81,6 +81,7 @@ function HostListItem({ i18n, host, isSelected, onSelect, detailUrl }) {
           {host.summary_fields.user_capabilities.edit ? (
             <Tooltip content={i18n._(t`Edit Host`)} position="top">
               <Button
+                aria-label={i18n._(t`Edit Host`)}
                 variant="plain"
                 component={Link}
                 to={`/hosts/${host.id}/edit`}
diff --git a/awx/ui_next/src/screens/Inventory/InventoryGroupHosts/InventoryGroupHostListItem.jsx b/awx/ui_next/src/screens/Inventory/InventoryGroupHosts/InventoryGroupHostListItem.jsx
index 17a2f85757..974595214e 100644
--- a/awx/ui_next/src/screens/Inventory/InventoryGroupHosts/InventoryGroupHostListItem.jsx
+++ b/awx/ui_next/src/screens/Inventory/InventoryGroupHosts/InventoryGroupHostListItem.jsx
@@ -73,6 +73,7 @@ function InventoryGroupHostListItem({
           {host.summary_fields.user_capabilities?.edit && (
             <Tooltip content={i18n._(t`Edit Host`)} position="top">
               <Button
+                aria-label={i18n._(t`Edit Host`)}
                 css="grid-column: 2"
                 variant="plain"
                 component={Link}
diff --git a/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupItem.jsx b/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupItem.jsx
index 45e1bee411..9ea542dad7 100644
--- a/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupItem.jsx
+++ b/awx/ui_next/src/screens/Inventory/InventoryGroups/InventoryGroupItem.jsx
@@ -54,7 +54,12 @@ function InventoryGroupItem({
         >
           {group.summary_fields.user_capabilities.edit && (
             <Tooltip content={i18n._(t`Edit Group`)} position="top">
-              <Button variant="plain" component={Link} to={editUrl}>
+              <Button
+                aria-label={i18n._(t`Edit Group`)}
+                variant="plain"
+                component={Link}
+                to={editUrl}
+              >
                 <PencilAltIcon />
               </Button>
             </Tooltip>
diff --git a/awx/ui_next/src/screens/Inventory/InventoryList/InventoryListItem.jsx b/awx/ui_next/src/screens/Inventory/InventoryList/InventoryListItem.jsx
index e83ffb2ef8..7457c43332 100644
--- a/awx/ui_next/src/screens/Inventory/InventoryList/InventoryListItem.jsx
+++ b/awx/ui_next/src/screens/Inventory/InventoryList/InventoryListItem.jsx
@@ -72,6 +72,7 @@ class InventoryListItem extends React.Component {
             {inventory.summary_fields.user_capabilities.edit ? (
               <Tooltip content={i18n._(t`Edit Inventory`)} position="top">
                 <Button
+                  aria-label={i18n._(t`Edit Inventory`)}
                   variant="plain"
                   component={Link}
                   to={`/inventories/${
diff --git a/awx/ui_next/src/screens/Job/JobOutput/JobOutput.test.jsx b/awx/ui_next/src/screens/Job/JobOutput/JobOutput.test.jsx
index 2d69b6cdd5..3e16411866 100644
--- a/awx/ui_next/src/screens/Job/JobOutput/JobOutput.test.jsx
+++ b/awx/ui_next/src/screens/Job/JobOutput/JobOutput.test.jsx
@@ -23,13 +23,13 @@ async function checkOutput(wrapper, expectedLines) {
 async function findScrollButtons(wrapper) {
   const pageControls = await waitForElement(wrapper, 'PageControls');
   const scrollFirstButton = pageControls.find(
-    'button[aria-label="scroll first"]'
+    'button[aria-label="Scroll first"]'
   );
   const scrollLastButton = pageControls.find(
-    'button[aria-label="scroll last"]'
+    'button[aria-label="Scroll last"]'
   );
   const scrollPreviousButton = pageControls.find(
-    'button[aria-label="scroll previous"]'
+    'button[aria-label="Scroll previous"]'
   );
   return {
     scrollFirstButton,
diff --git a/awx/ui_next/src/screens/Job/JobOutput/PageControls.jsx b/awx/ui_next/src/screens/Job/JobOutput/PageControls.jsx
index aec23d55e7..2cea2d0db0 100644
--- a/awx/ui_next/src/screens/Job/JobOutput/PageControls.jsx
+++ b/awx/ui_next/src/screens/Job/JobOutput/PageControls.jsx
@@ -1,4 +1,6 @@
 import React from 'react';
+import { withI18n } from '@lingui/react';
+import { t } from '@lingui/macro';
 import { Button as PFButton } from '@patternfly/react-core';
 import {
   PlusIcon,
@@ -22,32 +24,49 @@ const Button = styled(PFButton)`
 `;
 
 const PageControls = ({
+  i18n,
   onScrollFirst,
   onScrollLast,
   onScrollNext,
   onScrollPrevious,
 }) => (
   <Wrapper>
-    <Button variant="plain" css="margin-right: auto">
+    <Button
+      aria-label={i18n._(t`Toggle expand/collapse event lines`)}
+      variant="plain"
+      css="margin-right: auto"
+    >
       <PlusIcon />
     </Button>
     <Button
-      aria-label="scroll previous"
+      aria-label={i18n._(t`Scroll previous`)}
       onClick={onScrollPrevious}
       variant="plain"
     >
       <AngleUpIcon />
     </Button>
-    <Button aria-label="scroll next" onClick={onScrollNext} variant="plain">
+    <Button
+      aria-label={i18n._(t`Scroll next`)}
+      onClick={onScrollNext}
+      variant="plain"
+    >
       <AngleDownIcon />
     </Button>
-    <Button aria-label="scroll first" onClick={onScrollFirst} variant="plain">
+    <Button
+      aria-label={i18n._(t`Scroll first`)}
+      onClick={onScrollFirst}
+      variant="plain"
+    >
       <AngleDoubleUpIcon />
     </Button>
-    <Button aria-label="scroll last" onClick={onScrollLast} variant="plain">
+    <Button
+      aria-label={i18n._(t`Scroll last`)}
+      onClick={onScrollLast}
+      variant="plain"
+    >
       <AngleDoubleDownIcon />
     </Button>
   </Wrapper>
 );
 
-export default PageControls;
+export default withI18n()(PageControls);
diff --git a/awx/ui_next/src/screens/Job/JobOutput/PageControls.test.jsx b/awx/ui_next/src/screens/Job/JobOutput/PageControls.test.jsx
index 210c33feb9..86df76448f 100644
--- a/awx/ui_next/src/screens/Job/JobOutput/PageControls.test.jsx
+++ b/awx/ui_next/src/screens/Job/JobOutput/PageControls.test.jsx
@@ -1,5 +1,5 @@
 import React from 'react';
-import { mount } from 'enzyme';
+import { mountWithContexts } from '@testUtils/enzymeHelpers';
 import PageControls from './PageControls';
 
 let wrapper;
@@ -19,12 +19,12 @@ const findChildren = () => {
 
 describe('PageControls', () => {
   test('should render successfully', () => {
-    wrapper = mount(<PageControls />);
+    wrapper = mountWithContexts(<PageControls />);
     expect(wrapper).toHaveLength(1);
   });
 
   test('should render menu control icons', () => {
-    wrapper = mount(<PageControls />);
+    wrapper = mountWithContexts(<PageControls />);
     findChildren();
     expect(PlusIcon.length).toBe(1);
     expect(AngleDoubleUpIcon.length).toBe(1);
diff --git a/awx/ui_next/src/screens/Job/JobOutput/shared/OutputToolbar.jsx b/awx/ui_next/src/screens/Job/JobOutput/shared/OutputToolbar.jsx
index 81e75938bc..c09728c5ae 100644
--- a/awx/ui_next/src/screens/Job/JobOutput/shared/OutputToolbar.jsx
+++ b/awx/ui_next/src/screens/Job/JobOutput/shared/OutputToolbar.jsx
@@ -127,7 +127,11 @@ const OutputToolbar = ({ i18n, job, onDelete }) => {
           <Tooltip content={i18n._(t`Relaunch Job`)}>
             <LaunchButton resource={job} aria-label={i18n._(t`Relaunch`)}>
               {({ handleRelaunch }) => (
-                <Button variant="plain" onClick={handleRelaunch}>
+                <Button
+                  variant="plain"
+                  onClick={handleRelaunch}
+                  aria-label={i18n._(t`Relaunch`)}
+                >
                   <RocketIcon />
                 </Button>
               )}
@@ -138,7 +142,7 @@ const OutputToolbar = ({ i18n, job, onDelete }) => {
       {job.related?.stdout && (
         <Tooltip content={i18n._(t`Download Output`)}>
           <a href={`${job.related.stdout}?format=txt_download`}>
-            <Button variant="plain">
+            <Button variant="plain" aria-label={i18n._(t`Download Output`)}>
               <DownloadIcon />
             </Button>
           </a>
diff --git a/awx/ui_next/src/screens/Organization/OrganizationList/OrganizationListItem.jsx b/awx/ui_next/src/screens/Organization/OrganizationList/OrganizationListItem.jsx
index efcc1edff6..9c1f66ae11 100644
--- a/awx/ui_next/src/screens/Organization/OrganizationList/OrganizationListItem.jsx
+++ b/awx/ui_next/src/screens/Organization/OrganizationList/OrganizationListItem.jsx
@@ -93,6 +93,7 @@ function OrganizationListItem({
           {organization.summary_fields.user_capabilities.edit ? (
             <Tooltip content={i18n._(t`Edit Organization`)} position="top">
               <Button
+                aria-label={i18n._(t`Edit Organization`)}
                 variant="plain"
                 component={Link}
                 to={`/organizations/${organization.id}/edit`}
diff --git a/awx/ui_next/src/screens/Project/ProjectList/ProjectListItem.jsx b/awx/ui_next/src/screens/Project/ProjectList/ProjectListItem.jsx
index f66afdb4ac..ff67e3541b 100644
--- a/awx/ui_next/src/screens/Project/ProjectList/ProjectListItem.jsx
+++ b/awx/ui_next/src/screens/Project/ProjectList/ProjectListItem.jsx
@@ -132,6 +132,7 @@ class ProjectListItem extends React.Component {
                 <ProjectSyncButton projectId={project.id}>
                   {handleSync => (
                     <Button
+                      aria-label={i18n._(t`Sync Project`)}
                       css="grid-column: 1"
                       variant="plain"
                       onClick={handleSync}
@@ -147,6 +148,7 @@ class ProjectListItem extends React.Component {
             {project.summary_fields.user_capabilities.edit ? (
               <Tooltip content={i18n._(t`Edit Project`)} position="top">
                 <Button
+                  aria-label={i18n._(t`Edit Project`)}
                   css="grid-column: 2"
                   variant="plain"
                   component={Link}
diff --git a/awx/ui_next/src/screens/Team/TeamList/TeamListItem.jsx b/awx/ui_next/src/screens/Team/TeamList/TeamListItem.jsx
index 5c3d09ca19..002fe6b446 100644
--- a/awx/ui_next/src/screens/Team/TeamList/TeamListItem.jsx
+++ b/awx/ui_next/src/screens/Team/TeamList/TeamListItem.jsx
@@ -76,6 +76,7 @@ class TeamListItem extends React.Component {
             {team.summary_fields.user_capabilities.edit ? (
               <Tooltip content={i18n._(t`Edit Team`)} position="top">
                 <Button
+                  aria-label={i18n._(t`Edit Team`)}
                   variant="plain"
                   component={Link}
                   to={`/teams/${team.id}/edit`}
diff --git a/awx/ui_next/src/screens/Template/TemplateList/TemplateListItem.jsx b/awx/ui_next/src/screens/Template/TemplateList/TemplateListItem.jsx
index e22612c028..991f9b91c0 100644
--- a/awx/ui_next/src/screens/Template/TemplateList/TemplateListItem.jsx
+++ b/awx/ui_next/src/screens/Template/TemplateList/TemplateListItem.jsx
@@ -89,6 +89,7 @@ function TemplateListItem({ i18n, template, isSelected, onSelect, detailUrl }) {
               <LaunchButton resource={template}>
                 {({ handleLaunch }) => (
                   <Button
+                    aria-label={i18n._(t`Launch template`)}
                     css="grid-column: 1"
                     variant="plain"
                     onClick={handleLaunch}
@@ -102,6 +103,7 @@ function TemplateListItem({ i18n, template, isSelected, onSelect, detailUrl }) {
           {template.summary_fields.user_capabilities.edit ? (
             <Tooltip content={i18n._(t`Edit Template`)} position="top">
               <Button
+                aria-label={i18n._(t`Edit Template`)}
                 css="grid-column: 2"
                 variant="plain"
                 component={Link}
diff --git a/awx/ui_next/src/screens/User/UserList/UserListItem.jsx b/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
index 2783d378d9..0cbad39438 100644
--- a/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
+++ b/awx/ui_next/src/screens/User/UserList/UserListItem.jsx
@@ -71,6 +71,7 @@ class UserListItem extends React.Component {
             {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`}