Skip to content

test(WPB-23749): fix regression tests for AppLock #20908

Merged
JacquelineLehner merged 8 commits intodevfrom
23749-fix-and-reenable-app-lock
Apr 2, 2026
Merged

test(WPB-23749): fix regression tests for AppLock #20908
JacquelineLehner merged 8 commits intodevfrom
23749-fix-and-reenable-app-lock

Conversation

@JacquelineLehner
Copy link
Copy Markdown
Collaborator

@JacquelineLehner JacquelineLehner commented Mar 31, 2026

TaskWPB-23749 [Web/QA] Fix and re-enable TC-2752/2753 App lock

Summary

  • fix and re-enable flaky/broken test cases for AppLock
  • TC-2752 & TC-2753

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)


Mark Brockhoff and others added 3 commits March 31, 2026 14:13
…-2753)

This commit resolves the flakiness of the App Lock inactivity timeout
tests and re-enables them.

Since Playwright does not natively support backgrounding tabs, the
inactivity state is now reliably simulated by manually dispatching
`blur` and `focus` events on the document body.

Additionally, this introduces new testing utilities to support the
App Lock test suite:
- `handleAppLockState` helper for reusable app lock setup.
- `checkAnyIndexedDBExists` in a new `indexedDB.util.ts` to reliably
  verify database wipes during the forgotten passphrase reset flow.

Refs: WPB-23749
…-2753)

This commit resolves the flakiness of the App Lock inactivity timeout
tests and re-enables them.

Since Playwright does not natively support backgrounding tabs, the
inactivity state is now reliably simulated by manually dispatching
`blur` and `focus` events on the document body. Added missing TC-2761.

Additionally, this introduces new testing utilities to support the
App Lock test suite:
- `handleAppLockState` helper for reusable app lock setup.
- `checkAnyIndexedDBExists` in a new `indexedDB.util.ts` to reliably
  verify database wipes during the forgotten passphrase reset flow.

Refs: WPB-23749
zhannach
zhannach previously approved these changes Apr 1, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 288
  • Failed: 0
  • Skipped: 12
  • 🔁 Flaky: 3
  • 📊 Total: 303
  • Total Runtime: 355.5s (~ 5 min 56 sec)
specs/Calling/calling.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Calling > Verify in call reactions (tags: TC-8774, 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/Drive/editMultipartMessage-TC-8786.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Edit multipart message in a group conversation (tags: crit-flow-cells, regression, TC-8786)

@JacquelineLehner JacquelineLehner added this pull request to the merge queue Apr 2, 2026
Merged via the queue into dev with commit 52fac1c Apr 2, 2026
86 checks passed
@JacquelineLehner JacquelineLehner deleted the 23749-fix-and-reenable-app-lock branch April 2, 2026 09:49
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