Make WordPress Core

Opened 2 years ago

Closed 17 months ago

Last modified 17 months ago

#53484 closed defect (bug) (fixed)

unit test for current_datetime()

Reported by: pbearne's profile pbearne Owned by: hellofromtonya's profile hellofromTonya
Milestone: 6.0 Priority: normal
Severity: normal Version:
Component: Date/Time Keywords: has-patch has-unit-tests
Focuses: Cc:


Change History (7)

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

2 years ago

Added test for current datetime

Trac ticket:

#2 @desrosj
2 years ago

  • Component changed from Build/Test Tools to Date/Time
  • Milestone changed from Awaiting Review to 5.9

#3 @hellofromTonya
18 months ago

  • Milestone changed from 5.9 to 6.0
  • Owner set to hellofromTonya
  • Status changed from new to accepted

With 5.9 Beta 1 tomorrow, moving this ticket to 6.0 and assigning it to me for review.

#4 @SergeyBiryukov
17 months ago

#54869 was marked as a duplicate.

SergeyBiryukov commented on PR #1414:

17 months ago

Thanks for the PR! This looks good to me, but I think the test could be placed in a new file, tests/date/currentDatetime.php, for consistency with current_time() and other date/time tests. I can handle that on commit.

#6 @SergeyBiryukov
17 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 52611:

Date/Time: Add a unit test for the return type of current_datetime().

Follow-up to [45883].

Props pbearne.
Fixes #53484.

Note: See TracTickets for help on using tickets.