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

Conversation

@gw280
Copy link
Contributor

@gw280 gw280 commented Dec 3, 2019

  • Copies fuchsia_test.sh into the testing/ directory
  • Fixes a small issue with fuchsia_archive to ensure the cmx file is correctly named according to the target
  • Add a list of fuchsia unittest fars to run on CI
  • Add a GN build target to build all currently-enabled unittests for Fuchsia

@gw280 gw280 added the CQ+1 label Dec 3, 2019
@gw280 gw280 requested review from dnfield and iskakaushik December 3, 2019 22:20
@auto-assign auto-assign bot requested a review from cbracken December 3, 2019 22:20
@gw280
Copy link
Contributor Author

gw280 commented Dec 3, 2019

Eventually I'd like to make the list of fuchsia fars autogenerated by gn at build time based off the fuchsia_tests target, but I wanted to keep the complexity down for this PR and iterate later.

@dnfield
Copy link
Contributor

dnfield commented Dec 4, 2019

Is there a corresponding recipe change for this? Or is the plan to land this first and then do the recipe change?

@gw280
Copy link
Contributor Author

gw280 commented Dec 4, 2019

Is there a corresponding recipe change for this? Or is the plan to land this first and then do the recipe change?

Yes, there's a corresponding recipe change which I have confirmed works on LUCI when I trigger manual runs with this branch (and it enables the fml_tests at the same time). I will upload the CL once this change is reviewed and merged as there will likely be recipe changes if anything needs changing in this CL (such as the file locations etc).

Copy link
Contributor

@iskakaushik iskakaushik left a comment

Choose a reason for hiding this comment

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

LGTM modulo the nits.

@gw280 gw280 force-pushed the gwright-fuchsia-test-script branch from be08cef to 6bcc533 Compare December 5, 2019 01:29
- Copies fuchsia_test.sh into the testing/fuchsia directory
- Fixes a small issue with fuchsia_archive to ensure the cmx file is correctly named according to the target
- Add a list of fuchsia unittest fars to run on CI
- Add a GN build target to build all currently-enabled unittests for Fuchsia
@gw280 gw280 force-pushed the gwright-fuchsia-test-script branch from 489be10 to 06ef930 Compare December 5, 2019 01:54
@gw280 gw280 merged commit 253851e into flutter:master Dec 5, 2019
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
- Copies fuchsia_test.sh into the testing/fuchsia directory
- Fixes a small issue with fuchsia_archive to ensure the cmx file is correctly named according to the target
- Add a list of fuchsia unittest fars to run on CI
- Add a GN build target to build all currently-enabled unittests for Fuchsia
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants