diff --git a/src/frontend/apps/e2e/__tests__/app-impress/doc-header.spec.ts b/src/frontend/apps/e2e/__tests__/app-impress/doc-header.spec.ts index ff275443..2e5a187e 100644 --- a/src/frontend/apps/e2e/__tests__/app-impress/doc-header.spec.ts +++ b/src/frontend/apps/e2e/__tests__/app-impress/doc-header.spec.ts @@ -8,7 +8,7 @@ import { verifyDocName, } from './utils-common'; import { mockedAccesses, mockedInvitations } from './utils-share'; -import { createRootSubPage } from './utils-sub-pages'; +import { createRootSubPage, getTreeRow } from './utils-sub-pages'; test.beforeEach(async ({ page }) => { await page.goto('/'); @@ -73,17 +73,8 @@ test.describe('Doc Header', () => { await verifyDocName(page, '👍 Hello Emoji World'); // Check the tree - const docTree = page.getByTestId('doc-tree'); - await expect(docTree.getByText('Hello Emoji World')).toBeVisible(); - await expect(docTree.getByTestId('doc-emoji-icon')).toBeVisible(); - await expect(docTree.getByTestId('doc-simple-icon')).toBeHidden(); - - await page.getByTestId('home-button').click(); - - // Check the documents grid - const gridRow = await getGridRow(page, 'Hello Emoji World'); - await expect(gridRow.getByTestId('doc-emoji-icon')).toBeVisible(); - await expect(gridRow.getByTestId('doc-simple-icon')).toBeHidden(); + const row = await getTreeRow(page, 'Hello Emoji World'); + await expect(row.getByText('👍')).toBeVisible(); }); test('it deletes the doc', async ({ page, browserName }) => { diff --git a/src/frontend/apps/e2e/__tests__/app-impress/utils-sub-pages.ts b/src/frontend/apps/e2e/__tests__/app-impress/utils-sub-pages.ts index 236b455e..9ba38298 100644 --- a/src/frontend/apps/e2e/__tests__/app-impress/utils-sub-pages.ts +++ b/src/frontend/apps/e2e/__tests__/app-impress/utils-sub-pages.ts @@ -107,6 +107,20 @@ export const addChild = async ({ return name; }; +export const getTreeRow = async (page: Page, title: string) => { + const docTree = page.getByTestId('doc-tree'); + const row = docTree + .getByRole('treeitem') + .filter({ + hasText: title, + }) + .first(); + + await expect(row).toBeVisible(); + + return row; +}; + export const navigateToTopParentFromTree = async ({ page }: { page: Page }) => { await page.getByRole('link', { name: /Open root document/ }).click(); }; diff --git a/src/frontend/apps/impress/src/features/docs/doc-management/components/SimpleDocItem.tsx b/src/frontend/apps/impress/src/features/docs/doc-management/components/SimpleDocItem.tsx index 27b31cd1..8a390e10 100644 --- a/src/frontend/apps/impress/src/features/docs/doc-management/components/SimpleDocItem.tsx +++ b/src/frontend/apps/impress/src/features/docs/doc-management/components/SimpleDocItem.tsx @@ -4,20 +4,13 @@ import { css } from 'styled-components'; import { Box, Text } from '@/components'; import { useCunninghamTheme } from '@/cunningham'; -import { - Doc, - getEmojiAndTitle, - useDocUtils, - useTrans, -} from '@/docs/doc-management'; +import { Doc, useDocUtils, useTrans } from '@/docs/doc-management'; import { useResponsiveStore } from '@/stores'; import ChildDocument from '../assets/child-document.svg'; import PinnedDocumentIcon from '../assets/pinned-document.svg'; import SimpleFileIcon from '../assets/simple-document.svg'; -import { DocIcon } from './DocIcon'; - const ItemTextCss = css` overflow: hidden; text-overflow: ellipsis; @@ -45,10 +38,6 @@ export const SimpleDocItem = ({ const { untitledDocument } = useTrans(); const { isChild } = useDocUtils(doc); - const { emoji, titleWithoutEmoji: displayTitle } = getEmojiAndTitle( - doc.title || untitledDocument, - ); - return ( + ) : isChild ? ( + @@ -106,7 +87,7 @@ export const SimpleDocItem = ({ $css={ItemTextCss} data-testid="doc-title" > - {displayTitle} + {doc.title || untitledDocument} {(!isDesktop || showAccesses) && (