chore(e2e): Added e2e test for searching files in the conversation [WPB-21646] #20173

iskvortsov · 2026-01-22T10:36:53Z

WPB-21646 [Web] Search files in Conversation

Pull Request

Summary

Added e2e test for searching files in the conversation
Updated README.md for e2e tests

Security Checklist (required)

External inputs are validated & sanitized on client and/or server where applicable.
API responses are validated; unexpected shapes are handled safely (fallbacks or errors).
No unsafe HTML is rendered; if unavoidable, sanitization is applied and documented where it happens.
Injection risks (XSS/SQL/command) are prevented via safe APIs and/or escaping.

Accessibility (required)

I have read and this PR upholds our Accessibility Best Practices.

Standards Acknowledgement (required)

I have read and this PR upholds our Coding Standards and Tech Radar Choices.

Screenshots or demo (if the user interface changed)

Notes for reviewers

Trade-offs:
Follow-ups (linked issues):
Linked PRs (e.g. web-packages):

…est readme

…rsationFiles.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

This PR adds an end-to-end test for searching files in group conversations within the Cells feature. The test verifies that users can upload files (images and videos) to a group conversation and search for them using the Files tab search functionality.

Changes:

Added new e2e test searchFilesInGroupConversation-TC-8788.spec.ts that tests file upload and search in Cells group conversations
Extended page objects with methods for file search operations and video visibility checks
Exported VideoFileName constant for reuse across tests
Updated README documentation to use correct Nx commands

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
`apps/webapp/test/e2e_tests/utils/asset.util.ts`	Exported VideoFileName constant to make it available for test usage
`apps/webapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts`	New e2e test that verifies file search functionality in Cells group conversations
`apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts`	Added methods for file search, visibility check, and counting files
`apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversation.page.ts`	Updated image locator and added video visibility check method
`apps/webapp/test/e2e_tests/README.md`	Updated documentation with correct Nx command paths

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversation.page.ts

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

codecov · 2026-01-22T10:45:16Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.53%. Comparing base (c0be91d) to head (79237c9).

Additional details and impacted files

@@           Coverage Diff           @@
##              dev   #20173   +/-   ##
=======================================
  Coverage   45.53%   45.53%           
=======================================
  Files        1632     1632           
  Lines       40337    40337           
  Branches     8334     8334           
=======================================
  Hits        18366    18366           
  Misses      20040    20040           
  Partials     1931     1931

Flag	Coverage Δ
app_webapp	`43.75% <ø> (ø)`
lib_api_client	`50.27% <ø> (ø)`
lib_core	`58.90% <ø> (ø)`

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…PB-21646-new

…rsation.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversation.page.ts

…rsation.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

github-actions · 2026-01-22T16:24:48Z

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

✅ Passed: 110
❌ Failed: 5
⏭ Skipped: 12
🔁 Flaky: 6
📊 Total: 133
⏱ Total Runtime: 285.9s (~ 4 min 46 sec)

specs/AccountSettingsSpecs/accountSettings.spec.ts (❌ 1 failed, ⚠️ 0 flaky)

❌ account settings > I should not be able to change email of user managed by SCIM (tags: TC-60, regression)

specs/AppLock/AppLock.spec.ts (❌ 1 failed, ⚠️ 0 flaky)

❌ AppLock > Web: App should not lock if I switch back to webapp tab in time (during inactivity timeout) (tags: TC-2752, TC-2753, regression)

specs/ArchiveSpecs/archive.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Accessibility > I want to archive the 1on1 conversation from conversation details (tags: TC-105, regression)

specs/Authentication/authentication.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Authentication > Verify current browser is set as temporary device (tags: TC-3460, regression)

specs/Block/block.spec.ts (❌ 1 failed, ⚠️ 0 flaky)

❌ Block: User A and User B are NOT in the same team > Verify you can unblock someone from conversation list options (tags: TC-148, regression)

specs/ClearConversationContent/clearConversationContent.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Clear Conversation Content > I want to see incoming picture, ping and call after I clear content of a 1:1 conversation via conversation list (tags: TC-8779, regression)

specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts (❌ 1 failed, ⚠️ 0 flaky)

❌ Searching files in a group conversation (tags: crit-flow-cells, regression, TC-8788)

specs/CriticalFlow/Cells/uploadingFileInGroupConversation.spec.ts (❌ 1 failed, ⚠️ 0 flaky)

❌ Uploading an file in a group conversation (tags: crit-flow-cells, regression)

specs/Edit/edit.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Edit > I cannot edit another users message (tags: TC-683, regression)

specs/Reply/reply.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Reply > I should not be able to send a reply after I got removed from the conversation (tags: TC-3014, regression)

specs/SelfDeletingMessages/selfDeletingMessages.spec.ts (❌ 0 failed, ⚠️ 1 flaky)

⚠️ Self Deleting Messages > Verify sending ephemeral text message in group (tags: TC-658, regression)

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

apps/webapp/test/e2e_tests/README.md

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversation.page.ts

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Mark Brockhoff <95471369+markbrockhoff@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

apps/webapp/test/e2e_tests/README.md

…PB-21646-new

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts

…PB-21646-new

sonarqubecloud · 2026-01-23T14:39:07Z

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

thisisamir98 and others added 7 commits January 6, 2026 14:59

feat: Search files in cells conversation [WPB-21646]

a89aa46

