Make WordPress Core

Opened 3 months ago

Closed 3 months ago

#49846 closed defect (bug) (fixed)

Core TravisCI not submitting results to hosting automated tests.

Reported by: mikeschroder Owned by: mikeschroder
Milestone: 5.5 Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch commit
Focuses: Cc:


This is a followup to: https://github.com/WordPress/wpdev-docker-images/issues/22

For anyone who isn't familiar with the distributed automated tests, the WordPress Hosting Team has a distributed automated test project that collects PHPUnit test results from hosts.

As part of this, WordPress.org's TravisCI reports to that page. Currently, this is used to email hosts when Core's tests are passing, but theirs are not.

The packages required are there now (thanks @pento!), but it looks like the API key isn't making its way to the reporter:

I'll attach a patch shortly.

Attachments (1)

49846.diff (542 bytes) - added by mikeschroder 3 months ago.
Pass API key to reporter in docker-compose.

Download all attachments as: .zip

Change History (6)

3 months ago

Pass API key to reporter in docker-compose.

#1 @mikeschroder
3 months ago

Added 49846.diff.

This appears to work as expected on my Github fork with an alternate API key.

The tests still aren't able to pull envionment information for Imagick or MySQL (as is visible in these results), but it should restore result submission and pass/fail.

This ticket was mentioned in Slack in #hosting-community by mike. View the logs.

3 months ago

#3 @SergeyBiryukov
3 months ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 5.5

#4 @mikeschroder
3 months ago

Thanks much @SergeyBiryukov!

I'll get this committed.

#5 @mikeschroder
3 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 47556:

Tests: Pass API key to distributed test reporter.

Pass environment variable for test reporter API key to
docker-compose so that it can authenticate and submit results
to the hosting distributed tests.

Props SergeyBiryukov, mikeschroder.
Fixes #49846.

Note: See TracTickets for help on using tickets.