The reason I did it as its own package before was because test_rosbridge_library had message and service definitions and I didn't want those installed when you install rosbridge_library. Luckily, there's a NOINSTALL argument that can be passed to message and service generation to, well, not install it but stil generate it for testing purposes. The more you know!