Make WordPress Core

Opened 6 years ago

Closed 6 years ago

#39235 closed defect (bug) (fixed)

incorect @covers parm in unit test causes php covers report to fail

Reported by: pbearne's profile pbearne Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.8 Priority: normal
Severity: normal Version: 4.8
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:


Just comment text fix

Please do add so we can run the covers reports

Attachments (2)

adminbar.php.patch (425 bytes) - added by pbearne 6 years ago.
coverage_prams.patch (1.7 KB) - added by pbearne 6 years ago.
found some more

Download all attachments as: .zip

Change History (9)

#1 @pbearne
6 years ago

  • Keywords has-patch dev-feedback has-unit-tests added

6 years ago

found some more

#2 @pbearne
6 years ago

Found some more incorrect @coverage parms
Also removed leading whitespace from themeFile.php as this causing the cover report to fail

Last edited 6 years ago by pbearne (previous) (diff)

#3 follow-up: @netweb
6 years ago

  • Keywords dev-feedback has-unit-tests removed
  • Milestone changed from Awaiting Review to 4.8

Thanks for the patch and help testing this @pbearne

I've attached patch coverage_prams.patch to

The associated Travis CI job:

#4 in reply to: ↑ 3 @netweb
6 years ago

The Travis CI job failed:
The job exceeded the maximum time limit for jobs, and has been terminated.

Edit: I closed and reopened the PR to trigger a new Travis CI job, this should determine if the above failure was a once of isolated instance or not.

Last edited 6 years ago by netweb (previous) (diff)

#5 @pbearne
6 years ago

The coverage reports seem to very slow I am not sure how we can sort this unless we have full control of the box and can provide more resources

#6 @netweb
6 years ago

I've updated the pull request to utilise Travis CI Ubuntu Trusty and Google Compute Engine (~2xRAM/Similar CPU)

#7 @SergeyBiryukov
6 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 39918:

Build/Test Tools: Correctly reference function names in @covers entries.

Props pbearne.
Fixes #39235.

Note: See TracTickets for help on using tickets.