WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#37842 closed enhancement (fixed)

Make the tests for `comment_exists()` a little bit faster.

Reported by: Frank Klein Owned by: boonebgorges
Milestone: 4.7 Priority: normal
Severity: normal Version: 4.4
Component: Build/Test Tools Keywords: has-patch
Focuses: Cc:

Description

The current test methods in Tests_Admin_IncludesComment each create their own posts and comments.

It is possible to run the same tests by using a post with two comments attached, which is used by all tests.

Attachments (1)

37842.diff (4.3 KB) - added by Frank Klein 4 years ago.

Download all attachments as: .zip

Change History (4)

@Frank Klein
4 years ago

#1 @boonebgorges
4 years ago

  • Milestone changed from Awaiting Review to 4.7
  • Owner set to boonebgorges
  • Status changed from new to assigned

Dy-no-mite! Thanks for the patch.

#2 @boonebgorges
4 years ago

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

In 38372:

Use shared fixture in comment_exists() tests.

Props Frank Klein.
Fixes #37842.

#3 @DrewAPicture
4 years ago

  • Keywords has-patch added
Note: See TracTickets for help on using tickets.