search

britram / python-ipfix

IPFIX implementation for Python 3.x
GNU Lesser General Public License v3.0
40 stars 20 forks source link

Improve test coverage #8

Closed britram closed 10 years ago

britram commented 10 years ago

Doctest testing isn't full coverage (for some things: reader, writer, ieutils, this is okay; for others, it isn't). Add additional tests to the documentation or supplemental tests to increase coverage.

britram commented 10 years ago

Test coverage holes as noted by @rouge8:

ipfix/ie missing 100, 115, 163, 166, 169, 172, 184, 187, 233, 245, 269, 287-288, 291, 321 ipfix/ieutils missing 29-52, 56-63, 67-70 ipfix/message missing 194, 255, 266-269, 288, 305-334, 352, 359, 363, 408, 411, 420-422, 500, 518, 530-533, 556, 589, 613, 619, 652-655, 678, 684-689, 715-717, 721-722 ipfix/reader missing 42-44, 54-60, 71-77, 87 ipfix/template missing 35-36, 89, 102, 114, 196-197, 213, 258, 282-286, 302, 307-312, 315-324, 337-341, 379 ipfix/types missing 202, 205, 225-235, 271-274, 277-280, 349, 363-364, 371-372, 378-381 ipfix/v9pdu missing 52-70, 73, 77-78, 81-90, 118-154, 165, 177-178, 181-185, 198-207, 215-217, 232-265, 275, 280, 283-298, 301-317 ipfix/writer missing 40-43, 46-50, 63-65, 75, 87-88, 101-103, 107-109, 122-126, 138

britram commented 10 years ago

working on this for a 0.9.6 release -- will need some test utilities to make this work, though...

britram commented 10 years ago

Test coverage is much better in 0.9.7, but could still be improved; testing moved over to nose. Tests that don't fit in a module cleanly are now in testutils.py. reader and writer are not covered, but also aren't really part of the API yet, either.