feat(boxai-sidebar): PR fix

box · Dec 23, 2024 · 297d949 · 297d949
1 parent 837ccf2
commit 297d949
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/elements/content-sidebar/BoxAISidebarContent.tsx b/src/elements/content-sidebar/BoxAISidebarContent.tsx
@@ -6,7 +6,7 @@ import * as React from 'react';
 import flow from 'lodash/flow';
 import { useIntl } from 'react-intl';
 import { AgentsProvider, BoxAiAgentSelectorWithApi } from '@box/box-ai-agent-selector';
-import { IconButton, Text } from '@box/blueprint-web';
+import {IconButton, Text, Tooltip} from '@box/blueprint-web';
 import { Trash } from '@box/blueprint-web-assets/icons/Line';
 // @ts-expect-error - TS2305 - Module '"@box/box-ai-content-answers"' has no exported member 'ApiWrapperProps'.
 import { BoxAiContentAnswers, withApiWrapper, type ApiWrapperProps } from '@box/box-ai-content-answers'
@@ -23,7 +23,6 @@ import messages from '../common/messages';
 import sidebarMessages from './messages';
 
 import './BoxAISidebar.scss';
-import Tooltip, {TooltipPosition} from "../../components/tooltip/Tooltip";
 
 
 const MARK_NAME_JS_READY: string = `${ORIGIN_BOXAI_SIDEBAR}_${EVENT_JS_READY}`;
@@ -99,7 +98,7 @@ function BoxAISidebarContent(props: ApiWrapperProps) {
         <>
             { renderBoxAISidebarTitle() }
             { isResetChatEnabled &&
-                <Tooltip position={TooltipPosition.MIDDLE_LEFT} text={formatMessage(sidebarMessages.boxAISidebarClearConversationTooltip)} isTabbable={false}>
+                <Tooltip content={formatMessage(sidebarMessages.boxAISidebarClearConversationTooltip)}>
 					<IconButton
 							aria-label={formatMessage(sidebarMessages.boxAISidebarClear)}
 							icon={Trash}

diff --git a/src/elements/content-sidebar/__tests__/BoxAISidebar.test.tsx b/src/elements/content-sidebar/__tests__/BoxAISidebar.test.tsx
@@ -2,6 +2,7 @@ import React from 'react';
 import { userEvent } from '@testing-library/user-event';
 import { render, screen } from '../../../test-utils/testing-library';
 import BoxAISidebar, { BoxAISidebarProps } from '../BoxAISidebar';
+import messages from "../../common/content-answers/messages";
 
 jest.mock('@box/box-ai-agent-selector', () => ({
     ...jest.requireActual('@box/box-ai-agent-selector'),
@@ -169,4 +170,14 @@ describe('elements/content-sidebar/BoxAISidebar', () => {
         expect(screen.getByText('completed question')).toBeInTheDocument();
         expect(screen.queryByText('not completed question')).not.toBeInTheDocument();
     });
+
+    test('should display clear conversation tooltip', async () => {
+        await renderComponent();
+
+        const button = screen.getByRole('button', { name: 'Clear' });
+        await userEvent.hover(button);
+        const tooltip = await screen.findByRole('tooltip', { name: 'Clear conversation' });
+
+        expect(tooltip).toBeInTheDocument();
+    });
 });