Skip to content

Comments

test: add participant profile regression tests [WPB-19962]#20429

Merged
zhannach merged 3 commits intodevfrom
test/WPB-19962-participant-profile-regression-tests
Feb 20, 2026
Merged

test: add participant profile regression tests [WPB-19962]#20429
zhannach merged 3 commits intodevfrom
test/WPB-19962-participant-profile-regression-tests

Conversation

@zhannach
Copy link
Collaborator

@zhannach zhannach commented Feb 17, 2026

TaskWPB-19962 [Web/QA] Write the Participant Profile regression tests in Playwright

Pull Request

Summary

New Tests: Added E2E regression suite for Participant Profiles.
Fixture Update: Enhanced addMember in the createTeam fixture to support role specification.
Locator Logic: Refactored getLocatorByUser to select users regardless of their role (Admin vs. Member) or list placement.


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)

Standards Acknowledgement (required)


Screenshots or demo (if the user interface changed)

Notes for reviewers

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

@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.25%. Comparing base (f31810b) to head (966c106).
⚠️ Report is 11 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev   #20429   +/-   ##
=======================================
  Coverage   45.25%   45.25%           
=======================================
  Files        1633     1633           
  Lines       40245    40245           
  Branches     8325     8325           
=======================================
  Hits        18211    18211           
  Misses      20105    20105           
  Partials     1929     1929           
Flag Coverage Δ
app_webapp 43.43% <ø> (ø)
lib_api_client 50.17% <ø> (ø)
lib_core 58.86% <ø> (ø)

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.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 17, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 164
  • Failed: 7
  • Skipped: 12
  • 🔁 Flaky: 8
  • 📊 Total: 191
  • Total Runtime: 408.8s (~ 6 min 49 sec)
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/Authentication/authentication.spec.ts (❌ 1 failed, ⚠️ 0 flaky)
  • ❌ Authentication > Make sure user does not see data of user of previous sessions on same browser (tags: TC-1311, regression)
specs/Delete/delete.spec.ts (❌ 1 failed, ⚠️ 0 flaky)
  • ❌ Delete > Delete "For Everyone" works for file sharing (tags: TC-583, regression)
specs/Drive/replyingToMultipartMessage-TC-8787.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Replying to multipart message in a group conversation (tags: crit-flow-cells, regression, TC-8787)
specs/Folders/folders.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Folders > I want to move a 1:1 conversation to an existing custom folder (tags: TC-547, regression)
specs/HistoryBackup/historyBackup.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ History Backup > I want to see new name and system message of the renamed conversation when it was renamed after export (tags: TC-131, regression)
specs/Markdown/markdown.spec.ts (❌ 1 failed, ⚠️ 2 flaky)
  • ❌ Markdown > I want to write a url with markdown (Mixed with text) (tags: TC-1319, regression)
  • ⚠️ Markdown > I want to write a long code message (tags: TC-1316, regression)
  • ⚠️ Markdown > I want to edit a markdown message (tags: TC-1319, regression)
specs/noInternetCallGuard.spec.ts (❌ 1 failed, ⚠️ 0 flaky)
  • ❌ Starting call 1:1 call without internet (tags: )
specs/Reactions/reactions.spec.ts (❌ 2 failed, ⚠️ 2 flaky)
  • ❌ Reactions > Verify liking someone's message in 1:1 (tags: TC-1536, regression)
  • ❌ Reactions > I want to add/remove a reaction using emoji picker (tags: TC-1549, regression)
  • ⚠️ Reactions > Verify liking someone's location (tags: TC-1533, regression)
  • ⚠️ Reactions > Verify I can open like list by hovering the link in the tooltip of a reaction pill (tags: TC-1540, 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)

@zhannach zhannach force-pushed the test/WPB-19962-participant-profile-regression-tests branch from d48bd60 to 3007e7f Compare February 17, 2026 11:26
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
8.8% Duplication on New Code (required ≤ 5%)

See analysis details on SonarQube Cloud

@zhannach zhannach added this pull request to the merge queue Feb 20, 2026
Merged via the queue into dev with commit 8d99bf7 Feb 20, 2026
46 of 54 checks passed
@zhannach zhannach deleted the test/WPB-19962-participant-profile-regression-tests branch February 20, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants