Although ecm_optional_add_subdirectory(autotests) allows to skip
building the autotests with -DBUILD_autotests=FALSE, it doesn't seem
reasonable to have two options controlling similar behaviour and
BUILD_TESTING is the de-facto standard for cmake and already used
when discovering Qt5::Test.
Furthermore it avoids a cmake warning when BUILD_TESTING is set to
FALSE because Qt5::Test is passed to target_link_libraries but
isn't a valid target.