The user interface tests have been split up over multiple files. The tests concerning menu creation dialogs are placed in paddle_menu_manager_menu_form.test. This means that the name of paddle_menu_manager_ui.test is no longer correct, this does not contain all UI tests anymore.

I suggest to split these tests up in the following files:

  1. paddle_menu_manager_overviews.test: contains listings and overviews, these all need to have some pre-generated menus.
    • testLoadLanguage() -> this function is not correct, this actually tests the menus block.
    • testMenuOverview()
    • testMenuOverviewTypes()
  2. paddle_menu_manager_pages.test: contains simple page callbacks that do not require any pre-generated content.
    • testPlaceholderLink()

The documentation and classnames can be changed accordingly.

Comments

isolate’s picture

Assigned: Unassigned » isolate
isolate’s picture

Assigned: isolate » Unassigned
Status: Active » Needs review
pfrenssen’s picture

Assigned: Unassigned » pfrenssen

Reviewing.

pfrenssen’s picture

Assigned: pfrenssen » Unassigned
Status: Needs review » Fixed

Looks good, merged in 7.x-1.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Edit