Fix tests by using actual text

Author: badiuoanaalexandra

src/components/copy-button/copy-button.test.tsx

@@ -21,16 +21,16 @@ describe('CopyButton', () => {
       await userEvent.click(screen.getByTestId('copy-button'));
 
       await waitFor(() => {
-        expect(screen.getByRole('dialog', { hidden: true })).toBeInTheDocument();
+        expect(screen.getByText('Copied!')).toBeInTheDocument();
       });
 
       await waitFor(() => {
-        expect(screen.queryByRole('dialog', { hidden: true })).not.toBeInTheDocument();
+        expect(screen.queryByText('Copied!')).not.toBeInTheDocument();
       });
     });
   });
 
-  describe('all props', () => {
+  describe('all props',() => {
     const mockOnCopy = jest.fn();
     const props = {
       text: 'more copiable text',
@@ -59,11 +59,11 @@ describe('CopyButton', () => {
       await userEvent.click(screen.getByTestId('copy-button'));
 
       await waitFor(() => {
-        expect(screen.getByRole('dialog', { hidden: true })).toBeInTheDocument();
+        expect(screen.getByText('Copied!')).toBeInTheDocument();
       });
 
       await waitFor(() => {
-        expect(screen.queryByRole('dialog', { hidden: true })).not.toBeInTheDocument();
+        expect(screen.queryByText('Copied!')).not.toBeInTheDocument();
       });
     });
 

src/components/minimum-duration-loader/minimum-duration-loader.tsx

@@ -23,7 +23,7 @@ export default function MinimumDurationLoader({
 
   // If `isLoaded` is immediately true, load children right away.
   const [ wasLoaded ] = useState(isLoaded);
-  const prevIsLoaded = usePrevious(isLoaded);
+  const prevIsLoaded: boolean = usePrevious(isLoaded);
   const startTime = useRef(0);
   const delayedMountTimeoutRef = useRef(null);
   const [delayedLoaded, setDelayedLoaded] = useState(false);

src/components/popover/popover.test.tsx

@@ -161,7 +161,7 @@ describe('Popover', () => {
       const { baseElement } = render(<PopoverTest {...props} />);
       await user.click(screen.getByTestId('trigger'));
       await waitFor(() => {
-        expect(screen.getByRole('dialog', { hidden: true })).toBeInTheDocument();
+        expect(screen.getByText('Greetings')).toBeInTheDocument();
       });
       expect(baseElement).toMatchSnapshot();
     });

src/components/popover/popover.tsx

@@ -7,7 +7,7 @@ import { getTheme } from '../utils/styles';
 
 interface Props {
   children: ReactElement;
-  content: ReactNode | (() => ReactNode);
+  content: ReactNode;
   active: boolean;
   padding?: 'medium' | 'small' | 'none';
   coloring?: 'light' | 'dark' | 'warning' | 'error';

src/components/select/select.test.tsx

@@ -88,7 +88,7 @@ describe('Select', () => {
       const { baseElement } = render(<SelectTest {...props} />);
       await user.click(screen.getByTestId("trigger"));
       await waitFor(() => {
-        expect(screen.getByRole('listbox', { hidden: true })).toBeInTheDocument();
+        expect(screen.getByText('Humpback whale')).toBeInTheDocument();
       });
       expect(baseElement).toMatchSnapshot();
     });