Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#30769 closed defect (bug) (fixed)

Unit Tests: Update @covers in a couple of tests

Reported by: sgrant Owned by: boonebgorges
Priority: normal Milestone: 4.2
Component: Build/Test Tools Version: 4.2
Severity: normal Keywords:
Cc: Focuses:

Description

I found a couple of mislabeled @covers tags when running phpunit with coverage-html. The attached patch fixes those two tags.
And this is my first ticket in trac, so it's totally scary. :)

Attachments (1)

patch.diff (983 bytes ) - added by sgrant 12 years ago.
Update @covers tags

Download all attachments as: .zip

Change History (3)

@sgrant
12 years ago

Update @covers tags

#1 @boonebgorges
12 years ago

  • Milestone Awaiting Review4.2

And this is my first ticket in trac, so it's totally scary. :)

Welcome! And fear not :)

Good call on the incorrect annotation. For future reference, functions in the global scope must be referenced as sgrant notes. See https://phpunit.de/manual/3.7/en/appendixes.annotations.html#appendixes.annotations.covers

See also #26999.

#2 @boonebgorges
12 years ago

  • Owner set to boonebgorges
  • Resolutionfixed
  • Status newclosed

In 30976:

Use correct notation for global functions in @covers test annotations.

Props sgrant.
Fixes #30769.

Note: See TracTickets for help on using tickets.