Skip to content

test(backup mas): use checkDeviceIsConnectedKeyBackup instead at looking at the Security & Privacy tab#29146

Merged
florianduros merged 2 commits into
developfrom
florianduros/playright-backup-mas
Feb 10, 2025
Merged

test(backup mas): use checkDeviceIsConnectedKeyBackup instead at looking at the Security & Privacy tab#29146
florianduros merged 2 commits into
developfrom
florianduros/playright-backup-mas

Conversation

@florianduros
Copy link
Copy Markdown
Member

@florianduros florianduros commented Jan 30, 2025

Checklist

  • Tests written for new code (and old code if feasible).
  • New or updated public/exported symbols have accurate TSDoc documentation.
  • Linter and other CI checks pass.
  • I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

Key backup section will be removed in #29088. Use checkDeviceIsConnectedKeyBackup to look at the keybackup instead of using the Security & Privacy tab

@florianduros florianduros added the T-Task Tasks for the team like planning label Jan 30, 2025
@florianduros florianduros changed the title test(backup mas): use checkDeviceIsConnectedKeyBackup instead at looking at the *Security & Privacy* tab test(backup mas): use checkDeviceIsConnectedKeyBackup instead at looking at the Security & Privacy tab Jan 30, 2025
@florianduros florianduros force-pushed the florianduros/playright-backup-mas branch from b6bd4b7 to 698d72d Compare February 10, 2025 13:18
@florianduros florianduros force-pushed the florianduros/playright-backup-mas branch from 698d72d to dd7dd2c Compare February 10, 2025 15:26
// Wait for the ui to load
await expect(page.locator(".mx_MatrixChat")).toBeVisible();

// xxx: why the backup key is not in 4S?
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is no 4S at this point: recovery hasn't been set up.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Oh, okay, I'll remove my comment then

@florianduros florianduros added this pull request to the merge queue Feb 10, 2025
Merged via the queue into develop with commit 426a206 Feb 10, 2025
@florianduros florianduros deleted the florianduros/playright-backup-mas branch February 10, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants