WordPress.org

Make WordPress Core

Opened 3 months ago

Last modified 3 months ago

#54359 new defect (bug)

test for wp_get_direct_update_https_url

Reported by: pbearne Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Build/Test Tools Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

just yet another test

Change History (3)

This ticket was mentioned in PR #1795 on WordPress/wordpress-develop by pbearne.


3 months ago

  • Keywords has-patch has-unit-tests added

#2 @prbot
3 months ago

joyously commented on PR #1795:

This does not test the case for when the environment variable is not set.
Since the function is supposed to return a URL, and it can return an empty string, it seems like that is beyond the unit test (for what happens with an empty URL), but at least it should assert that there is no URL in that case.

#3 @prbot
3 months ago

pbearne commented on PR #1795:

added a test to cover empty state

Note: See TracTickets for help on using tickets.