Make WordPress Core

Opened 7 months ago

Closed 8 days ago

Last modified 8 days ago

#53484 closed defect (bug) (fixed)

unit test for current_datetime()

Reported by: pbearne Owned by: 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.

7 months ago

Added test for current datetime

Trac ticket:

#2 @desrosj
7 months ago

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

#3 @hellofromTonya
2 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
8 days ago

#54869 was marked as a duplicate.

#5 @prbot
8 days ago

SergeyBiryukov commented on PR #1414:

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
8 days 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.