Make WordPress Core

Opened 5 years ago

Closed 3 years ago

#10 closed defect (fixed)

Correct and make use of TestPluginMenus

Reported by: duck_ Owned by: westi
Milestone: Fix Existing Tests Priority: normal
Component: Test Framework Keywords: has-patch


Currently some incorrect assertions for sub menus added to custom top level menus.

_WPEmptyBlog sets the site url to be example.com so we should be able to use that instead of null.host

Also see #WP14159

Attachments (2)

testpluginmenus.diff (5.8 KB) - added by duck_ 5 years ago.
ut.10.diff (793 bytes) - added by duck_ 5 years ago.

Download all attachments as: .zip

Change History (5)

#1 @duck_
5 years ago


Works great on its own, but noticed that it sometimes fails when run with all other tests. I believe this is because user with ID 1 is deleted at some previous point (so problems with separation of tests) by seeing that var_dump( get_userdata(1) ); is false. Patch attached to make a new administrator user which is deleted at the end of the test instead of relying on user ID 1.

5 years ago

#2 @nacin
3 years ago

Already fixed.

#3 @nacin
3 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.