Skip to content

Latest commit

 

History

History
69 lines (43 loc) · 3.75 KB

Testing-presubmit-Engine-PRs-with-the-Flutter-framework.md

File metadata and controls

69 lines (43 loc) · 3.75 KB

Running Framework presubmit tests on Engine PRs

This documentation describes how to run flutter/flutter presubmit checks on flutter/engine PRs before submitting them.

Overview

  1. Create your engine pull request with the test: all label.
  2. Wait for all presubmit checks on your flutter/engine PR to be green.
  3. Determine the commit hash for your flutter/engine PR.
  4. Create and upload a flutter/flutter PR, (OR run tests locally).
  5. Wait for flutter/flutter presubmits/tests to run ☕.

1. Create your engine pull request with the test: all label.

When creating your PR, add the test: all label before submitting the PR to CI. This will ensure that all builds required for framework testing are triggered.

If you sent out your PR without adding the test: all label, you can add it, then re-push your branch to re-trigger presubmits.

By default, not all builds and tests are run in engine presubmits. When our CI is able to determine that certain shards are unaffected by a change, via a runIf clause in our .ci.yaml, for example, it will be skipped. Many framework tests, however, assume all build products are present and will trigger a flutter precache, which will fail with a 404 on missing build artifacts.

2. Wait

Contemplate the nature of the universe or why it is that this workflow was necessary for your situation. Could tests have been added to the engine? If not, get youself some coffee and a cookie. You'll need them.

3. The commit hash

  1. Go to the "Commits" tab in the GitHub UI for you Engine PR.
  2. Click the button to copy the most recent commit hash to your clipboard.
Screenshot 2023-08-04 at 12 54 55 PM

4. Create and upload a flutter/flutter PR.

Edit your flutter/flutter checkout as follows:

  1. bin/internal/engine.version should be edited to contain the commit hash from (2).
  2. bin/internal/engine.realm should be edited to contain the string flutter_archives_v2.

To run flutter/flutter presubmits on CI, you can accomplish these two edits directly in the GitHub editor UI, if desired. Otherwise, upload a flutter/flutter PR with these changes.

You can also build apps, and run tests locally at this point.

5. Wait for flutter/flutter presubmits to run ☕.

The flutter/flutter presubmit checks will run. There will be at least two failures:

  1. A Flutter CLI test will ensure that a PR with a non-empty engine.realm file will fail a presubmit check.
  2. The fuchsia_precache test will fail because Fuchsia artifacts are not uploaded from Engine presubmit runs.

Any other failures are possibly due to the changes to flutter/engine, so deflake and examine them carefully.

6. Devicelab tests

A subset of devicelab tests are available for optionally running in presubmit on flutter/flutter PRs. They are the tests listed in the flutter/flutter .ci.yaml file that are prefixed with Linux_android, Mac_android, and Mac_ios.

To run one of these tests, remove the line presubmit: false from the .ci.yaml file under the test you'd like to run. For an example, see the PR here.

Screenshot 2023-09-21 at 3 19 51 PM

This will trigger the devicelab test to run. The test will show up in the list of presubmit checks, and you can click through to the LUCI page to see the results.