Make WordPress Core

Opened 7 years ago

Last modified 6 years ago

#47880 new task (blessed)

Extend unit tests for Site Health component.

Reported by: peterwilsoncc Owned by:
Priority: normal Milestone: Future Release
Component: Site Health Version: 5.2
Severity: normal Keywords: needs-unit-tests needs-patch
Cc: Focuses:

Description

A file for site health check unit tests has been added in tests/phpunit/tests/site-health.php.

At the time of writing it only includes tests for various states of cron, expanding these would help with future development of the component.

Change History (12)

#1 @Clorith
7 years ago

  • Milestone Awaiting Review5.3
  • Type enhancementtask (blessed)

Agreed, some inspiration can be taken from the plugin (https://github.com/WordPress/health-check/tree/master/tests/phpunit), although the tests there also need to be expanded upon as they're fairly limited at this time, and relate more to user experience (in the relation of the Site Health Checks).

I'm going to flag this for 5.3, to get some traction, but with the knowledge it may end up getting bumped to 5.4 as well, and change it to a task as unit tests can be added throughout betas so we don't stop any momentum gained.

#2 @desrosj
7 years ago

  • Keywords needs-patch added
  • Version5.2

This ticket was mentioned in Slack in #core-site-health by afragen. View the logs.


7 years ago

#4 @afragen
7 years ago

  • Milestone 5.35.4

This ticket was mentioned in Slack in #core-site-health by peterwilsoncc. View the logs.


7 years ago

#6 @rukkykf
7 years ago

I'd like to get to work on this. Will take a while though, I'm still working on getting familiar with the codebase.

This ticket was mentioned in Slack in #core-site-health by clorith. View the logs.


7 years ago

This ticket was mentioned in Slack in #core-site-health by afragen. View the logs.


7 years ago

#9 @afragen
7 years ago

  • Milestone 5.45.5

This ticket was mentioned in Slack in #core-site-health by afragen. View the logs.


6 years ago

This ticket was mentioned in Slack in #core-site-health by afragen. View the logs.


6 years ago

#12 @desrosj
6 years ago

  • Milestone 5.5Future Release

This still needs a working patch to be considered. I'm going to move this to future release, but because this just aims to add tests, it can be moved back whenever it's deemed ready.

Note: See TracTickets for help on using tickets.