Skip to content

Conversation

@JacquelineLehner
Copy link
Collaborator

@JacquelineLehner JacquelineLehner commented Feb 6, 2026

TaskWPB-19954 [Web/QA] Write the History Backup regression tests in Playwright

https://wearezeta.atlassian.net/browse/WPB-19954

Pull Request

Summary

  • Add regression tests for creating and restoring history backups

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)


Notes for reviewers

  • code duplication cannot be reduced any further since it only refers to the login of users

@JacquelineLehner JacquelineLehner changed the title test(historyBackup): add regression tests for history backup and restore test(WPB-19954): add regression tests for history backup and restore Feb 6, 2026
@codecov
Copy link

codecov bot commented Feb 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 45.25%. Comparing base (8b2e1c1) to head (88a5761).
⚠️ Report is 1 commits behind head on dev.

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

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 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 6, 2026

🔗 Download Full Report Artifact

🧪 Playwright Test Summary

  • Passed: 157
  • Failed: 4
  • Skipped: 11
  • 🔁 Flaky: 10
  • 📊 Total: 182
  • Total Runtime: 563.5s (~ 9 min 24 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/Block/block.spec.ts (❌ 0 failed, ⚠️ 1 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 group conversation via conversation list (tags: TC-156, regression)
specs/CriticalFlow/personalAccountLifecycle-TC-8638.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Personal Account Lifecycle (tags: TC-8638, crit-flow-web)
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)
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/HistoryBackup/historyBackup.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ History Backup > I want to have the same mute or archive state of a conversation after import (tags: TC-133, regression)
specs/noInternetCallGuard.spec.ts (❌ 1 failed, ⚠️ 0 flaky)
  • ❌ Starting call 1:1 call without internet (tags: )
specs/Reactions/reactions.spec.ts (❌ 2 failed, ⚠️ 1 flaky)
  • ❌ Reactions > Verify liking someone's location (tags: TC-1533, regression)
  • ❌ Reactions > I want to add/remove a reaction using emoji picker (tags: TC-1549, regression)
  • ⚠️ Reactions > Verify liking someone's link preview in 1:1 (tags: TC-1527, regression)
specs/Reply/reply.spec.ts (❌ 0 failed, ⚠️ 2 flaky)
  • ⚠️ Reply > I want to reply to a picture (tags: TC-3002, regression)
  • ⚠️ Reply > I want to reply with mention and tap on the mention in the reply opens the user profile (tags: TC-3016, regression)
specs/Search/search.spec.ts (❌ 0 failed, ⚠️ 1 flaky)
  • ⚠️ Search > Verify you can search by partial username of unconnected user (tags: TC-1657, regression)

@JacquelineLehner JacquelineLehner force-pushed the 19954-history-backup branch 2 times, most recently from 0a1f0c2 to be051b8 Compare February 9, 2026 10:08
@JacquelineLehner JacquelineLehner force-pushed the 19954-history-backup branch 2 times, most recently from 0d8cc44 to ade0b7e Compare February 10, 2026 17:10
@JacquelineLehner JacquelineLehner marked this pull request as ready for review February 10, 2026 17:11
Copilot AI review requested due to automatic review settings February 10, 2026 17:11
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds Playwright regression coverage around history backup export and restore flows, plus shared utilities/page objects to support those scenarios.

Changes:

  • Introduces a reusable createAndSaveBackup helper for exporting and saving history backups.
  • Adds a new HistoryBackup E2E spec covering restore scenarios (wrong account, wrong password, conversation rename, mute/archive state, etc.).
  • Extends conversation list/details page objects with additional locators and a notifications helper.

Reviewed changes

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

Show a summary per file
File Description
apps/webapp/test/e2e_tests/utils/userActions.ts Adds shared backup export/download helper for E2E tests
apps/webapp/test/e2e_tests/specs/HistoryBackup/historyBackup.spec.ts New regression suite for history backup/restore scenarios
apps/webapp/test/e2e_tests/specs/CriticalFlow/backupRestoration-TC-8634.spec.ts Reuses the shared backup helper (removes local duplication)
apps/webapp/test/e2e_tests/pageManager/webapp/pages/conversationList.page.ts Adds locator for muted conversation badge
apps/webapp/test/e2e_tests/pageManager/webapp/pages/conversationDetails.page.ts Adds locators + helper for notifications, rename, delete group

This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
Copilot AI review requested due to automatic review settings February 11, 2026 13:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
Copilot AI review requested due to automatic review settings February 11, 2026 13:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
This commit introduces a new test suite `historyBackup.spec.ts` to
verify the history export and import functionality.

Implemented scenarios include:
- Importing backups after changing account credentials (email/password)
  (@TC-118).
- Verifying cross-account restoration is prevented (@TC-125).
- Verifying restoration with an incorrect password is prevented
  (@TC-135).
- Ensuring conversation states (renames, system messages, mute, archive)
  are preserved after restore (@TC-131, @TC-133).
- Verifying deleted groups do not reappear after restoration (@TC-1097).
- added Test steps

Refs: WPB-19954
Copilot AI review requested due to automatic review settings February 11, 2026 14:52
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

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