From 293308e683e51a1f70afb20e5d7991b091ecbec6 Mon Sep 17 00:00:00 2001
From: Juri Solovjov <jsolovjo@redhat.com>
Date: Mon, 20 Jan 2025 14:20:47 +0100
Subject: [PATCH] Add data-testid to the elements missing it on Preferece pages

---
 .../hawtio/src/plugins/connect/ConnectPreferences.tsx | 11 +++++++----
 packages/hawtio/src/preferences/HomePreferences.tsx   |  8 +++++---
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/packages/hawtio/src/plugins/connect/ConnectPreferences.tsx b/packages/hawtio/src/plugins/connect/ConnectPreferences.tsx
index 0a1aaa06c..2b4d3fd70 100644
--- a/packages/hawtio/src/plugins/connect/ConnectPreferences.tsx
+++ b/packages/hawtio/src/plugins/connect/ConnectPreferences.tsx
@@ -186,7 +186,9 @@ const JolokiaForm: React.FunctionComponent = () => {
         />
       </FormGroup>
       <FormGroup fieldId='jolokia-form-apply'>
-        <Button onClick={applyJolokia}>Apply</Button>
+        <Button data-testid='apply-jolokia' onClick={applyJolokia}>
+          Apply
+        </Button>
         <FormHelperText>
           <HelperText>
             <HelperTextItem>Restart Hawtio with the new values in effect.</HelperTextItem>
@@ -215,16 +217,17 @@ const ResetForm: React.FunctionComponent = () => {
 
   const ConfirmClearModal = () => (
     <Modal
+      data-testid='clear-connections-modal'
       variant={ModalVariant.small}
       title='Clear saved connections'
       titleIconVariant='danger'
       isOpen={isConfirmModalOpen}
       onClose={confirmClear}
       actions={[
-        <Button key='reset' variant='danger' onClick={clear}>
+        <Button key='reset' data-testid='clear-btn' variant='danger' onClick={clear}>
           Clear
         </Button>,
-        <Button key='cancel' variant='link' onClick={confirmClear}>
+        <Button key='cancel' data-testid='cancel-btn' variant='link' onClick={confirmClear}>
           Cancel
         </Button>,
       ]}
@@ -236,7 +239,7 @@ const ResetForm: React.FunctionComponent = () => {
   return (
     <FormSection title='Reset' titleElement='h2'>
       <FormGroup label='Clear saved connections' fieldId='reset-form-clear'>
-        <Button variant='danger' onClick={confirmClear}>
+        <Button data-testid='clear-btn' variant='danger' onClick={confirmClear}>
           Clear
         </Button>
         <ConfirmClearModal />
diff --git a/packages/hawtio/src/preferences/HomePreferences.tsx b/packages/hawtio/src/preferences/HomePreferences.tsx
index d00ca57e6..8d1f47778 100644
--- a/packages/hawtio/src/preferences/HomePreferences.tsx
+++ b/packages/hawtio/src/preferences/HomePreferences.tsx
@@ -40,6 +40,7 @@ const UIForm: React.FunctionComponent = () => {
   return (
     <FormGroup label='Default vertical nav state' fieldId='ui-form-vertical-nav-switch'>
       <Switch
+        data-testid='switch-vertical-nav-state'
         label='Show vertical navigation'
         labelOff='Hide vertical navigation'
         isChecked={showVerticalNav}
@@ -65,16 +66,17 @@ const ResetForm: React.FunctionComponent = () => {
 
   const ConfirmResetModal = () => (
     <Modal
+      data-testid='reset-settings-modal'
       variant={ModalVariant.small}
       title='Reset settings'
       titleIconVariant='danger'
       isOpen={isConfirmResetOpen}
       onClose={confirmReset}
       actions={[
-        <Button key='reset' variant='danger' onClick={reset}>
+        <Button key='reset' data-testid='reset-btn' variant='danger' onClick={reset}>
           Reset
         </Button>,
-        <Button key='cancel' variant='link' onClick={confirmReset}>
+        <Button key='cancel' data-testid='cancel-btn' variant='link' onClick={confirmReset}>
           Cancel
         </Button>,
       ]}
@@ -88,7 +90,7 @@ const ResetForm: React.FunctionComponent = () => {
   return (
     <React.Fragment>
       <FormGroup label='Reset settings' fieldId='reset-form-reset'>
-        <Button variant='danger' onClick={confirmReset}>
+        <Button data-testid='reset-btn' variant='danger' onClick={confirmReset}>
           Reset
         </Button>
         <FormHelperText>