diff --git a/awx/ui_next/src/components/DetailList/CodeDetail.js b/awx/ui_next/src/components/DetailList/CodeDetail.js index 4bba4416c8..5e926c6f64 100644 --- a/awx/ui_next/src/components/DetailList/CodeDetail.js +++ b/awx/ui_next/src/components/DetailList/CodeDetail.js @@ -17,6 +17,7 @@ import Popover from '../Popover'; function CodeDetail({ value, label, mode, rows, helpText, dataCy }) { const labelCy = dataCy ? `${dataCy}-label` : null; const valueCy = dataCy ? `${dataCy}-value` : null; + const editorId = dataCy ? `${dataCy}-editor` : 'code-editor'; return ( <> @@ -27,12 +28,13 @@ function CodeDetail({ value, label, mode, rows, helpText, dataCy }) { data-cy={labelCy} >
- {label} - + {helpText && ( )} @@ -45,6 +47,7 @@ function CodeDetail({ value, label, mode, rows, helpText, dataCy }) { data-cy={valueCy} > (props.isEncrypted || props.isNotConfigured) && ` - color: var(--pf-global--Color--400); + color: var(--pf-global--disabled-color--100); `} `; diff --git a/awx/ui_next/src/components/ExecutionEnvironmentDetail/ExecutionEnvironmentDetail.js b/awx/ui_next/src/components/ExecutionEnvironmentDetail/ExecutionEnvironmentDetail.js index d59097310f..f6c2a02d31 100644 --- a/awx/ui_next/src/components/ExecutionEnvironmentDetail/ExecutionEnvironmentDetail.js +++ b/awx/ui_next/src/components/ExecutionEnvironmentDetail/ExecutionEnvironmentDetail.js @@ -30,6 +30,7 @@ function ExecutionEnvironmentDetail({ virtualEnvironment, verifyMissingVirtualEnv, helpText, + dataCy, }) { const config = useConfig(); const docsLink = `${getDocsBaseUrl( @@ -51,7 +52,7 @@ function ExecutionEnvironmentDetail({ } helpText={helpText} - dataCy="execution-environment-detail" + dataCy={dataCy} /> ); } @@ -89,7 +90,7 @@ function ExecutionEnvironmentDetail({ } - dataCy="missing-execution-environment-detail" + dataCy={`missing-${dataCy}`} /> ); } @@ -113,7 +114,7 @@ function ExecutionEnvironmentDetail({ } - dataCy="execution-environment-detail" + dataCy={dataCy} /> ); } @@ -127,6 +128,7 @@ ExecutionEnvironmentDetail.propTypes = { virtualEnvironment: string, verifyMissingVirtualEnv: bool, helpText: string, + dataCy: string, }; ExecutionEnvironmentDetail.defaultProps = { @@ -135,6 +137,7 @@ ExecutionEnvironmentDetail.defaultProps = { virtualEnvironment: '', verifyMissingVirtualEnv: true, helpText: '', + dataCy: 'execution-environment-detail', }; export default ExecutionEnvironmentDetail; diff --git a/awx/ui_next/src/components/JobList/JobListItem.js b/awx/ui_next/src/components/JobList/JobListItem.js index 40feb84a90..e244c08ab9 100644 --- a/awx/ui_next/src/components/JobList/JobListItem.js +++ b/awx/ui_next/src/components/JobList/JobListItem.js @@ -124,6 +124,7 @@ function JobListItem({ )} @@ -214,6 +215,7 @@ function JobListItem({ )} {credentials && credentials.length > 0 && ( diff --git a/awx/ui_next/src/components/LaunchButton/ReLaunchDropDown.js b/awx/ui_next/src/components/LaunchButton/ReLaunchDropDown.js index 28d969e5fa..c5fac44a4f 100644 --- a/awx/ui_next/src/components/LaunchButton/ReLaunchDropDown.js +++ b/awx/ui_next/src/components/LaunchButton/ReLaunchDropDown.js @@ -15,7 +15,7 @@ function ReLaunchDropDown({ isPrimary = false, handleRelaunch, isLaunching, - + id = 'relaunch-job', ouiaId, }) { const [isOpen, setIsOpen] = useState(false); @@ -75,7 +75,7 @@ function ReLaunchDropDown({ toggleIndicator={null} onToggle={onToggle} aria-label={t`relaunch jobs`} - id="relaunch_jobs" + id={id} isPrimary > {t`Relaunch`} @@ -97,7 +97,7 @@ function ReLaunchDropDown({ toggleIndicator={null} onToggle={onToggle} aria-label={t`relaunch jobs`} - id="relaunch_jobs" + id={id} > diff --git a/awx/ui_next/src/components/Lookup/ExecutionEnvironmentLookup.js b/awx/ui_next/src/components/Lookup/ExecutionEnvironmentLookup.js index b7d4ed8a90..8ab4ad0203 100644 --- a/awx/ui_next/src/components/Lookup/ExecutionEnvironmentLookup.js +++ b/awx/ui_next/src/components/Lookup/ExecutionEnvironmentLookup.js @@ -20,6 +20,7 @@ const QS_CONFIG = getQSConfig('execution_environments', { }); function ExecutionEnvironmentLookup({ + id, globallyAvailable, helperTextInvalid, isDefaultEnvironment, @@ -154,7 +155,7 @@ function ExecutionEnvironmentLookup({ const renderLookup = () => ( <> } helperTextInvalid={helperTextInvalid} @@ -231,6 +232,7 @@ function ExecutionEnvironmentLookup({ } ExecutionEnvironmentLookup.propTypes = { + id: string, value: ExecutionEnvironment, popoverContent: string, onChange: func.isRequired, @@ -243,6 +245,7 @@ ExecutionEnvironmentLookup.propTypes = { }; ExecutionEnvironmentLookup.defaultProps = { + id: 'execution-environments', popoverContent: '', isDefaultEnvironment: false, isGlobalDefaultEnvironment: false, diff --git a/awx/ui_next/src/components/Lookup/InventoryLookup.js b/awx/ui_next/src/components/Lookup/InventoryLookup.js index 58811a9a61..105f62da4b 100644 --- a/awx/ui_next/src/components/Lookup/InventoryLookup.js +++ b/awx/ui_next/src/components/Lookup/InventoryLookup.js @@ -236,6 +236,7 @@ function InventoryLookup({ } InventoryLookup.propTypes = { + fieldId: string, value: Inventory, onChange: func.isRequired, required: bool, @@ -247,6 +248,7 @@ InventoryLookup.propTypes = { }; InventoryLookup.defaultProps = { + fieldId: 'inventory', value: null, required: false, isOverrideDisabled: false, diff --git a/awx/ui_next/src/components/Lookup/Lookup.js b/awx/ui_next/src/components/Lookup/Lookup.js index 34160968f1..c4d1554020 100644 --- a/awx/ui_next/src/components/Lookup/Lookup.js +++ b/awx/ui_next/src/components/Lookup/Lookup.js @@ -128,7 +128,7 @@ function Lookup(props) {