I'm trying to set up a new development environment for MusicBrainz using the VM detailed at http://wiki.musicbrainz.org/MusicBrainz_Server/Setup, and before I start making any changes to the code I want to check that my environment is good by running the test suite. However, prove -l t aborts with:

...
t/tests.t .. 1/363 There is no configuration in DBDefs for database READWRITE but one is required at /home/musicbrainz/musicbrainz-server/lib/MusicBrainz/Server/DatabaseConnectionFactory.pm line 47.
...

This is misleading, because contrary to this message, READWRITE does appear in DBDefs.pm. After some digging, it turned out that this was because MusicBrainz::Server::Test aliases READWRITE to TEST, and it was TEST that was missing from DBDefs.

Should MusicBrainz::Server::Test check that TEST is defined before trying to create the alias? Should method MusicBrainz::Server::DatabaseConnectionFactory::alias throw an exception if the target database is undefined?