ecm_add_test method is added

Authored by himanshuvishwakarma on Mar 16 2018, 2:55 PM.



In this diff,
I removed the enable testing & add-testing and others method for the adding the unit test. Now, we have to add the test by using the one method i.e. ecm_add_test in cmake.

by this change, extra-cmake-modules is required for the building test. So I updated the HACKING manual also.

Diff Detail

R2 GCompris
Lint Skipped
Unit Tests Skipped
Restricted Application added a project: KDE Edu. · View Herald TranscriptMar 16 2018, 2:55 PM
Restricted Application added a subscriber: KDE Edu. · View Herald Transcript
himanshuvishwakarma requested review of this revision.Mar 16 2018, 2:55 PM
jjazeix added inline comments.Mar 16 2018, 7:07 PM

I'd rather not force to install ECM if not needed for building the main executable.
On other OS than linux, it may be difficult to find and install.


do we really need KDEInstallDirs and KDECMakeSettings?


Okay, i will make it optional, If we make ECM as optional then our test will also be optional.


Yaa, I think so because of the without KDEInstallDirs and KDECMakeSettings, ecm_add_test method of the module ECMAddTests is not working. : )

jjazeix accepted this revision.Mar 17 2018, 8:30 PM

I pushed an equivalent of it in
Now unit tests are optional so you need to pass -DBUILD_TESTING to build the library and unit tests.
I didn't added KDEInstallDirs and KDECMakeSettings as it was not needed

This revision is now accepted and ready to land.Mar 17 2018, 8:30 PM
jjazeix closed this revision.Mar 17 2018, 8:30 PM
himanshuvishwakarma edited the summary of this revision. (Show Details)

Now, extra-cmake-modules are not compulsory to compile the application but it required to run the unit test and other things.

I am sorry, I haven't seen that you push it and make necessary changes as required.