why? tests are enabled by default anyway, using the condition is common practice and that way it's clear which dependencies are needed just for tests and which are needed for building. Also, it would allow for faster builds if required and I don't think it hurts readability of the spec file