Skip to content

Conversation

@ricardoamador
Copy link
Contributor

@ricardoamador ricardoamador commented Feb 8, 2023

The Linux_Android android_defines_test is consistently failing on release branches.

For the linux_android defines test the use_emulator flag made it into the release branch however none of the supporting changes did. This cherry-picks two commits that are needed to support the testing with the emulator.

Commits are:
f989d55
22bbdf0

List which issues are fixed by this PR. You must list at least one issue.
Fixes #120287

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.

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, including Features we expect every widget to implement.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is test-exempt.
  • All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

* Round 2 with dimensions

* Try with stringified keys

* Forgot a comma

* trying a list of maps.

* Try key value pairs

* Try with kvm only

* try with swarming_dimensions

* Updated with fix in swarming.star

* Attempting to remove the bring up property to verify test is running.

* Remove android from target tags to avoid adb command automatically running in test_utils call.

* Remove the duplicated dependencies in the defines target that exist in the platform
* Testing whether emulator is possible.

* Adding changes to see if emulator can be used from recipe.

* adding emulator support.

* Add the emulator flag for testing.

* Using string for boolean since it cannot be parsed in properties

* Checking to see if these changes are being used.

* Updated bool back to string

* Remove trailing whitespace from file.
@flutter-dashboard
Copy link

This pull request was opened from and to a release candidate branch. This should only be done as part of the official Flutter release process. If you are attempting to make a regular contribution to the Flutter project, please close this PR and follow the instructions at Tree Hygiene for detailed instructions on contributing to Flutter.

Reviewers: Use caution before merging pull requests to release branches. Ensure the proper procedure has been followed.

@flutter-dashboard flutter-dashboard bot added the c: contributor-productivity Team-specific productivity, code health, technical debt. label Feb 8, 2023
@ricardoamador ricardoamador deleted the flutter-3.7-candidate.1 branch February 8, 2023 18:11
@ricardoamador ricardoamador restored the flutter-3.7-candidate.1 branch February 8, 2023 20:00
@ricardoamador ricardoamador reopened this Feb 8, 2023
@CaseyHillers
Copy link
Contributor

@ricardoamador this requires a CP request issue to be attached. Please file one at https://github.com/flutter/flutter/issues/new?assignees=&labels=cp%3A+review&template=7_cherry_pick.yml&title=%5BCP%5D+%3Ctitle%3E

In the description, include why this is needed, and links to example breakages.

@ricardoamador
Copy link
Contributor Author

ricardoamador commented Feb 8, 2023

Added a Cherry-pick request.
#120320

@godofredoc
Copy link
Contributor

Landing with failures on codesigning tests as they are not supposed to run/pass on presubmit.

@godofredoc godofredoc merged commit f09b73e into flutter:flutter-3.7-candidate.1 Feb 9, 2023
@ricardoamador
Copy link
Contributor Author

Thanks @godofredoc!

engine-flutter-autoroll added a commit to engine-flutter-autoroll/plugins that referenced this pull request Feb 22, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 25, 2023
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Feb 26, 2023
@ricardoamador ricardoamador deleted the flutter-3.7-candidate.1 branch September 25, 2023 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: contributor-productivity Team-specific productivity, code health, technical debt.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants