Believe it or not, it should actually be a build depend. Test dependencies are really just great dependencies needed at runtime. In this case, rostest exports the add_rostest macro, which needs to be available when building no matter what. There's a thread on this somewhere in the buildsystem SIG. One of the many reasons I think more people should be involved in the catkin discussion.