Skip to content
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

More updates to the Robolectric test harness #11068

Merged
merged 1 commit into from
Aug 17, 2019
Merged

More updates to the Robolectric test harness #11068

merged 1 commit into from
Aug 17, 2019

Conversation

mklim
Copy link
Contributor

@mklim mklim commented Aug 16, 2019

Previously the test wasn't correctly re-building the engine when its
files changed on multiple runs of testing/run_tests.py. It looks like
this is because the test build target wasn't depending on the entire
engine Android dependency, so some code changes were being ignored.
Update the build.

Copy link
Contributor

@matthew-carroll matthew-carroll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Previously the test wasn't correctly re-building the engine when its
files changed on multiple runs of `testing/run_tests.py`. It looks like
this is because the test build target wasn't depending on the entire
engine Android dependency, so some code changes were being ignored.
Update the build.
@mklim mklim merged commit e23a2e2 into flutter:master Aug 17, 2019
@mklim mklim deleted the more_test_updates branch August 17, 2019 00:09
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Aug 17, 2019
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Aug 17, 2019
[email protected]:flutter/engine.git/compare/4d5c38e215e9...e23a2e2

git log 4d5c38e..e23a2e2 --no-merges --oneline
2019-08-17 [email protected] More updates to the Robolectric test harness (flutter/engine#11068)
2019-08-16 [email protected] Minor update to the Robolectric test harness (flutter/engine#11067)
2019-08-16 [email protected] Roll src/third_party/dart 2a3b844b41..8ab978b6d4 (7 commits)
2019-08-16 [email protected] Roll fuchsia/sdk/core/linux-amd64 from 7b00W... to -UaaS... (flutter/engine#11065)

The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff ([email protected]), and stop
the roller if necessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants