Fix running the tests with CMAKE_RUNTIME_OUTPUT_DIRECTORY set

Authored by heikobecker on Apr 18 2019, 1:08 PM.



With the newer ECM requirement introduced in [1]
CMAKE_RUNTIME_OUTPUT_DIRECTORY gets set [2], which causes the tests
to fail because they don't find the test executables. Adding the
NAME and COMMAND keywords, which aren't really optional according to
cmake's docs, makes them pass again.

[1] cb9697bc1a5ee2b31d554d6ae68c9c6e12206eea

Diff Detail

R467 K3b
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
heikobecker requested review of this revision.Apr 18 2019, 1:08 PM
heikobecker created this revision.
kossebau accepted this revision.Apr 18 2019, 1:16 PM

My bad, I know about this need for the newer add_test signature, but seems somehow missed it here. Thanks for fixing.

This revision is now accepted and ready to land.Apr 18 2019, 1:16 PM

Fixed the commit which introduced the newer ECM requirement

This revision was automatically updated to reflect the committed changes.