Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[Android] Add target to have linux_android_emulator_tests run on AVDs with Android 33 & 34 #48811

Merged
merged 8 commits into from
Dec 11, 2023

Conversation

camsim99
Copy link
Contributor

@camsim99 camsim99 commented Dec 7, 2023

Adds target (Linux linux_android_emulator_tests_api_33) to have Android emulator testst run on AVDs with Android 33 so that the differences between those tests runs and those running on AVDs with Android 34 (Linux linux_android_emulator_tests) may be observed.

Related to flutter/flutter#137947.

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
  • I listed at least one issue that this PR fixes in the description above.
  • I added new tests to check the change I am making or feature I am adding, or the PR is test-exempt. See testing the engine for instructions on writing and running engine tests.
  • I updated/added relevant documentation (doc comments with ///).
  • I signed the CLA.
  • All existing and new tests are passing.

@camsim99 camsim99 marked this pull request as ready for review December 8, 2023 16:19
@camsim99 camsim99 added autosubmit Merge PR when tree becomes green via auto submit App and removed autosubmit Merge PR when tree becomes green via auto submit App labels Dec 11, 2023
Adds comment for context.
@camsim99 camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 11, 2023
@auto-submit auto-submit bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Dec 11, 2023
Copy link
Contributor

auto-submit bot commented Dec 11, 2023

auto label is removed for flutter/engine/48811, due to - The status or check suite Linux mac_unopt has failed. Please fix the issues identified (or deflake) before re-applying this label.

@camsim99 camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label Dec 11, 2023
@auto-submit auto-submit bot merged commit a4f87af into flutter:main Dec 11, 2023
@camsim99 camsim99 added the revert Label used to revert changes in a closed and merged pull request. label Dec 11, 2023
@camsim99
Copy link
Contributor Author

camsim99 commented Dec 11, 2023

We missed the name mismatch 😭 will rename and reland. I am now, though, confused as to how my test run passed...

auto-submit bot pushed a commit that referenced this pull request Dec 11, 2023
… on AVDs with Android 33 & 34 (#48811)"

This reverts commit a4f87af.
@auto-submit auto-submit bot removed the revert Label used to revert changes in a closed and merged pull request. label Dec 11, 2023
auto-submit bot added a commit that referenced this pull request Dec 11, 2023
…n on AVDs with Android 33 & 34" (#48895)

Reverts #48811
Initiated by: camsim99
This change reverts the following previous change:
Original Description:
Adds target (`Linux linux_android_emulator_tests_api_33`)  to have Android emulator testst run on AVDs with Android 33 so that the differences between those tests runs and those running on AVDs with Android 34 (`Linux linux_android_emulator_tests`) may be observed.

Related to flutter/flutter#137947.

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Dec 11, 2023
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Dec 11, 2023
…139941)

flutter/engine@5c1f13e...d001419

2023-12-11 98614782+auto-submit[bot]@users.noreply.github.com Reverts "[Android] Add target to have linux_android_emulator_tests run on AVDs with Android 33 & 34" (flutter/engine#48895)
2023-12-11 skia-flutter-autoroll@skia.org Roll Dart SDK from 611fca7f0c20 to 53dfedb82222 (1 revision) (flutter/engine#48891)
2023-12-11 skia-flutter-autoroll@skia.org Roll Skia from e55567fceb5f to 17bd70326a49 (22 revisions) (flutter/engine#48890)
2023-12-11 43054281+camsim99@users.noreply.github.com [Android] Add target to have linux_android_emulator_tests run on AVDs with Android 33 & 34 (flutter/engine#48811)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
@camsim99
Copy link
Contributor Author

Getting re-landed in #48936.

auto-submit bot pushed a commit that referenced this pull request Dec 13, 2023
…n on AVDs with Android 33 & 34" (#48936)

Re-lands #48811 with corrected config file name.

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
auto-submit bot added a commit that referenced this pull request Dec 13, 2023
…_tests run on AVDs with Android 33 & 34"" (#48988)

Reverts #48936
Initiated by: loic-sharma
This change reverts the following previous change:
Original Description:
Re-lands #48811 with corrected config file name.

[C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants