-
Notifications
You must be signed in to change notification settings - Fork 29.7k
[Android 16] Update mockito to 5.17.0 from 4.7.0, prework for android 36 support #167573
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
[Android 16] Update mockito to 5.17.0 from 4.7.0, prework for android 36 support #167573
Conversation
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.
Can you add this to be the PR description: "Deleted mockito-inline because it is automatically used after 5.2.0 according to Maven docs: https://mvnrepository.com/artifact/org.mockito/mockito-inline/5.2.0"
done |
gmackall
left a comment
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.
LGTM - was this config doing anything? Or were we just being explicit about not using a manifest, and now it is implicit?
Best I can tell Config.NONE did nothing and the manifest attribute has been deprecated since 2018 and we had no instances of a configured manifest to migrate. |
|
autosubmit label was removed for flutter/flutter/167573, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label. |
…lutter#167573) This updates all the dependencies in engine/src/flutter/shell/platform/android/test_runner/build.gradle and ignores the play store deprecration so I could get android studio to give me other lint help. This work is required to unblock @jesswrd's work to update the engine to use android 36. Deleted mockito-inline because it is automatically used after 5.2.0 according to Maven docs: https://mvnrepository.com/artifact/org.mockito/mockito-inline/5.2.0" - **Remove config none annotations that have been deprecrated since roboletrics 4.0.1** - **Update all dependencies used in test runner, passes for spellcheckplugintest** ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement]. - [x] I signed the [CLA]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I added new tests to check the change I am making, or this PR is [test-exempt]. - [x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported. - [x] All existing and new tests are passing.
This updates all the dependencies in engine/src/flutter/shell/platform/android/test_runner/build.gradle and ignores the play store deprecration so I could get android studio to give me other lint help.
This work is required to unblock @jesswrd's work to update the engine to use android 36.
Deleted mockito-inline because it is automatically used after 5.2.0 according to Maven docs: https://mvnrepository.com/artifact/org.mockito/mockito-inline/5.2.0"
Pre-launch Checklist
///).