From ed26004cd46964a47d45a964769bb8e3dcf67393 Mon Sep 17 00:00:00 2001 From: Abhishek Jakhar Date: Tue, 20 Oct 2020 08:20:35 +0530 Subject: [PATCH] fix CopyTextButton test case warnings --- .../CopyTextButton/CopyTextButton.test.tsx | 32 ++++++++++--------- 1 file changed, 17 insertions(+), 15 deletions(-) diff --git a/packages/core/src/components/CopyTextButton/CopyTextButton.test.tsx b/packages/core/src/components/CopyTextButton/CopyTextButton.test.tsx index cab992e11a..07bce97d74 100644 --- a/packages/core/src/components/CopyTextButton/CopyTextButton.test.tsx +++ b/packages/core/src/components/CopyTextButton/CopyTextButton.test.tsx @@ -15,8 +15,9 @@ */ import React from 'react'; -import { render, fireEvent } from '@testing-library/react'; -import { wrapInTestApp } from '@backstage/test-utils'; +import { fireEvent } from '@testing-library/react'; +import { act } from 'react-dom/test-utils'; +import { renderInTestApp } from '@backstage/test-utils'; import { CopyTextButton } from './CopyTextButton'; import { ApiRegistry, @@ -55,29 +56,30 @@ const apiRegistry = ApiRegistry.from([ ]); describe('', () => { - it('renders without exploding', () => { - const { getByDisplayValue } = render( - wrapInTestApp( - - - , - ), + it('renders without exploding', async () => { + const { getByDisplayValue } = await renderInTestApp( + + + , ); getByDisplayValue('mockText'); }); it('displays tooltip on click', async () => { + jest.useFakeTimers(); document.execCommand = jest.fn(); - const rendered = render( - wrapInTestApp( - - - , - ), + const rendered = await renderInTestApp( + + + , ); const button = rendered.getByTitle('mockTooltip'); fireEvent.click(button); + act(() => { + jest.runAllTimers(); + }); expect(document.execCommand).toHaveBeenCalled(); rendered.getByText('mockTooltip'); + jest.useRealTimers(); }); });