-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix: bypassing app lock after timeout #2324
Conversation
Test Results645 tests +1 645 ✔️ +1 6m 42s ⏱️ - 1m 21s Results for commit c3ec807. ± Comparison against base commit d3d6bf7. This pull request removes 12 and adds 13 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Codecov Report
@@ Coverage Diff @@
## develop #2324 +/- ##
=============================================
- Coverage 41.27% 41.26% -0.01%
- Complexity 1051 1052 +1
=============================================
Files 328 328
Lines 11924 11924
Branches 1589 1585 -4
=============================================
- Hits 4922 4921 -1
Misses 6533 6533
- Partials 469 470 +1
Continue to review full report in Codecov by Sentry.
|
APKs built during tests are available here. Scroll down to Artifacts! |
Build 1290 succeeded. The build produced the following APK's: |
PR Submission Checklist for internal contributors
The PR Title
SQPIT-764
The PR Description
What's new in this PR?
Issues
There was an issue with locking logic that allowed the user to bypass entering passcode by reseting the timer.
Solutions
Removed keeping the timestamp in the data store, because according to acceptance criteria: "If the user kills the app, then a password is required even if the minute is not over", so it was additionally making the logic more complex than it should be. Thanks of that, the logic responsible for checking the lock and app visibility state and calculating the lock timeout could be simplified to not operate on timestamps but just provide a delay.
Testing
Test Coverage (Optional)
How to Test
PR Post Submission Checklist for internal contributors (Optional)
PR Post Merge Checklist for internal contributors
References
feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764
.