On Mon, Aug 18, 2014 at 12:26 PM, Franck Joncourt <[email protected]> wrote:
> Hi Michael,
>
> [...]
> > I think using the c unit framework would be a good addition. It would
> > likely help test certain portions of the code more rigorously - would you
> > agree?
>
> Yes, I do. The goal is only to test functions to make sure they do what
> they
> should do and as expected. I think it could be useful to reach code not
> easily
> accessible from integration tests. It could also be extended further once
> we
> get a 100% coverage :)
>
>
Sounds good.
> > I'm not well versed in the c unit framework though, and I need to
> > work on that. I wonder which portions of the code could use the framework
> > the most?
>
> I have not checked the full lcov report yet to find out which functions
> would
> take benefit from unit testing. We can easily write many tests, but it can
> become quite time consuming. I think, first we can try to add this
> framework
> and see how to integrate it easily in the development process and take
> care to
> increase fwknop code coverage.
>
> I will be glad to have a look at it this week if you do not mind.
>
>
Sure that would be great. Perhaps c unit tests should be gated by a
./configure command line switch and corresponding #if blocks (like the
--enable-libfiu-support switch)?
Thanks,
--Mike
> Regards,
>
> --
> Franck Joncourt
>
> ------------------------------------------------------------------------------
>
> _______________________________________________
> Fwknop-discuss mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/fwknop-discuss
>
>
--
Michael Rash | Founder
http://www.cipherdyne.org/
Key fingerprint = 53EA 13EA 472E 3771 894F AC69 95D8 5D6B A742 839F
------------------------------------------------------------------------------
Slashdot TV.
Video for Nerds. Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Fwknop-discuss mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fwknop-discuss
