> On Sept. 17, 2013, 6:26 p.m., Alexander Neundorf wrote: > > The macro does more than the name implies, additionally to marking it as > > test it also actually adds the test. > > So I'd prefer a different name. > > > > Having said that, the CMakeLists.txt in the various tests/ subdirs in KDE > > frameworks look all quite similar, they more or less all feature a quite > > similar macro, foo_unit_tests(). Maybe a more advanced function can be > > written which can be used in all those places ? > > Alexander Richardson wrote: > I just played around with adding a new macro to replace the > <FRAMEWORK_NAME>_UNIT_TESTS macros which are in every autotests directory: > http://paste.kde.org/p11adaa09/ > > Could something like this be considered for ecm? If so I will add some > documentation and create a review request for ecm.
Neundorf: well, the difference between mark_as_test and mark_as_autotest is that it marks it as a test. Actually I think that mark_as_test is in fact a bit misleading, because it's not marking it as a test, only making it build as a test. About foo_unit_tests, I don't hate the idea, but then I don't think it pays off that much, if we go with that, we can just keep kde4_add_unit_test just as well. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112772/#review40240 ----------------------------------------------------------- On Sept. 17, 2013, 12:35 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112772/ > ----------------------------------------------------------- > > (Updated Sept. 17, 2013, 12:35 p.m.) > > > Review request for Build System and KDE Frameworks. > > > Description > ------- > > Creates a function that will define a unit test with the project name (like > it's being done in KF5) and calls ecm_mark_as_test. > > This should help simplify the tests creation that at the moment is quite > verbose. > > > Diffs > ----- > > modules/ECMMarkAsTest.cmake f1e53e4 > > Diff: http://git.reviewboard.kde.org/r/112772/diff/ > > > Testing > ------- > > Ported some tests in KF5, it worked. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ Kde-buildsystem mailing list Kde-buildsystem@kde.org https://mail.kde.org/mailman/listinfo/kde-buildsystem
