From f43fdd872a28adcd5a18c625b2e5b0bb2bb0a6fb Mon Sep 17 00:00:00 2001
From: Hiroshi Ogawa <hi.ogawa.zz@gmail.com>
Date: Tue, 19 Dec 2023 20:36:26 +0900
Subject: [PATCH] fix(ui): show file item when search filter matches only test
 cases (#4736)

---
 packages/ui/client/components/TaskTree.vue |  2 +-
 test/ui/test/ui.spec.ts                    | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/packages/ui/client/components/TaskTree.vue b/packages/ui/client/components/TaskTree.vue
index ebef882b5efa..a8608c200161 100644
--- a/packages/ui/client/components/TaskTree.vue
+++ b/packages/ui/client/components/TaskTree.vue
@@ -17,7 +17,7 @@ const { task, indent = 0, nested = false, search, onItemClick } = defineProps<{
   <!-- maybe provide a KEEP STRUCTURE mode, do not filter by search keyword  -->
   <!-- v-if = keepStructure ||  (!search || caseInsensitiveMatch(task.name, search)) -->
   <TaskItem
-    v-if="!search || caseInsensitiveMatch(task.name, search)"
+    v-if="!nested || !search || caseInsensitiveMatch(task.name, search)"
     v-bind="$attrs"
     :task="task"
     :style="{ paddingLeft: `${indent * 0.75 + 1}rem` }"
diff --git a/test/ui/test/ui.spec.ts b/test/ui/test/ui.spec.ts
index 440009312135..770f348da099 100644
--- a/test/ui/test/ui.spec.ts
+++ b/test/ui/test/ui.spec.ts
@@ -53,4 +53,22 @@ test.describe('ui', () => {
     await page.getByTestId('btn-console').click()
     await page.getByText('/(?<char>\\w)/').click()
   })
+
+  test('file-filter', async ({ page }) => {
+    await page.goto(pageUrl)
+
+    // match all files when no filter
+    await page.getByPlaceholder('Search...').fill('')
+    await page.getByText('PASS (3)').click()
+    await expect(page.getByText('fixtures/sample.test.ts', { exact: true })).toBeVisible()
+
+    // match nothing
+    await page.getByPlaceholder('Search...').fill('nothing')
+    await page.getByText('No matched test').click()
+
+    // searching "add" will match "sample.test.ts" since it includes a test case named "add"
+    await page.getByPlaceholder('Search...').fill('add')
+    await page.getByText('PASS (1)').click()
+    await expect(page.getByText('fixtures/sample.test.ts', { exact: true })).toBeVisible()
+  })
 })