[tor-dev] Bored C programmers? I've got some warnings for you....
nickm at torproject.org
Thu Sep 8 15:11:03 UTC 2016
On Wed, Aug 31, 2016 at 2:16 PM, Nick Mathewson <nickm at torproject.org> wrote:
> Hi, all!
> I've just turned on some code  that makes all of the "bug" warnings
> that occur during the unit tests get logged to console. Previously,
> all warnings from the unit tests were off by default.
> Now there are a lot of warnings from the unit tests! For each one, we
> should either:
> 1) Decide that the warning message wouldn't actually reflect a bug
> in Tor, and change it.
> 2) Decide that there's a bug in the unit tests (or elsewhere) and fix it.
> 3) Decide that _in this case_, the unit tests are correctly testing
> code that *would* indicate a real bug in Tor. In this case, we should
> use the functions in log_test_helpers.c to temporarily suppress the
> log messages, and make sure that this message (and only this message!)
> is logged.
> This is going to be a fair bit of work, but with any luck, we can
> actually fix it all before the next 029 alpha comes out, and improve
> our confidence in our unit tests.
We're down to 17 of these.
More information about the tor-dev