Make WordPress Core

Opened 2 years ago

Closed 2 years ago

#57452 closed defect (bug) (fixed)

Incorrect inline comment in Tests_Ajax_wpAjaxReplytoComment::test_as_subscriber

Reported by: davidbinda's profile david.binda Owned by: audrasjb's profile audrasjb
Milestone: 6.2 Priority: normal
Severity: trivial Version:
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

The inline comment in Tests_Ajax_wpAjaxReplytoComment::test_as_subscriber says:

// Become an administrator.

But the code does:

$this->_setRole( 'subscriber' );

The code seems to match the intention of the Tests_Ajax_wpAjaxReplytoComment::test_as_subscriber test, so the inline comment should be adjusted accordingly

Attachments (1)

57452.diff (449 bytes) - added by david.binda 2 years ago.

Download all attachments as: .zip

Change History (4)

@david.binda
2 years ago

#1 @SergeyBiryukov
2 years ago

  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 6.2

#2 @audrasjb
2 years ago

  • Owner set to audrasjb
  • Status changed from new to accepted

Thanks for the patch!
Looks good to go. Marking for commit.

#3 @audrasjb
2 years ago

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

In 55058:

Build/Test Tools: Fix an incorrect inline comment in Tests_Ajax_wpAjaxReplytoComment.

This changeset replaces "Become an administrator" with "Become a subscriber" in an inline comment located in Tests_Ajax_wpAjaxReplytoComment::test_as_subscriber.

Props davidbinda.
Fixes #57452.

Note: See TracTickets for help on using tickets.