Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#44521 closed enhancement (fixed)

Add documentation to the PHPUnit factory objects

Reported by: andizer Owned by: pento
Priority: normal Milestone: 5.1
Component: Build/Test Tools Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

I discovered that the php unit test factories for WordPress aren't documented well. To make it easier for every one I decided to add some documentation for it.

Another thing I was wondering about is the old php4 way to declare methods (without access modifiers) and var declarations on class attributes. Is it desirable to modernize these or should I keep this untouched. Probably it is worth to make a separate issue for it.

Attachments (2)

44521.diff​ (15.1 KB ) - added by andizer 8 years ago.
44521.2.diff​ (15.2 KB ) - added by andizer 8 years ago.

Download all attachments as: .zip

Change History (8)

@andizer
8 years ago

#1 @andizer
8 years ago

  • Keywords has-patch added

#2 @JPry
8 years ago

@andizer

One thing I noticed when doing a quick skim of your patch: The docblock for WP_UnitTest_Factory_Callback_After_Create::__construct() has string as the parameter type. This should actually be callable. While this type hint is only valid in PHP 5.4+, using it in the docblock won't have any issue.

@andizer
8 years ago

#3 @andizer
8 years ago

@JPry Thanks. You are right, it makes more sense to have it named callable. I've changed it.

#4 @netweb
8 years ago

  • Component General → Build/Test Tools
  • Milestone Awaiting Review → 5.0

#5 @jorbin
8 years ago

  • Milestone 5.0 → 5.1

#6 @pento
8 years ago

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

In 44497:

Docs: Add docblocks for the PHPUnit factory objects.

Props andizer.
Fixes #44521.

Note: See TracTickets for help on using tickets.