Add tests and deprecate prop

Author: rebeccaalpert

packages/react-core/src/components/ClipboardCopy/ClipboardCopyButton.tsx

@@ -11,6 +11,8 @@ export interface ClipboardCopyButtonProps
   children: React.ReactNode;
   /** ID of the copy button */
   id: string;
+  /** @deprecated ID of the content that is being copied */
+  textId?: string;
   /** Additional classes added to the copy button */
   className?: string;
   /** Exit delay on the copy button tooltip */

packages/react-core/src/components/ClipboardCopy/__tests__/ClipboardCopy.test.tsx

@@ -13,7 +13,7 @@ jest.mock('../ClipboardCopyButton', () => ({
       <p>{`entryDelay: ${entryDelay}`}</p>
       <p>{`maxWidth: ${maxWidth}`}</p>
       <p>{`position: ${position}`}</p>
-      <p>{`button-ariaLabel: ${ariaLabel}`}</p>
+      <p>{`ariaLabel: ${ariaLabel}`}</p>
       <div>{`children: ${children}`}</div>
       <button onClick={onClick}>Test CCB clicker</button>
     </div>
@@ -177,7 +177,7 @@ test('Renders content passed to additionalActions when variant is inline-compact
 test('Passes hoverTip to ClipboardCopyButton by default', () => {
   render(<ClipboardCopy hoverTip="hover tip">{children}</ClipboardCopy>);
 
-  expect(screen.getByText('button-ariaLabel: hover tip')).toBeVisible();
+  expect(screen.getByText('ariaLabel: hover tip')).toBeVisible();
   expect(screen.getByText('children: hover tip')).toBeVisible();
 });
 
@@ -188,7 +188,7 @@ test('Passes hoverTip to ClipboardCopyButton when variant is inline-compact', ()
     </ClipboardCopy>
   );
 
-  expect(screen.getByText('button-ariaLabel: hover tip')).toBeVisible();
+  expect(screen.getByText('ariaLabel: hover tip')).toBeVisible();
   expect(screen.getByText('children: hover tip')).toBeVisible();
 });
 
@@ -265,6 +265,12 @@ test('Passes position to ClipboardCopyButton when variant is inline-compact', ()
   expect(screen.getByText('position: bottom')).toBeVisible();
 });
 
+test('Passes copyAriaLabel to ClipboardCopyButton', () => {
+  render(<ClipboardCopy copyAriaLabel="Copy text">{children}</ClipboardCopy>);
+
+  expect(screen.getByText('ariaLabel: Copy text')).toBeVisible();
+});
+
 test('Passes toggleAriaLabel to ClipboardCopyToggle when variant is expansion', () => {
   render(
     <ClipboardCopy variant="expansion" toggleAriaLabel="toggle label">

packages/react-core/src/components/ClipboardCopy/__tests__/__snapshots__/ClipboardCopy.test.tsx.snap

@@ -18,7 +18,7 @@ exports[`Matches snapshot 1`] = `
         <input
           aria-invalid="false"
           aria-label="Copyable input"
-          data-ouia-component-id="OUIA-Generated-TextInputBase-34"
+          data-ouia-component-id="OUIA-Generated-TextInputBase-35"
           data-ouia-component-type="PF6/TextInput"
           data-ouia-safe="true"
           id="text-input-generated-id"
@@ -42,7 +42,7 @@ exports[`Matches snapshot 1`] = `
           position: top
         </p>
         <p>
-          button-ariaLabel: Copy to clipboard
+          ariaLabel: Copy to clipboard
         </p>
         <div>
           children: Copy to clipboard