We periodically perform a manual two-way sync. Below is an outline of the process interspersed with some tips.
Clone the Puppeteer git repository and checkout the release tag you want to vendor into mozilla-central.
% git checkout tags/v10.0 -b sync-v10.0
Apply any recent changes in
remote/test/puppeteerto the Puppeteer branch created above.
You might want to [install the project] at this point and make sure unit tests pass. Check the project’s
package.jsonfor relevant testing commands.
You should use this as basis for a PR to the Puppeteer project once you are satisfied that the two-way sync will be successful in mozilla-central. See their CONTRIBUTING.md.
Typically, the changes we push to Puppeteer include unskipping newly passing unit tests for Firefox along with minor fixes to the tests or to Firefox-specific browser-fetching and launch code.
Be sure to run tests against both Chromium and Firefox in the Puppeteer repo. You can specify your local Firefox build when you do so:
% BINARY=<path-to-objdir-binary> npm run funit
Now back in mozilla-central, you can run the following mach command to copy over the Puppeteer branch you just prepared. The mach command has flags to specify a local or remote repository as well as a commit.
% ./mach remote vendor-puppeteer
By default, this command also installs the newly-pulled Puppeteer package in order to generate a new
package-lock.jsonfile for the purpose of pinning Puppeteer dependencies for our CI. There is a
--no-installoption if you want to skip this step; for example, if you want to run installation separately at a later point.
Go through the changes under
remote/test/puppeteer/testand unskip any newly-skipped tests (e.g. change
it). A mass-change with
awkmight be useful here.
Why do we do this? The Puppeteer team runs their unit tests against Firefox in their CI with many tests skipped. In contrast, we leave these tests unskipped in Mozilla CI and track test expectation metadata in puppeteer-expected.json instead.
./mach puppeteer-test(see Testing) to run Puppeteer tests against both Chromium and Firefox in headless mode. Again, only running a subset of tests against Firefox is fine – at this point you just want to check that the typescript compiles and the browser binaries are launched successfully.
Next you want to update the test expectation metadata: test results might have changed, tests may have been renamed, removed or added. The easiest way to do this is to run the Puppeteer test job on try (see Testing). You will find the new test metadata as an artifact on that job and you can copy it over into your sync patch if it looks reasonable.
Examine the job logs and makes sure the run didn’t get interrupted early by a crash or a hang, especially if you see a lot of
TEST-UNEXPECTED-MISSINGin the Treeherder Failure Summary. You might need to add new test skips or fix some new bug in the unit tests. This is the fun part.
Once you are happy with the metadata and are ready to submit the sync patch up for review, run the Puppeteer test job on try again with
--rebuild 10to check for stability.