Added e2e test for searching files in the conversation, updated e2e t…

49e7f06

…est readme

Merge branch 'dev' into WPB-21646-new

ac06588

Merge branch 'dev' into WPB-21646-new

d4d3f45

Update apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConve…

0e2aeaa

…rsationFiles.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Merge branch 'dev' into WPB-21646-new

ace6160

Merge branch 'dev' into WPB-21646-new

5f26f86

iskvortsov requested review from a team and otto-the-bot as code owners January 22, 2026 10:36

Copilot AI review requested due to automatic review settings January 22, 2026 10:36

otto-the-bot added type: chore 🧹 👕 size: M app: webapp labels Jan 22, 2026

Copilot started reviewing on behalf of iskvortsov January 22, 2026 10:37 View session

Merge branch 'dev' into WPB-21646-new

c4af923

Copilot AI reviewed Jan 22, 2026

View reviewed changes

Improved TC-8788

d087d17

iskvortsov and others added 2 commits January 22, 2026 11:47

Merge branch 'WPB-21646-new' of github.com:wireapp/wire-webapp into W…

1c16301

…PB-21646-new

Update apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConve…

1952c86

…rsation.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 22, 2026 10:48

Copilot started reviewing on behalf of iskvortsov January 22, 2026 10:48 View session

markbrockhoff reviewed Jan 22, 2026

View reviewed changes

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts Outdated Show resolved Hide resolved

Copilot AI reviewed Jan 22, 2026

View reviewed changes

iskvortsov and others added 2 commits January 22, 2026 17:05

Applied review suggestions

b72ef46

Update apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConve…

a45b5bd

…rsation.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 22, 2026 16:09

Merge branch 'dev' into WPB-21646-new

328dfb9

Copilot started reviewing on behalf of iskvortsov January 22, 2026 16:10 View session

iskvortsov requested a review from markbrockhoff January 22, 2026 16:12

Update apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConve…

b6d4ba1

…rsation.page.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI reviewed Jan 22, 2026

View reviewed changes

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts Outdated Show resolved Hide resolved

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts Show resolved Hide resolved

Update apps/webapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFile…

79ad097

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 22, 2026 16:19

Copilot started reviewing on behalf of iskvortsov January 22, 2026 16:20 View session

Copilot AI reviewed Jan 22, 2026

View reviewed changes

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts Outdated Show resolved Hide resolved

Update apps/webapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFile…

a9faaf8

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 22, 2026 16:29

Copilot started reviewing on behalf of iskvortsov January 22, 2026 16:30 View session

Copilot AI reviewed Jan 22, 2026

View reviewed changes

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts Show resolved Hide resolved

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts Outdated Show resolved Hide resolved

Update apps/webapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFile…

8018d99

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 23, 2026 08:47

Copilot started reviewing on behalf of iskvortsov January 23, 2026 08:47 View session

Copilot AI reviewed Jan 23, 2026

View reviewed changes

...ebapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFilesInGroupConversation-TC-8788.spec.ts Show resolved Hide resolved

aweiss-dev approved these changes Jan 23, 2026

View reviewed changes

markbrockhoff requested changes Jan 23, 2026

View reviewed changes

Update apps/webapp/test/e2e_tests/specs/CriticalFlow/Cells/searchFile…

ddbf27a

…sInGroupConversation-TC-8788.spec.ts Co-authored-by: Mark Brockhoff <95471369+markbrockhoff@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 23, 2026 10:19

Copilot started reviewing on behalf of iskvortsov January 23, 2026 10:19 View session

Copilot AI reviewed Jan 23, 2026

View reviewed changes

apps/webapp/test/e2e_tests/README.md Outdated Show resolved Hide resolved

iskvortsov added 3 commits January 23, 2026 15:14

Applied more review suggestions

db31323

Merge branch 'WPB-21646-new' of github.com:wireapp/wire-webapp into W…

036aef4

…PB-21646-new

Merge branch 'dev' into WPB-21646-new

b0605b3

Copilot AI review requested due to automatic review settings January 23, 2026 14:23

Copilot started reviewing on behalf of iskvortsov January 23, 2026 14:23 View session

Copilot AI reviewed Jan 23, 2026

View reviewed changes

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts Show resolved Hide resolved

apps/webapp/test/e2e_tests/pageManager/webapp/cells/cellsConversationFiles.page.ts Outdated Show resolved Hide resolved

iskvortsov added 2 commits January 23, 2026 15:35

Changed placeholder value and improved locators

4e77575

Merge branch 'WPB-21646-new' of github.com:wireapp/wire-webapp into W…

79237c9

…PB-21646-new

chore(e2e): Added e2e test for searching files in the conversation [WPB-21646] #20173

Are you sure you want to change the base?

chore(e2e): Added e2e test for searching files in the conversation [WPB-21646] #20173

Conversation

iskvortsov commented Jan 22, 2026 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull Request

Summary

Security Checklist (required)

Accessibility (required)

Standards Acknowledgement (required)

Screenshots or demo (if the user interface changed)

Notes for reviewers

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

codecov bot commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

github-actions bot commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🧪 Playwright Test Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

iskvortsov commented Jan 22, 2026 •

edited by github-actions bot

Loading

codecov bot commented Jan 22, 2026 •

edited

Loading

github-actions bot commented Jan 22, 2026 •

edited

Loading