search

farsightsec / wdns

low-level DNS library
Apache License 2.0
18 stars 5 forks source link

Add libmy coverage, following fstrm example. #30

Closed alesage closed 5 years ago

alesage commented 5 years ago

This uses the libmy code coverage macro, following the example in fstrm. To enable, use --enable-code-coverage during configure, note that we newly require the installation of lcov .

djw1149 commented 5 years ago

I tried code coverage on macOS. I had to "$ brew install lcov" then "$ ./configure --enable-code-coverage". "$ make check-code-coverage" worked, producing ./wdns-0.9.1-coverage/index.html which looked like a traffic light festival (e.g. as expected).

alesage commented 5 years ago

Confused b/c I thought I was paying attention to this but now I see the dir has lately changed, thanks @reedjc .

reedjc commented 5 years ago

I am not sure about a recent change, as the directory ./t/ has been that way since the wdns tests added in 2015. The change here looks fine. Thanks.