Make WordPress Core

Opened 5 years ago

Closed 14 months ago

#53746 closed task (blessed) (maybelater)

Tidy up the tearDown() methods

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

Description (last modified by johnbillion)

The tearDown() method in several of the test classes perform redundant operations, for example removing or reinstating hooks, which are already taken care of in the base test class.

This means there's code that can be removed from these methods to tidy them up.

Related: #53011

Previously: [50463]

Change History (9)

This ticket was mentioned in PR #1514 on WordPress/wordpress-develop by johnbillion.


5 years ago
#1

  • Keywords has-patch has-unit-tests added

#2 @johnbillion
5 years ago

  • Description modified (diff)

hellofromtonya commented on PR #1514:


5 years ago
#3

There are a lot of global resetting happening throughout the test classes. In some cases these are duplicated for similar tests such as with the themes group. We may want to think about pulling these resets back into the framework itself instead of distributed within individual tests. This is likely beyond the scope of this PR/ticket, but does fit into Trac https://core.trac.wordpress.org/ticket/53011.

#4 @hellofromTonya
5 years ago

  • Milestone 5.96.0

5.9 is in Feature Freeze and focus is on defects. With less than a week until Beta 1, there's likely not time to in the 5.9 cycle to focus on this initiative. Let's get it done in 6.0.

As tests can be committed at anytime into trunk, if someone has the time, please pull it back in and commit.

This ticket was mentioned in Slack in #core by mike. View the logs.


4 years ago

#6 @Toro_Unit
4 years ago

@johnbillion

Hello. What is the current status of this ticket and want to have this patch committed in 6.0 ?

#7 @johnbillion
4 years ago

  • Milestone 6.0Future Release

#9 @johnbillion
14 months ago

  • Milestone Future Release
  • Resolutionmaybelater
  • Status newclosed
Note: See TracTickets for help on using tickets.