Remove Mockito dependency from project #13838
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Woohoo! Thank you! Heads up that this will go out in 7.29, which likely won't be until after the holidays :) |
AbandonedCart
pushed a commit
to SamSprung/Signal-Android
that referenced
this pull request
Jan 31, 2025
Fix derivation of RRP in RefreshAttributesJob. Fix incorrect RRP bugs and reglock state with reregistering. Update translations and other static files. Bump version to 7.28.1 Fix large message size calculation to use bytes. Update translations and other static files. Bump version to 7.28.2 Fix bug with ellipsizing on media messages. Bump version to 7.28.3 Translate missing recipients into unknown recipients. Bump version to 7.28.4 Treat all groups updates as directionless for backupv2. Remove Mockito dependency from project. Resolves signalapp#13838 Fix gv1 updates in backupv2. Fix empty profile names during backupv2 export. Move from AssertJ to AssertK. Resolves signalapp#13841 Fix dangling call ringers. Isolated tests for RecurringInAppPaymentRepository. Add verification metadata for windows. Don't show linked device bottom sheets if you've seen them before. Add internal UI for importing backup with different credentials. Improve internal backup import UI tool. Implement initial support for IAP data. Upgrade to AGP 8.7.2 Upgrade to AGP 8.7.3 Isolated tests for OneTimeInAppPaymentRepository. Move some things to lib.versions.toml Add AAPT metadata for witness verification. Use more plugin aliases. Upgrade to kotlin 2.1.0 Update a bunch of libraries. Update compileSdk to 35. Add .kotlin to gitignore. Update more libraries. Ensure that the V262 database migration runs. Update our base themes to avoid crashing MaterialAlertDialog. Revert "Improve table display in Spinner." This reverts commit df96b05. Fix cutoff string in help section. Fix typo in sql query. Add trigger definitions to logs. Capitalize log section title. Fix small gap in call menu. Update string translation comments. Fix leaked refreshActiveSubscription disposable. Fix ability to share text stories. Fixes signalapp#13855 Resolves signalapp#13879 Initialize database error handler with application instance. Fix text overflow length calculation. Fix overflow handling in condensed mode. Ensure filter is not retriggered when formatting. Fixes signalapp#13876 Fix proximity sensor for voice notes. Ensure new manifest is saved after rotation. Inline and enable the SSRE2 config. Exclude unregistered group members from invalid collisions check. Fixes signalapp#13866 Stop in-chat notification sounds if notifications are disabled. Fix bug showing null string on empty search. Fixes signalapp#13854 Resolves signalapp#13878 Revert "Update our base themes to avoid crashing MaterialAlertDialog." This reverts commit 446c7d6. Add missing required material3 dialog theme attribute. Disable the SSRE2 capability for now. Fix benchmark build. Add loading state to toggle switch and enforce when changing call link admin settings. Update translations and other static files. Update baseline profile. Co-Authored-By: Greyson Parrelli <greyson@signal.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contributor checklist
Fixes #1234syntaxDescription
This is the final pull request in a series, to move off of Mockito. Prior work went in via these commits:
After the current PR,
mockitocan't be found in the project:Testing
This still passes:
./gradlew qa