WordPress.org
Get WordPress

Make WordPress Core

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#50649 closed defect (bug) (fixed)

Travis: Download Chromium for e2e and JS tests only, take 2

Reported by: SergeyBiryukov Owned by: SergeyBiryukov
Priority: normal Milestone: 5.5
Component: Build/Test Tools Version:
Severity: normal Keywords:
Cc: Focuses:

Description

Background: #49621

[48177] removed PUPPETEER_SKIP_CHROMIUM_DOWNLOAD from .travis.yml, reverting [47460] and causing unnecessary downloads of ~120 MB over and over again on every run.

The tests ​still pass with that environment variable restored, so this change appears to be accidental.

Change History (2)

#1 @SergeyBiryukov
6 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 48458:

Tests: Download Chromium for e2e and JS tests only.

This reduces the chance of random failures due to network timeouts and avoids the overhead of unnecessarily downloading ~120 MB for PHP tests.

Follow-up to [47449], [47459], [47867], [48177].

Fixes #50649.

#2 @desrosj
6 years ago

In 49227:

Build/Test Tools: Download Chromium for E2E tests only.

This prevents Chromium from being downloaded in workflows where it is not required.

See #49621, #50649, #50401.

Note: See TracTickets for help on using tickets.