This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Make robolectric tests run against SDK 29 by default #17996
Merged
Conversation
This file contains hidden or 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
blasten
reviewed
Apr 28, 2020
| @@ -0,0 +1,2 @@ | |||
| # Match the value at shell/platform/android/embedding_bundle/build.gradle. | |||
| sdk=28 No newline at end of file | |||
blasten
approved these changes
Apr 28, 2020
blasten
left a comment
blasten
left a comment
There was a problem hiding this comment.
Thanks for catching this one up! It took me a while, and I couldn't figure out when I tried it.
Member
Author
|
Needs flutter/buildroot#367 |
xster
changed the title
Member
Author
|
ok, I had to rescope this. We need to keep the android-all-4 in there because this test https://github.com/flutter/engine/blob/master/shell/platform/android/test/io/flutter/plugin/platform/PlatformPluginTest.java#L18 uses SDK=16. I changed it a bit. We used to test everything on SDK=16 but the old shadows have some buggy behaviors. Made tests run against SDK=29 (our engine's normal targetSDK) instead. Left the second jar in the test. |
xster
added
the
waiting for tree to go green
Contributor
|
This pull request is not suitable for automatic merging in its current state.
|
fluttergithubbot
removed
the
waiting for tree to go green
xster
added
the
waiting for tree to go green
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
This was referenced May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
This was referenced May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 9, 2020
This was referenced May 9, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 9, 2020
GaryQian
pushed a commit
to flutter/flutter
that referenced
this pull request
May 11, 2020
* 9ea2db5 Add FlMessageCodec, FlBinaryCodec, FlStringCodec (flutter/engine#18186) * f422757 Roll src/fuchsia/sdk/linux from MhpFP... to c1q_S... (flutter/engine#18222) * 2ab918c Roll src/third_party/skia a14084ba1b41..8f6c3ed7c7be (1 commits) (flutter/engine#18223) * 40167b6 Make robolectric tests run against SDK 29 by default (flutter/engine#17996) * e5b0db6 Roll src/third_party/skia 8f6c3ed7c7be..b55372444d1b (4 commits) (flutter/engine#18224) * ed08c3e Roll src/fuchsia/sdk/mac from 4MCVP... to T5tT0... (flutter/engine#18228) * 6767517 Roll src/third_party/skia b55372444d1b..ac09f7cd7a28 (2 commits) (flutter/engine#18229) * 983de2c Remove pipeline in favor of layer tree holder (flutter/engine#17688) * a1218dd Roll src/third_party/skia ac09f7cd7a28..c683912173bb (2 commits) (flutter/engine#18230) * 68bf137 skip painting clipped out pictures (flutter/engine#18204) * 7035255 make compiler worker count configurable (flutter/engine#17616) * c8ff03c Publish validation layer deps as part of the fuchsia artifacts (flutter/engine#18214) * 576f0e1 Roll src/third_party/skia c683912173bb..7359165e660c (1 commits) (flutter/engine#18234) * 1b3b4ec skip font loading tests for safari (flutter/engine#18232) * 9319d7c Roll src/third_party/skia 7359165e660c..6913d1bb1d7a (1 commits) (flutter/engine#18237) * 1b56f35 Roll src/third_party/dart 617bc54b715d..2a14a62112e6 (30 commits) (flutter/engine#18239) * ff6942f Add fontFeatures and decorationThickness to textstyle (flutter/engine#18235) * 4418ce8 Revert "Remove pipeline in favor of layer tree holder (#17688)" (flutter/engine#18242) * 9d8daf2 Roll src/third_party/skia 6913d1bb1d7a..bf1904fd4898 (3 commits) (flutter/engine#18243) * Updated bin/internal/fuchsia-linux.version * Updated bin/internal/fuchsia-mac.version
This was referenced May 16, 2020
wandyers
pushed a commit
to wandyers/engine
that referenced
this pull request
May 23, 2020
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
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.
This lets us just have one copy of the robolectric android classes.
Let me know if you want me to do the CIPD change here or separately.