-
Notifications
You must be signed in to change notification settings - Fork 187
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test recovery key #2429
Test recovery key #2429
Conversation
… Node decide how to navigate.
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
trySend(backupStateMapper.map(status)) | ||
} | ||
} | ||
service.backupStateListener(listener) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd have offer this as extension methods as we already do that in several places (RoomList, Timeline...)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, sorry, I forgot this: 7d4cf6d
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #2429 +/- ##
========================================
Coverage 72.44% 72.45%
========================================
Files 1357 1357
Lines 32261 32263 +2
Branches 6259 6259
========================================
+ Hits 23373 23375 +2
Misses 5658 5658
Partials 3230 3230 ☔ View full report in Codecov by Sentry. |
e13a590
to
7d4cf6d
Compare
|
Complement for #2421: handle @ganfra 's remarks, and add Maestro test on entering Recovery Key to verify the session.
The RecoveryKey has been added to the GH secret, but will be visible on Maestro recording, this is OK I think, this is just a test account. The account password is still not visible.