Add test framework to Dawg - Githubissues

CartwrightLab / dawg

Simulating Sequence Evolution

GNU General Public License v2.0

11 stars 3 forks source link

Add test framework to Dawg #12

Closed zmertens closed 8 years ago

zmertens commented 8 years ago

This PR addresses #11 by adding a test structure to CMake (using the Boost unit test libraries). The test structure is taken from DeNovoGear.

Some things to point out:

The examples which are read by parse_test.cpp are specified to the ${CMAKE_SOURCE_DIR}/tests/Unit/dwg path.
There still may be some unnecessary references to DeNovoGear that are left over in CMake files

reedacartwright commented 8 years ago

Why are we using 'Dwg' instead of Dawg in the tests?

zmertens commented 8 years ago

Why are we using 'Dwg' instead of Dawg in the tests?

~~I think there is a name collision conflict with the name of the executable.~~ Updated.

reedacartwright commented 8 years ago

Jenkins, okay to test

dng-jenkins commented 8 years ago

Can one of the admins verify this patch?

reedacartwright commented 8 years ago

@zryan3 This patch now conflicts.

reedacartwright commented 8 years ago

Can we make it such that the TestU01 depending unit tests are skipped if the library is not found?

reedacartwright commented 8 years ago

TestU01 can't be found on Freebsd because it is installed in /usr/local/include/TestU01 directory.

If you add a PATH_SUFFIXES argument to FIND_PATH in your FindTestU01 model, it should be able to be found.

zmertens commented 8 years ago

~~Updated (also didn't amend the commit this time)~~

Need to make some more minor changes (no longer require TestU01)

reedacartwright commented 8 years ago

Looks like it worked. Yay!