From 327bcfaad8241fae149fded1567d3ee4e57ab1eb Mon Sep 17 00:00:00 2001 From: Kevin Date: Wed, 25 Jun 2025 12:33:14 -0500 Subject: [PATCH] fix: fixing another e2e test --- e2e/playwright/desktop-export.spec.ts | 2 +- e2e/playwright/projects.spec.ts | 4 ++-- src/components/Explorer/utils.test.ts | 31 +++++++++++++++------------ 3 files changed, 20 insertions(+), 17 deletions(-) diff --git a/e2e/playwright/desktop-export.spec.ts b/e2e/playwright/desktop-export.spec.ts index 7c7f595e3..7755a1ad9 100644 --- a/e2e/playwright/desktop-export.spec.ts +++ b/e2e/playwright/desktop-export.spec.ts @@ -95,7 +95,7 @@ test( await u.openFilePanel() // Click on the other file - const otherKclButton = page.getByRole('button', { name: 'other.kcl' }) + const otherKclButton = scene.getFileRowFromExplorer('other.kcl') await otherKclButton.click() // Close the file pane diff --git a/e2e/playwright/projects.spec.ts b/e2e/playwright/projects.spec.ts index 116af3a40..b4f58d7ff 100644 --- a/e2e/playwright/projects.spec.ts +++ b/e2e/playwright/projects.spec.ts @@ -1092,7 +1092,7 @@ test(`Create a few projects using the default project name`, async ({ test( 'File in the file pane should open with a single click', { tag: '@desktop' }, - async ({ context, homePage, page }, testInfo) => { + async ({ context, homePage, page, scene }, testInfo) => { const projectName = 'router-template-slate' await context.folderSetupFn(async (dir) => { await fsp.mkdir(`${dir}/${projectName}`, { recursive: true }) @@ -1120,7 +1120,7 @@ test( await page.getByRole('button', { name: 'Project Files' }).click() - const file = page.getByRole('button', { name: 'otherThingToClickOn.kcl' }) + const file = scene.getFileRowFromExplorer('otherThingToClickOn.kcl') await expect(file).toBeVisible() await file.click() diff --git a/src/components/Explorer/utils.test.ts b/src/components/Explorer/utils.test.ts index 42ed598df..64b7f4d8e 100644 --- a/src/components/Explorer/utils.test.ts +++ b/src/components/Explorer/utils.test.ts @@ -1,4 +1,7 @@ -import { addPlaceHoldersForNewFileAndFolder, isRowFake } from '@src/components/Explorer/utils' +import { + addPlaceHoldersForNewFileAndFolder, + isRowFake, +} from '@src/components/Explorer/utils' import type { FileExplorerEntry } from '@src/components/Explorer/utils' import type { FileEntry } from '@src/lib/project' @@ -98,15 +101,15 @@ describe('Explorer utils.ts', () => { { path: `/${parentPath}/.zoo-placeholder-folder`, name: '.zoo-placeholder-folder', - children : [] + children: [], }, { path: `/${parentPath}/.zoo-placeholder-file.kcl`, name: '.zoo-placeholder-file.kcl', - children : null + children: null, }, ] - const children : FileEntry[] = [] + const children: FileEntry[] = [] const actual = addPlaceHoldersForNewFileAndFolder(children, parentPath) expect(actual).toBe(expectedResult) expect(children).toStrictEqual(expectedList) @@ -120,45 +123,45 @@ describe('Explorer utils.ts', () => { { path: `/${parentPath}/.zoo-placeholder-folder`, name: '.zoo-placeholder-folder', - children : [] + children: [], }, { path: `/${parentPath}/part001`, name: 'part001', - children : [ + children: [ { path: `/${parentPath}/part001/.zoo-placeholder-folder`, name: '.zoo-placeholder-folder', - children : [] + children: [], }, { path: `/${parentPath}/part001/.zoo-placeholder-file.kcl`, name: '.zoo-placeholder-file.kcl', - children : null + children: null, }, - ] + ], }, { path: `/${parentPath}/main.kcl`, name: 'main.kcl', - children : null + children: null, }, { path: `/${parentPath}/.zoo-placeholder-file.kcl`, name: '.zoo-placeholder-file.kcl', - children : null + children: null, }, ] - const children : FileEntry[] = [ + const children: FileEntry[] = [ { path: `/${parentPath}/part001`, name: 'part001', - children : [] + children: [], }, { path: `/${parentPath}/main.kcl`, name: 'main.kcl', - children : null + children: null, }, ] const actual = addPlaceHoldersForNewFileAndFolder(children, parentPath)