From d7cb7b1be5778c4b692543e20ef9e0bc7cf258ff Mon Sep 17 00:00:00 2001
From: Eric Olkowski <thatblindgeye@gmail.com>
Date: Wed, 24 Jan 2024 15:03:41 -0500
Subject: [PATCH] Updated tests per Austin feedback

---
 .../HelperText/__tests__/HelperTextItem.test.tsx  | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/packages/react-core/src/components/HelperText/__tests__/HelperTextItem.test.tsx b/packages/react-core/src/components/HelperText/__tests__/HelperTextItem.test.tsx
index 4ef5fb26b7a..81bc38d3275 100644
--- a/packages/react-core/src/components/HelperText/__tests__/HelperTextItem.test.tsx
+++ b/packages/react-core/src/components/HelperText/__tests__/HelperTextItem.test.tsx
@@ -33,7 +33,7 @@ test('Renders custom className', () => {
 test('Does not render screen reader text by default', () => {
   render(<HelperTextItem>help test text 1</HelperTextItem>);
 
-  expect(screen.queryByText('help test text 1')?.querySelector('.pf-v5-screen-reader')).not.toBeInTheDocument();
+  expect(screen.queryByText('status')).not.toBeInTheDocument();
 });
 
 Object.values(['indeterminate', 'warning', 'success', 'error']).forEach((variant) => {
@@ -52,10 +52,21 @@ Object.values(['indeterminate', 'warning', 'success', 'error']).forEach((variant
         text
       </HelperTextItem>
     );
-    expect(screen.getByText('text').querySelector('span')).toHaveTextContent(`: ${variant} status;`);
+    expect(screen.getByText(`: ${variant} status;`)).toBeInTheDocument();
   });
 });
 
+test('Renders custom screen reader text', () => {
+  render(
+    <HelperTextItem variant="error" screenReaderText="danger">
+      help test text 1
+    </HelperTextItem>
+  );
+
+  expect(screen.queryByText(': error status;')).not.toBeInTheDocument();
+  expect(screen.getByText(': danger;')).toBeInTheDocument();
+});
+
 test('Renders id when id is passed', () => {
   render(<HelperTextItem id="text-item">help test text 1</HelperTextItem>);
   expect(screen.getByText('help test text 1').parentElement).toHaveAttribute('id', 'text-item');