PelionIoT / mbed-trace

mbed trace library
Apache License 2.0
18 stars 15 forks source link

Improved test coverage and fixed valgrind error. #24

Closed tommikas closed 8 years ago

tommikas commented 8 years ago

Improved test coverage by enabling a test for a function that was previously not compiled. Fixed valgrind error by reducing the size of a very long string a little (still a very long string).

jupe commented 8 years ago

almost :D

Code coverage enforcement failed for the following metrics:
14:31:00     Lines's stability is 82.88 and set mininum stability is 83.77.
14:31:00     Conditionals's stability is 63.89 and set mininum stability is 64.08.
tommikas commented 8 years ago
07:51:03 Overall coverage rate:
07:51:03   lines......: 85.6% (250 of 292 lines)
07:51:03   functions..: 91.3% (21 of 23 functions)

which means

07:51:03     Lines's new health minimum is: 85.62
07:51:03     Conditionals's new health minimum is: 65.56
07:51:03     Lines's new stability minimum is: 85.62
07:51:03     Conditionals's new stability minimum is: 65.56
tommikas commented 8 years ago

test this please

jupe commented 8 years ago

lgtm :+1: