Fix VRT failures in CI

[tofumatt]

Summary

Addresses issue:

• Fix issues where the VRT test fails in CI before the tests have all completed #9528

PR Author Checklist

• My code is tested and passes existing unit tests.
• My code has an appropriate set of unit tests which all pass.
• My code is backward-compatible with WordPress 5.2 and PHP 7.4.
• My code follows the WordPress coding standards.
• My code has proper inline documentation.
• I have added a QA Brief on the issue linked above.
• I have signed the Contributor License Agreement (see https://cla.developers.google.com/).

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

Code Reviewer Checklist

• Run the code.
• Ensure the acceptance criteria are satisfied.
• Reassess the implementation with the IB.
• Ensure no unrelated changes are included.
• Ensure CI checks pass.
• Check Storybook where applicable.
• Ensure there is a QA Brief.
• Ensure there are no unexpected significant changes to file sizes.

Merge Reviewer Checklist

• Ensure the PR has the correct target branch.
• Double-check that the PR is okay to be merged.
• Ensure the corresponding issue has a ZenHub release assigned.
• Add a changelog message to the issue.

[eugene-manuilov]

All Run commands must be combined into one.

[eugene-manuilov]

Group ARGs together:

ARG ...
ARG ...

ENV ...

[eugene-manuilov]

There is the newer version of backstopjs - 6.3.2. We should probably upgrade it as well, shouldn't we?

[tofumatt]

Oh yes, sorry, I missed that 👍🏻

[eugene-manuilov]

Also, there are two ca-certificates packages are being installed. We need to cleanup installed packages to have no duplicates.

[eugene-manuilov]

Plus we can probably move PUPPETEER_SKIP_CHROMIUM_DOWNLOAD from RUN to a separate ENV PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=....

[eugene-manuilov]

@benbowler are all those lib* packages that you added in the dockerfile file really needed? Or it was copy/pasted from somewhere?

[benbowler]

@benbowler are all those lib* packages that you added in the dockerfile file really needed? Or it was copy/pasted from somewhere?

@eugene-manuilov they are from the puppeteer documentation linked in the code comment:

In gitlab-ci.yml we need to install some packages to make it possible to launch headless Chrome in your docker env.