WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#50649 closed defect (bug) (fixed)

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

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

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
5 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to 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
5 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.