Skip to content

test: re-enable group call test in clearing conversation tests [WPB-23752]#20632

Merged
markbrockhoff merged 1 commit intodevfrom
mb/fix-group-call-crit-flow-test
Mar 10, 2026
Merged

test: re-enable group call test in clearing conversation tests [WPB-23752]#20632
markbrockhoff merged 1 commit intodevfrom
mb/fix-group-call-crit-flow-test

Conversation

@markbrockhoff
Copy link
Collaborator

@markbrockhoff markbrockhoff commented Mar 10, 2026

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

Pull Request

Summary

  • Only wait 5s in createTeam fixture if one feature was actually set to true
  • Re-enable previously skipped tests which depended on enterprise features

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):

@sonarqubecloud
Copy link

@github-actions
Copy link
Contributor

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 257
  • Failed: 1
  • Skipped: 15
  • 🔁 Flaky: 7
  • 📊 Total: 280
  • Total Runtime: 480.3s (~ 8 min 0 sec)
specs/Authentication/authentication.spec.ts (❌ 1 failed, ⚠️ 1 flaky)
  • ❌ Authentication > Verify current browser is set as temporary device (tags: TC-3460, regression)
  • ⚠️ Authentication > Make sure user does not see data of user of previous sessions on same browser (tags: TC-1311, 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 group conversation via conversation list (tags: TC-156, regression)
specs/Conversations/conversations.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Conversations > I want to see a system message with all group members mentioned on creating a group (tags: TC-2965, regression)
specs/HistoryBackup/historyBackup.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ History Backup > I want to import a backup that I exported when I was using a different email/password (tags: TC-118, regression)
specs/Ping/ping.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Ping > Verify I can receive ping several times in a row (tags: TC-1491, regression)
specs/Reactions/reactions.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Reactions > Verify likes are reset if you edited message (tags: TC-1538, regression)
specs/Reply/reply.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Reply > I want to see a placeholder text as quote when original message is not available anymore (tags: TC-2994, regression)

@markbrockhoff markbrockhoff enabled auto-merge March 10, 2026 16:22
@markbrockhoff markbrockhoff added this pull request to the merge queue Mar 10, 2026
Merged via the queue into dev with commit 3cc9bae Mar 10, 2026
52 of 53 checks passed
@markbrockhoff markbrockhoff deleted the mb/fix-group-call-crit-flow-test branch March 10, 2026 16:53
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