Skip to content

test: re-enable TC 156 group call after clearing conversation [WPB-23752]#20563

Draft
markbrockhoff wants to merge 3 commits intodevfrom
mb/update-linked-jira-tickets
Draft

test: re-enable TC 156 group call after clearing conversation [WPB-23752]#20563
markbrockhoff wants to merge 3 commits intodevfrom
mb/update-linked-jira-tickets

Conversation

@markbrockhoff
Copy link
Collaborator

@markbrockhoff markbrockhoff commented Mar 3, 2026

TaskWPB-23752 [Web/QA] Fix and re-enable TC-156 Group call after clearing conversation

Pull Request

Summary

  • Fixes / Adds some jira links in comments
  • Re-enables previously skipped test which was blocked by enterprise capabilities

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)

@markbrockhoff markbrockhoff changed the title test: re-enable TC-156 group call after clearing conversation [WPB-23752] test: re-enable TC 156 group call after clearing conversation [WPB-23752] Mar 3, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 188
  • Failed: 1
  • Skipped: 12
  • 🔁 Flaky: 11
  • 📊 Total: 212
  • Total Runtime: 421.9s (~ 7 min 2 sec)
specs/Archive/archive.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Archive > I want to archive the 1on1 conversation from conversation details (tags: TC-105, regression)
specs/Authentication/authentication.spec.ts (❌ 1 failed, ⚠️ 2 flaky)
  • ❌ Authentication > Make sure user does not see data of user of previous sessions on same browser (tags: TC-1311, regression)
  • ⚠️ Authentication > Verify current browser is set as temporary device (tags: TC-3460, regression)
  • ⚠️ Authentication > I want to keep my history after refreshing the page on permanent device (tags: TC-3472, regression)
specs/CriticalFlow/groupCalls-TC-8632.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Planning group call with sending various messages during call (tags: TC-8632, crit-flow-web)
specs/Delete/delete.spec.ts (❌ 0 failed, ⚠️ 2 flaky)
  • ⚠️ Delete > Delete "For Everyone" works for link preview (tags: TC-581, regression)
  • ⚠️ Delete > Delete "For Everyone" works for video messages (tags: TC-585, regression)
specs/Drive/uploadingFileInGroupConversation-TC-8785.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Uploading an file in a group conversation (tags: crit-flow-cells, regression, TC-8785)
specs/Notifications/notifications.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Notifications > Verify I see notification when I receive a connection request (tags: TC-1462, regression)
specs/ParticipantProfile/participantProfile.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Participant Profile > Verify I can see participant profile of user I blocked in a group conversation (tags: TC-1479, regression)
specs/Reply/reply.spec.ts (❌ 0 failed, ⚠️ 2 flaky)
  • ⚠️ Reply > I should not see the quoted message when searching for original message in collections (tags: TC-2996, regression)
  • ⚠️ Reply > I should not be able to send a reply after I got removed from the conversation (tags: TC-3014, regression)

arjita-mitra
arjita-mitra previously approved these changes Mar 3, 2026
@arjita-mitra
Copy link
Contributor

arjita-mitra commented Mar 3, 2026

LGTM although its draft didn't need approval :-)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2026

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.

3 participants