Move SimpleTest out of "Site building"

I'm re-opening this. Had been meaning to open this for ages and glad someone else thought of it.

From UMN we know that some users often do one or both of these two things:

1. look in site building before anywhere else when they want to do something
2. enable all modules

A combination of these two means a lot of users lost in the middle of simpletests either looking for something else, or they want to 'ensure their site code is working as designed' - perhaps some HTML or CSS, or a new theme etc.

At least in 'reports' it's obvious that this is a diagnostic tool. You can use watchdog, phpinfo() and a bunch of other stuff while developing, but there's a reason it's all in reports.

There's also the question of whether SimpleTest should be included in non -dev tarballs in the same way we switch E_ALL off, I'm not sure if that's up for discussion.

I'd vote for creating a development menu. It could then be used by other related modules like devel.

+1 for admin/devel[opment|oper]. Who rolls the patch?

Patch creates a development menu in system.module and changes the main SimpleTest interface to be under it.

It also updates simpletest.test.

This works fine, but has the ugly side-effect of leaving a "Development" menu item there with nothing underneath it when you first install Drupal. This is not a fault of this patch, since moving all of the menu items from, say, "Content management" will have the same effect.

I suggested adding an access callback that both checks the permission AND whether or not there are sub-items, so that these "empty" items would be removed from the UI.

Well, system_admin_menu_block_page is a legit page callback. The menu system has no way of knowing that that callback is any different from the "My account" callback, which doesn't have any sub-items.

Might be worth looking at the "Create content" menu item's page/access callbacks and seeing how it handles this.

This will need to update http://drupal.org/node/230269#comment-867768 if committed.

Re-rolled and addressed #9.

Corrects all references to admin/build/testing and simpletest.test passes.

If we're moving it, we can't keep the admin > site building > SimpleTest directions anyway - would have to be admin > development > Testing

This addresses the issues brought up in #16 and and the > replacement for ">>".

Looks good to me, +1 for the idea of moving to admin/development.

I'm putting this in CNW because I implemented in #296693: Restrict access to empty top level administration pages a way to properly hide empty administrative sections (please review).

I believe the >> is used since that is the character used in garland. I see that in a number of INSTALL.txt files and such.

If we are waiting on other issue then this shouldn't be needs work it should be postponed.

It seems this has been done.