From bc08c02b896db8a9691bb89c79603b25e32e5594 Mon Sep 17 00:00:00 2001
From: mabashian <mabashia@redhat.com>
Date: Thu, 6 Feb 2020 10:40:24 -0500
Subject: [PATCH] Adds ID's to toolbar(s) and action buttons within
 visualizer/output toolbars.

---
 .../Job/WorkflowOutput/WorkflowOutputToolbar.jsx      |  6 +++---
 .../VisualizerToolbar.jsx                             | 11 ++++++++---
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/awx/ui_next/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.jsx b/awx/ui_next/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.jsx
index 3f0410ce55..26907ab3e4 100644
--- a/awx/ui_next/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.jsx
+++ b/awx/ui_next/src/screens/Job/WorkflowOutput/WorkflowOutputToolbar.jsx
@@ -65,7 +65,7 @@ function WorkflowOutputToolbar({ i18n, job }) {
   const totalNodes = nodes.reduce((n, node) => n + !node.isDeleted, 0) - 1;
 
   return (
-    <Toolbar>
+    <Toolbar id="workflow-output-toolbar">
       <ToolbarJob>
         <StatusIconWithMargin status={job.status} />
         <b>{job.name}</b>
@@ -76,7 +76,7 @@ function WorkflowOutputToolbar({ i18n, job }) {
         <VerticalSeparator />
         <Tooltip content={i18n._(t`Toggle Legend`)} position="bottom">
           <ActionButton
-            id="workflow-output-legend-button"
+            id="workflow-output-toggle-legend"
             isActive={showLegend}
             onClick={() => dispatch({ type: 'TOGGLE_LEGEND' })}
             variant="plain"
@@ -86,7 +86,7 @@ function WorkflowOutputToolbar({ i18n, job }) {
         </Tooltip>
         <Tooltip content={i18n._(t`Toggle Tools`)} position="bottom">
           <ActionButton
-            id="workflow-output-tools-button"
+            id="workflow-output-toggle-tools"
             isActive={showTools}
             onClick={() => dispatch({ type: 'TOGGLE_TOOLS' })}
             variant="plain"
diff --git a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/VisualizerToolbar.jsx b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/VisualizerToolbar.jsx
index 701f7d1250..cf7d2fedbb 100644
--- a/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/VisualizerToolbar.jsx
+++ b/awx/ui_next/src/screens/Template/WorkflowJobTemplateVisualizer/VisualizerToolbar.jsx
@@ -48,7 +48,7 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
   const totalNodes = nodes.reduce((n, node) => n + !node.isDeleted, 0) - 1;
 
   return (
-    <div>
+    <div id="visualizer-toolbar">
       <div css="align-items: center; border-bottom: 1px solid grey; display: flex; height: 56px; padding: 0px 20px;">
         <div css="display: flex;">
           <b>{i18n._(t`Workflow Visualizer`)}</b>
@@ -61,6 +61,7 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
           <VerticalSeparator />
           <Tooltip content={i18n._(t`Toggle Legend`)} position="bottom">
             <ActionButton
+              id="visualizer-toggle-legend"
               isActive={showLegend}
               onClick={() => dispatch({ type: 'TOGGLE_LEGEND' })}
               variant="plain"
@@ -70,6 +71,7 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
           </Tooltip>
           <Tooltip content={i18n._(t`Toggle Tools`)} position="bottom">
             <ActionButton
+              id="visualizer-toggle-tools"
               isActive={showTools}
               onClick={() => dispatch({ type: 'TOGGLE_TOOLS' })}
               variant="plain"
@@ -77,14 +79,15 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
               <WrenchIcon />
             </ActionButton>
           </Tooltip>
-          <ActionButton variant="plain" isDisabled>
+          <ActionButton id="visualizer-documentation" variant="plain" isDisabled>
             <BookIcon />
           </ActionButton>
-          <ActionButton variant="plain" isDisabled>
+          <ActionButton id="visualizer-launch" variant="plain" isDisabled>
             <RocketIcon />
           </ActionButton>
           <Tooltip content={i18n._(t`Delete All Nodes`)} position="bottom">
             <ActionButton
+              id="visualizer-delete-all"
               aria-label={i18n._(t`Delete all nodes`)}
               isDisabled={totalNodes === 0}
               onClick={() =>
@@ -100,6 +103,7 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
           </Tooltip>
           <VerticalSeparator />
           <Button
+            id="visualizer-save"
             aria-label={i18n._(t`Save`)}
             variant="primary"
             onClick={onSave}
@@ -108,6 +112,7 @@ function VisualizerToolbar({ i18n, onClose, onSave, template }) {
           </Button>
           <VerticalSeparator />
           <Button
+            id="visualizer-close"
             aria-label={i18n._(t`Close`)}
             onClick={onClose}
             variant="plain"