search

mitchcurtis / slate

Pixel Art Editor
GNU General Public License v3.0
1.07k stars 103 forks source link

Add testing to cmake #118

Closed wow2006 closed 5 years ago

mitchcurtis commented 5 years ago

Thanks! Looks mostly good, just a few questions.

wow2006 commented 5 years ago

I just remove the empty string from add_executable.

mitchcurtis commented 5 years ago

What about the other comment? "test-app" is duplicated twice - and that happens for each test. I dunno if it matters but it looks fishy. :)

wow2006 commented 5 years ago

yes we need one Name and second for command add_test

It just shortcuts for this command

add_test(NAME test-app COMMAND test-app)

You can run all test cases using

> cd build
> cmake path/to/main/CMakeLists.txt
> make
> ctest
mitchcurtis commented 5 years ago

OK, thanks! :)